Ausgabe über JLabel

[Damon_179]

Hey,
da ich noch recht neu hier im Forum bin, noch ein Anfänger bin und außerdem dies hier meine erste Frage im Forum ist die ich hier stelle, bitte ich um Rücksicht falls ich etwas nicht richtig beachtet habe.

Folgendes Problem: Ich erstelle ein recht simples Programm als Frame, welches nach Eingabe von Stückzahl, Stückpreis und Rabattsatz den Einkaufspreis berechnet und über ein JLabel ausgeben soll.
Zusätzlich soll - wenn eines der Textfelder leer ist - eine Fehlermeldung ausgegeben werden (das ist aber soweit kein Problem für mich).
Das einzige was mich stört ist, dass das Programm beim Berechnen abstürzt und folgendes in der Konsole ausgibt:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at gui.Einkauf.berechnen(Einkauf.java:147)
at gui.Einkauf.access$2(Einkauf.java:133)
at gui.Einkauf$4.keyPressed(Einkauf.java:98)
at java.awt.Component.processKeyEvent(Unknown Source)
at javax.swing.JComponent.processKeyEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.KeyboardFocusManager.redispatchEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source)
at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Ich hoffe ihr könnt mir eventuell weiterhelfen, danke!

 

[Hellosager]

Dein JLabel lblAusgabe wird nirgendwo initialisiert.
Es fehlt also irgendwo die Zeile

lblAusgabe = new JLabel();

Das kannst du theoretisch schon direkt nach der Deklaration machen.
Dann sollte es klappen.

 

[Damon_179]

Aber ich initialisiere mein JLabel für den Output doch an dieser stelle:

JLabel lblAusgabe = new JLabel("");
    lblAusgabe.setBounds(10, 104, 220, 20);
    contentPane.add(lblAusgabe);

 

[njans]

private JLabel lblAusgabe; das ist eine Instanzvariable.

Wenn du
JLabel lblAusgabe = new JLabel("");
Aufrufst, dann deklarierst und initialisierst du eine lokale Variable, innerhalb deiner Methode.
Deshalb hat @Hellosager auch den Typ bei der Zuweisung weggelassen.

 

[Hellosager]

Alles klar, das hab ich übersehen.
Mach folgendes draus:

JLabel lblAusgabe = new JLabel("");
    lblAusgabe.setBounds(10, 104, 220, 20);
    contentPane.add(lblAusgabe);

schon wird es funktionieren.

Edit: mist zu langsam^^

 

[njans]

Nein genau das sollte er eben nicht machen.

lblAusgabe = new JLabel("");
   lblAusgabe.setBounds(10, 104, 220, 20);
   contentPane.add(lblAusgabe);

greift auf seine Instanzvariable zu. Wenn du den Typ vor der Initialisierung angibst, wird daraus eine lokale Variable die am Ende der Methode weggeschmissen wird, da er damit nichts mehr macht.

 

[Hellosager]

Er will ja auf die Instanzvariable zugreifen...
Die war nur vorher nicht initialisiert, weil er eine lokale Variable angelegt hat statt mit der Zeile zu initialisieren. Deswegen gabs NullPointer.
Habs damit getestet es funktioniert.

 

[njans]

Dann testest du das Gegenteil von dem, was du in deinem Vorpost beschreibst.
In seinem code steht

JLabel lblAusgabe = new JLabel("");
   lblAusgabe.setBounds(10, 104, 220, 20);
   contentPane.add(lblAusgabe);

Und das funktioniert offensichtlicher weise nicht. In deinem Post schlägst du vor, genau diesen Code zu verwenden. Copy & Paste Fehler?

 

[Hellosager]

Das tut mir Leid ich hab gar nicht verstanden was du meintest. Natürlich muss es so aussehen:

lblAusgabe = new JLabel("");
    lblAusgabe.setBounds(10, 104, 220, 20);
    contentPane.add(lblAusgabe);

Schande auf mein Haupt...

 

[Damon_179]

Vielen dank für die weiteren Antworten.
Ich habe es nun so gemacht wie von euch beschrieben, allerdings wird mir nun kein Output geliefert - das JLabel erzeugt keinen Text bzw. Ergebnis. Fehler im Quellcode werden mir auch nirgends angezeigt.
Der Code der Musterlösung (Programmieren lernen mit Java, 3. Auflage - 2015) funktioniert allerdings, obwohl er jetzt identisch mit meinem korregierten Code ist, oder?.

Die Muserlösung:

package gui;

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

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.JLabel;
import javax.swing.JTextField;
import javax.swing.JButton;

import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.text.DecimalFormat;

public class Einkauf extends JFrame {

    private JPanel contentPane;
    private JTextField tfStueckzahl;
    private JTextField tfStueckpreis;
    private JTextField tfRabatt;
    private JLabel lblErgebnis;

    /**
     * Launch the application.
     */
    public static void main(String[] args) {
    EventQueue.invokeLater(new Runnable() {
        public void run() {
        try {
            Test frame = new Test();
            frame.setVisible(true);
        } catch (Exception e) {
            e.printStackTrace();
        }
        }
    });
    }

    /**
     * Create the frame.
     */
    public Einkauf() {
    setTitle("Einkaufspreis");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 514, 178);
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    setContentPane(contentPane);
    contentPane.setLayout(null);

    JLabel lblStueckzahl = new JLabel("St\u00FCckzahl");
    lblStueckzahl.setBounds(10, 11, 80, 14);
    contentPane.add(lblStueckzahl);

    JLabel lblStueckpreis = new JLabel("St\u00FCckpreis");
    lblStueckpreis.setBounds(127, 11, 88, 14);
    contentPane.add(lblStueckpreis);

    JLabel lblRabatt = new JLabel("Rabatt (%)");
    lblRabatt.setBounds(246, 11, 74, 14);
    contentPane.add(lblRabatt);

    tfStueckzahl = new JTextField();
    tfStueckzahl.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
        if (e.getKeyCode() == KeyEvent.VK_ENTER) {
            tfStueckpreis.requestFocus();
        }
        }
    });
    tfStueckzahl.setBounds(10, 25, 86, 20);
    contentPane.add(tfStueckzahl);
    tfStueckzahl.setColumns(10);

    tfStueckpreis = new JTextField();
    tfStueckpreis.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
        if (e.getKeyCode() == KeyEvent.VK_ENTER) {
            tfRabatt.requestFocus();
        }
        }
    });
    tfStueckpreis.setBounds(127, 25, 86, 20);
    contentPane.add(tfStueckpreis);
    tfStueckpreis.setColumns(10);

    tfRabatt = new JTextField();
    tfRabatt.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
        if (e.getKeyCode() == KeyEvent.VK_ENTER) {
        berechnen();
        }
        }
    });
    tfRabatt.setBounds(246, 25, 86, 20);
    contentPane.add(tfRabatt);
    tfRabatt.setColumns(10);

    JButton btnBerechnen = new JButton("Berechnen");
    btnBerechnen.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
        berechnen();
        }
    });
    btnBerechnen.setBounds(352, 24, 118, 23);
    contentPane.add(btnBerechnen);

    JButton btnEnde = new JButton("Ende");
    btnEnde.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
        System.exit(0);
        }
    });
    btnEnde.setBounds(368, 97, 89, 23);
    contentPane.add(btnEnde);

    lblErgebnis = new JLabel("");
    lblErgebnis.setBounds(10, 64, 322, 14);
    contentPane.add(lblErgebnis);
    }

    private void berechnen() {
    int stueckzahl;
    double stueckpreis, rabattsatz, bruttopreis, rabatt, einkaufspreis;
    if (tfStueckzahl.getText().equals("") || tfStueckpreis.equals("")
        || tfRabatt.equals("")) {
        lblErgebnis.setText("Die Eingaben sind unvollständig!");
    } else {
        stueckzahl = Integer.parseInt(tfStueckzahl.getText());
        stueckpreis = Double.parseDouble(tfStueckpreis.getText());
        rabattsatz = Double.parseDouble(tfRabatt.getText());
        bruttopreis = stueckzahl * stueckpreis;
        rabatt = bruttopreis * rabattsatz / 100;
        einkaufspreis = bruttopreis - rabatt;
        DecimalFormat f = new DecimalFormat("#0.00");
        lblErgebnis.setText("Der Einkaufspreis beträgt "
            + f.format(einkaufspreis) + " €");
    }
    tfStueckzahl.requestFocus();
    tfStueckzahl.selectAll();
    }

}

Tut mir Leid, falls ich etwas übersehen habe, es ist nun schon etwas spät und eigentlich habe ich ja frei, aber das lässt mich gerade nicht los.

 

[Damon_179]

Un hier noch einmal mein neuer, aktualisierter, welcher immer noch nicht funktioniert:

package gui;

import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.text.DecimalFormat;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.UIManager;
import javax.swing.border.EmptyBorder;

public class Einkauf extends JFrame {

    private JPanel contentPane;
    private JTextField tfStueckzahl;
    private JTextField tfStueckpreis;
    private JTextField tfRabatt;
    private JLabel lblAusgabe;

    /**
     * Launch the application.
     */
    public static void main(String[] args) {
        try {
            UIManager.setLookAndFeel("com.sun.java.swing.plaf.windows.WindowsLookAndFeel");
        } catch (Throwable e) {
            e.printStackTrace();
        }
    EventQueue.invokeLater(new Runnable() {
        public void run() {
        try {
            Einkauf frame = new Einkauf();
            frame.setVisible(true);
        } catch (Exception e) {
            e.printStackTrace();
        }
        }
    });
    }

    /**
     * Create the frame.
     */
    public Einkauf() {
        setTitle("Einkaufspreis");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 600, 200);
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    setContentPane(contentPane);
    contentPane.setLayout(null);
   
    JLabel lblStueckzahl = new JLabel("St\u00FCckzahl");
    lblStueckzahl.setBounds(10, 11, 110, 20);
    contentPane.add(lblStueckzahl);
   
    tfStueckzahl = new JTextField();
    tfStueckzahl.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
        if (e.getKeyCode() == KeyEvent.VK_ENTER) {
            tfStueckpreis.requestFocus();
        }
        }
    });
    tfStueckzahl.setBounds(10, 28, 110, 20);
    contentPane.add(tfStueckzahl);
    tfStueckzahl.setColumns(10);
   
    JLabel lblStckpreis = new JLabel("St\u00FCckpreis");
    lblStckpreis.setBounds(165, 14, 110, 17);
    contentPane.add(lblStckpreis);
   
    tfStueckpreis = new JTextField();
    tfStueckpreis.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
        if (e.getKeyCode() == KeyEvent.VK_ENTER) {
            tfRabatt.requestFocus();
        }
        }
    });
    tfStueckpreis.setBounds(165, 28, 110, 20);
    contentPane.add(tfStueckpreis);
    tfStueckpreis.setColumns(10);
   
    tfRabatt = new JTextField();
    tfRabatt.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
        if (e.getKeyCode() == KeyEvent.VK_ENTER) {
        berechnen();
        }
        }
    });
    tfRabatt.setBounds(325, 28, 110, 20);
    contentPane.add(tfRabatt);
    tfRabatt.setColumns(10);
   
    JLabel lblRabatt = new JLabel("Rabatt (%)");
    lblRabatt.setBounds(325, 14, 110, 17);
    contentPane.add(lblRabatt);
   
    JButton btnBerechnen = new JButton("Berechnen");
    btnBerechnen.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
            berechnen();
        }
    });
    btnBerechnen.setBounds(485, 27, 89, 23);
    contentPane.add(btnBerechnen);
   
    JButton btnEnde = new JButton("Ende");
    btnEnde.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent arg0) {
            System.exit(0);
        }
    });
    btnEnde.setBounds(485, 100, 89, 23);
    contentPane.add(btnEnde);
   
    lblAusgabe = new JLabel("");
    lblAusgabe.setBounds(10, 104, 220, 20);
    contentPane.add(lblAusgabe);
    }
   
    private void berechnen() {
      int stueckzahl;
      double stueckpreis, rabattsatz, bruttopreis, rabatt, einkaufspreis;
      if (tfStueckzahl.getText().equals("") || tfStueckpreis.equals("")
          || tfRabatt.equals("")) {
          lblAusgabe.setText("Die Eingaben sind unvollständig!");
      } else {
          stueckzahl = Integer.parseInt(tfStueckzahl.getText());
          stueckpreis = Double.parseDouble(tfStueckpreis.getText());
          rabattsatz = Double.parseDouble(tfRabatt.getText());
          bruttopreis = stueckzahl * stueckpreis;
          rabatt = bruttopreis * rabattsatz / 100;
          einkaufspreis = bruttopreis - rabatt;
          DecimalFormat f = new DecimalFormat("#0.00");
          lblAusgabe = new JLabel("");
          lblAusgabe.setText("Der Einkaufspreis beträgt " + f.format(einkaufspreis) + " €");
      }
      tfStueckzahl.requestFocus();
      tfStueckzahl.selectAll();
      }

  }

 

[Hellosager]

Du initialisierts dein Ausgabelabel jetzt zwei mal.
Das erste mal im Konstruktor, was soweit richtig ist.
Und das zweite mal in deiner Methode berechnen direkt vor der Ausgabe im Label.
Da du vor der Ausgabe im Label sozusagen ein komplett anderes JLabel erstellst, hat dieses auche keine Informationen mehr über Layout, wo es sich befindet und wurde auch nicht mit add(JLabel) erneut hinzugefügt. Es wird schlicht nicht angezeigt.
Langes Gerede, einfach

lblAusgabe = new JLabel("");

in der Methode berechnen entfernen (nur da!) und es klappt.

 

[Damon_179]

Ah okay stimmt, danke jetzt habe ich es!