2016-09-09 10 views
0

私はFlappyBirdsをJavaで作っています。しかし、私は問題を抱えています。KeyListenerが遅れています

私は整数変数 'status'を持っています。 status0の場合、ゲームは実行されます。status1の場合、ウィンドウは "game over, press s to begin"と表示されます。私のKeyListenerは、ゲームプレイに適しています。

しかし、私がゲームを失ってstatus1になると、sボタンをもう一度押すと、いつかボタンを押し続ける必要があります。すぐには反応しません。開始時にstatus1です。その間にsを押すと、すぐに復帰しますが、失うとステータスが再び1になると、問題が発生します。

package flappybirds; 

import java.awt.Color; 
import java.awt.Font; 
import java.awt.Graphics; 
import java.awt.Rectangle; 
import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 
import java.awt.event.KeyEvent; 
import java.awt.event.KeyListener; 
import java.util.ArrayList; 
import javax.swing.JFrame; 
import javax.swing.JPanel; 
import javax.swing.Timer; 
import java.util.Random; 
import java.util.logging.Level; 
import java.util.logging.Logger; 


public class FlappyBirds extends JPanel implements  ActionListener,KeyListener { 
    public int WIDTH=800,HEIGHT=800,x=WIDTH/2-10,y=HEIGHT/2-10 , gravity=0,  ticks=0,status=1; 
    public static FlappyBirds flappy; 
    public boolean jump; 
    public boolean gameOver=true ; 
    public ArrayList<Rectangle>walls; 
    public Rectangle bird; 

    public Random generator; 
    public FlappyBirds(){ 
     Timer t=new Timer(10,this); 
     JFrame window=new JFrame("Flappy Birds"); 
     window.setSize(WIDTH ,HEIGHT); 
     window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     window.add(this); 
     window.setVisible(true); 
     window.addKeyListener(this); 
     walls=new ArrayList<Rectangle>(); 
     generator=new Random(); 
     t.start(); 
    } 

    public static void main(String[] args) { 
     flappy=new FlappyBirds(); 
    } 

    @Override 
    public void paintComponent(Graphics g){ 
     super.paintComponent(g); 
     bird=new Rectangle(x,y,15,15); 
     g.setColor(new Color(0,200,255)); 
     g.fillRect(0,0,WIDTH,HEIGHT); 
     g.setColor(Color.ORANGE); 
     g.fillRect(0, HEIGHT-100, WIDTH,100); 
     g.setColor(Color.GREEN); 
     g.fillRect(0,HEIGHT-110,WIDTH,10); 
     g.setColor(Color.red); 
     g.fillRect(bird.x,bird.y,bird.width,bird.height); 
     g.drawString(Integer.toString(status),200,200); 
     g.setColor(new Color(200,200,40)); 

     if(status==0){ 
      for(Rectangle rect:walls){ 
       paintWall(rect,g); 
      } 
     } 

     if(status==1){ 
      g.setFont(new Font("Arial",50,50)); 
      g.setColor(Color.RED); 
      g.drawString("Press 'S' to begin",400,400); 
     } 

     g.setColor(Color.RED); 
    } 

    @Override 
    public void actionPerformed(ActionEvent e) { 

     repaint(); 
     if(status==0){ 
      addWall(true); 
      addWall(true); 

      ticks ++; 
      if(jump){ 
       if(ticks%8==0){ 
        gravity=-10; 
       } 

      } 
      else if(ticks%8==0 && gravity<15 && jump==false){ 
       gravity+=2; 

      } 
      if(ticks==1000){ 
       ticks=0; 
      } 
      y += gravity; 

      for(Rectangle rect:walls){ 
       rect.x -=10; 
      } 

      for(int i=0;i<walls.size();i++){ 
       Rectangle r=walls.get(i); 

       if(r.x+r.width<0){ 

        addWall(false); 
       } 
      } 


      for(Rectangle column:walls){ 
       if(column.intersects(bird) || bird.y>=HEIGHT-120 ||  bird.y<=3){ 
        gameOver=true; 
        status=1; 
        if(!jump){ 
         if(y>=HEIGHT-120){ 
          y=HEIGHT-120; 
         } 
         if(column.intersects(bird)){ 
          y=HEIGHT/2-10; 
         } 
         if(bird.y<=3){ 
          y=HEIGHT/2-10; 
         } 
        }else if(jump){ 
         y=HEIGHT/2-10; 
        } 
       }else{ 
        gameOver=false; 
       } 
      } 
      collide(); 

     } 
    } 


    @Override 
    public void keyTyped(KeyEvent e) {} 

    @Override 
    public void keyPressed(KeyEvent e) { 
     int c=e.getKeyCode(); 
     if(status==0){ 
      if(c==KeyEvent.VK_SPACE){ 
       jump=true; 
      } 
     } 
     if(status==1){ 
      if(c==KeyEvent.VK_S){ 

       status=0; 
      } 
     } 
    } 
    public void collide(){ 
     if(status==0){ 
      if(gameOver){ 
       walls.clear(); 
       addWall(true); 
       addWall(true); 
       gameOver=false; 

      } 
     } 
    } 

    @Override 
    public void keyReleased(KeyEvent e) { 
     jump=false; 
    } 
    public void paintWall(Rectangle rect,Graphics g){ 
     g.fillRect(rect.x, rect.y, rect.width, rect.height); 
    } 
    public void paintSpace(Rectangle rect,Graphics g){ 
     g.setColor(new Color(0,200,225)); 
     g.fillRect(rect.x,rect.y,rect.width,rect.height); 
    } 
    public void addWall(boolean oldwall){ 
     int width=100; 
     final int height=350; 
     int yloc=100+generator.nextInt(150); 
     int space=100; 
     if(oldwall){ 
      if(generator.nextBoolean()){ 
       walls.add(new Rectangle(WIDTH+width+ (walls.size()*200),HEIGHT- height-yloc,width,height)); 
       walls.add(new Rectangle(WIDTH+width+(walls.size()-1)*200,- yloc, width,height- space)); 
      }else{ 
       walls.add(new Rectangle(WIDTH+width+ (walls.size()*200),HEIGHT-height+yloc,width,height)); 
       walls.add(new Rectangle(WIDTH+width+ (walls.size()-1)*200,+yloc, width,height- space)); 
      } 
     }else{ 
      if(generator.nextBoolean()){ 
       walls.add(new  Rectangle(walls.get(walls.size()-1).x+600,HEIGHT-height- yloc,width,height+space)); 
       walls.add(new Rectangle(walls.get(walls.size()-2).x+600,- yloc,width,height-space)); 
      }else{ 
       walls.add(new  Rectangle(walls.get(walls.size()-1).x+600,HEIGHT- height+yloc,width,height+space)); 
       walls.add(new  Rectangle(walls.get(walls.size()-2).x+600,yloc,width,height-space)); 
      } 
     } 
    } 
} 
+0

あなたの質問に答えるのに十分な情報はありません。あなたが示したコード部分には何も問題はありません。つまり、問題は他のどこかから来ています。たぶんあなたのゲームループが2回目にあまりにも多くのメモリを占有したり、ゲーム自体に切り替わるコードが何らかの遅れやキーリスナーが削除されてしばらくしてから再割り当てされた場合には、ランダムに可能性はありますが、情報がほとんどないため、私はそれが役に立たないと思います。 – Mark

+0

ソースコード全体を投稿できますか? –

+0

これは読めるコードがたくさんあり、多くの人がその時間を見つけるわけではありません:Dあなたはちょっとデバッグを試みて、おおよそあなたのコードの中に問題があり、その部分だけを投稿してください。単にIDEのデバッガを使用するか、テスト出力を入れてください(後でもう一度取り出してください)。私はおそらく、keyPressedメソッドが遅すぎるか、ステータスが0になると実行される必要があるコードを見て開始するでしょう。 – Mark

答えて

0

間違いなく遅延の原因はタイマーです:

タイマーがイベント10ms毎(すなわちactionPerformedを呼び出す)を立ち上げ、継続的に実行されています。 ゲームが終了してもが実行されているため、ActionEventイベントがイベントキューに蓄積され、あなたが押した 'S'はその処理が完了するまで大きな遅延があります。

私はあなたをお勧めします。

  1. とすぐにゲームオーバーですと、タイマーを停止します。
  2. ゲームが再開されるとすぐに、すべてのインスタンス変数を初期状態に再初期化します。たとえば、initというプライベートメソッドを実装し、x,ygravitytickswalls、そしてもちろんtimer.start()の呼び出しを初期化します。
  3. コンストラクタの末尾にinitを、の後にkeyPressedを呼び出します。
+0

コンストラクタでタイマーを停止する必要がありますか?私はそうするとき、それは働いていないので、私はこれを求めています。 –

+0

いいえ。タイマーはコンストラクタで開始されなければならず、 'gameOver = true'を指定すると停止します。 –

関連する問題