I need help with my script. Debugger sends no errors but it's still not working. Help!

Voldrania

So basically what i'm trying to do is if the object (Rigidbody2D in this case) is pickable, whenever I hold it with the mouse button it should equal the position vectors of the object and the mouse so I can move it around. However, due to probably my errors, nothing happens while I click the object.

Here is the entire script:

Hold is an InputAction for LMB and mouse_entered is the signal for detecting whether the mouse is in or outside of the object. If it's in, it should emit that signal and also report some inputevents.

Also the RigidBody2D in this case is being instanced as a PackedScene in a different scene.

Here is the code for that:

Anybody can figure out the issue, please let me know. I'm a newbie to game development and i'm still learning.

TwistedTwigleg

While I am not positive, I think the issue is that you are not setting the position of the Rigidbody2D to the mouse position, you are instead just setting a variable, objPos, to the position of the mouse event. While objPos contains the position of the RigidBody2D when initialized, I believe GetGlobalPosition returns a copy of the position rather than a pointer (or C# equivalent). This means changing the return value of GetGlobalPosition will not change the position of the RigidBody2D.

My C# skills are a little rusty and I'm not positive I have the syntax right, but replacing the _Mouse_Has_Entered code in the Rigidbody2D with the following should work (untested):

public void _Mouse_Has_Entered()
{
	if (Input.IsActionPressed("Hold"))
	{
		Vector2 mousePos = GetGlobalMousePosition
		
		// Set the position of the RigidBody to the mouse 
		GlobalPosition = mousePos
	}
}

Hopefully this helps a bit, and welcome to the forums!

Voldrania

OFC!! I just realized!! OMG thanks man, i'm such an idiot haha.. -.-

Voldrania

That's called being stressed out and trying to rush the learning process.

Voldrania

Actually, it didn't work. I would guess it's the mouse entered signal that's the problem because I tried putting the Mouse's If function into the _Process(float delta) method and then it started working. But do you know any other way I could pick up collision then? Or rather, mouse overlap on an object?

UPDATE: Found the issue! Added console.writeline method at the start of the function. The signal isn't outputting anything, that's the issue. So how do I check for collision, overlap with mouse?

UPDATE2: Nevermind, the console writeline method didn't interact with godots built in. How do I print out with C#?

Losing my mind on this haha..

cybereality

I don't use C# but this looks like the way:

GD.print("Hello, Godot!");

TwistedTwigleg

No worries about the stress, it happens to all of us. I find one of the best ways to deal with programming issues that I cannot figure out is to step away from it for a bit, maybe make some coffee, and then try to tackle the issue with fresh eyes.

As for how to print to the console, I believe @cybereality is correct, that GD.print is what you need to use.

For methods to detect if the mouse is over a RigidBody2D, unfortunately I do not know right off, as I mostly do 3D game development. You might be able to use _input and then do a bounding box check, but that seems a tad overkill. If I remember correctly, either on the RigidBody2D node or the CollisionShape2D node there is a checkbox to allow it to be picked (something like pickable if I recall correctly) that may need to be checked before the mouse_enter and mouse_exit signals are sent.

Voldrania

Thanks @cyberreality That worked perfect. @TwistedTwigleg Yeah I have checked the pickable but it still won't send anything, as I tried now to put GD.print("Mouse detected") in the mouse entered function that's connected to the signal, and it doesn't detect anything. So i'll probably have to figure out something. Anyway, thanks for helping me out guys!

cybereality

I've used "mouse_entered" on Area2D nodes, that seems to work reliably. You can probably just add an Area2D child to your object and use that for mouse detection.

Voldrania

@cyberreality idk if im connecting this right but ANYTHING I TRY doesn't work.. I've tried a million stuff now, even Area2D like in this script... How do you connect it? I suppose you use GDScript though.

cybereality

I always use "self" as the second parameter for connect (probably "this" in C#).

Not sure what it's supposed to do though.

Voldrania

Anyway, thanks for trying guys. I'll probably test different stuff till I get it functioning.

Voldrania

@cybereality @TwistedTwigleg Hey guys, sorry for keeping you busy wtih my questions but I solved it now. I simply connected the mouse entered signal on the control node on my bullet texture to a function on the scene node via the inspector tab or whatever it is. But now I have got another issue. When I execute it, all instances of that PackedScene I have loaded in go to that position. And I only want it to move the single one I click on. Any ideas? I was thinking myself of if I could create a for loop so every individual scene created has a different name, but I don't think that will solve it.

cybereality

You just need to keep track of the currently selected object. I've done this before by changing a global variable (e.g. a variable on the main script, not the object) on mouse enter/exit.