Probleme mit else if

[Mr.Ich]

Hey,
also ich habe mich jetzt seid längerem nicht mehr mit Java beschäftigt, möchte nun aber wieder anfangen. Zur Übung wollte ich einen einfachen Taschenrechner programmieren. Ich bin allerdings noch nicht weit gekommen, da sofort Probleme auftraten. Ich verstehe einfach nicht warum eclipse hier das "else" von "else if" unterschlängelt:

import javax.swing.JOptionPane;

public class Test {
public static void main(String[] args) {


JOptionPane.showMessageDialog(null, "Hallo, dies ist ein Taschenrechner.");
String Operator = JOptionPane.showInputDialog("Welchen Rechenoperator benötigen sie? (+ - * /)");	

if (Operator.equals("+"));
{
String AFrage1 = JOptionPane.showInputDialog("Geben Sie jetzt die erste Zahl ein.");	
String AFrage2 = JOptionPane.showInputDialog("Geben Sie jeztz die zweite Zahl ein.");
int AZahl1 = Integer.parseInt(AFrage1);
int AZahl2 = Integer.parseInt(AFrage2);
JOptionPane.showMessageDialog(null,  AZahl1 + AZahl2);
}

	
else if (Operator.equals("-"))
{
	
}



}
}

Hoffe ihr könnt mir helfen

 

[bone2]

if (Operator.equals("+"));

da is ein semikolon zuviel

 

[Mr.Ich]

Achjaa klar! xD
Manchmal hat man aber auch echt ein Brett vorm Kopf XD





Ich habe übrigens schon wieder ein neues Problem:

else if (Operator.equals("/"))
{
String DFrage1 = JOptionPane.showInputDialog("Geben Sie jetzt ihre erste Zahl ein.");
String DFrage2 = JOptionPane.showInputDialog("Geben Sie jetzt ihre zweite Zahl ein.");
Double DZahl1 = Double.parseDouble(DFrage1);
Double DZahl2 = Double.parseDouble(DFrage1);
JOptionPane.showMessageDialog(null, DZahl1/DZahl2);
}

Egal was für Zahlen man eingibt, es kommt immer 1.0 raus. ;(

 

[pl4gu33]

probiers mal so

Double DZahl2 = Double.parseDouble(DFrage2);

 

[MiMij]

Und gewoehne dir direkt die Code Formatierung an. Also den Code einruecken, dann hat man direkt ne bessere Uebersicht.
Variablen schreibt man im uebrigen in Java klein.

 

[SlaterB]

Variablen unbedingt klein schreiben

der Fehler mit den falschen Variablen könnte gar nicht auftreten, wenn man eines der Grundmittel der Programmierung,
Untermethoden, verwendet:

double d2 = frageDoubleAb("Text");
// Methode erstellt JOptionPane, übernimmt das Parsen, gibt double zurück

 

[MiMij]

Achso was mir noch auffaellt, aber nur eine Kleinigkeit, das abfragen der Zahlen (Dialog und auslesen der Zahl) geschieht immer unabhaenig vom Operator.
D.h. du kannst diese Abfragen und das Auslesen auch schon vor dem If machen. Somit hast du keinen doppelten unnuetzen Code und umso weniger Zeilen Code umso weniger Fehler

Edit:
Wenn du wirklich fast wieder bei 0 anfangen musst, schreibe doch den Taschenrechner erst nur mit der Plus Operation. Funktioniert diese, kannst du deinen Code weiter ausbreiten mit Minus. Dann wieder testen usw. Erst einfachen Code schreiben der funktioniert und dann weiter ausbreiten

 

[Mr.Ich]

Das war ja schon wieder so ein kleiner sch*** Fehler den ich da hatte xD
Und ich guck ne halbe Stunde lang wie blöd auf den Script und find den
Fehler einfach nicht xD

Und danke für eure zahlreichen Tipps. Werde sie beherzigen

 

[Mr.Ich]

Also mein Taschenrechner ist jetzt fertig. Nur eine letzde Sache fehlt noch. Ich würde gerne haben, dass wenn man mit der
Rechnung fertig ist und der einem das Ergebnis gesagt hat, dass man dann automatisch wieder zum Anfang kommt, zwei neue
Zahlen eingeben kann und er es wieder rechnet. Unendlich oft. Also im Prinzip einen Neustart Knopf. Dazu habe ich allerdings nicht die richtige Schleife gefunden.
Hoffen ihr könnt mir helfen.


PS: Das ist der Taschenrechner. Bin immer offen für Verbesserungvorschläge

import javax.swing.JOptionPane;

public class Taschenrechner {
public static void main(String[] args) {


JOptionPane.showMessageDialog(null, "Hallo, dies ist ein Taschenrechner.");
String Operator = JOptionPane.showInputDialog("Welchen Rechenoperator benötigen sie? (+ - * /)");	

if (Operator.equals("+"))
{
String AFrage1 = JOptionPane.showInputDialog("Geben Sie jetzt ihre erste Zahl ein.");	
String AFrage2 = JOptionPane.showInputDialog("Geben Sie jeztz ihre zweite Zahl ein.");

try {
Double AZahl1 = Double.parseDouble(AFrage1);
Double AZahl2 = Double.parseDouble(AFrage2);
JOptionPane.showMessageDialog(null,  AZahl1+AZahl2);
}
 
catch (Exception e) 
{
JOptionPane.showMessageDialog(null, "Geben Sie zwei ZAHLEN ein!");
}

}

	
else if (Operator.equals("-"))
{
String SFrage1 = JOptionPane.showInputDialog("Geben Sie jetzt ihre erste Zahl ein.");
String SFrage2 = JOptionPane.showInputDialog("Geben Sie jetzt ihre zweite Zahl ein.");
try {
	double SZahl1 = Double.parseDouble(SFrage1);
	double SZahl2 = Double.parseDouble(SFrage2);
	JOptionPane.showMessageDialog(null, SZahl1 - SZahl2);
} 
catch (Exception e) 
{
JOptionPane.showMessageDialog(null, "Geben Sie zwei ZAHLEN ein!");	
}
}


else if (Operator.equals("*"))
{
String MFrage1 = JOptionPane.showInputDialog("Geben Sie jetzt ihre erste Zahl ein.");
String MFrage2 = JOptionPane.showInputDialog("Geben Sie jetzt ihre zweite Zahl ein.");
try {
	double MZahl1 = Double.parseDouble(MFrage1);
	double MZahl2 = Double.parseDouble(MFrage2);
	JOptionPane.showMessageDialog(null, MZahl1 * MZahl2);
} 
catch (Exception e) 
{
JOptionPane.showMessageDialog(null, "Geben Sie zwei ZAHLEN ein!");
}
}

else if (Operator.equals("/"))
{
String DFrage1 = JOptionPane.showInputDialog("Geben Sie jetzt ihre erste Zahl ein.");
String DFrage2 = JOptionPane.showInputDialog("Geben Sie jetzt ihre zweite Zahl ein.");
try {
	double DZahl1 = Double.parseDouble(DFrage1);
	double DZahl2 = Double.parseDouble(DFrage2);
	JOptionPane.showMessageDialog(null, DZahl1 / DZahl2);
} 
catch (Exception e) 
{
JOptionPane.showMessageDialog(null, "Geben Sie zwei ZAHLEN ein!");
}
}

else
{
JOptionPane.showMessageDialog(null, "Das ist keiner der gennanten Rechenoperatoren!");
}







}
}

 

[njans]

Stichwort: Schleife

 

[Mujahiddin]

import javax.swing.JOptionPane;


public class Taschenrechner {
	
	public static void main(String[] args) {
		JOptionPane.showMessageDialog( null, "Hallo, dies ist ein Taschenrechner." );
		do
			invokeTaschenrechner();
		while( JOptionPane.showConfirmDialog( null, "Möchten Sie fortführen?", "Fortführen", JOptionPane.YES_NO_OPTION ) == 0 );
	}
	
	private static void invokeTaschenrechner() {
		String Operator = JOptionPane.showInputDialog( "Welchen Rechenoperator benötigen sie? (+ - * /)" );
		
		if( Operator.equals( "+" ) ) {
			String AFrage1 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre erste Zahl ein." );
			String AFrage2 = JOptionPane.showInputDialog( "Geben Sie jeztz ihre zweite Zahl ein." );
			
			try {
				Double AZahl1 = Double.parseDouble( AFrage1 );
				Double AZahl2 = Double.parseDouble( AFrage2 );
				JOptionPane.showMessageDialog( null, AZahl1 + AZahl2 );
			}
			
			catch( Exception e ) {
				JOptionPane.showMessageDialog( null, "Geben Sie zwei ZAHLEN ein!" );
			}
			
		}
		
		else if( Operator.equals( "-" ) ) {
			String SFrage1 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre erste Zahl ein." );
			String SFrage2 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre zweite Zahl ein." );
			try {
				double SZahl1 = Double.parseDouble( SFrage1 );
				double SZahl2 = Double.parseDouble( SFrage2 );
				JOptionPane.showMessageDialog( null, SZahl1 - SZahl2 );
			} catch( Exception e ) {
				JOptionPane.showMessageDialog( null, "Geben Sie zwei ZAHLEN ein!" );
			}
		}
		
		else if( Operator.equals( "*" ) ) {
			String MFrage1 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre erste Zahl ein." );
			String MFrage2 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre zweite Zahl ein." );
			try {
				double MZahl1 = Double.parseDouble( MFrage1 );
				double MZahl2 = Double.parseDouble( MFrage2 );
				JOptionPane.showMessageDialog( null, MZahl1 * MZahl2 );
			} catch( Exception e ) {
				JOptionPane.showMessageDialog( null, "Geben Sie zwei ZAHLEN ein!" );
			}
		}
		
		else if( Operator.equals( "/" ) ) {
			String DFrage1 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre erste Zahl ein." );
			String DFrage2 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre zweite Zahl ein." );
			try {
				double DZahl1 = Double.parseDouble( DFrage1 );
				double DZahl2 = Double.parseDouble( DFrage2 );
				JOptionPane.showMessageDialog( null, DZahl1 / DZahl2 );
			} catch( Exception e ) {
				JOptionPane.showMessageDialog( null, "Geben Sie zwei ZAHLEN ein!" );
			}
		}
		
		else {
			JOptionPane.showMessageDialog( null, "Das ist keiner der gennanten Rechenoperatoren!" );
		}
	}
}

Und bitte die Tipps der anderen beachten.
Variablen mit kleinem Buchstaben beginnen!

 

[hüteüberhüte]

[JAVA=10] while( JOptionPane.showConfirmDialog( null, "Möchten Sie fortführen?", "Fortführen", JOptionPane.YES_NO_OPTION ) == 0 );[/code]

Solche Literale immer vermeiden, später weiß keiner mehr, was damit gemeint ist, oder ein Programm funktioniert nicht mehr, weil sich Konstanten für etwas geändert haben und und und

 

[Mujahiddin]

Solche Literale immer vermeiden, später weiß keiner mehr, was damit gemeint ist, oder ein Programm funktioniert nicht mehr, weil sich Konstanten für etwas geändert haben und und und

Normalerweise mach ich das auch, aber bei JOptionPanes ist es so, dass von links nach rechts gezählt wird. (Also X=-1, Yes=0, No=1, Cancel=2)

 

[AquaBall]

Du solltest Tipps schon anschauen und wahrnehmen:

das abfragen der Zahlen (Dialog und auslesen der Zahl) geschieht immer unabhaenig vom Operator.
D.h. du kannst diese Abfragen und das Auslesen auch schon vor dem If machen. Somit hast du keinen doppelten unnuetzen Code und umso weniger Zeilen Code umso weniger Fehler

Ohne irgendeine Überprüfung:

import javax.swing.JOptionPane;
 
public class Taschenrechner {
    
    public static void main(String[] args) {
        JOptionPane.showMessageDialog( null, "Hallo, dies ist ein Taschenrechner." );
        do
            invokeTaschenrechner();
        while( JOptionPane.showConfirmDialog( null, "Möchten Sie fortführen?", "Fortführen", JOptionPane.YES_NO_OPTION ) == 0 );
    }
    
    private static void invokeTaschenrechner() {
        String Operator = JOptionPane.showInputDialog( "Welchen Rechenoperator benötigen sie? (+ - * /)" );
        
        String AFrage1 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre erste Zahl ein." );
        String AFrage2 = JOptionPane.showInputDialog( "Geben Sie jeztz ihre zweite Zahl ein." );
            
        try {
            Double AZahl1 = Double.parseDouble( AFrage1 );
            Double AZahl2 = Double.parseDouble( AFrage2 );
        } catch( Exception e ) {
            JOptionPane.showMessageDialog( null, "Geben Sie zwei ZAHLEN ein!" );
        }
            
        switch Operator.toCharArray()[0]
        case '+': JOptionPane.showMessageDialog( null, AZahl1 + AZahl2 );
           break;
        case '-': JOptionPane.showMessageDialog( null, AZahl1 - AZahl2 );
           break;
        case '*': JOptionPane.showMessageDialog( null, AZahl1 * AZahl2 );
           break;
        case '/': JOptionPane.showMessageDialog( null, AZahl1 / AZahl2 );
           break;
        sefault:
            JOptionPane.showMessageDialog( null, "Das ist keiner der gennanten Rechenoperatoren!" );
        }
    }
}

 

[Wishmaster51]

Normalerweise mach ich das auch, aber bei JOptionPanes ist es so, dass von links nach rechts gezählt wird. (Also X=-1, Yes=0, No=1, Cancel=2)

Trotzdem ist dies für den Leser des Codes nicht immer sofort ersichtlich.

 

[Wishmaster51]

Du solltest Tipps schon anschauen und wahrnehmen:


Ohne irgendeine Überprüfung:

Ich war mal so frei, das switch-Statement zu ändern (benötigt Java7):

import javax.swing.JOptionPane;
 
public class Taschenrechner {
    
    public static void main(String[] args) {
        JOptionPane.showMessageDialog( null, "Hallo, dies ist ein Taschenrechner." );
        do
            invokeTaschenrechner();
        while( JOptionPane.showConfirmDialog( null, "Möchten Sie fortführen?", "Fortführen", JOptionPane.YES_NO_OPTION ) ==  JOptionPane.YES_OPTION );
    }
    
    private static void invokeTaschenrechner() {
        String Operator = JOptionPane.showInputDialog( "Welchen Rechenoperator benötigen sie? (+ - * /)" );
        
        String AFrage1 = JOptionPane.showInputDialog( "Geben Sie jetzt ihre erste Zahl ein." );
        String AFrage2 = JOptionPane.showInputDialog( "Geben Sie jeztz ihre zweite Zahl ein." );
            
        try {
            Double AZahl1 = Double.parseDouble( AFrage1 );
            Double AZahl2 = Double.parseDouble( AFrage2 );
            
            switch (Operator)
            {
            case "+":
                        JOptionPane.showMessageDialog( null, AZahl1 + AZahl2 );
                        return;
            case "-":
                       JOptionPane.showMessageDialog( null, AZahl1 - AZahl2 );
                       return;
            case "*":
                       JOptionPane.showMessageDialog( null, AZahl1 * AZahl2 );
                       return;
            case "/":
                       JOptionPane.showMessageDialog( null, AZahl1 / AZahl2 );
                       return;
            default:
                       JOptionPane.showMessageDialog( null, "Das ist keiner der gennanten Rechenoperatoren!" );
                       return;
        }
        } catch( Exception e ) {
            JOptionPane.showMessageDialog( null, "Geben Sie zwei ZAHLEN ein!" );
        }
    }
}