Thread: Help with Graphics class and keyPressed method

Hi, here is the lab assignment that I'm currently working on : Using nested loops and objects in an applet

Right now I'm stuck at these two steps :

Now add some more code to your paint method so that it displays the terrain type where the player is currently located. This can be displayed near the bottom of the applet. To do this you use the drawString method of the Graphics class.

Next, fill in keyPressed so that the user can move by using the arrow keys. Don't forget to repaint so they get an update on what kind of terrain they are on. The user should be able to move around now (after clicking the applet to gain focus).

import java.applet.*;
import java.awt.*;
import java.awt.event.*;
 
public class TreasureApplet extends Applet implements KeyListener {
    private Island island;
 
    public void init() {
        island = new Island(10);
        addKeyListener(this);
    }
 
    public void keyPressed(KeyEvent e) {
        if (island.currentLocation() == Island.WATER) {
            return;
        }
        if (island.currentLocation() == Island.TREASURE) {
            return;
        }
        if (island.currentLocation() == Island.PIRATE) {
            return;
        }
        if (e.getKeyCode() == KeyEvent.VK_LEFT) {
            island.moveWest();
        }
        repaint();
        if (e.getKeyCode() == KeyEvent.VK_RIGHT) {
            island.moveEast();
        }
        repaint();
 
        if (e.getKeyCode() == KeyEvent.VK_UP) {
            island.moveNorth();
        }
        repaint();
 
        if (e.getKeyCode() == KeyEvent.VK_DOWN) {
            island.moveSouth();
        }
        repaint();
    }
 
    public void keyReleased(KeyEvent e) {
    }
 
    public void keyTyped(KeyEvent e) {
    }
 
    public void paint(Graphics g) {
        int x = 0;
        while(x < 10) {
            int y = 0;
            while(y < 10) {
                if (island.terrainAt(x,y) == Island.WATER) {
                    //draw the water
                    g.setColor(new Color(0,0,210));
                    g.fillRect(x*40, y*40, 40, 40);
                    g.drawString("Splash! You got to the water.", 50, 430);
                }
                else if (island.terrainAt(x,y) == Island.SAND) {
                    //draw the sand
                    g.setColor(new Color(160,82,45));
                    g.fillRect(x*40, y*40, 40, 40);
                    g.drawString("Ouch. Sand got into your sandals.", 50, 430);
                }
                else if (island.terrainAt(x,y) == Island.TREE) {
                    //draw the treasure
                    g.setColor(new Color(50,205,50));
                    g.fillRect(x*40, y*40, 40, 40);
                    g.drawString("You are standing by a tree.", 50, 430);
                }
                else if (island.terrainAt(x,y) == Island.TREASURE) {
                    //draw the treasure
                    g.setColor(new Color(225,215,0));
                    g.fillRect(x*40, y*40, 40, 40);
                    g.drawString("Congratulations. You found the treasure", 50, 430);
                }
                else {
                    //draw the pirate
                    g.setColor(new Color(0,0,0));
                    g.fillRect(x*40, y*40, 40, 40);
                    g.drawString("It's the pirate. Run!", 50, 430);
                }
 
                //draw a square so the user can distinguish locations clearly
                g.setColor(Color.BLACK);
                g.drawRect(x*40, y*40, 40, 40);
 
                y++;
            }
            x++;
        }
    }
}

When I try to compile and display the applet, things got really weird when all 5 drawString strings appear at the same time. I mean, they sit on top of each other. It's weird because I have already used an if-else sentence to classify them, how in the world they still appear all at once?

Another thing I would like to ask is the keyPressed method. I think I have wrote the code wrong somewhere because I don't see the player moving at all. Please take a look and give me some suggestions on how to change it. Thank you all so much.

Thank you. I know how to solve my problem now. First, to classify and display message, I have to use "island.currentLocation" and not "island.terrainAt". Second, I did NOT click on the applet to gain focus, so that's why I don't see any movement lol. Anw, thanks a lot for a very nice suggestion. I'll keep that in mind.