配列内の文字をランダムに表示したい

Question

#困っていること
最初にラベルに配列Janken内の文字列をランダムに表示し、更にボタンを押すたびに新たに配列Jankenの文字列をランダムに表示するプログラムを作っているのですが、エラーが出て実行できません。

#該当のソースコード

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;

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

public class L5enshu3 {
    JLabel lb;
    Random rndm;
    String[] Janken={"グーでした","チョキでした","パーでした"};     //配列Jankenを定義
    public static void main(String[] args) {
        new L5enshu3();
    }
    public L5enshu3(){
        JFrame f;
        f=new JFrame("乱数");
        f.setSize(400,200);

        JButton b;
        b=new JButton();

        b.setText("押してください");
        int num;
        lb=new JLabel();
        f.setLayout(new BorderLayout());
        f.add(b, BorderLayout.CENTER);
        f.add(lb, BorderLayout.SOUTH);
        rndm=new Random();     //1行下　配列Janken[0]～配列Janken[2]までを1つランダムに生成
        num=rndm.Janken(2);    //エラー　メソッドJanken(int)は型Randomで未定義です
        lb.setText(num);       //エラー　型JLabelのメソッドsetText(String)は引数(int)に適用できません
        b.addActionListener(new L5enshu3click());
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.setVisible(true);
        f.setVisible(true);
    }
    class L5enshu3click implements ActionListener{
        public void actionPerformed(ActionEvent e){
            int num;
            rndm=new Random();     //ボタンが押されたら配列から新たにランダムに選ぶ
            num=rndm.Janken;       //エラー　Jankenは解決できないか、フィールドではありません
            lb.setText(num);       //エラー　型JLabelのメソッドsetText(String)は引数intに適用できません
        }
    }
}

#エラーメッセージ(ソースコード横のコメントにも記載済み)

32行目  num=rndm.Janken(2);    
メソッドJanken(int)は型Randomで未定義です

33行目 lb.setText(num)
型JLabelのメソッドsetText(String)は引数(int)に適用できません

43行目   num=rndm.Janken;
Jankenは解決できないか、フィールドではありません

44行目 lb.setText(num);
型JLabelのメソッドsetText(String)は引数intに適用できません   

Answer 1

rndm.nextInt(Janken.length) で 0~2 の整数値num を取得して、
Janken[num] をラベルに登録すれば良いです。

気になること

• 配列オブジェクトの名前はJankenよりjankenの方が良いです。慣習上、Javaでは大文字始まりのシンボルは主にクラス名に用います。
• Randomインスタンスはクラスで一回だけ生成すれば良いです。乱数を得る度に生成するのはリソースの無駄ですし、後からシード値を指定することも困難です。

Comments

[退会済みユーザー]

ご回答ありがとうございます。また時間のある時に試してみようと思います。

[退会済みユーザー]

ありがとうございます。解決しました！