Area 2D > Collision shape Question

IvanGrozny

Hi all !

I am making a (player controlled) BigBasket ,which goes left-right and collects falling fruit.

The problem I am having is with Area2D detection of my basket. My Basket detects a fruit has fallen at start, even though there is no fruit.

Even if I reduce my collisionSphere2D I still have a detection of collision.

The entire code is :

extends KinematicBody2D

const BASKET_SPEED = 200

var BasketFull = 0
var BasketCount = 0
var screen_size = Vector2(1280, 720)

var BasketPosition_x = round(screen_size[0]/2)	
var BasketPosition_y = round(screen_size[1]*0.7)
var kosaraPosition = Vector2(BasketPosition_x, BasketPosition_y)

func _ready():
	set_process(true)

	
func _process(delta):
	movementBasket(delta)

func movementBasket(delta):	
	if (Input.is_action_pressed("ui_left")):
	    BasketPosition_x += BASKET_SPEED * delta*(-1)
		
	if (Input.is_action_pressed("ui_right")):
		#print("registriran move_right")
		BasketPosition_x += BASKET_SPEED * delta
	
	kosaraPosition = Vector2(BasketPosition_x, BasketPosition_y)
	get_node("../BigBasket").set_pos(kosaraPosition)

func _on_Area2D_body_enter( body ):
	BasketFull = 1
	BasketCount +=1
	var TextBasket = str("Basket has caught the ballon : ", BasketCount , " times")
	get_node("../Label").set_text(TextBasket)
	pass # replace with function body

Bu the problem is not in code, I'm 100% sure.

lonesurvivor

I guess the problem here is that you have some PhysicsBody2D that is colliding with your Area2D when you start the game and therefore reports a collision to your callback function. There is a RigidBody2D in your node list, i guess that is causing the problem. I don't know what you are using it for (doen't look to me like it is necessary for your goal), but if you want to keep it you need to check from which body the collision was reported. You can do that with area.get_name() for example.

Rebel_X

From my experience, you will get nothing but problems if you make Area2D inherit Physics Body or vice versa. My suggestion is that, make your hierarchy of nodes as follows:

Node2D
- Sprite2D
- KinematicBody2D
- - CollisionShape2D or CollisionPolygon2D
- Area2D
- -CollisionShape2D

Node2D: Big Basket.

KinematicBody2D: for moving Big Basket, use set_pos() for parent node and move(0,0) for Kinematic to detect collisions and block movement without really moving since the parent moved using transform. You can test for collision and don't process set_pos() for parent if the KinematicBody2D is_colliding()

Area2D: for catching balloons or fruits. Note: that on the signal Body Enter for the Area2D, you have to test for and ignore the sibling KinematicBody2D from being detected. It can be done simply by:

func _on_body_enter(body):
		if (body.get_parent() == self):
				return
... rest of code to collect balloons or fruit...

That is, if the script is assigned to the parent Node2D, and the signal connection is done to point to the script in Node2D.