Question

我正试图为我的Java编程简介编写交通信号灯模拟程序。当我运行小程序时，即使我还没有点击任何JButton，所有三个指示灯仍然亮起。当我这样做的时候，applet确实会在它做某事时暂时变成空白，如果我按下GO，那么这辆车确实会移动。我想知道我是否需要重置图形类中的颜色，或者使用switch语句（我不确定该怎么做）并开始使用我在其他地方看到的颜色例子。我的代码中有什么不正确的东西会妨碍我获得我想要的结果吗？任何帮助将不胜感激。谢谢。

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

/**
 * Class TrafficLights - At the click of a button, change the traffic lights 
 * 
 * @author Mickey Mouse
 * @version JDK 8
 * @course CSCI 1130-01
 * @date 10-2-15
 */
public class TrafficLights extends JApplet implements ActionListener {
    private JButton WAIT, STOP, GO;

    private boolean clickWAIT = false;
    private boolean clickSTOP = false;
    private boolean clickGO = false;

    private int carX = 200;

    /**
     * Called by the browser or applet viewer to inform this JApplet that it
     * has been loaded into the system. It is always called before the first 
     * time that the start method is called.
     */
    public void init()
    {
        setLayout( new FlowLayout()); // changes the layout from BorderLayout to FlowLayout

        WAIT = new JButton ("WAIT"); //adds WAIT label to button
        WAIT.setForeground(Color.yellow); //changes the label to yellow 

        //adds the WAIT JButton to the screen
        add (WAIT); 
        WAIT.addActionListener(this);


        GO = new JButton ("GO"); //adds GO label to button 
        GO.setForeground(Color.green); //changes the label to green

        //adds the button to the screen
        add (GO);
        GO.addActionListener(this);


        STOP = new JButton ("STOP"); //adds STOP label to button 
        STOP.setForeground(Color.red); //changes the label to red 

        //adds STOP JButton to screen 
        add (STOP);
        STOP.addActionListener(this); 


    }



    /**
     * Called by the browser or applet viewer to inform this JApplet that it 
     * should start its execution. It is called after the init method and 
     * each time the JApplet is revisited in a Web page. 
     */
    public void start()
    {
        // provide any code requred to run each time 
        // web page is visited
    }

    /** 
     * Called by the browser or applet viewer to inform this JApplet that
     * it should stop its execution. It is called when the Web page that
     * contains this JApplet has been replaced by another page, and also
     * just before the JApplet is to be destroyed. 
     */
    public void stop()
    {
        // provide any code that needs to be run when page
        // is replaced by another page or before JApplet is destroyed 
    }

    /**
     * Paint method for applet.
     * 
     * @param  g   the Graphics object for this applet
     */
    public void paint(Graphics g)
    {
        super.paint(g);

        //declares and retrieves the images from their file locations 
        Image img = getImage(getDocumentBase(), "stoplights.png");
        Image img2 = getImage(getDocumentBase(), "car.jpg");

        g.drawImage( img, 50, 100, 300, 350, 0, 0, 5000, 5000, this ); //draws and resizes the stoplights image 
        g.drawImage( img2, carX, 400, 1000, 1000, 0, 0, 5000, 5000, this); //draws and resizes the car image 

        //draw and fill an oval red for the STOP stoplight when STOP is pressed 
        if (clickSTOP == true);
        {
        g.drawOval(63, 112, 30, 30); 
        g.setColor(Color.red);
        g.fillOval(63, 112, 30, 30);
        clickSTOP = false;
        }


        //draw and fill an oval yellow for the WAIT stoplight when WAIT is pressed
        if (clickWAIT == true); 
        {
        g.setColor(Color.black);
        g.drawOval(63, 148, 30, 30);
        g.setColor(Color.orange);
        g.fillOval(63, 148, 30, 30); 
        clickWAIT = false;
        }

        //draw and fill an oval green for the GO stoplight when GO is pressed 
        if (clickGO == true);
        {
        g.setColor(Color.black);
        g.drawOval(63, 184, 30, 30);
        g.setColor(Color.green);
        g.fillOval(63, 184, 30, 30); 
        clickGO = false;
        }



    }


     public void actionPerformed(ActionEvent event)
    {
       /*
        * Links the JButtons and the graphic sequences to display the lights
        * 
        */
       if(event.getSource() == GO) //display green if GO is clicked 
       {
           clickGO = true;
           carX -=15; 
           repaint();
           }
           if (event.getSource() == WAIT) //display yellow if WAIT is clicked
           {
               clickWAIT = true;
               repaint();
               }
               if (event.getSource() == STOP) //display red if STOP is clicked
               {
                   clickSTOP = true;
                   repaint();
                   }

    } 


    /**

Answer 1

当我运行小程序时，即使我还没有点击任何JButton，所有三个指示灯仍然亮起

这是因为在每个;语句之后你有一个if，这基本上是你的逻辑短路，所以跟随它的代码总是被执行，而不管变量的状态如何...

if (clickSTOP == true);

所以，相反，你应该使用更像

的东西

if (clickSTOP == true)
{
    g.setColor(Color.red);
    g.fillOval(63, 112, 30, 30);
    clickSTOP = false;
}

您也在使用paint方法修改程序的状态，这通常是不鼓励的，因为paint应该这样做，绘制当前状态。

改变状态的最佳位置是ActionListener。

而不是有三个状态变量，通常描述同一个事物，你应该有一个状态变量，所以状态只能是go或wait或stop ，绝不是三者的组合。

同样，您可以使用ButtonGroup和JToggleButton或JRadioButton s，这将允许按钮携带一定数量的状态信息，因为只有按钮可以一次选择。

有关详细信息，请查看How to Use the ButtonGroup Component和How to Use Buttons, Check Boxes, and Radio Buttons

当我这样做的时候，applet确实会变得空白，就像它做某事
一样

这可能是因为您正在paint方法中加载图片，但作为一般规则，您应该避免覆盖paint这样的顶级容器。

使用If或Switch语句更改交通灯

1 个答案: