0

ユーザーがクリックしたボタンに基づいてコーヒーカップの画像を画面に配置するプログラムがありますが、画像が常に表示されるとは限らず、表示されても動きません。誰かが私のコードが間違っている場所を見つけるのを手伝ってくれますか? ありがとうございました！

import java.awt.*;
import java.awt.event.*;
import java.applet.*;

public class MoveIt extends Applet implements ActionListener

{
    //Declare Variables
    private Image cup;
    private Panel keyPad;
    public int top = 15;
    public int left = 15;
    private boolean foundKey;

    public void init()
    {
        cup = getImage(getDocumentBase(), "cup.gif");

        Canvas myCanvas = new Canvas();

        keyPad = new Panel();

        keysArray = new Button[5];


        //Create Buttons
        keysArray[0] = new Button("Up");
        keysArray[1] = new Button("Left");
        keysArray[2] = new Button("Right");
        keysArray[3] = new Button("Down");
        keysArray[4] = new Button("Center");

        keysArray[0].setActionCommand("Up");
        keysArray[1].setActionCommand("Left");
        keysArray[2].setActionCommand("Right");
        keysArray[3].setActionCommand("Down");
        keysArray[4].setActionCommand("Center");

        setBackground(Color.blue);

        //Sets the layout manager for the frame
        setLayout(new BorderLayout());

        //Sets the layout manager for the Panel
        keyPad.setLayout(new BorderLayout());

        //Add Buttons to keypad panel
        keyPad.add(keysArray[0], BorderLayout.NORTH);
        keyPad.add(keysArray[1], BorderLayout.WEST);
        keyPad.add(keysArray[2], BorderLayout.EAST);
        keyPad.add(keysArray[3], BorderLayout.SOUTH);
        keyPad.add(keysArray[4], BorderLayout.CENTER);

        add(myCanvas, BorderLayout.NORTH);
        add(keyPad, BorderLayout.SOUTH);

        //adds actionlistener to the buttons
        for(int i = 0; i < keysArray.length; i++)
           keysArray[i].addActionListener(this);
    }

    //puts image on the canvas
    public void paint( Graphics g )
    {
       g.drawImage( cup, left, top, this );
    }

    public void actionPerformed(ActionEvent e)
    {
        String arg = e.getActionCommand();


        if(arg == "Up") top = top - 15;
        if(arg == "Left") left = left - 15;
        if(arg == "Right") left = left + 15;
        if(arg == "Down") top = top + 15;
        if(arg == "Center")
        {
            top = 60;
            left = 125;
        }
        repaint();
    }
}
4

2 に答える 2

3

これには非常に多くの問題がありますが、根本的な原因から始めましょう....

これは、Java での比較のしくみでif (arg == "Up") {はありません。String代わりに、"Up".equals(arg)またはケースを気にしない場合は使用する必要があります"Up".equalsIgnoreCase(arg)

また、ユーザーが一度に複数のボタンを押すことができる可能性は非常に低いため、代わりに

if ("Up".equals(arg)) {...}
if ("Left".equals(arg)) {...}

if-elseステートメントを使用する必要があります

if ("Up".equals(arg)) {
    //...
} else if ("Left".equals(arg)) {
    //..
} ...

更新しました

私が推奨しようとしているもののいくつかはレンドンによって議論され始めたので、彼に+1.

  • (個人的には) AWT を避け、代わりに Swing を使用してください。AWT は 13 年以上前に Swing に置き換えられました
  • 最上位のコンテナに直接ペイントすることは避けてください。通常、これらのコンテナーの上には多くのレイヤーが配置されていますが、これらのコンテナーに直接ペイントすることで台無しにすることができます。代わりに、コンテナーを作成します ( のように、そのJPanel上にアプリケーションをビルドします)。これにより、将来的に移植性と再利用性が向上します
  • super.paintXxx非常に正当な理由がなく、これらのメソッドが行うすべての作業を処理する準備ができていない限り、呼び出す必要があります。
  • 「マジック」ナンバーは避けましょう。代わりに、絶対値に依存してください。この場合、ペイント先のコンテナの幅と高さ、および画像のサイズです。これにより、「中心」が実際に「中心」になることが保証されます

次の例は、これらのアイデアのいくつかを示しています。基本的にControlPane、ボタンとCupPane. 自己完結型で、カップのCupPaneペイントと配置を担当します

ここに画像の説明を入力

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JApplet;
import javax.swing.JButton;
import javax.swing.JPanel;

public class MoveIt extends JApplet {

    public void init() {
        setLayout(new BorderLayout());
        add(new ControlPane());
    }

    public class CupPane extends JPanel {

        private Image cup;
        public int top = 15;
        public int left = 15;

        public CupPane() {
            cup = getImage(getDocumentBase(), "cup.png");
            setBackground(Color.BLUE);
        }

        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            g.drawImage(cup, left, top, this);
        }

        public void moveUp() {
            top -= 15;
            repaint();
        }

        public void moveLeft() {
            left -= 15;
            repaint();
        }

        public void moveRight() {
            left += 15;
            repaint();
        }

        public void moveDown() {
            top += 15;
            repaint();
        }

        public void center() {
            int width = getWidth();
            int height = getHeight();
            top = (height - cup.getHeight(this)) / 2;
            left = (width - cup.getWidth(this)) / 2;
        }
    }

    public class ControlPane extends JPanel {

        private JPanel keyPad;
        public int top = 15;
        public int left = 15;
        private boolean foundKey;
        private JButton[] keysArray;
        private CupPane cupPane;

        public ControlPane() {
            keyPad = new JPanel();
            keysArray = new JButton[5];
            cupPane = new CupPane();

            //Create Buttons
            keysArray[0] = new JButton("Up");
            keysArray[1] = new JButton("Left");
            keysArray[2] = new JButton("Right");
            keysArray[3] = new JButton("Down");
            keysArray[4] = new JButton("Center");

            keysArray[0].setActionCommand("Up");
            keysArray[1].setActionCommand("Left");
            keysArray[2].setActionCommand("Right");
            keysArray[3].setActionCommand("Down");
            keysArray[4].setActionCommand("Center");

            setBackground(Color.blue);

            //Sets the layout manager for the frame
            setLayout(new BorderLayout());

            //Sets the layout manager for the Panel
            keyPad.setLayout(new BorderLayout());

            //Add Buttons to keypad panel
            keyPad.add(keysArray[0], BorderLayout.NORTH);
            keyPad.add(keysArray[1], BorderLayout.WEST);
            keyPad.add(keysArray[2], BorderLayout.EAST);
            keyPad.add(keysArray[3], BorderLayout.SOUTH);
            keyPad.add(keysArray[4], BorderLayout.CENTER);

            add(cupPane, BorderLayout.CENTER);
            add(keyPad, BorderLayout.SOUTH);

            ActionHandler handler = new ActionHandler();
            //adds actionlistener to the buttons
            for (int i = 0; i < keysArray.length; i++) {
                keysArray[i].addActionListener(handler);
            }

        }

        protected class ActionHandler implements ActionListener {

            @Override
            public void actionPerformed(ActionEvent e) {
                String arg = e.getActionCommand();

                if ("Up".equals(arg)) {
                    cupPane.moveUp();
                } else if ("Left".equals(arg)) {
                    cupPane.moveLeft();
                } else if ("Right".equals(arg)) {
                    cupPane.moveRight();
                } else if ("Down".equals(arg)) {
                    cupPane.moveDown();
                } else if ("Center".equals(arg)) {
                    cupPane.center();
                }
                repaint();
            }
        }
    }
}

詳細については、Performing Custom PaintingおよびPaint in AWT and Swingを参照してください。

于 2013-07-23T23:45:17.573 に答える
2

私が理解している場合は、次のようなものがあります。

______________________
|         ___        | 
|        |img|       | 
|                    | 
|____________________| 
|  <-     ^    ->    | 
|_________v__________|

アプローチを再編成することを提案します。

  1. 描画専用のクラスを作成します (例: DrawingArea)。
  2. moveLeft()、moveRight()、moveUp()などのアクションごとにメソッドを作成します。

  3. 次に、メソッドでactionPerformed().

    public void actionPerformed(ActionEvent e)
{
  Button source = (Button)e.getSource();

  if(source == buttonUp)         // keysArray[0] in your case
    drawingArea.moveUp();
  else if(source == buttonLeft)  // keysArrays[1]
    drawingArea.moveLeft();
  else if(source == buttonRight) // keysArrays[2]
    drawingArea.moveRight();
  // etc.

    }

  4. DrawingAreaでオブジェクトを作成しMoveItて追加します。

    DrawingArea drawingArea = new DrawingArea();
add(drawingArea, BorderLayout.CENTER);
于 2013-07-23T23:41:53.930 に答える