Taschenrechner

[Krypthor]

Hi,
ich hab aus spaß mal einen kleinen Rechner Programmiert der (erstmal) nur Addiert.
Leider bin ich auf ein kleines Problem gestoßen und ich weis nicht wie ich das lösen kann.

Hier der Code:

public class Rechner extends JFrame implements ActionListener {
	
	
	public Rechner() 
	{
	//Plus Knopf
	JButton plus = new JButton("+");
	plus.setBounds(335,525,60,24);
	
		
	//Erstes Eingabefeld	
	JTextField eingabe1 = new JTextField();
	eingabe1.setBounds(300,525,30,25);
	add(eingabe1);
	eingabe1.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	
	//Zweites Eingabefeld
	JTextField eingabe2 = new JTextField();
	eingabe2.setBounds(400,525,30,25);
	add(eingabe2);
	eingabe2.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	setLayout(null);
	setTitle("Rechner");	
    setSize(800,600);
    setLocation(300,300);
    setResizable(false);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    plus.addActionListener(this);
    add(plus);
    setVisible(true);
    
    int z1 = Integer.parseInt(eingabe1.getText());
    int z2 = Integer.parseInt(eingabe2.getText());
    int eins = z1 + z2; 
    String aus = Integer.toString(eins);
    
    
	}
    	   
    public void actionPerformed (ActionEvent e)
    {
    	//Ausgabefeld
    	final JTextField ausgabe = new JTextField();
    	ausgabe.setBounds(450,525,30,25);
    	add(ausgabe);
    	ausgabe.setText(aus); // Hier liegt das Problem
    }
}

Er schreibt den String einfach nicht in das JTextField, bzw er lädt den String "aus" nicht zur
public void. Ich kenne mich leider noch nicht so gut mit Java aus und gegooglet hab ich, leider nichts gefunden. Es wäre auch nicht schlimm wenn er den String (also die Lösung) "aus" nicht in einem TextField ausgibt.

Würde mich über Hilfe freuen,

Mfg,
Krypthor

 

[Michael...]

Ist der Code überhaupt ausführbar? Spätestens in Zeile 54 sollte eine Exception fliegen.

Dann würde ich mir noch überlegen, ob es wirklich notwendig ist bei jedem Button Klick ein neues Textfeld zur Ausgabe zu erzeugen.

 

[Krypthor]

Sorry hab mich falsch ausgedrückt, ist nicht ausführbar, das möchte ich ja ändern
Und wie viel schwieriger ist es das ohne den Button zu programmieren?

 

[Michael...]

Warum dann soviel Code, wenn er dann doch nicht ausführbar ist ;-)
Klein anfangen:
Erst das Layout mit den Eingabefeldern, dem Button und dem Ausgabefeld programmieren. Wenn das passt, ActionListener an den Button und die Werte der Eingabefelder per System.out und evtl. im Ausgabefeld ausgeben...

 

[SlaterB]

String aus muss als Instanzattribut außerhalb des Konstruktors definiert werden

 

[Krypthor]

Hab ich gemacht, funktioniert aber immer noch nicht, woran kann das liegen?
Hier der aktuelle Code:

public class Rechner extends JFrame implements ActionListener {
	
	String aus;
	JTextField ausgabe;
	int eins;
	public Rechner() 
	{
	//Plus Knopf
	JButton plus = new JButton("+");
	plus.setBounds(335,525,60,24);
	
		
	//Erstes Eingabefeld	
	JTextField eingabe1 = new JTextField();
	eingabe1.setBounds(300,525,30,25);
	add(eingabe1);
	eingabe1.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	
	//Zweites Eingabefeld
	JTextField eingabe2 = new JTextField();
	eingabe2.setBounds(400,525,30,25);
	add(eingabe2);
	eingabe2.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	//Ausgabe
	final JTextField ausgabe = new JTextField();
	ausgabe.setBounds(450,525,30,25);
	
	setLayout(null);
	setTitle("Rechner");	
    setSize(800,600);
    setLocation(300,300);
    setResizable(false);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    plus.addActionListener(this);
    add(plus);
    add(ausgabe);
    setVisible(true);
    
    int z1 = Integer.parseInt(eingabe1.getText());
    int z2 = Integer.parseInt(eingabe2.getText());
    int eins = z1 + z2;
    String aus = new String(Integer.toString(eins));
	}
	
    public void actionPerformed (ActionEvent e)
    {
    	ausgabe.setText(aus);
    }
}

was hab ich falsch gemacht?

 

[SlaterB]

in Zeile 64 definierst du eine gleichnamige lokale Variable, das Instanzattribut bleibt null

 

[Krypthor]

    int z1 = Integer.parseInt(eingabe1.getText());
    int z2 = Integer.parseInt(eingabe2.getText());
    int eins = z1 + z2;
    String aus = Integer.toString(eins);
	}
	
    public void actionPerformed (ActionEvent e)
    {
    	ausgabe.setText(aus);
    }

Habs jetzt so geändert, passiert trozdem nichts.
Kann es vielleicht sein das er das nicht ins Textfield schreibt?
Danke schonmal für die bisherigen Antworten aber bitte erklärt mir genauer was ich machen muss.

 

[SlaterB]

new String() war wirklich übel, aber eine lokale Variable ist es immer noch, du musst das Wort String am Anfang entfernen

ist das wirklich ein sinnvolles Vorgehen für dich, jede kritische Code-Zeilen eine halbe Stunde übers Forum zu klären?
ohne Grundkenntnisse zu Variablen, Klassen usw. kannst du kaum effektiv programmieren

ach ja, die Berechnung sollte besser in den Listener, denn direkt im Konstruktor wird zeitlich in den Textfeldern noch nichts drinstehen,
wenn dann im ActionListener, dann könnte es auch wieder eine lokale Variable werden

 

[njans]

Hast du denn das gemacht, was SlaterB vorgelschalgen hat?

Du hast da:

 final JTextField ausgabe = new JTextField();

Damit überschreibst du den Sichtbarkeitsbereich der Instanzvariable ausgabe. Daher ist die immer null
nehm mal das "final JTextField" weg.

 

[Krypthor]

Ah sehr gut jetzt kommt schonmal keine Fehlermeldung mehr.
Entschuldigt bitte das ich das so durcheinander geschmissen habe.

public class Rechner extends JFrame implements ActionListener {
	
	String aus;
	JTextField ausgabe;
	int eins;
	public Rechner() 
	{
	//Plus Knopf
	JButton plus = new JButton("+");
	plus.setBounds(335,525,60,24);
	
		
	//Erstes Eingabefeld	
	JTextField eingabe1 = new JTextField();
	eingabe1.setBounds(300,525,30,25);
	add(eingabe1);
	eingabe1.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	
	//Zweites Eingabefeld
	JTextField eingabe2 = new JTextField();
	eingabe2.setBounds(400,525,30,25);
	add(eingabe2);
	eingabe2.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	//Ausgabe
	ausgabe = new JTextField();
	ausgabe.setBounds(450,525,30,25);
	
	setLayout(null);
	setTitle("Rechner");	
    setSize(800,600);
    setLocation(300,300);
    setResizable(false);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    plus.addActionListener(this);
    add(plus);
    add(ausgabe);
    setVisible(true);
    
    int z1 = Integer.parseInt(eingabe1.getText());
    int z2 = Integer.parseInt(eingabe2.getText());
    int eins = z1 + z2;
    aus = Integer.toString(eins);
	}
	
    public void actionPerformed (ActionEvent e)
    {
    	ausgabe.setText(aus);
    }
}

Wenn ich jetzt auf den button drücke erscheint leider immer noch nichts im Textfield.
Aber eigentlich müsste doch jetzt alles klappen?

 

[SlaterB]

ach ja, die Berechnung sollte besser in den Listener, denn direkt im Konstruktor wird zeitlich in den Textfeldern noch nichts drinstehen,
wenn dann im ActionListener, dann könnte es auch wieder eine lokale Variable werden

im Moment Exception beim Parsen des String "" im Konstruktor, String aus bleibt null, keine Anzeige

 

[Krypthor]

Ich habs! Danke euch!

 

[Krypthor]

Ich hab doch nochmal ein kleines Problem^^
und zwar hab ich jetzt +,-,* und / hinzugefügt, aber das Ergebnis wird immer "0" obwohl
ich die Rechnung in den Listener gestellt habe.

public class Rechner extends JFrame implements ActionListener {
	
	String aus;
	String aus2;
	String aus3;
	String aus4;
	JTextField eingabe1;
	JTextField eingabe2;
	JTextField ausgabe;
	int z1;
	int z2;
	int eins;
	int zwei;
	int drei;
	int vier;
	JButton plus;
	String[] rechenart = {"+", "-", "*", ":"}; 
	JComboBox wahl = new JComboBox(rechenart);
	
	
	public Rechner() 
	{
	
	//Plus Knopf
	plus = new JButton("=");
	plus.setBounds(430,525,60,24);
	
		
	//Erstes Eingabefeld	
	eingabe1 = new JTextField();
	eingabe1.setBounds(300,525,30,25);
	add(eingabe1);
	eingabe1.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	
	//Zweites Eingabefeld
	eingabe2 = new JTextField();
	eingabe2.setBounds(392,525,30,25);
	add(eingabe2);
	eingabe2.addKeyListener(new KeyAdapter() {
        public void keyTyped(KeyEvent e) {
          char c = e.getKeyChar();      
          if (!((Character.isDigit(c) ||
             (c == KeyEvent.VK_BACK_SPACE) ||
             (c == KeyEvent.VK_DELETE)))) {
               getToolkit().beep();
               e.consume();
          }
        }
      });
	
	//Ausgabe
	ausgabe = new JTextField();
	ausgabe.setBounds(500,525,30,25);
	
	setLayout(null);
	setTitle("Rechner");	
    setSize(800,600);
    setLocation(300,300);
    setResizable(false);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    plus.addActionListener(this);
    add(plus);
    add(ausgabe);
    wahl.setBounds(335,525,50,24);
    add(wahl);
    setVisible(true);
    
    
	}
	
    public void actionPerformed (ActionEvent e)
    {
    	z1 = Integer.parseInt(eingabe1.getText());
    	z2 = Integer.parseInt(eingabe2.getText());
    	eins = z1 + z2;
    	zwei = z1 - z2;
    	drei = z1 * z2;
    	vier = z1 / z2;
    	
    	if(wahl.getSelectedIndex() == 0) 
        	aus = Integer.toString(eins);
        	ausgabe.setText(aus);
        if(wahl.getSelectedIndex() == 1);
        	aus2 = Integer.toString(zwei);
        	ausgabe.setText(aus2);
        if(wahl.getSelectedIndex() == 2);
        	aus3 = Integer.toString(drei);
        	ausgabe.setText(aus3);
        if(wahl.getSelectedIndex() == 3);
        	aus4 = Integer.toString(vier);
        	ausgabe.setText(aus4);
    }
}

 

[SlaterB]

nicht immer, bei 100 und 5 kommt 20 raus, oder?

Semikolon ; direkt in der if-Zeile ist ganz schlecht,
zudem immer Klammern verwenden! sonst wird maximal der nächste Befehl dem if zugeordnet,
Einrückung hat keinerlei Bedeutung außer Verschleierung von Fehlern,
wenn richtig gemacht natürlich begrüßenswert, aber überlasse das besser nur IDEs wie Eclipse,
dann kann man Fehler wie falsche ifs gleich erkennen, so sieht es im Moment aus:

        if (wahl.getSelectedIndex() == 0) aus = Integer.toString(eins);
        ausgabe.setText(aus);
        if (wahl.getSelectedIndex() == 1) ;
        aus2 = Integer.toString(zwei);
        ausgabe.setText(aus2);
        if (wahl.getSelectedIndex() == 2) ;
        aus3 = Integer.toString(drei);
        ausgabe.setText(aus3);
        if (wahl.getSelectedIndex() == 3) ;
        aus4 = Integer.toString(vier);
        ausgabe.setText(aus4);

im Moment wird IMMER als letztes Zeile 102 (bzw. 11 in meinem Code-Block),
> ausgabe.setText(aus4);
ausgeführt, und dank schlechtem if auch Zeile 101 davor

 

[Michael...]

Ich würde für den Anfang weniger Wert auf das Layout legen und versuchen mit weniger überflüssigen Hilfsvariablen aus zu kommen. Ab und zu ein System.out.println(...); mit den interessanten Werten hilft Fehler aufzuspüren.

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JTextField;
public class Rechner extends JFrame {
	
	private JTextField field1, field2;
	private JComboBox combo;
	private JButton button;
	private JLabel label;
	
	public Rechner() {
		setLayout(new GridLayout(5,1));
		field1 = new JTextField();
		field2 = new JTextField();
		combo = new JComboBox(new String[] {"+", "-", "*", "/"});
		button = new JButton("=");
		label = new JLabel();
		add(field1);
		add(combo);
		add(field2);
		add(button);
		add(label);
		
		button.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				int value1;
				int value2;
				try {
					value1 = Integer.parseInt(field1.getText());
					value2 = Integer.parseInt(field2.getText());
				}
				catch(NumberFormatException exc) {
					System.out.println("Es wurde ein ungültiger Wert eingegen!");
					label.setText("ERROR");
					exc.printStackTrace();
					return;
				}
				String ergebnisString = null;
				switch (combo.getSelectedIndex()) {
				case 0: ergebnisString = String.valueOf(value1 + value2); break;
				case 1: ergebnisString = String.valueOf(value1 - value2); break;
				case 2: ergebnisString = String.valueOf(value1 * value2); break;
				case 3: ergebnisString = String.valueOf(value1 / value2); break;
				default: ergebnisString = "ERROR";
				}
				System.out.println(ergebnisString);
				label.setText(ergebnisString);
			}
		});
	}
	
	public static void main(String[] args) {
		JFrame frame = new Rechner();
		frame.setBounds(0, 0, 150, 200);
		frame.setLocationRelativeTo(null);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
	}
}

 

[Krypthor]

Hier nochmal eine Frage, ich möchte den Satz des Pythagoras berechnen, habe 3 felder erstellt für a, b und c und je das welches freigelassen wird soll berechnet werden. (das das nur bei rechtwinkeligen dreiecken geht lasse ich fürs erste außer acht, sonst wirds zu komplitziert)
Nur leider egal was ich eingebe und welches ich freilasse wird immer c ausgerechnet. Das liegt wahrscheinlich daran,
dass c zuerst abgefragt wird?! Aber wie kann ich das ändern?

 add(plus3);
    plus3.addActionListener (new ActionListener() {
    	public void actionPerformed (ActionEvent e)
    	{
    		a1 = ausgabea.getText();
    		b1 = ausgabeb.getText();
    		c1 = ausgabec.getText();
    		
    		if(c1.equals("")) {
    			a2 = Double.parseDouble(a1);
    			b2 = Double.parseDouble(b1);
    			a3 = a2 * a2;
    			b3 = b2 * b2;
    			c4 = a3 + b3;
    			c5 = Math.sqrt(c4);
    			c5 = Math.round(c5*100)/100.0;
    			c = String.valueOf(c5);
    			ausgabec.setText(c);
    			System.out.print("c erfolgreich berechnet!");
    		}
    		else if(a1.equals("")) {
    			b2 = Double.parseDouble(b1);
        		c2 = Double.parseDouble(c1);
        		c3 = c2 * c2;
        		b3 = b2 * b2;
        		a4 = c3 - b3;
        		a5 = Math.sqrt(a4);
        		a5 = Math.round(a5*100)/100.0;
        		a = String.valueOf(c5);
        		ausgabec.setText(a);
        		System.out.print("a erfolgreich berechnet!");
    		}
    		else if(b1.equals("")) {
				a2 = Double.parseDouble(b1);
				c2 = Double.parseDouble(c1);
				a3 = a2 * a2;
				c3 = c2 * c2;
				b4 = c3 - a3;
				b5 = Math.sqrt(b4);
				b5 = Math.round(b5*100)/100.0;
				b = String.valueOf(b5);
				ausgabec.setText(b);
				System.out.print("b erfolgreich berechnet!");
    		}
    		
    		
    	}
    });

 

[Kevin94]

Wenn man mal davon absieht, dass das alles Spagetti-Code ist, hast du den klassischen copy&paste Fehler. Du schreibst das Ergebnis immer in [c]ausgabec[/c].

 

[Krypthor]

oh ^^
naja ich habs geändert, aber wisst ihr vielleicht wie man es so erstellen kann, das nur rechtwinkelige dreiecke berechnet werden?

veränderter code:

    add(plus3);
    plus3.addActionListener (new ActionListener() {
    	public void actionPerformed (ActionEvent e)
    	{
    		a1 = ausgabea.getText();
    		b1 = ausgabeb.getText();
    		c1 = ausgabec.getText();
    		
    		if(c1.equals("")) {
    			a2 = Double.parseDouble(a1);
    			b2 = Double.parseDouble(b1);
    			a3 = a2 * a2;
    			b3 = b2 * b2;
    			c4 = a3 + b3;
    			c5 = Math.sqrt(c4);
    			c5 = Math.round(c5*100)/100.0;
    			c = String.valueOf(c5);
    			ausgabec.setText(c);
    			System.out.print("c erfolgreich berechnet!");
    		}
    		else if(a1.equals("")) {
    			b2 = Double.parseDouble(b1);
        		c2 = Double.parseDouble(c1);
        		c3 = c2 * c2;
        		b3 = b2 * b2;
        		a4 = c3 - b3;
        		a5 = Math.sqrt(a4);
        		a5 = Math.round(a5*100)/100.0;
        		a = String.valueOf(a5);
        		ausgabea.setText(a);
        		System.out.print("a erfolgreich berechnet!");
    		}
    		else if(b1.equals("")) {
				a2 = Double.parseDouble(a1);
				c2 = Double.parseDouble(c1);
				a3 = a2 * a2;
				c3 = c2 * c2;
				b4 = c3 - a3;
				b5 = Math.sqrt(b4);
				b5 = Math.round(b5*100)/100.0;
				b = String.valueOf(b5);
				ausgabeb.setText(b);
				System.out.print("b erfolgreich berechnet!");
    		}
    		
    		
    	}
    });

 

[Kevin94]

Ich versteh deine Frage nicht ganz. Du gibst dem Programm zwei Zahlen und verlangst dann von dem Programm das es deine Gedanken liest ob du mit diesen zwei Zahlen wirklich zwei Seiten eines Rechtwinkligen Dreiecks gemeint hast, oder wie?

 

[TKausL]

Ich versteh deine Frage nicht ganz. Du gibst dem Programm zwei Zahlen und verlangst dann von dem Programm das es deine Gedanken liest ob du mit diesen zwei Zahlen wirklich zwei Seiten eines Rechtwinkligen Dreiecks gemeint hast, oder wie?

Frage ich mich auch ???:L

Zur Info: Bei einem Dreieck brauchst du Mindestens 3 Werte (Seitenlänge, Winkel) um die restlichen Werte zu errechnen, und dadurch dann herauszufinden ob das Dreieck rechtwinklig ist.

 

[Krypthor]

nein ich möchte einen Code erstellen, das das Programm dann anhand der zwei eingetragenen Werte erkennt ob das Dreieck rechtwinkelig ist.

 

[Gast2]

Anhand von zwei Seitenlängen kannst du nicht entscheiden ob ein Rechteck rechtwinklig ist oder nicht.

 

[Krypthor]

es soll ja im nachhinein bestimmt werden, wenn die dritte Seitenlänge schon ausgerechnet ist

 

[TKausL]

Du kannst aber die Dritte Seitenlänge nicht ausrechnen.

Nochmal: Du brauchst MINDESTENS 3 Werte zum berechnen der anderen.
Bei Pythagoras weißt du 2 Längen UND du weißt, dass das Dreieck rechtwinklig ist, hast also den dritten wert von 90°.
Wie also willst du jetzt die 3. Länge errechnen, wenn du garnicht weißt, ob es rechtwinklig ist? Pythagoras kann nur auf Rechtwinklige Dreiecke angewendet werden.

 

[Krypthor]

Stimmt, klingt logisch die Sache muss ich nochmal überdenken^^