0

これは 12 年生の OPP 課題です。

現在、ボールが時々反射せずにブロックに入ってしまうという問題があります。

これが私のコードです：

メインがボールがブロックに当たったかどうかをチェックする方法:

public void theBall(){
    //System.out.println(blocks.size());
    if(!started){
        balls.get(0).withPaddle(paddle.returnX());
    }
    if(started){
        for(int i=0;i<balls.size();i++){
            if(balls.get(i).returnX()>=0 && balls.get(i).returnX()<=800 && balls.get(i).returnY()>=0){
                balls.get(i).move();
            }
            if(balls.get(i).returnX()+balls.get(i).returnDx()<= 0){
                balls.get(i).collide(1);//collide with left wall
            }
            if(balls.get(i).returnX()+10+balls.get(i).returnDx() >= 800){
                balls.get(i).collide(2);//collide with right wall
            }
            if(balls.get(i).returnY()-balls.get(i).returnDy() <= 0){
                balls.get(i).collide(3);//collide with up wall
            }

            if(paddle.returnX() < balls.get(i).returnX() && balls.get(i).returnX()< paddle.returnX()+paddle.returnSize() && balls.get(i).returnY()-balls.get(i).returnDy() >= 545){//fix this && balls.get(i).returnY()-balls.get(i).returnDy() <= 560
                balls.get(i).collidePaddle(balls.get(i).returnX()-paddle.returnX());//collide with paddle
                //System.out.println(ball.returnDy());
            }
            for(int j=0;j<blocks.size();j++){
                //blocks.get(j).getHit(balls.get(i));
                if(balls.get(i).returnX()+10+balls.get(i).returnDx() >= blocks.get(j).returnX() && balls.get(i).returnX()+10+balls.get(i).returnDx() <= blocks.get(j).returnX()+5 && balls.get(i).returnY() >= blocks.get(j).returnY() && balls.get(i).returnY() < blocks.get(j).returnY()+50){
                    if(!blocks.get(j).returnHit()){
                        balls.get(i).collide(2);//collide with right wall (block's left)
                        blocks.get(j).getHit(balls.get(i));
                        System.out.println("boiboiboiboibo1");
                    }
                }

                if(balls.get(i).returnX()+balls.get(i).returnDx() <= blocks.get(j).returnX()+50 && balls.get(i).returnX()+balls.get(i).returnDx() >= blocks.get(j).returnX()+45 && balls.get(i).returnY() >= blocks.get(j).returnY() && balls.get(i).returnY() < blocks.get(j).returnY()+50){
                    if(!blocks.get(j).returnHit()){
                        balls.get(i).collide(1);//collide with left wall (block's right)
                        blocks.get(j).getHit(balls.get(i));
                        System.out.println("boiboiboiboibo2");
                    }
                }

                if(balls.get(i).returnX() >= blocks.get(j).returnX() && balls.get(i).returnX() < blocks.get(j).returnX()+50 && balls.get(i).returnY()-balls.get(i).returnDy() <= blocks.get(j).returnY()+50 && balls.get(i).returnY()-balls.get(i).returnDy() >= blocks.get(j).returnY()+45){
                    if(!blocks.get(j).returnHit()){
                        balls.get(i).collide(3);//collide with up wall (block's bottom)
                        blocks.get(j).getHit(balls.get(i));
                        System.out.println("boiboiboiboibo3");
                    }
                }
                if(balls.get(i).returnX() >= blocks.get(j).returnX() && balls.get(i).returnX() < blocks.get(j).returnX()+50 && balls.get(i).returnY()+10-balls.get(i).returnDy() >= blocks.get(j).returnY() && balls.get(i).returnY()+10-balls.get(i).returnDy() <= blocks.get(j).returnY()+5){
                    if(!blocks.get(j).returnHit()){
                        balls.get(i).collide(4);//collide with down wall (block's top)
                        blocks.get(j).getHit(balls.get(i));
                        System.out.println("boiboiboiboibo4");
                    }
                }
            }
            for(int j=0;j<specialBlocks.size();j++){
                if(balls.get(i).returnX()>=specialBlocks.get(j).returnX() && balls.get(i).returnX()<=specialBlocks.get(j).returnX()+100 && balls.get(i).returnY()>=specialBlocks.get(j).returnY() && balls.get(i).returnY()<=specialBlocks.get(j).returnY()+100){
                    oneAndThreeBalls(i, j);
                    balls.get(i).changeType(j);
                }
            }
        }
    }
}

ボールがどのように反射するか:

public void collide(int collideWith){
    if(collideWith==1){//collide with left wall
        if(90<direction && direction<180){
            direction = 180-direction;
        }
        if(180<direction && direction<270){
            direction = 540-direction;
        }
    }
    if(collideWith==2){//collide with right wall
        if(0<direction && direction<90){
            direction = 180-direction;
        }
        if(270<direction && direction<360){
            direction = 540-direction;
        }
    }
    if(collideWith==3){//collide with up wall
        direction = 360-direction;
    }
    if(collideWith==4){//collide with down wall
        direction = 360-direction;

    }
}

バグが境界の二重チェックによるものであることは明らかです。ボールがブロックの角にある場合に発生します。しかし、それがボールの位置が反映されているかどうかを確認するために私が考えることができる唯一の方法です。このバグを修正する方法についてのアイデアはありますか? ありがとうございました。

4

1 に答える 1

1

なぜそうしないのですか：

1) バウンスを行った後、ゲームの次のティックまで、そのボールの衝突チェックを停止します。そうすれば、ボールはティックごとに 1 回だけ跳ね返ることができます。

2) ブロックと衝突して跳ね返った後、ボールがブロックに重ならなくなるまで、その特定のブロックとボールの間の衝突を無視します。そうすれば、同じ衝撃でボールが誤ってブロックから 2 回跳ね返ることはありません。

ところで、数字などをわずかに変更しただけで、コピーして貼り付けたコードがたくさんあります。わずかに異なるパラメーターで呼び出し続けるメソッドの観点から言い換える方法を見つけた場合、コードはよりきれいになり、問題の根底にある対称性もより明白になります。

于 2013-02-25T04:03:39.163 に答える