My Run Method Does Not Work Witn Changing JLabel Images From The Other Class?

[Alerhau]

I guess the title says most of it. What I am trying to do is changing a JLabels icon to another using only another classes public stuff. (Both images, JLabels). In order to do this I am using this piece of code (Note, this is in a class which implements Runnable, so this is into the run(); method):

Thread.sleep(1000);

Engine Eng = new Engine();

Eng.Grid[1][1].setIcon(Eng.Sprite[0])

Why is it not working? Any soloution, please help!

[Alerhau]

I really need quick help.

[kammce]

What are your errors? I am guessing that the Engine class is one of your own design, no?

[Norm]

Are you using the GUI's thread to do it? Or on your own thread? Have you told the GUI it has new stuff to show?

You are creating a new Engine object. Is there an old one still around that you are looking at expecting it to be changed vs the new one where the changes are being made.

Perhaps a small program (SSCCE) that shows the problem

[Alerhau]

What are your errors? I am guessing that the Engine class is one of your own design, no?

No errors at all. It goes through the run method just fine, I've tested the run method, which loops at the same time as I move my character over the giant grid. In the end of the loop I have a JOptionPane which tells me each time it's finished & that works. So then I know that it is going through the code I have in my first post, but not using it. Well I don't know if it's using it. But I know that the JOptionPane pops up.

Are you using the GUI's thread to do it? Or on your own thread? Have you told the GUI it has new stuff to show?

It's another class, which I call "Run" that is the thread class. Then I just create a thread object for that class & I do "RunThread.start();".

You are creating a new Engine object. Is there an old one still around that you are looking at expecting it to be changed vs the new one where the changes are being made.

How do you mean?

[Norm]

Is there an old one still around that you are looking at expecting it to be changed vs the new one where the changes are being made.

How many different places in your code do you create an Engine object?
Has one been created and is being shown when your posted code is executed which creates a new one.

Otherwise make a small program (SSCCE) that compiles and executes and shows the problem.

[Alerhau]

How many different places in your code do you create an Engine object?

Only that place. (The "Place" is in a different class, called Run which has the Runnable implemented)

Has one been created and is being shown when your posted code is executed which creates a new one.

What do you mean?

[Norm]

Can you explain what "is not working" means?

What should the result of the setIcon method call be?

Eng.Grid[1][1].setIcon(Eng.Sprite[0])

Why do you do this outside of the Eng class?
Why isn't it done in the Eng class's constructor?

Grid[1][1].setIcon(Sprite[0]);

[Alerhau]

I want this to work because I need Java to change one icon each second that goes, I also need the movement of my game to work at the same time.

So that I can both move my character around on the Grid & change icons every at the same time.

If I just call the Thread.sleep() into the main class (Engine) Everything in the class will sleep.

If there any way to do it into my main class with just making some stuff to sleep at the side of my movement?

[Norm]

Use a Timer to do something every time period.

It is almost impossible to suggest what to change in your code after seeing 3 lines of code.

If you could make a SSCCE to demonstrate the problem we could make better suggestions.

[Alerhau]

How would I make the Timer? Does the timer update, do something every time period? Like every 10 seconds it changes a JLabel icon?

[Alerhau]

Got it to work! Thanks!

[Alerhau]

Thanks for all help, the timer worked awsomely amazing!