[Maxtremus]

I'm developing an aplication that uses only one Image object to handle images and show it in the screen, my problem is that it loads the same image file every loop, how do I fix it? Here's a short example of the code:

int x = 0;
Image img;
while(end == false){
    if(key == LEFT_PRESSED){
        x--;
    }
    if(key == RIGHT_PRESSED){
        x++;
    }

    switch(x){
        case 1:
            img = Image.createImage("/img01.png");
            break;
        case 2:
            img = Image.createImage("/img02.png");
            break;
        case 3:
            img = Image.createImage("/img03.png");
            break;
        case 4:
            img = Image.createImage("/img04.png");
            break;
        //case 5 ...
        //case 6 ...
        //case 7 ...
    }
    graphic.drawImage(img, 0, 0, Graphics.LEFT | Graphics.TOP);

}

[sharvan1981]

Hi,

You should do null of the image object on every event like that

int x = 0;
Image img;
while(end == false){
if(key == LEFT_PRESSED){
x--;
}
if(key == RIGHT_PRESSED){
x++;
}

switch(x){
case 1:
img=null;
img = Image.createImage("/img01.png");
break;
case 2:
img=null;
img = Image.createImage("/img02.png");
break;
case 3:
img=null;
img = Image.createImage("/img03.png");
break;
case 4:
img=null;
img = Image.createImage("/img04.png");
break;
//case 5 ...
//case 6 ...
//case 7 ...
}
graphic.drawImage(img, 0, 0, Graphics.LEFT | Graphics.TOP);

}

[grahamhughes]

Originally Posted by Maxtremus

I'm developing an aplication that uses only one Image object to handle images and show it in the screen, my problem is that it loads the same image file every loop, how do I fix it?

There's nothing obviously wrong with the code you posted... but your problem may lie elsewhere.

Where is this code? It looks like it's from the run() method of a class than extends GameCanvas and implements Runnable, yes? In that case, you need to call flushGraphics().

No, you don't need to set the variable to null.

Loading images can be time consuming on some devices... unless the images are very large, you might consider loading them (for example, into an Image[]) ahead of time.

Cheers,
Graham.

[Maxtremus]

Originally Posted by grahamhughes

There's nothing obviously wrong with the code you posted... but your problem may lie elsewhere.

Where is this code? It looks like it's from the run() method of a class than extends GameCanvas and implements Runnable, yes? In that case, you need to call flushGraphics().

No, you don't need to set the variable to null.

Loading images can be time consuming on some devices... unless the images are very large, you might consider loading them (for example, into an Image[]) ahead of time.

Cheers,
Graham.

Yes, it is in the run() method, I'm also using low level graphics programming (GameCanvas) because I want to make some effects after.

Using the Image[] does not fit here, because I'll have so many images, and loading them all before may take a long time to load and cause an Out of Memory exception, thats because I want a "real time" loading.

[fletcher.jonnes]

Once I have a problem with the method run and a parameter. A function call with a parameter inside a run method, every time the call was with the first value of the parameter, to get it "working" I've changed the parameter for a class atributte. Maybe this help you.

[grahamhughes]

Possible problems:

1. You aren't calling flushGraphics()... until you do this, nothing you draw will appear.

2. An exception is being thrown, and you're ignoring it.

3. "x" doesn't have the value you think it has.

4. You have two variables called "img", and one is hiding the other.

When you do this:

img = Image.createImage("/img02.png");

there are only two possible outcomes. Either:

1. img will refer to a new Image object, representing the data in /img02.png; or

2. An exception is thrown, and img is unchanged.

Nothing else can happen.

Cheers,
Graham.