Teleporting problem?

assuming that you are using something like getOneIntersectingObject() and your teleport object is called Teleport.class you would introduce a boolean variable like: private boolean teleported = false ; // declared as a class variable .... if (getOneIntersectingObject(Teleport.class) && ! teleported){ teleported = true; // do the action of teleporting to the other teleporter } if ( teleported && !getOneIntersectingObject(Teleport.class)) // player is no longer touching it teleported = false;

Hmm... actually, trying this results in my character getting stuck at the teleporter she is teleporting to. This is the code, which is slightly modified from yours as I use 'can_use_teleporter' instead of 'teleported' (so its essentially opposite)

I am not sure if this will correct the functionality of it, but it appears that line 48 should be '&&', not '||'. Same with line 67. I think the coding had been made more complicated than need be because of seperate classes (redportal/blueportal) for the portals. You can make them both the same class and add the Color of the portal as a parameter in the constructor of the 'Portal' class -- public Portal(Color color). You can add an instance variable of Portal class to the code declared with -- Portal portalLink; -- and add the methods -- public void setPortalLink(Portal portal) -- and -- public Portal getPortalLink() -- to facilitate setting and retrieving the value of the variable of the objects. When creating the portals, create both -- Portal rPortal = new Portal(Color.red); Portal bPortal = new Portal(Color.blue); rPortal.setPortalLink(bPortal); bPortal.setPortalLink(rPortal) -- then add them to the world. Once done, you can easily find out where to teleport to when intersecting any portal using one code-set. Also, you only need one code-set to check if teleport complete (not intersecting any portal object to reset the can_use_portal variable).

I didn't realize you had two different classes for the teleports you would probably need two boolean variables and do something like: private canUseRedPortal = true; private canUseBluePortal = true; ..... if (canUseRedPortal && getOneIntersectingObject(redportal.class)!= null){ canUseBluePortal = false; // since you are teleported to blue portal setLocation(bportal.getX(), bportal.getY()); } // just teleported to red portal so it is availabe as soon as you step off it if (!canUseRedPortal && getOneIntersectingObject(redportal.class)== null) canUseRedPortal = true; // set true when you are not touching teleporter if (canUseBluePortal && getOneIntersectingObject(blueportal.class) != null){ canUseRedPortal = false; //since you are teleporting to red portal setLocation(rportal.getX(),rportal.getY()); } // just teleported to blue portal so it is available as soon as you step off it if (!canUseBluePortal && getOneIntersectingObject(blueportal.class) == null) canUseBluePortal = true;

For some reason I have a feeling that I should be able to use just 1 boolean that states whether or not I am allowed to use a portal of any kind (if colliding with red or blue). This is my current code for the player (named chell) in it's current state. At the moment when chell touches either portal she becomes 'stuck' to the red portal as if its constantly teleporting chell to the red portal. Thank you for all the help so far guys! I linked it to pastie as it's quite a bit of code to put in the post itself.

Lines 50 and 65, change '||' to '&&'. You want to set canUsePortal true if you can't currently see any portal. At the moment you're setting it true if you can't see the blue portal OR you can't see the red portal, which is always going to be the case (you'll never be able to see them both at the same time).

I had mentioned this two days ago. The same post continued which is illustrated in the 'Teleport Demo' that I am uploading now.