Problem beim Währungsrechner - Bitte um Hifle

[Cyberbob_at_tot]

Guten morgen,
lernen gerade in der Schule Java,
und sollen einen Währungsrechner machen.

Dabei habe ich aber 2 Probleme,
das erste wie kann ich beim Borderlayout 2 Textfields in North packen ?
Und beim ausführen, und dann beim berechnen, oder da wo er eigentlich rechnen sollte bekomme ich nur Fehler ohne ende, leider weis ich nicht wieso ?

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.text.*;

public class Rechner extends JPanel implements ActionListener 
{
	
	private JButton b1;
	private JComboBox auswahlvor;
	private JComboBox auswahlnach;
	private JTextField eingabe;
	private JTextField ausgabe;
	
	public Rechner()
		{
		setLayout(new BorderLayout());
		
		add(new JLabel("   >-->-->--> "), BorderLayout.CENTER);
		
		JTextField eingabe = new JTextField();
		add(eingabe, BorderLayout.NORTH);
		
		JTextField ausgabe = new JTextField();
		add(ausgabe, BorderLayout.NORTH);
		
		String[] items = {"Euro","Dollar"};
		JComboBox auswahlvor  = new JComboBox(items);
		JComboBox auswahlnach = new JComboBox(items);
		auswahlvor.setSelectedIndex(0);
		auswahlnach.setSelectedIndex(0);
		add(auswahlvor, BorderLayout.WEST);
		add(auswahlnach, BorderLayout.EAST);
		
		JButton b1 = new JButton("Umrechnen");
		b1.addActionListener(this);
		add(b1, BorderLayout.SOUTH);
		}
		
	public void actionPerformed(ActionEvent e)
		{
		double betrag, ergebnis;		 
		Double fc;					// Deklaration der Hüllklassenvariable
		String str;					// Deklaration einer Stringvariable
		str = eingabe.getText();	// Auslesen des Textfeldes 
		fc = new Double(str);		// Wert wird Hüllklasse übergeben
		betrag = fc.floatValue();	// Wert(string) wird in Float gewandelt
		DecimalFormat f = new DecimalFormat ("###,##0.00"); //Ausgabeformat
		
		if(auswahlvor.getSelectedIndex()==0 & auswahlnach.getSelectedIndex()==1)
			{
			ergebnis = betrag / (double) 2;
			ausgabe.setText((new Double(ergebnis)).toString());
			}
		if(auswahlvor.getSelectedIndex()==1 & auswahlnach.getSelectedIndex()==0)
			{
			ergebnis = betrag * (double) 2;
			ausgabe.setText((new Double(ergebnis)).toString());
			}
		if(auswahlvor.getSelectedIndex()==1 & auswahlnach.getSelectedIndex()==1)
			{
			ausgabe.setText("Das geht nicht");
			}
		if(auswahlvor.getSelectedIndex()==0 & auswahlnach.getSelectedIndex()==0)
			{
			ausgabe.setText("Das geht nicht");
			}
		}
	
	
	public static void main(String[] args)
		{
		JFrame frame = new JFrame("Währungsrechner");
		frame.getContentPane().add(new Rechner());
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setSize(200, 120);
		frame.setVisible(true);
		}
}

Danke für eure Hilfe!

Cyberbob

 

[bygones]

wenn du im mehrere Elemente in NORTH willst musst du dort wieder einen neuen Container reinsetzen (also neues JPanel z.b.) in diesem JPanel kannst du dann die Elemente so reinsetzen wie du willst.

auf den ersten Blick:

   private JTextField eingabe;

Im Knostruktor definierst du eine lokale Variable eingabe. Dadurch ist die Instanzvariable oben nie initialisiert und wirft eine NullPointerException in der actionPerformed Methode

JTextField eingabe = new JTextField();

 

[Oni]

du kannst nur eine komponente pro Borderlayout-richtung packen.

deshalb solltest du ein neues panel erstellen und die beiden textfelder auf das panel packen. das panel addes du dann nach north. ( geiles deutsch )

was für eine fehlermeldung kommt den?

 

[Cyberbob_at_tot]

sorry hat ein bischen länger gedauert!

Bekomme beim Ausführen, und wenn ich dann rechnen will, die Fehlermeldung in der DosBox kann ich leider nicht hier reinkopieren. Ist auch zu vie und sagt mir gerade nicht viel!

 

[The_S]

Doch, DOS-Box in den gewünschten Stellen markieren (ganz normal mit der Maus) und Enter drücken => kopiert! Ansonsten schreib die obersten 5 Zeilen hier rein.

 

[Cyberbob_at_tot]

mmm wußte ich noch garnicht, in der normales CMD geht das auch, starte das Programm aber aus JCreater herraus, da gehts nicht!

Hier aber nun ein Screenshot

bin nicht so der hellste in Java, könnt ihr mir Lösungsvorschläge geben was das neue Panel und die Deklaration angeht ?

Besten dank!

Cyberbob

 

[Oni]

gucks dir mal an und frag wenn du etwas von den änderungen nicht verstehst.

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.text.*;

public class Rechner extends JPanel implements ActionListener
{

   private JButton b1;
   private JComboBox auswahlvor;
   private JComboBox auswahlnach;
   private JTextField eingabe;
   private JTextField ausgabe;

   public Rechner()
      {
      setLayout(new BorderLayout());

      add(new JLabel("   >-->-->--> "), BorderLayout.CENTER);

      JPanel p = new JPanel();          // neues panel

      eingabe = new JTextField(5);//geändert länge textfield
      p.add(eingabe);                    //geändert textfield aufs panel

      ausgabe = new JTextField(5);//geändert länge textfield
      p.add(ausgabe);                    //geändert textfield aufs panel

      add(p, BorderLayout.NORTH); // neues panel hinzufügen

      String[] items = {"Euro","Dollar"};
      auswahlvor  = new JComboBox(items);  //geändert initialisierung ist schon oben
      auswahlnach = new JComboBox(items);  //geändert initialisierung ist schon oben
      auswahlvor.setSelectedIndex(0);
      auswahlnach.setSelectedIndex(0);
      add(auswahlvor, BorderLayout.WEST);
      add(auswahlnach, BorderLayout.EAST);

      JButton b1 = new JButton("Umrechnen");
      b1.addActionListener(this);
      add(b1, BorderLayout.SOUTH);
      }

   public void actionPerformed(ActionEvent e)
      {
      double betrag, ergebnis;
      Double fc;               // Deklaration der Hüllklassenvariable
      String str = "";               // Deklaration einer Stringvariable
      str = eingabe.getText();   // Auslesen des Textfeldes
      fc = new Double(str);      // Wert wird Hüllklasse übergeben
      betrag = fc.floatValue();   // Wert(string) wird in Float gewandelt
      DecimalFormat f = new DecimalFormat ("###,##0.00"); //Ausgabeformat

      if(auswahlvor.getSelectedIndex()==0 & auswahlnach.getSelectedIndex()==1)
         {
         ergebnis = betrag / (double) 2;
         ausgabe.setText((new Double(ergebnis)).toString());
         }
      if(auswahlvor.getSelectedIndex()==1 & auswahlnach.getSelectedIndex()==0)
         {
         ergebnis = betrag * (double) 2;
         ausgabe.setText((new Double(ergebnis)).toString());
         }
      if(auswahlvor.getSelectedIndex()==1 & auswahlnach.getSelectedIndex()==1)
         {
         ausgabe.setText("Das geht nicht");
         }
      if(auswahlvor.getSelectedIndex()==0 & auswahlnach.getSelectedIndex()==0)
         {
         ausgabe.setText("Das geht nicht");
         }
      }


   public static void main(String[] args)
      {
      JFrame frame = new JFrame("Währungsrechner");
      frame.getContentPane().add(new Rechner());
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.setSize(200, 120);
      frame.setVisible(true);
      }
}

 

[The_S]

Du kannst im "cmd von JCreator" genau auf die selbe weiße kopieren, weil der JCreator nichts anderes macht, als die Konsole aufzurufen und das Ding dann da ganz normal auszuführen.

 

[Cyberbob_at_tot]

Cool, wenn man es sieht im Code ist echt einfach zu verstehen. Ist mir jetzt alles klar. Besten dank für eure Hilfe.

 

[SnooP]

Relevant ist meist vor allem die erste Zeile bzw. die erste Exception - in deinem Fall NullPointerException, d.h. Zugriff auf ein Objekt, was nicht existiert... die Antwort wurde dir auch schon gegeben... - du hast doppelt deklariert.
Wenn du eine Instanzvariable eingabe oder ausgabe etc. machst, darfst du im Konstruktor nicht nochmal:
JTextField eingabe = ... machen - dadurch erzeugst du, wie oben schon erwähnst eine lokale Variable eingabe die nur für den Konstruktor bzw. die Methode selbst gilt... in deinem Fall möchtest du aber, dass im Konstruktor die Instanzvariablen gesetzt werden, sprich: this.eingabe = new ...
Man sollte sich an der Stelle auch das this. angwöhnen, weil es verdeutlicht, dass man auf eine Instanzvariable zugreifen möchte... hat dann in Umgebungen wie Eclipse auch den Vorteil, dass die vorhandenen Instanzvariablen in einer Combobox angezeigt und ausgewählt werden können

Merke: NullPointer -> Objekt nicht da... Zeile angucken wo's auftritt und mal schaun wo das Objekt instanziert (new...) wird.