"ActionListener" funktioniert nicht

[francisca93]

Hallo,
ich möchte, dass beim Drücken von "Return" aus einem Textfield der Wert ausgelesen und als Variable in einer Instanz gespeichert wird. Der Index des Arrays der Instanz soll vorher aus einer Jlist mit "getSelectedIndex()" ausgelesen werden:

JTextField textFieldAnzahl = new JTextField();
            textFieldAnzahl.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    int n = listProdukte.getSelectedIndex();
                    w[n].Stueckzahl = Integer.parseInt (textFieldAnzahl.getText());
                                                       
                    double gesamtpreis = 0;
                    for(int i=0; i<w.length; i++)
                        gesamtpreis = (gesamtpreis + w[i].Stueckpreis*w[i].Stueckzahl);
                    double preis = w[n].Stueckpreis*w[n].Stueckzahl;
                   
                    lblGesamtpreisAnz.setText("" + gesamtpreis);
                    lblPreisAnz.setText("" + preis);
                }
            });

Blöderweise passiert überhaupt nichts, der ActionListener scheint gar nicht anzuspringen. Dafür kommen unten eine Menge Meldungen in roter Schrift, wenn ich Return drücke:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
    at praktikum_3.Praktikum_3$2.actionPerformed(Praktikum_3.java:113)
    at javax.swing.JTextField.fireActionPerformed(Unknown Source)
    at javax.swing.JTextField.postActionEvent(Unknown Source)
    at javax.swing.JTextField$NotifyAction.actionPerformed(Unknown Source)

(und noch ein paar hundert Zeilen...)

Was könnte denn da falsch sein?

 

[Robat]

Die "Meldungen in roter Schrift" nennen sich Exceptions. Aus denen kann man eine ganze Menge ablesen, wenn man weiß wo

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException

Hier siehst du bspw. dass es sich um eine NullpointerException handelt was bedeutet dass du versuchst auf ein Objekt zuzugreifen (bspw eine Methode aufzurufen) welches noch gar nicht initialisiert ist. Demnach ist dieses Objekt null.

t praktikum_3.Praktikum_3$2.actionPerformed(Praktikum_3.java:113)

Hier siehst du dass es sich um die Zeile 113 in der Klasse Praktikum_3 handelt. Schreib am Besten mal welche Zeile das genau ist dann kann man gemeinsam schauen wo der Fehler liegt.

Gruß Robert

 

[francisca93]

Danke. Also die Zeile 113 ist diese: int n = listProdukte.getSelectedIndex();

Die listProdukte ist weiter unten im Text als JList definiert. Ich setze mal den ganzen Text hin:

package praktikum_3;

import java.awt.BorderLayout;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.JList;
import javax.swing.JLabel;
import javax.swing.JTextField;
import javax.swing.DefaultListModel;
import javax.swing.event.ListSelectionListener;
import javax.swing.event.ListSelectionEvent;
import javax.swing.ListSelectionModel;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.text.*;




public class Praktikum_3 extends JFrame {
   
    private JPanel contentPane;
    private JList listProdukte;
    private JLabel lblProdukt;
    private JLabel lblStueckpreis;
    private JLabel lblAnzahl;
    private JLabel lblPreis;
    private JLabel lblGesamtpreis;
    private JLabel lblStueckpreisAnz;
    private JLabel lblPreisAnz;
    private JLabel lblGesamtpreisAnz;
    private JTextField textFieldAnzahl;
   
    DecimalFormat f = new DecimalFormat("#0.00");
    /**
     * Launch the application.
     */
    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    Praktikum_3 frame = new Praktikum_3();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    /**
     * Create the frame.
     */
    public Praktikum_3() {
       
        init();
    }   
   
    public Werkzeug w[] = {
            new Werkzeug("Schraubenzieher", 8.95),
            new Werkzeug("Hammer", 6.49),
            new Werkzeug("Maurerkelle", 9.95),
            new Werkzeug("Zollstock", 2.49),
            new Werkzeug("Hobel", 15.95)
    };
   
        public void init(){   
            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            setBounds(100, 100, 450, 300);
            contentPane = new JPanel();
            contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
            setContentPane(contentPane);
            contentPane.setLayout(null);
                       
            lblProdukt = new JLabel("Produkt:");
            lblProdukt.setBounds(30, 30, 55, 16);
            contentPane.add(lblProdukt);
           
            lblStueckpreis = new JLabel("Stueckpreis:");
            lblStueckpreis.setBounds(204, 70, 83, 16);
            contentPane.add(lblStueckpreis);
           
            lblAnzahl = new JLabel("Anzahl:");
            lblAnzahl.setBounds(204, 114, 55, 16);
            contentPane.add(lblAnzahl);
           
            lblPreis = new JLabel("Preis:");
            lblPreis.setBounds(204, 162, 55, 16);
            contentPane.add(lblPreis);
           
            lblGesamtpreis = new JLabel("Gesamtpreis:");
            lblGesamtpreis.setBounds(204, 219, 83, 16);
            contentPane.add(lblGesamtpreis);
           
            lblStueckpreisAnz = new JLabel("New label");
            lblStueckpreisAnz.setBounds(324, 70, 55, 16);
            contentPane.add(lblStueckpreisAnz);
           
            lblPreisAnz = new JLabel("New label");
            lblPreisAnz.setBounds(324, 162, 55, 16);
            contentPane.add(lblPreisAnz);
           
            lblGesamtpreisAnz = new JLabel("New label");
            lblGesamtpreisAnz.setBounds(324, 219, 55, 16);
            contentPane.add(lblGesamtpreisAnz);
                           
            JTextField textFieldAnzahl = new JTextField();
            textFieldAnzahl.addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e) {
                    int n = listProdukte.getSelectedIndex();
                    w[n].Stueckzahl = Integer.parseInt (textFieldAnzahl.getText());
                                                       
                    double gesamtpreis = 0;
                    for(int i=0; i<w.length; i++)
                        gesamtpreis = (gesamtpreis + w[i].Stueckpreis*w[i].Stueckzahl);
                    double preis = w[n].Stueckpreis*w[n].Stueckzahl;
                   
                    lblGesamtpreisAnz.setText("" + gesamtpreis);
                    lblPreisAnz.setText("" + preis);
                }
            });
           
            textFieldAnzahl.setBounds(320, 112, 59, 20);
            contentPane.add(textFieldAnzahl);
            textFieldAnzahl.setColumns(10);
           
            DefaultListModel listModel = new DefaultListModel();
           
            JList listProdukte = new JList(listModel);
            listProdukte.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
            listProdukte.addListSelectionListener(new ListSelectionListener() {
                public void valueChanged(ListSelectionEvent e) {
                    int i = listProdukte.getSelectedIndex();
                    if ((i>=0)&&(i<=w.length)){
                        lblStueckpreisAnz.setText("" + w[i].Stueckpreis);   
                        textFieldAnzahl.setText("" + w[i].Stueckzahl);
                    }   
                }
            });
           
            listProdukte.setBounds(30, 69, 145, 166);
            contentPane.add(listProdukte);
                                   
             for (int i=0; i<w.length; i++)
            listModel.addElement(w[i].name);       
                   
        }
               
}

package praktikum_3;

public class Werkzeug {
   
    String name;
    double Stueckpreis;
    int Stueckzahl;
   
    public Werkzeug(String n, double s){
        name = n;
        Stueckpreis = s;
        Stueckzahl = 0;
    }
}

 

[Robat]

int n = listProdukte.getSelectedIndex();

Wie erwartet ist also deine listProdukte noch null in dem Moment wo der Listener aufgerufen wird.

Das solltest du relativ leicht beheben können in dem du die Initialisierung des listModels und der listProdukte

DefaultListModel listModel = new DefaultListModel();
JList listProdukte = new JList(listModel);

vor die des textFieldAnzahl schiebst.

 

[francisca93]

Alles klar, das war es