Values of a float not updating in classes besides the one it is defined in :(

[wreed12345]

So I have been trying to fix this for nearly an hour now, but I am truly clueless.... In my class MainGame i define 2 floats:

1   2

private float levelX = -500; private float levelY = -500;

and I also define a getter for both of those. When you press the up arrow key the levelY is decreased (using libgdx so it may be irregular seeing something like this) And I do the same for the rest of my arrow keys. Now the problem is when I am trying to use the value of the levelX or levelY in another class it always returns as -500 even though if I print out the value in the MainGame class it is changed from that original -500 but not in the other class.... Is this some stupid error I am not picking up on here or is it something deeper ?

[matheus23]

How are you incrementing/decreasing those values?

This is definitely a language-problem.

[wreed12345]

pretty much its just this for each key:

1   2   3   4   5

if ((Gdx.input.isKeyPressed(Keys.W))) {                levelY -= playerSpeed;                up = true;             } else                up = false;

where

1	playerSpeed = 2;

Games published by our own members! Check 'em out!

[ra4king]

Are you also updating the values in the other class?

[Gamerulf]

1   2   3   4   5   6   7   8   9   10   11

public float getLevelY() { System.out.println("getY called"); return levelY; } public float getLevelX() { System.out.println("getX called"); return levelX; }

Then run game and see if those methods are called.

Im new to this so that's all I can come up with.

[wreed12345]

@ra4king I never knew I had to do that in past experiences it automatically did that... i think

@gamerulf
both get statements are being called repeatedly when trying that

[cubus]

only 2 possible reasons for that:

1) the values get reset between changing and getting

2) you are changing the values in one instance and getting them from another instance
print out the instance address in the key handler method and in the getters. i'm pretty sure they are not equal...

[Gamerulf]

This would be a whole lot easier if you'd show us the classes.

[wreed12345]

I am in the process of cleaning all of this code up so here is the main class:
http://pastebin.java-gaming.org/7ab3539984a
and the class im calling it from:
http://pastebin.java-gaming.org/ab359489a48
The latter needs to be re organized so please ignore that for now although it should be readable

Before everything in my game was in one class for some stupid reason, so i am in this spring cleaning mode lol

[Sammidysam]

I think you should pass levelX and levelY as an argument instead of creating a main instance.  I feel like trouble awaits when you do that.

Games published by our own members! Check 'em out!

[wreed12345]

I tried doing that earlier and it worked, although i was really curious of why this was not working out

[Gamerulf]

Passing it like an argument? I think im unsure of what an argument is.

Like "main.levelX" ?

[ra4king]

Err you seem to not understand OOP. I advise you to learn more about Java before making games.

In your code, you are never calling mainGame.update() inside Resource however you claimed to be calling it, which leads me to believe you are using a different instance in a separate class. This means you don't understand how different instances are separate from each other.

[Sparky83]

It is like ra4king says. Because you have a new instance of ressource in GameMain and in Ressource and you also create an instance of its own at the very beginning here:

1   2   3   4   5   6   7   8

public class Resource {    public int x, y;    int width = 64;    int height = 58;    boolean drawResource = true;    float resourceTimer = 0;    GameMain mainGame = new GameMain(); // HERE

So you have two independent instances of GameMain.

[wreed12345]

@sparky83 you are definately right about that but how do I actually avoid creating a new instance of that? Do i have to use static getters / setters or something. I feel very foolish for not realizing this 

[Sammidysam]

I would base all game logic and rendering in separate classes than Main, and just use Main for the game loop.

[wreed12345]

but still, if i create one class containing Level with has levelX and levelY how do I acess those without creating new instances

[Sammidysam]

Arguments, I guess.  I don't have a good enough grasp of your code base to suggest other alternatives.

[Grunnt]

I suggest you first do some Java tutorials as your questions are not related to game development but to Java and OO programming in general. A good place to start are the Java tutorials by Oracle:http://docs.oracle.com/javase/tutorial/

[wreed12345]

Does anyone have ideas on solving this? I feel like static Is a bad idea and in some cases arguments won't solve this...

[vbrain]

Does anyone have ideas on solving this? I feel like static Is a bad idea and in some cases arguments won't solve this...

Static is not a bad idea. You should really study more about Java before attempting a game.

[Gamerulf]

I had 150hours of programming in school before we even touched graphical/visual programming.
And that doesnt include all the homework we had...

And Im still lost

[wreed12345]

Would you recommend diverging else over static?

[Jimmt]

Would you recommend diverging else over static?

What?
What does else have to do with static?

[wreed12345]

Would you recommend anything* else over static, sorry auto correct changed it

[Sammidysam]

Well you could make lots of classes and work around that.

[Jimmt]

Shouldn't use static just because "normal" variables aren't working for you. levelX and levelY sound like iffy variables to make static, unless you only are ever going to have one level total (also, shouldn't you have a level class? and a player class?)
Referring to the original question, are you somehow calling onCreate() excessively? By the method name I wouldn't think so but I'm only familiar with desktop libgdx. When is the getter called in relation to the setter?

[Gamerulf]

The other class doesnt store it. He uses main.getLevelX() as "values" for the conditions of an if-statement.
If I remember correctly..

[vbrain]

Dear God, it sounds to me like you need to completely redesign and restructure your entire code.

[wreed12345]

@vbrain is my coding really that bad?