2

私はこれに対する答えをかなり探しましたが、いくつか編集しましたが、うまくいきません。私のブール値メソッド getColor() に何か問題があると思います。別のクラスのループで使用しようとしています。

AvatarPanel クラスの getColor メソッドを取得します

    public boolean setColor(boolean good)
{

    if (good == true)
        what--;
        avatars.get(0).setBackground(colors.get(what));
    repaint();
    if (good == false)  
        what++;
        avatars.get(0).setBackground(colors.get(what));
    repaint();

    return good;
}

getColor を使用しようとしている Background クラスの attackPerformed メソッドを次に示します。メソッドの 1 つを間違って記述したのか、attackPerformed にリスナーが必要なのかはわかりません。ゲームはまだ正常に実行されており、正常に動作しています。これらが少なくとも正しく書かれていることを確認するための助けが欲しかった.

バックグラウンド コンストラクターを表示するのは良い考えかもしれないと判断しました。バックグラウンドコンストラクターにあると確信していますが、attackPerformed を呼び出す必要がある場所がどこにあるのかわかりません。attackPerformed メソッドを以下に示します。コンストラクターが一番上にあります。

public class Background extends JPanel implements Constants, ActionListener
{
   private static final long serialVersionUID = 1L;

   private final Color BACK_COLOR = Color.GRAY;

   private ArrayList<Sprite> sprites;

   private Avatar avatar;

   /** Constructor  */
   public Background()
   {

       sprites = new ArrayList<Sprite>();
       reset(0);
       setBackground(BACK_COLOR);


       KeyboardFocusManager manager =KeyboardFocusManager.getCurrentKeyboardFocusManager();
       manager.addKeyEventDispatcher(new KeyEventDispatcher()
       {
        public boolean dispatchKeyEvent(KeyEvent event)
        {
            int x = 0, y = 0;
            double angle = 0;
            Point p = avatar.getPosition();

            switch (event.getKeyCode())
            {
            case KeyEvent.VK_DOWN:
                y=6;
                angle=270;
                break;

            case KeyEvent.VK_UP:
                y = -6;
                angle = 90;
                break;

            case KeyEvent.VK_LEFT:
                x = -6;
                break;

            case KeyEvent.VK_RIGHT:
                x = 6;
                angle = 180;
                break;
            }

            avatar.setParameters();
            p.x += x;
            p.y += y;
            avatar.setPosition(p);
            avatar.setAngle(angle);

            if (!getBounds().contains(avatar.getBounds()))
            {
                avatar.restore();
                Toolkit.getDefaultToolkit().beep();
                return true;
            }
            repaint();
            return true;            
        }



    });
    Timer timer = new Timer(15, this);
    timer.start();



} //end of constructor  



public void attackPerformed()
{
    for(Sprite s : sprites)
    {
        AvatarPanel panel = new AvatarPanel();
        Rectangle r = new Rectangle(sprites.get(0).getBounds());
        Rectangle p = new Rectangle(sprites.get(1).getBounds());

            if(r.intersects(p))
            {
                panel.setColor(false);
            }               
    }   
}


public void actionPerformed(ActionEvent e)
{
    for (Sprite s : sprites)
    {
        if (s instanceof Move)
        {
            s.setParameters();
            for (int i=0; i<3; i++)
            {
                ((Move)s).nextPosition();
                if (!getBounds().contains(s.getBounds()))
                {
                    ((Move) s).moveFailed();
                }
                else break;
            }
        }
    }
    repaint();
}




public @Override void paintComponent(Graphics g)
{
   super.paintComponent(g); /* Let the parent class do its painting */

   for (Sprite sprite: sprites)
   {
       sprite.draw(g);
   }
 }

public void reset(int level)
{
    Sprite sprite = null;

    int x, y, angle;
    double scale, min, max;

    sprites.clear();
    int[] figures = FIGURES[level];
    for (int f=0; f<figures.length; f++)
    {
        for (int i=0; i<figures[f]; i++)
        {
            x = (int)(Math.random()*600 + 100);
            y = (int)(Math.random()*400 + 100);

            switch (f)
            {
                case 0:
                    sprite = avatar = new Avatar(x,y);
                    break;
                case 1:

                    sprite = new Predator(x,y);
                    break;
                case 2:
                    sprite = new Mine(x,y);
                    break;
                case 3:
                    sprite = new Shield(x,y);
                    break;
            }

            angle = (int)(Math.random()*360);
            sprite.setAngle(angle);

            min = sprite.getMinScaleFactor();
            max = sprite.getMaxScaleFactor();
            scale = Math.random() * (max - min) + min;
            sprite.setScale(scale);

            sprites.add(sprite);
            repaint();
        }
    }
}

}

4

2 に答える 2

2

ifとelseのケースは非常に似ているように見えます。これに単純化することができます。

public boolean setColor(boolean good)
{
    what+=good?-1:+1;
    avatars.get(0).setBackground(colors.get(what));
    repaint();
    return good;
}
于 2013-03-16T19:20:27.480 に答える
1

repaint（）メソッドはifステートメントの外で実行されるため、角かっこを使用することを強くお勧めします。ブール値は2つの可能な値しか持てないため、次の場合は2番目の代わりに「else」を使用できます。

public boolean setColor(boolean good)
{
    if (good == true)
    {
        what--;
        avatars.get(0).setBackground(colors.get(what));
        repaint();
    }
    else 
    {    
        what++;
        avatars.get(0).setBackground(colors.get(what));
        repaint();
    }
于 2013-03-16T19:15:40.387 に答える