Seems ok to me. I need to ask, is this in your world or actor class? I certainly have no problems from the World. I'd suggest you change the code slightly: this compacts the code a little and you ensure you have one set of Mi coordinates. It's 'possible' that between your mouseX and mouseY variable settings, the mouse could have moved, distorting the actual position! In your gotoAngle computation, i'm not familiar with the mathematic logic there (but is y and x round the right way? It probably is, but want to check). Where are getY() and getX() coming from??? If you can answer questions, I'll try to help.. If you stick a before the gotoAngle, you should see the position constantly changing. If you don't, you know there is something wrong in terms of where you've put this code (i.e. actor vs world).

I think you will need to set up two integer instance variables to hold the most current mouseX and mouseY coordinates. Then when the mouseMoved(this) call returns null, you can use those values to traverse around.

@sp33dy This code is in an actor class, the player. The player turns to face the mouse, by working out the mouse position in relation to it's own current position and then uses the x and y distances to work out the angle to the mouse (gotoAngle).

As dan has hinted at; is your problem that the MouseInfo is sometimes null, hence the direction breaks? If so, you'll need two instance variables (i.e. private int x, y = 0) variables and set them to the last MouseInfo x and y position. You'd then add an else statement to your null check to set position based on the last recorded x/y position. If not, you need to diagnose why the mouse position isn't changing (it should). I was originally worried that your actor was't receiving the mouse input, but you aren't relying on a press, over or click (so all should be working). Are you sure your Act method is calling this method?? Otherwise, I need more information to help you :(

@Gazzzah, I think you are looking for something like the following Then somewhere (after the call to the findMouse() method) you should have something to actually move the actor, like:I set it up to somewhat gradually rotate to the proper angle, and not move if actually at the location of the mouse. Oh, and yes, this would go in the class of the actor that follows the mouse.

Play around with the adjustments to r in the setRotation statement (r - 2). Try (r - 5); unless you want your actor to immediately turn toward the mouse; in which case you would just use 'setRotation(gotoAngle);'.

Hooray! That problem is solved. Thanks for your help guys. danpost, that turn towards equation is not what I was after but funnily enough will come in handy later, thanks. Now I'm still having one more problem with updating mouseinfo. If I click, it sets a boolean leftClicked to true (and same with rightClicked) using the following code. Don't worry about "released", that's not relevant. My problem is that if I left or right click it will stay that way until I move the mouse. How might I fix this?

Try in line 1. I am assuming it does not matter where the mouse is clicked, just that it is clicked. Then for your method click()

I am not sure if this is what you wanted. Your post was a bit vague. I was unsure what you meant by 'it (what) will stay that way (how) until I move the mouse'. I was thinking leftClicked and rightClicked will retain a true value??? But, if that were the case, you could always set them back to false once you have used them.

The problem, I believe, stems from the fact that you are calling getMouseInfo three different times. After the first time, it clears to null, until a mouse event occurs to put information back into it. With that being said, the code provided above should work for you.

Ahhh, yes that makes sense now. So the mouse info stays null until it changes. I'll try the alternative you suggested next I get the chance. And by 'it will stay that way...' I was referring to the leftClicked boolean (or rightClicked) staying true until the mouse was moved. An thus the effect of leftClicked being true continues as if held down. I'll try mouseClicked(null) and hopefully that'll work.