0

エラー:奇妙なことに、エラーが発生しません。

私が作ろうとしているもの: (コードの動作を説明する // メッセージも追加しました。)

私が作ろうとしていることはかなり単純です。

1) JTextField に名前を入力し、Enter キーを押すと、名前が JTextArea に表示されます。名前が JTextArea にあると、JTextField は空になるので、別の名前を入力できます。JTextArea に名前のリストが表示されるはずです。

2) ボタン kiesWin を押して、プログラムにリストからランダムに人物を選択させます。（ここでエラーになります）

3) ボタン resetL を押してプログラムをリセットし、新しいリストを作成してランダムな勝者を選択できるようにします。

問題:ボタン Kies (翻訳: 選択) を押すと、ArrayList からランダムな名前が選択され、JTextField textvak2 にランダムな名前が表示されます。しかし、ボタン Kies を押しても、プログラムは何もしません。そして、ArrayList からランダムに選択された名前が表示されるはずです。

これは正しく動作しないクラスです: (私は思う)

// This is the button that chooses a random name from the ArrayList.
// The random chosen name should appear in the JTextField textvak2. (but it doesn't)
// This is also the part where it goes wrong at the moment.
class Kies extends OnthoudNaam implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
        Random r = new Random();
        if (lijst.size() > 0) {
            int n = r.nextInt(lijst.size());
            Naam kiesNaam = lijst.get(n);
            textvak2.setText(kiesNaam.getIngevoerdNaam());
            }
    }
}

コード全体が必要な場合:

import javax.swing.*;
import javax.swing.border.*;
import java.awt.*;
import java.awt.event.*;
import java.util.*;

// Main method to make the frame.
public class Loterij3 extends JFrame {
public static void main( String args[] ) {
    JFrame frame = new Loterij3();
    frame.setExtendedState( frame.MAXIMIZED_BOTH );
    frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
    frame.setTitle( "My Lottery!" );
    frame.setContentPane( new Paneel() );
    frame.setVisible( true );
}
}

// This is the Panel that goes into the frame.
class Paneel extends JPanel {
private Boven boven;
JTextArea textvak1;
JTextField textvak2;
OnthoudNaam onthoudNaam = new OnthoudNaam();
JTextField invoervak1; // JTextField from class Boven.

public Paneel() {
setLayout( new BorderLayout() ); // using border Layout.
setBackground( Color.LIGHT_GRAY );

boven = new Boven(); 

textvak1 = new JTextArea();
add( new JScrollPane( textvak1 ) );
textvak1.setBackground( Color.WHITE );

textvak2 = new JTextField();
textvak2.setHorizontalAlignment(JTextField.CENTER); 

add( boven, BorderLayout.NORTH ); // Where the class Boven should be.
add( textvak1, BorderLayout.CENTER );
add( textvak2, BorderLayout.SOUTH );
}

// This is the class Boven (Translation up or upper).
// This is where the JButtons, JTextField and JLabels are.
public class Boven extends JPanel {
JButton kiesWin, resetL;
JLabel label1;

public Boven() {
    setBackground( Color.LIGHT_GRAY );
    setLayout( new GridLayout( 1, 4, 100, 5 ) ); // using GridLayout.
    Border border = 
        BorderFactory.createEmptyBorder( 10, 10, 10, 10 );
    setBorder( border );

    kiesWin = new JButton("Kies een Winnaar!");
    kiesWin.addActionListener( new Kies() );
    resetL = new JButton("Reset alles");
    resetL.addActionListener( new Reset() );
    label1 = new JLabel("Voer Persoon in en druk op enter: ", JLabel.RIGHT);
    invoervak1 = new JTextField( 20 );
    invoervak1.addActionListener( new InvoerVakHandler() );

    add( label1 );
    add( invoervak1 );
    add( kiesWin );
    add( resetL );
    }
}

// The class Naam (translation = name).
// This is what the ArrayList should remember
// In other words ArrayList should remember the Names I put in the JTextField.
class Naam {
    private String ingevoerdNaam;

    public Naam( String ingevoerdNaam) {
        this.ingevoerdNaam = ingevoerdNaam;
    }

    public String getIngevoerdNaam() {
        return ingevoerdNaam;
    }

    public String toString() {
        return ingevoerdNaam;
    }
}

// This is my ArrayList,
// This should remember the names I type in the JTextField.
class OnthoudNaam extends JPanel {
    protected ArrayList<Naam> lijst;

    public OnthoudNaam() {
        lijst = new ArrayList<Naam>();
        }

        public void voegNaamToe(Naam x ) {
        lijst.add(x);
        }

        public String toString() {
        StringBuffer buffer = new StringBuffer();
        for(Naam x : lijst ) {
        buffer.append( x );
        buffer.append( "\n" );
    }
    return buffer.toString();
}
}

// This is the JTextField where I enter the names.
// The Name I fill in the JTextField should be remembered by the ArrayList.
// The Name I fill in should be put in the JTextArea.
public class InvoerVakHandler implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
        String invoer = invoervak1.getText();
        Naam Naam = new Naam( invoer );
        onthoudNaam.voegNaamToe( Naam );
        textvak1.setText( onthoudNaam.toString() );
        invoervak1.setText( "" );
    }
}
    // This is the button that chooses a random name from the ArrayList.
    // This is also the part where it goes wrong at the moment.
class Kies extends OnthoudNaam implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
        Random r = new Random();
        if (lijst.size() > 0) {
            int n = r.nextInt(lijst.size());
            Naam kiesNaam = lijst.get(n);
            textvak2.setText(kiesNaam.getIngevoerdNaam());
            }
    }
}

// This should become the button that resets everything so you can start over.
class Reset implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
    }
}
}
4

1 に答える 1

1

Random#nextInt正の数が必要ですが、の初期サイズはList lijst例外0です

ドキュメントはこれを明確に述べています

スロー: IllegalArgumentException - n が正でない場合

最初にエントリがあることを確認しますList。

if (lijst.size() > 0) {
   int n = r.nextInt(lijst.size());
   Naam kiesNaam = lijst.get(n);
   textvak2.setText(kiesNaam.getIngevoerdNaam());
}

余談: からオブジェクトをキャストするのではなく、オブジェクトをList抽出してNaamそのgetIngevoerdNaamメソッドを使用します。

覚えておいてくださいデバッガーはあなたの友達です

于 2013-05-09T10:28:51.097 に答える