Discussions

Thank you it works now

There are many things that can be improved upon. Using 'getObjects(Shape.class).size()' would be better than having a field keep track of objects in the world. Having 'inWorld' in the actor class is meaningless as all actors that are acted on ARE in the world. Line 10 above, will still error when the shape is removed by the mouse (switch the order of lines 9 and 10). I am not sure if you want a constant value for 'colourChance' or not; but, if the color of the squares are to change back and forth during execution, remove the field and use 'Greenfoot.getRandomNumber(2)<1' anywhere in the c

Thanks so much for the quick and helpful replies - really appreciate it! It's working great now. :)

Thank you mr GreenGoo

I see one problem immediately. Neither barrel nor target should be sub-classes of player (they both should 'extend Actor'). Move the methods related to the angle of the barrel from the player class to the barrel class. Move the 'checkTarget' method from the player class to the target class and call it from the act method, there.

thank you very much

Use the GreenfootImage 'scale' method on the image.

Make two classes, one for each player. Then make a method to control, move, turn (how do you want) the first class. Copy it into the other and change the key bindings. If you have more questions, ask.

thank you

Here is what I see: your act method has three method calls in it. First a call to 'move' (all well and good). Then, a call to attackCharacter, which is an empty method. Finally, a call to 'hitCharacter', which returns the value of a boolean field. This boolean value is not used in any way and the 'moveTowardCharacter' is never called (anywhere).

It seems a bit wasteful moving back and forth, like that, we could replace the code from line 11 on with: <Code Omitted>

One thing I would change is line 32, which I would make: <Code Omitted>

Thanks , works perfectly!

@Gevater_Tod4711, Thanks a ton! it works fine now :D @danpost, thanks for showing me how to slow it down it was going a bit fast!

To get the x value in world coordinates of the far right side of the scrolling area, then with the new methods, you can use: <Code Omitted>