なぜか一回のクリックでイベントが複数回起こる。

Question

コードは長いですが、簡単なゲームをつくろうとしています。このコードで実行⇒スタート(ボタン）⇒右上の画像ボタン（最初は別の画像で、押すと消える）を押すと元のウィンドウは消え、別ウィンドウに画像が表示されるイベント処理があります。１回目は成功するのですが、なぜか２回目からはこの動作が複数回起き（２回程度）、続けていると１回のクリックに対する動作の回数が増えていき、最終的にはウィンドウの嵐になります。対処法を教えてください。
コードは長いですが、バグに関係あるのはタイマー処理に関係する部分と、CATCH１～３（）のところだと思ってます。

package sample;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;
import java.util.Timer;
import java.util.TimerTask;

import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

class TASK extends TimerTask{

// タイマー処理のメソッドを誘発させる
public void run(){
	Fishing Act = new Fishing();     // これができなければ詰んでた
	Act.ROLL();
}

}

class TASK2 extends TimerTask{

// タイマー処理のメソッドを誘発させる
public void run(){
	
	Fishing Act = new Fishing();     
	Act.ROLL2();
}

}

public class Fishing extends JFrame implements ActionListener
{
private static final long serialVersionUID = 1L;

//クラス変数定義
public static int flag=0;
static int gamescore =0;
static int alltime =0; //60sカウントに使う。
static int[] fish_score1={0,50,50,100,100};
static int[] fish_score2={150,150,200,200,250};
static int[] fish_score3={300,300,400,400,500};
static int[] fish_scoreQ={-50,-50,-100,0,1000};
static ImageIcon[] fish1={new ImageIcon("N0.jpg"),new ImageIcon("N1.jpg"),new ImageIcon("N2.jpg"),new ImageIcon("N3.jpg"),new ImageIcon("N4.jpg"),};
static ImageIcon[] fish2={new ImageIcon("R0.jpg"),new ImageIcon("R1.jpg"),new ImageIcon("R2.jpg"),new ImageIcon("R3.jpg"),new ImageIcon("R4.jpg"),};
static ImageIcon[] fish3={new ImageIcon("SR0.jpg"),new ImageIcon("SR1.jpg"),new ImageIcon("SR2.jpg"),new ImageIcon("SR3.jpg"),new ImageIcon("SR4.jpg"),};
static ImageIcon[] fishQ={new ImageIcon("SC0.jpg"),new ImageIcon("SC1.jpg"),new ImageIcon("SC2.jpg"),new ImageIcon("SC3.jpg"),new ImageIcon("SC4.jpg"),};
static JButton[] Bbt={new JButton(new ImageIcon("しーん.jpg")),new JButton(new ImageIcon("！.jpg")),
new JButton(new ImageIcon("！！.jpg")),new JButton(new ImageIcon("！！！.jpg")),new JButton(new ImageIcon("？.jpg"))};
ImageIcon Titlegazo = new ImageIcon("タイトル画像.jpg");
ImageIcon RULE1 = new ImageIcon("ルール１.jpg"); ImageIcon RULE2 = new ImageIcon("ルール2.jpg");
ImageIcon NINGEN = new ImageIcon("釣りbasic2.jpg"); ImageIcon SEA = new ImageIcon("海.jpg");
static JButton CHANGEBt=Bbt[0];
JButton bt1 = new JButton("スタート"); JButton bt2 = new JButton("あそびかた");
JButton bt3 = new JButton("つぎへ"); JButton bt4 = new JButton("タイトルへ");
JButton bt5 =new JButton("もどる");
JPanel p4 = new JPanel(); Timer timer2 = new Timer(true); Timer timer1 = new Timer(true);

// タイマー処理（本命）
public void ROLL(){
timer2.cancel();
int x= Atrandom();
//p4.revalidate();
//p4.remove(CHANGEBt); p4.add(Bbt[x]);
CHANGEBt.setBounds(0,0,0,0);
CHANGEBt=Bbt[x];
CHANGEBt.setBounds(378,120,375,280);
p4.revalidate();
p4.repaint();
switch(x){
case 1:
{
timer1.schedule(new TASK2(),600); break;
}
case 2:
{
timer1.schedule(new TASK2(),400); break;
}

}
	               

}

//もう一つのタイマー処理
public void ROLL2(){
//if(flag%2==0){
timer1.cancel();
CHANGEBt.setBounds(0,0,0,0);
CHANGEBt=Bbt[0];
CHANGEBt.setBounds(378,120,375,280);
p4.revalidate();
p4.repaint();
// System.exit(0);
//Fishing Act = new Fishing(); Act.MAINGAME();
//}
}

//1～4までの非等倍乱数を得る。
public int Atrandom(){
int Ra[]={1,1,1,1,1,1,1,1,2,2,2,2,1,3,3,2,1,2,1,2}; Random r=new Random(); //4,4,4,4,4
int A=r.nextInt(20); int B = Ra[A];
return B;
}

//タイトル画面
public void TITLE(){
setBounds(550, 100, 770, 800);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setTitle("魚釣りゲーム～タイトル～");
JLabel titleLabel = new JLabel(Titlegazo);
JPanel p1 = new JPanel();
p1.setLayout(null);
bt1.setBounds(310,580,130,30); bt2.setBounds(310,650,130,30);
bt1.addActionListener(this ); bt1.addActionListener(event -> dispose());
bt2.addActionListener(this ); bt2.addActionListener(event -> dispose());
p1.add(bt1); p1.add(bt2);
p1.add(titleLabel); titleLabel.setBounds(-10,0,770,760);
add("Center",p1);
setVisible(true);
}

//ルール説明1
public void Explain1(){
setBounds(550, 100, 770, 800);
setTitle("魚釣りゲームのルール説明");
JPanel p2 = new JPanel();
p2.setLayout(null);
bt3.setBounds(100,540,130,30);
bt3.addActionListener(this ); bt3.addActionListener(event -> dispose());
p2.add(bt3);
JLabel RURELabel1 = new JLabel(RULE1);
p2.add(RURELabel1); RURELabel1.setBounds(0,150,760,460);
add("Center",p2);
setVisible(true);
}

//ルール説明２
public void Explain2(){
setBounds(550, 100, 770, 800);
setTitle("魚釣りゲームのルール説明");
JPanel p3 = new JPanel();
p3.setLayout(null);
bt4.setBounds(50,520,180,30);
bt4.addActionListener(this ); bt4.addActionListener(event -> dispose());
p3.add(bt4);
JLabel RURELabel2 = new JLabel(RULE2);
p3.add(RURELabel2); RURELabel2.setBounds(0,150,760,460);
add("Center",p3);
setVisible(true);

}

// ゲーム画面
public void MAINGAME(){
setBounds(550, 100, 770, 800);
setTitle("魚釣りゲーム");
p4.setLayout(null);
JLabel SCORE_Label = new JLabel();
SCORE_Label.setText("SCORE: "+gamescore+"点");
p4.add(SCORE_Label); SCORE_Label.setBounds(100,0,400,400);
JLabel SEA_Label = new JLabel(SEA);
p4.add(SEA_Label); SEA_Label.setBounds(195,400,560,350);
JLabel NINGEN_Label = new JLabel(NINGEN);
p4.add(NINGEN_Label); NINGEN_Label.setBounds(0,180,380,380);
p4.add(CHANGEBt); CHANGEBt.setBounds(378,120,375,280);
CHANGEBt.addActionListener(this ); CHANGEBt.addActionListener(event -> dispose());
p4.add(Bbt[1]); p4.add(Bbt[2]); p4.add(Bbt[3]); p4.add(Bbt[4]); //サイズで表示非表示を表現するなら、ボタンは全部Addされていないといけない
Bbt[1].addActionListener(this ); Bbt[1].addActionListener(event -> dispose()); Bbt[3].addActionListener(this ); Bbt[3].addActionListener(event -> dispose());
Bbt[2].addActionListener(this ); Bbt[2].addActionListener(event -> dispose()); Bbt[4].addActionListener(this ); Bbt[4].addActionListener(event -> dispose());
add("Center",p4); //saigo
setVisible(true); //saigo
Random r = new Random(); int R;
R=100*(r.nextInt(50) +1); // 100<= R <= 5000
alltime = alltime + R; // 総合時間を加算

timer2.schedule(new TASK(),R);  //タイマー処理は最後にコンパイラされる

}

// !ボタンが押されたとき
public void CATCH1(){
//flag=flag+1;
setBounds(550, 100, 770, 800); setTitle("魚釣りゲーム");
JPanel p5 = new JPanel(); p5.setLayout(null);
Random r = new Random(); int B= r.nextInt(5);
bt5.setBounds(280,650,130,30);
bt5.addActionListener(this ); bt5.addActionListener(event -> dispose());
p5.add(bt5);
ImageIcon A = fish1[B]; gamescore= gamescore+fish_score1[B];
JLabel CATCH_Label = new JLabel(); CATCH_Label.setText("釣り上げた！");
p5.add(CATCH_Label); CATCH_Label.setBounds(305,120,100,100);
JLabel A_Label = new JLabel(A);
p5.add(A_Label); A_Label.setBounds(150,200,400,400);
add("Center",p5); setVisible(true);

}

//!!ボタンが押されたとき
public void CATCH2(){
flag=flag+1;
setBounds(550, 100, 770, 800); setTitle("魚釣りゲーム");
JPanel p6 = new JPanel(); p6.setLayout(null);
Random r = new Random(); int B= r.nextInt(5);
bt5.setBounds(280,650,130,30);
bt5.addActionListener(this ); bt5.addActionListener(event -> dispose());
p6.add(bt5);
ImageIcon A = fish2[B]; gamescore=gamescore+fish_score2[B];
JLabel CATCH_Label = new JLabel(); CATCH_Label.setText("釣り上げた！");
p6.add(CATCH_Label); CATCH_Label.setBounds(305,120,100,100);
JLabel A_Label = new JLabel(A);
p6.add(A_Label); A_Label.setBounds(150,200,400,400);
add("Center",p6); setVisible(true);

}

//!!!ボタンが押されたとき
public void CATCH3(){
flag=flag+1;
setBounds(550, 100, 770, 800); setTitle("魚釣りゲーム");
JPanel p7 = new JPanel(); p7.setLayout(null);
Random r = new Random(); int B= r.nextInt(5);
bt5.setBounds(280,650,130,30);
bt5.addActionListener(this ); bt5.addActionListener(event -> dispose());
p7.add(bt5);
ImageIcon A = fish3[B]; gamescore=gamescore+fish_score3[B];
JLabel CATCH_Label = new JLabel(); CATCH_Label.setText("釣り上げた！");
p7.add(CATCH_Label); CATCH_Label.setBounds(305,120,100,100);
JLabel A_Label = new JLabel(A);
p7.add(A_Label); A_Label.setBounds(150,200,400,400);
add("Center",p7); setVisible(true);

}

// クリックからメソッドを誘導する
public void actionPerformed(ActionEvent e) {
if(e.getSource()==bt2){
Fishing ive = new Fishing(); ive.Explain1();
}
else if(e.getSource()==bt3){
Fishing ive = new Fishing(); ive.Explain2();
}
else if(e.getSource()==bt4){
Fishing ive = new Fishing(); ive.TITLE();
}
else if(e.getSource()==bt1){
Fishing ive = new Fishing(); ive.MAINGAME(); // スタート→メイン画面へ遷移
}
else if(e.getSource()==Bbt[1]){
Fishing ive = new Fishing(); ive.CATCH1();
}
else if(e.getSource()==Bbt[2]){
Fishing ive = new Fishing(); ive.CATCH2();
}
else if(e.getSource()==Bbt[3]){
Fishing ive = new Fishing(); ive.CATCH3();
}
else if(e.getSource()==bt5){ // 釣り上げ→メイン画面へ遷移
if(alltime<60000)
{
//flag=flag+1;
CHANGEBt=Bbt[0];
Fishing ive = new Fishing(); ive.MAINGAME();
}
else
{
//game end
}

}

}

//メインメソッド
public static void main(String[] args)
{
Fishing set = new Fishing(); set.TITLE();
}
}

Comments

[shiracamus]

インデントが正しく表示されるように、コードをcodeブロックで囲んでください。

[jimbe]

前の回答で「TimerTaskでnew Fishing()はよくない」と言われた意味をお考えください.

Answer 1

かなりひどいコードです.
命名規則を無視し変数名はテキトウ, スコープを理解せずオブジェクトをそこら中で作りまくり, static 変数だらけです.

Fishing を new するのはメインメソッドでの1回のみとなるように, スコープを理解して修正してみてください.

---

import java.awt.CardLayout;
import java.awt.event.*;
import java.util.Random;
import javax.swing.*;

public class JFishing extends JFrame {
  /** 魚 */
  private static class Fish {
    private ImageIcon icon;
    private int score;
    Fish(String iconName, int score) {
      icon = new ImageIcon(iconName);
      this.score = score;
    }
    Icon getIcon() { return icon; }
    int getScore() { return score; }
  }
  /** 釣り上げチャンス */
  private static class CatchChance {
    private static Random random = new Random(System.currentTimeMillis());
    private int ratio;
    private Icon icon;
    private int time;
    private Fish[] fishes;
    /**
     * コンストラクタ
     * @param ratio 確率(全体に占める割合)
     * @param iconName イメージファイル名
     * @param time 継続時間
     * @param fishes 釣れる可能性のある魚の配列
     */
    CatchChance(int ratio, String iconName, int time, Fish ... fishes) {
      this.ratio = ratio;
      this.icon = new ImageIcon(iconName);
      this.time = time;
      this.fishes = fishes;
    }
    int getRatio() { return ratio; }
    Icon getIcon() { return icon; }
    int getTime() { return time; }
    Fish getFish() {
      if(fishes.length == 0) return null;
      int j = random.nextInt(fishes.length);
      return fishes[j];
    }
  };
  /** 魚が掛かっていない状態 */
  private static class NormalState extends CatchChance {
    private Random random = new Random(System.currentTimeMillis());
    NormalState(String iconName) {
      super(0, iconName, 0);
    }
    /** 魚が掛かっていない時間 */
    int getTime() {
      return 100 * (random.nextInt(50) + 1); // 100 - 5000
    }
  }
  private static final CatchChance CATCH_CHANCES[] = {
      new NormalState("しーん.jpg"),
      new CatchChance(11,"！.jpg",    600, new Fish("N0.jpg" ,  0), new Fish("N1.jpg" , 50), new Fish("N2.jpg" ,  50), new Fish("N3.jpg" ,100), new Fish("N4.jpg" , 100)),
      new CatchChance( 7,"！！.jpg",  400, new Fish("R0.jpg" ,150), new Fish("R1.jpg" ,150), new Fish("R2.jpg" , 200), new Fish("R3.jpg" ,200), new Fish("R4.jpg" , 250)),
      new CatchChance( 2,"！！！.jpg",300, new Fish("SR0.jpg",300), new Fish("SR1.jpg",300), new Fish("SR2.jpg", 400), new Fish("SR3.jpg",400), new Fish("SR4.jpg", 500)),
      new CatchChance( 0,"？.jpg",      0, new Fish("SC0.jpg",-50), new Fish("SC1.jpg",-50), new Fish("SC2.jpg",-100), new Fish("SC3.jpg",  0), new Fish("SC4.jpg",1000))
  };

  public static void main(String[] args) {
    new JFishing().setVisible(true);
  }

  //private int flag = 0; //使い道不定
  private int gamescore = 0;
  private int alltime = 0; //60sカウントに使う。

  private CardLayout mgr;
  private CatchPanel catchPanel;

  JFishing() {
    super();
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(550, 100, 770, 800);

    mgr = new CardLayout();
    getContentPane().setLayout(mgr);
    add(new TitlePanel(),   "Title");
    add(new ExplainPanel(), "Explain");
    add(new GamePanel(),    "Game");

    catchPanel = new CatchPanel();
    add(catchPanel, "Catch");

    showTitlePanel();
  }
  private void showTitlePanel() {
    gamescore = 0;
    alltime = 0;
    mgr.show(getContentPane(), "Title");
  }
  private void showExplainPanel() {
    mgr.show(getContentPane(), "Explain");
  }
  private void showGamePanel() {
    mgr.show(getContentPane(), "Game");
  }
  private void showCatchPanel(Fish fish) {
    catchPanel.setFish(fish);
    mgr.show(getContentPane(), "Catch");
  }

  /** タイトルパネル */
  private class TitlePanel extends JPanel {
    TitlePanel() {
      super(null);
      setTitle("魚釣りゲーム～タイトル～");

      JLabel titleLabel = new JLabel(new ImageIcon("タイトル画像.jpg"));
      titleLabel.setBounds(-10, 0, 770, 760);
      add(titleLabel);

      JButton startButton = new JButton("スタート");
      startButton.setBounds(310, 580, 130, 30);
      startButton.addActionListener(e -> showGamePanel()); // スタート→メイン画面へ遷移
      add(startButton);

      JButton explainButton = new JButton("あそびかた");
      explainButton.setBounds(310, 650, 130, 30);
      explainButton.addActionListener(e -> showExplainPanel());
      add(explainButton);
    }
  }

  /** ルール説明パネル */
  private class ExplainPanel extends JPanel {
    private Icon rule1Icon = new ImageIcon("ルール１.jpg");
    private Icon rule2Icon = new ImageIcon("ルール2.jpg");
    private JLabel ruleLabel;
    private JButton titleButton;
    private JButton nextButton;

    ExplainPanel() {
      super(null);
      setTitle("魚釣りゲームのルール説明");

      ruleLabel = new JLabel(rule1Icon);
      ruleLabel.setBounds(0, 150, 760, 460);
      add(ruleLabel);

      titleButton = new JButton("タイトルへ");
      titleButton.setBounds(50, 520, 180, 30);
      titleButton.addActionListener(e -> showTitlePanel());
      add(titleButton);

      nextButton = new JButton("つぎへ");
      nextButton.setBounds(100, 540, 130, 30);
      nextButton.addActionListener(e -> showRule2());
      add(nextButton);

      addComponentListener(new ComponentAdapter() {
        @Override
        public void componentShown(ComponentEvent e) {
          showRule1();
        }
      });
    }
    private void showRule1() {
      ruleLabel.setIcon(rule1Icon);
      nextButton.setVisible(true);
      titleButton.setVisible(false);
    }
    private void showRule2() {
      ruleLabel.setIcon(rule2Icon);
      nextButton.setVisible(false);
      titleButton.setVisible(true);
    }
  }

  /** ゲームパネル */
  private class GamePanel extends JPanel {
    private long starttime;
    private CatchAction catchAction;
    private JLabel scoreLabel;

    GamePanel() {
      super(null);
      setTitle("魚釣りゲーム");

      scoreLabel = new JLabel();
      scoreLabel.setBounds(100, 0, 400, 400);
      add(scoreLabel);

      JLabel seaLabel = new JLabel(new ImageIcon("海.jpg"));
      seaLabel.setBounds(195, 400, 560, 350);
      add(seaLabel);

      JLabel fisherLabel = new JLabel(new ImageIcon("釣りbasic2.jpg"));
      fisherLabel.setBounds(0, 180, 380, 380);
      add(fisherLabel);

      catchAction = new CatchAction(CATCH_CHANCES);
      catchAction.setOnCatchListener(fish -> {
        gamescore += fish.getScore();
        rewriteScoreLabel();
        showCatchPanel(fish);
      });

      JButton catchButton = new JButton(catchAction);
      catchButton.setBounds(378, 120, 375, 280);
      add(catchButton);

      addComponentListener(new ComponentAdapter() {
        @Override
        public void componentShown(ComponentEvent e) {
          starttime = System.currentTimeMillis();
          rewriteScoreLabel();
          catchAction.start();
        }
        @Override
        public void componentHidden(ComponentEvent e) {
          alltime += (int)(System.currentTimeMillis() - starttime);
          catchAction.stop();
        }
      });
      rewriteScoreLabel();
    }
    private void rewriteScoreLabel() {
      scoreLabel.setText("SCORE: " + gamescore + "点");
    }
  }
  /** 釣り上げ */
  private static class CatchAction extends AbstractAction {
    interface OnCatchListener {
      void onCatch(Fish fish);
    }
    private OnCatchListener listener;
    private CatchChance[] catchChances;
    private int totalRatio;
    private Random random = new Random(System.currentTimeMillis());
    private Timer timer;
    private CatchChance chance;

    CatchAction(CatchChance[] catchChances) {
       this.catchChances = catchChances;
       totalRatio = 0;
       for(CatchChance cr : catchChances) totalRatio += cr.getRatio();

       chance = catchChances[0];
       putValue(LARGE_ICON_KEY, chance.getIcon());
    }
    void setOnCatchListener(OnCatchListener listener) {
      this.listener = listener;
    }
    void start() {
      chance = catchChances[0];
      putValue(LARGE_ICON_KEY, chance.getIcon());
      int delay = chance.getTime();
      timer = new Timer(delay, new ActionListener() {
        public void actionPerformed(ActionEvent e) {
          timer.stop();
          chance = (chance == catchChances[0] ? getChance() : catchChances[0]);
          putValue(LARGE_ICON_KEY, chance.getIcon());
          timer.setInitialDelay(chance.getTime());
          timer.restart();
        }
      });
      timer.setRepeats(false);
      timer.start();
    }
    private CatchChance getChance() {
      int v = random.nextInt(totalRatio);
      int i = 1;
      while(v>=catchChances[i].getRatio()) v-=catchChances[i++].getRatio();
      return catchChances[i];
    }
    void stop() {
      if(timer != null) {
        timer.stop();
        timer = null;
      }
    }
    @Override
    public void actionPerformed(ActionEvent e) {
      Fish fish = chance.getFish();
      if(fish != null && listener != null) {
        stop();
        listener.onCatch(fish);
      }
    }
  }
  /** 釣り上げパネル */
  private class CatchPanel extends JPanel {
    private JLabel imageLabel;
    CatchPanel() {
      JButton returnButton = new JButton("もどる");
      returnButton.setBounds(280, 650, 130, 30);
      returnButton.addActionListener(e -> {
        if(alltime < 60000) {
          showGamePanel();
        } else {
          //game end
        }
      });
      add(returnButton);

      imageLabel = new JLabel();
      imageLabel.setBounds(150, 200, 400, 400);
      add(imageLabel);

      JLabel catchLabel = new JLabel("釣り上げた！");
      catchLabel.setBounds(305, 120, 100, 100);
      add(catchLabel);
    }
    void setFish(Fish fish) {
      imageLabel.setIcon(fish.getIcon());
    }
  }
}

Comments

[Pcios]

こんなにたくさん…！ありがとうございます！
もう自分が書いたコードとはかけ離れてしまいましたが、私のコードのバグは、static変数が多すぎてバグったのですかね？
取り敢えず、明日くらいにじっくり確認させていただきます！

[jimbe]

コードを全く変えてしまったのは申し訳ありません.
物が大量に詰まっている家の中では, 無くしモノを探す前に片付けなければなりません.
家の主なら何処に何があるのか分かっているかも知れませんが, 他人には分かりません.
だからこそ, (多少長くなっても)変数名やメソッド名, クラス名等に命名規則に基づいた意味のある名前をつけたり, 分けられるものはクラスに分けたりするのです.

[jimbe]

ご質問の原因は特定していませんが, static が多いというのは直接の原因ではありません.
まず問題なのは, 回答に書きました通り Fishing を多量に生成していることです.
オブジェクトに慣れていない方がよく勘違いされますのは, new HogeHoge によって生成されるモノが全て「一つのモノ」「同じモノ」だと思うことです.
実際には逆で, 全て別々のモノです.

例えば, ある個所(A)で new iPhone として得た iPhone と, 別の個所(B)で new iPhone として得た iPhone は, 同じ iPhone の設計図(クラス)から作られた iPhone (オブジェクト) ではありますが, モノとしては別々です.
A の iPhone のアドレス帳に友人を登録しても, B の iPhone のアドレス帳にはその友人は登録されないでしょう.
ところが java 的に, アドレス帳が static であったとすると, B の iPhone のアドレス帳にも友人が登録されているようになります.
「なんだ, じゃあ全部 static にして, 必要になったら new iPhone で作れば良いじゃないか」...とは言えません.
どの iPhone からでも同じアドレス帳に登録が出来るとしても, 友人を登録する度に iPhone を買うでしょうか? 一回電話をするだけの為に買うでしょうか? iPhone は(コストが)高いです. どんどん浪費することになるでしょう. (Fishing が継承している JFrame も背後に大量のコード・データがあり, コストが高いです.)
そんなことをしなくても, 既にポケットに iPhone が入っているなら, 隣の部屋に iPhone があるなら, それを取って使えば良いだけです. 同じ iPhone を使い続ければ, アドレス帳が static でもそうでなくても関係ありません.

それが「Fishing を new するのはメインメソッドでの1回のみとなるように」と回答に書かせていただいたことです.

[Pcios]

とても分かりやすい説明、ありがとうございます！お陰様で理解出来ました！オブジェクトをstaticにすると1つのものになると説明していただきましたが、普通にインスタンス化した場合(例えばFishing frame = new frame()とした場合)は、このオブジェクトframeはstaticになってるのでしょうか？それともスコープはインスタンス化したブロック内に限られるのでしょうか？もしstaticなオブジェクトを作るにはどうすれば良いですか？

[Pcios]

画面生成用としてframeオブジェクトを作ったので、もしこのframeオブジェクトがstaticなら、あらゆるメソッドから操作できれば画面遷移が簡単に出来るのでは？と思ったので。。

[jimbe]

static な変数は, "static" と書いたものだけです. static と付いていなければ, それはスコープを抜ければ消えることになります.

画面遷移の例ですが, 私のコードでは画面の遷移は "showほにゃららPanel" というメソッドを呼ぶだけで出来ています. ですが JFishing は static ではありません.
「(static で用いることで)あらゆるメソッドから操作できる」ことが「簡単になる」とは限りません. というより, それでは問題が大きかったのでオブジェクトが生まれたと言えるかと思います.
この辺りは書き始めると長くなります. (というか書き始めてみたのですが止めました.)
オブジェクトがなぜあるのか等は歴史をお調べになってみてください.

[Pcios]

staticなオブジェクトを作るには、static Fishing frame= new Fishing();とすれば良いのでしょうか？

[jimbe]

書き方としてはそうなります.
なお, static なのはオブジェクトではなく変数です. オブジェクトが変数に入れられている場合は, オブジェクトの生存期間はその変数と共になります.

[jimbe]

現状をどう表現すれば良いかと考えていましたが,
[ルーブ・ゴールドバーグ・マシン]( https://ja.wikipedia.org/wiki/%E3%83%AB%E3%83%BC%E3%83%96%E3%83%BB%E3%82%B4%E3%83%BC%E3%83%AB%E3%83%89%E3%83%90%E3%83%BC%E3%82%B0%E3%83%BB%E3%83%9E%E3%82%B7%E3%83%B3 )
があっているかもしれません.

Answer 2

jimbeさんの素敵なコードの後に出すのは気が引けますが、ちょうど中間あたりを狙ってみました。

変更点

• 各画面をパネルに分割
  分割すると管理が楽になります。
• 画面遷移は同じくカードレイアウトを使用
  遷移の常套手段です。
• 全部同じでは芸がないので釣り上げ画面はダイアログに
  ImageIconなども簡単に入れられて楽でした。
• 変数名やメソッドはできるだけ元のままにして、何がどこへいったかをわかりやすく
  ケース（Fish fish FISH等の大文字小文字の法則）くらいは標準的なものでないとムズムズします。
• 内部クラスはお好みでないようなので使わない（ボタンのアクションだけはラムダを使用）
  内部クラスはコンストラクタで渡したりがなくなるので楽ですが、クラスが長くなるので一長一短ですね。
• タイマーをjavax.swing.Timerに変更
  特に理由がなければこちらのほうがいいです。
• static撲滅
  staticが全部ダメというわけではないのですが、はじめのうちはstatic void mainでしか使わないぐらいでいいと思います。

import java.awt.CardLayout;
import java.awt.Component;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.Timer;


public class Fishing extends JFrame {

    public static void main(String[] args) {
        new Fishing().setVisible(true);
    }

    private Fishing() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(550, 100, 770, 800);

        getContentPane().setLayout(new CardLayout());
        add(new TitlePanel(this), "タイトル"); // 自分自身(this)をコンストラクタでわたす
        add(new Explain1Panel(this), "ルール１");
        add(new Explain2Panel(this), "ルール２");
        add(new MainGamePanel(this), "ゲーム");

        showPanel("タイトル");
    }

    void showPanel(String name) { // 画面遷移
        CardLayout cardLayout = (CardLayout) getContentPane().getLayout();
        cardLayout.show(getContentPane(), name);

        for (Component c : getContentPane().getComponents()) { // ウィンドウタイトル変更
            if (c.isVisible()) setTitle(c.getName());
        }
    }
}

class TitlePanel extends JPanel {

    TitlePanel(Fishing fishing) { // 表示しているFishingをコンストラクタでもらう
        setName("魚釣りゲーム～タイトル～"); // ウィンドウタイトル用
        setLayout(null);

        JButton bt1 = new JButton("スタート");
        bt1.setBounds(310, 580, 130, 30);
        bt1.addActionListener((e) -> fishing.showPanel("ゲーム")); // 引数でもらったFishingのメソッドを呼ぶ
        add(bt1);

        JButton bt2 = new JButton("あそびかた");
        bt2.setBounds(310, 650, 130, 30);
        bt2.addActionListener((e) -> fishing.showPanel("ルール１"));
        add(bt2);

        JLabel titleLabel = new JLabel(new ImageIcon("タイトル画像.jpg"));
        titleLabel.setBounds(-10, 0, 770, 760);
        add(titleLabel);
    }
}

class Explain1Panel extends JPanel {

    Explain1Panel(Fishing fishing) {
        setName("魚釣りゲームのルール説明");
        setLayout(null);

        JButton bt3 = new JButton("つぎへ");
        bt3.setBounds(100, 540, 130, 30);
        bt3.addActionListener((e) -> fishing.showPanel("ルール２"));
        add(bt3);

        JLabel RURELabel1 = new JLabel(new ImageIcon("ルール１.jpg"));
        RURELabel1.setBounds(0, 150, 760, 460);
        add(RURELabel1);
    }
}

class Explain2Panel extends JPanel {

    Explain2Panel(Fishing fishing) {
        setName("魚釣りゲームのルール説明");
        setLayout(null);

        JButton bt4 = new JButton("タイトルへ");
        bt4.setBounds(50, 520, 180, 30);
        bt4.addActionListener((e) -> fishing.showPanel("タイトル"));
        add(bt4);

        JLabel RURELabel2 = new JLabel(new ImageIcon("ルール2.jpg"));
        RURELabel2.setBounds(0, 150, 760, 460);
        add(RURELabel2);
    }
}

class MainGamePanel extends JPanel implements ActionListener {
    private int[] Ra = { 1, 1, 1, 1, 1, 1, 1, 1, 2, 2, 2, 2, 1, 3, 3, 2, 1, 2, 1, 2 };

    private int[] fish_score1 = { 0, 50, 50, 100, 100 };
    private int[] fish_score2 = { 150, 150, 200, 200, 250 };
    private int[] fish_score3 = { 300, 300, 400, 400, 500 };

    private ImageIcon[] fish1 = { new ImageIcon("N0.jpg"), new ImageIcon("N1.jpg"), new ImageIcon("N2.jpg"), new ImageIcon("N3.jpg"), new ImageIcon("N4.jpg"), };
    private ImageIcon[] fish2 = { new ImageIcon("R0.jpg"), new ImageIcon("R1.jpg"), new ImageIcon("R2.jpg"), new ImageIcon("R3.jpg"), new ImageIcon("R4.jpg"), };
    private ImageIcon[] fish3 = { new ImageIcon("SR0.jpg"), new ImageIcon("SR1.jpg"), new ImageIcon("SR2.jpg"), new ImageIcon("SR3.jpg"), new ImageIcon("SR4.jpg"), };

    private JButton[] Bbt = { new JButton(new ImageIcon("しーん.jpg")), new JButton(new ImageIcon("！.jpg")), new JButton(new ImageIcon("！！.jpg")), new JButton(new ImageIcon("！！！.jpg")), new JButton(new ImageIcon("？.jpg")) };
    private JLabel SCORE_Label = new JLabel();

    private Random random = new Random();
    private Timer timer1;
    private Timer timer2;
    private int gamescore = 0;
    private int alltime = 0;


    MainGamePanel(Fishing fishing) {
        setName("魚釣りゲーム");
        setLayout(null);

        SCORE_Label.setBounds(100, 0, 400, 400);
        add(SCORE_Label);
        addScore(0);

        JLabel SEA_Label = new JLabel(new ImageIcon("海.jpg"));
        SEA_Label.setBounds(195, 400, 560, 350);
        add(SEA_Label);

        JLabel NINGEN_Label = new JLabel(new ImageIcon("釣りbasic2.jpg"));
        NINGEN_Label.setBounds(0, 180, 380, 380);
        add(NINGEN_Label);

        for (JButton b : Bbt) {
            b.setBounds(378, 120, 375, 280);
            b.setVisible(false);
            b.addActionListener(this);
            add(b);
        }
        Bbt[0].setVisible(true);

        timer1 = new Timer(0, new TASK(this)); // delayはROLL ROLL2で設定
        timer2 = new Timer(0, new TASK2(this));
        ROLL2();
    }

    void ROLL() {
        timer1.stop();
        int x = Atrandom();
        changeButton(x);

        int delay;
        switch (x) {
            case 1:
                delay = 600;
                break;
            case 2:
                delay = 400;
                break;
            default:
                delay = 200;
                break;
        }

        timer2.setInitialDelay(delay);
        alltime += delay;
        timer2.start();
    }

    void ROLL2() {
        timer2.stop();
        changeButton(0);

        int R = 100 * (random.nextInt(50) + 1);
        alltime += R;
        timer1.setInitialDelay(R);
        timer1.start();
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        JButton b = (JButton) e.getSource();
        if (b == Bbt[0]) return; // 連打okなのか？？

        timer1.stop();
        timer2.stop();
        if (b == Bbt[1]) CATCH1();
        else if (b == Bbt[2]) CATCH2();
        else if (b == Bbt[3]) CATCH3();

        ROLL2();
    }

    private void addScore(int score) {
        gamescore += score;
        SCORE_Label.setText("SCORE: " + gamescore + "点");
    }

    private void changeButton(int index) { // ボタンの変更 Visible式
        for (JButton b : Bbt) {
            b.setVisible(false);
        }
        Bbt[index].setVisible(true);
    }

    private int Atrandom() {
        return Ra[random.nextInt(Ra.length)];
    }

    private void CATCH1() {
        int B = random.nextInt(5);
        JOptionPane.showMessageDialog(this, fish1[B], "釣り上げた！", JOptionPane.PLAIN_MESSAGE);
        // ダイアログは処理をブロックする 閉じるまで↓に行かない
        addScore(fish_score1[B]);
    }

    private void CATCH2() {
        int B = random.nextInt(5);
        JOptionPane.showMessageDialog(this, fish2[B], "釣り上げた！", JOptionPane.PLAIN_MESSAGE);
        addScore(fish_score2[B]);
    }

    private void CATCH3() {
        int B = random.nextInt(5);
        JOptionPane.showMessageDialog(this, fish3[B], "釣り上げた！", JOptionPane.PLAIN_MESSAGE);
        addScore(fish_score3[B]);
    }
}

class TASK implements ActionListener {
    private MainGamePanel mainGamePanel;

    TASK(MainGamePanel mainGamePanel) {
        this.mainGamePanel = mainGamePanel;
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        mainGamePanel.ROLL();
    }
}

class TASK2 implements ActionListener {
    private MainGamePanel mainGamePanel;

    TASK2(MainGamePanel mainGamePanel) {
        this.mainGamePanel = mainGamePanel;
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        mainGamePanel.ROLL2();
    }
}

Comments

[Pcios]

回答ありがとうございます！！
明日実際実行してみます！
ちなみになんですが、1回目の動作はうまくいくのに2回目からうまくいかなくなるってことが起きたとしたら、その原因って主にどんなものがありますか？

[TN8001]

Fishingが複数残っていてCATCH()が重複して呼ばれているんだろうとは思いますが、詳しく追う気になりません^^;

もしかして現コードをどうにか直そうと思っています？
ここまで書いたのをどうにかしたい気持ちはわかります。
しかしあまりにもイレギュラーな作りなので、jimbeさんや私のコードを参考にしていただいて作り直すのを強く勧めます。