Can anyone tell me what mistake I made here?: Page 2

Can anyone tell me what mistake I made here?

TwistedTwigleg

It might be due to drag-and-drop, since both Imgur and Vanilla use drag-in-drop for image uploading. That would also explain why one link goes to Imgur, and another to the forums server.

Edit: Unfortunately, its part of the Advance editor and there is no great way to disable it, that I can see anyway. I might be able to modify the Advance editor though, but I haven't looked into it yet. I suppose we could also disable drag-and-drop for the Imgur plugin to resolve the issue of double image uploads.

UnknownUser

that I can put in the code of "mob" to make it work?

SIsilicon28

Could you elaborate please?

UnknownUser

@SIsilicon28 said: Could you elaborate please?

So how can I make it work, do I have to put something in the mob script or something?

SIsilicon28

Remember this? What I was suggesting to you was a way to figure out the root of this problem. This is not relevant to the mob script itself. It was only to add typed scripting to the mob variable and see where in your code you assigned something you shouldn't.

Ok, I know what to do. Could you please share you entire script? That can help us better to help you. Don't share it in screenshots please. ;)

UnknownUser

how can I do that? I copy it and paste it?

Megalomaniak

Yes then after pasting it, on the line in front and behind the pasted code you add ~~~ and it gets formatted as a code block.

UnknownUser

func _on_mobtimer_timeout():
	$mobpath/mobspawnlocation.offset = randi()
# warning-ignore:shadowed_variable
	var mob = mob.instance()
	add_child(mob)
	var direction = $mobpath/mobspawnlocation.rotation + PI / 2
	mob.position = $mobpath/mobspawnlocation.position
	direction += rand_range(-PI / 4, PI / 4)
	mob.rotation = direction
	mob.linear_velocity = Vector2(rand_range(mob.min_speed, mob.max_speed), 0).rotated(direction)

(TwistedTwigleg edit: Fixed the code formatting)

UnknownUser

that I can change or modify to make it work?

Megalomaniak

That is just a single function from the script, mind copying and pasting the whole thing as per @SIsilicon28 request?

UnknownUser

then would it be the function of mobtimer to change something?

SIsilicon28

We need the entire script if you don't mind.

UnknownUser

extends Node

export var mob: PackedScene = preload("res://mob.tscn")

var score 

func _ready():
	randomize()
	new_game()

func game_over():
	$scoretimer.stop()
	$mobtimer.stop()

func new_game():
	score = 0
	$ninga.start($starposition.position)
	$startTimer.start()

func _on_mobtimer_timeout():
	$mobpath/mobspawnlocation.offset = randi()
# warning-ignore:shadowed_variable
	var mob = mob.instance()
	add_child(mob)
	var direction = $mobpath/mobspawnlocation.rotation + PI / 2
	mob.position = $mobpath/mobspawnlocation.position
	direction += rand_range(-PI / 4, PI / 4)
	mob.rotation = direction
	mob.linear_velocity = Vector2(rand_range(mob.min_speed, mob.max_speed), 0).rotated(direction)

func _on_scoretimer_timeout():
	score += 1

func _on_startTimer_timeout():
	$mobtimer.start()
	$mobtimer.start()

TwistedTwigleg

@alex0264 - Try this:

extends Node

export var mob_scene: PackedScene = preload("res://mob.tscn")

var score 

func _ready():
	randomize()
	new_game()

func game_over():
	$scoretimer.stop()
	$mobtimer.stop()

func new_game():
	score = 0
	$ninga.start($starposition.position)
	$startTimer.start()

func _on_mobtimer_timeout():
	$mobpath/mobspawnlocation.offset = randi()
	var mob_clone = mob_scene.instance()
	add_child(mob_clone)
	var direction = $mobpath/mobspawnlocation.rotation + PI / 2
	mob_clone.position = $mobpath/mobspawnlocation.position
	direction += rand_range(-PI / 4, PI / 4)
	mob_clone.rotation = direction
	mob_clone.linear_velocity = Vector2(rand_range(mob_clone.min_speed, mob_clone.max_speed), 0).rotated(direction)

func _on_scoretimer_timeout():
	score += 1

func _on_startTimer_timeout():
	$mobtimer.start()

UnknownUser

if it works, thank you very much

SIsilicon28

@TwistedTwigleg , don't forget to explain what you changed in the script. ;)

TwistedTwigleg

@SIsilicon28 said: @TwistedTwigleg , don't forget to explain what you changed in the script. ;)

Good idea! When I posted I didn't have the time, but now that I do...

I changed the name of the exported PackedScene from mob to mob_scene to make it easier to tell what variable holds what.
I changed the name of the instanced scene from mob (line 23 in the original) to mob_clone. I also updated mob.instanced to mob_scene.instanced.

And that was basically it. There wasn't really a lot to change that I could see, outside of both the local variable and class variable sharing the name mob, which I am fairly positive was causing the issues. By changing both of these variables to use unique names, the issue where GDScript gets confused and errors when spawning should be fixed.

UnknownUser

wow, thank you very much, really

« Previous Page