Question

I don’t know why but nothing is appearing?

I suppose to have a applet of a house.

import javax.swing.*;
import java.awt.*;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.*;


public class color extends JApplet
{
    public void init()
    {
        addMouseListener(new MyMouseListener());
        getContentPane().setBackground(Color.white);
    }
    public class MyMouseListener implements MouseListener
    {
    public void mouseClicked(MouseEvent e) 
    {
           int x = e.getX();
           int y = e.getY();
           boolean closeDoors = true;

           if(x>330 && x<280 && y>20 && y<20)
           {
               closeDoors = false;
               repaint();

           }

    }
    public void mouseEntered(MouseEvent e) { }
    public void mouseExited(MouseEvent e) { }
    public void mousePressed(MouseEvent e) { }
    public void mouseReleased(MouseEvent e) { }
    }
    public void paint ( Graphics g, boolean closeDoors)
           {
                super.paint (g);

                do
                {
               g.drawLine (35, 50, 570, 50);
                g.drawLine (35, 50, 250, 0);
                g.drawLine (250, 0, 570, 50);
                g.drawRect (50, 50, 500, 350);
                g.fillRect (100, 75, 80, 80);
                g.fillRect (400, 75, 80, 80);
                g.fillRect (240, 200, 125, 200);
                }
                 while (closeDoors = true);

                 if (closeDoors = false);
                {
                g.drawLine (35, 50, 570, 50);
                g.drawLine (35, 50, 250, 0);
                g.drawLine (250, 0, 570, 50);
                g.drawLine (180, 120, 100, 120);
                g.drawLine (400, 120, 480, 120);
                g.drawLine (140, 75, 140, 160);
                g.drawLine (450, 75, 450, 160);
                g.drawRect (50, 50, 500, 350);
                g.drawRect (100, 75, 80, 80);
                g.drawRect (400, 75, 80, 80);
                g.drawRect (240, 200, 125, 200);
                g.drawOval (330,280, 20, 20);
                }
            }
}

Answer 1

I’ll try to help get you on the right track 🙂

You may already know this, but if your not using an IDE, I recommend using appletviewer to develop your applets instead of with a browser. Just food for thought 🙂

First of all, Toader Mihai Claudiu‘s suggestion is correct. Change

            do
            {
           g.drawLine (35, 50, 570, 50);
            g.drawLine (35, 50, 250, 0);
            g.drawLine (250, 0, 570, 50);
            g.drawRect (50, 50, 500, 350);
            g.fillRect (100, 75, 80, 80);
            g.fillRect (400, 75, 80, 80);
            g.fillRect (240, 200, 125, 200);
            }
             while (closeDoors = true);

             if (closeDoors = false);
            {

into

            if (closeDoors)
            {
           g.drawLine (35, 50, 570, 50);
            g.drawLine (35, 50, 250, 0);
            g.drawLine (250, 0, 570, 50);
            g.drawRect (50, 50, 500, 350);
            g.fillRect (100, 75, 80, 80);
            g.fillRect (400, 75, 80, 80);
            g.fillRect (240, 200, 125, 200);
            }
            else
            {

Otherwise, you’re going to be painting as long as closeDoors is true. You just need to paint once. Java will ask you to paint again when it has to (for instance, when you call repaint()).

Also, set closeDoors as a member variable. In other words, have:

public class color extends JApplet
{
  public boolean closeDoors = false;

And when you switch the value of closeDoors in the click listener, you can simplify it as:

       int x = e.getX();
       int y = e.getY();

       if(x>330 && x<280 && y>20 && y<20)
       {
           closeDoors = !closeDoors;
           repaint();

       }

That will, when you click in your specific area, invert the value of closeDoors. In other words, if closeDoors is true, it will be set to false, and vice versa.

Note, your code if(x>330 && x<280 && y>20 && y<20) probably won’t work at all, since y cannot be greater than 20 and less than 20 at the same time, ever. I’ll let you play with that to figure out what works :-).

Hope this helps.

---

Just a minor detail, but you should probably call your class Color instead of color to follow Java’s standard naming convention, or call it something else if you don’t want to clash with java.awt.Color.