issue with "other"

[Rick-V]

hello i'm pretty new on GMS and recently i found a problem:
i was trying to make a movement like chess, here is the code, casella_movimento stay for "movement_slot"

step event:

mb_left = mouse_check_button(mb_left);

///slots
if(mov == true){
instance_create(x+40,y,obj_casella_movimento);
instance_create(x,y+40,obj_casella_movimento);
instance_create(x-40,y,obj_casella_movimento);
instance_create(x,y-40,obj_casella_movimento);
    mov = false;
}

slot collision event:

///movement
if(mb_left){
    x = other.x;
    y = other.y;
    mov = true;
}

the problem is that the player doesn't move at the slot's x, y.
Could anybody help me please?
thanks a lot!

 

[JFitch]

mb_left is a built in constant. You can't use it as a variable.

 

[Rick-V]

actually in my code i wrote mb_left in italian (mb_sinistra), so the built in constant is not the problem

 

[JFitch]

if(mb_sinistra){
   x = other.x;
   y = other.y;
   mov = true;
}

Your code is saying "if the object calling this code is colliding with a slot and if the left mouse button is held with the mouse anywhere on the screen, move the object calling this code to the slot's position." Is that what you're trying to do?

 

[Rick-V]

thanks, i tried to fix it but it doesn't seem working anyway

///movimento
if(mb_sinistra){
    if(position_meeting(mouse_x,mouse_y,obj_casella_movimento)){
        x = other.x;
        y = other.y;
        mov = true;
    }
}

 

[Paskaler]

You'll have to get the ID of the instance:

///movimento
if(mb_sinistra){
  var inst = instance_place(mouse_x, mouse_y, obj_casella_movimento);
 if(instance_exists(inst)){
       x = inst.x;
       y = inst.y;
       mov = true;
   }
}

Also, other is only usable from within a with statement to target the instance outside the statement and in the collision event to target the instance the instance that called the event collided with.

 

[Rick-V]

You'll have to get the ID of the instance:

///movimento
if(mb_sinistra){
  var inst = instance_place(mouse_x, mouse_y, obj_casella_movimento);
 if(instance_exists(inst)){
       x = inst.x;
       y = inst.y;
       mov = true;
   }
}

Also, other is only usable from within a with statement to target the instance outside the statement and in the collision event to target the instance the instance that called the event collided with.

thanks! now it works perfectly!
I have still a question: it's like the mask of the slot is moved from the center of the sprite(both sprites are centered)