if schleife

[Anxiety]

Hallo,

ich möchte mich schonmal im vorfeld entschuldigen... das ist eine wirklich dumme Frage aber es dreht sich um eine simple if-Schleife. Problem ist das Lösen einer quadratischen Gleichung.

public class QuadGleichung {

     /**
     * Methode um eine quadratische Gleichung zu lösen
     */
    public static void main() {
        double a;
        double b;
        double c;
        double p;
        double q;
        double D; // Diskriminante
        double x1; //1. Ergebniss
        double x2; //2. Ergebniss
                
        a = TastaturEingabe.readDouble("ax²+bx+c=0\nBitte a eingeben.\na: ");
        b = TastaturEingabe.readDouble("Bitte b eingeben.\nb: ");
        c = TastaturEingabe.readDouble("Bitte c eingeben.\nc: ");
      
        p=b/a;
        q=c/a;
        
        D = Math.sqrt(Math.pow((p/2),2)-q);
        
        if (a> 0.0000001){
        
                if (D>0) {
                System.out.println("\n2 reele Lösungen");
                x1=(-p/2)+Math.sqrt(Math.pow((p/2),2)-q);
                x2=(-p/2)-Math.sqrt(Math.pow((p/2),2)-q);
                x1=Math.round( x1 * 100. ) / 100.; //runden auf 2 Nachkommastellen
                x2=Math.round( x2 * 100. ) / 100.;
                System.out.println("\nx1= " + x1 + " x2= " + x2);
                     }
                     
                     if (D==0) {
                         System.out.println("\n1 reele Doppellösunge");
                         x1=(-p/2)+Math.sqrt(Math.pow((p/2),2)-q);
                         x2=(-p/2)-Math.sqrt(Math.pow((p/2),2)-q);
                         x1=Math.round( x1 * 100. ) / 100.; //runden auf 2 Nachkommastellen
                         x2=Math.round( x2 * 100. ) / 100.;
                         System.out.println("\nx1= " + x1 + " x2= " + x2);
                          } 
                          
                          if (D<0) {
                              System.out.println("\n1 komplexe Lösungen");
                              x1=(-p/2)+Math.sqrt(Math.pow((p/2),2)-q);
                              x2=(-p/2)-Math.sqrt(Math.pow((p/2),2)-q);
                              x1=Math.round( x1 * 100. ) / 100.; //runden auf 2 Nachkommastellen
                              x2=Math.round( x2 * 100. ) / 100.;
                              System.out.println("\nx1= " + x1 + " x2= " + x2);
                         }          
        else {
        System.out.println(a);
    }
        } 
     
    } 
}

Das Problem ist nun... dass zunächst der Fall wenn die Diskriminante kleiner als 0 ist nicht beachtet wird. Die beiden anderen Fälle funktionieren und es wird auch das richtige Ergebnis ausgegeben. Ausserdem gehört das "else" (ganz unten) noch zum ersten if (a> 0.0000001)... wird allerdings immer ausgegeben.

Ich vermute ein simples klammerungsproblem... Kann mir jmd helfen?

Anxiety

 

[The_S]

www.if-schleife.de

Das else hast du eine Klammer zu früh gesetzt. => Es wird ausgeüfhrt wenn D < 0 ist. Warum die letzte Abfrage nicht ausgeführt wird ... vermutlich weil D nicht kleiner als 0 ist. Außerdem würde ich hier mit else if und else arbeiten und nicht mit 3 if-anweisungen. Sowas ist (in deinem Fall zwar nicht, aber allgemein) fehleranfälliger.

 

[Anxiety]

Wie meinst du das mit dem else eine Klammer zu früh gesetzt? Wie sehe das ganze deiner Meinung nach mit dem if else konstrukt aus? Das habe ich nämlich auch probiert aber da brachte der Compiler einen Fehler das es diesen ausdruck nicht gibt.

 

[The_S]

So

public class QuadGleichung {

	/**
	 * Methode um eine quadratische Gleichung zu lösen
	 */
	public static void main() {

		double a;
		double b;
		double c;
		double p;
		double q;
		double D; // Diskriminante
		double x1; // 1. Ergebniss
		double x2; // 2. Ergebniss

		a = TastaturEingabe.readDouble("ax²+bx+c=0\nBitte a eingeben.\na: ");
		b = TastaturEingabe.readDouble("Bitte b eingeben.\nb: ");
		c = TastaturEingabe.readDouble("Bitte c eingeben.\nc: ");

		p = b / a;
		q = c / a;

		D = Math.sqrt(Math.pow((p / 2), 2) - q);

		if (a > 0.0000001) {

			if (D > 0) {
				System.out.println("\n2 reele Lösungen");
				x1 = (-p / 2) + Math.sqrt(Math.pow((p / 2), 2) - q);
				x2 = (-p / 2) - Math.sqrt(Math.pow((p / 2), 2) - q);
				x1 = Math.round(x1 * 100.) / 100.; // runden auf 2
													// Nachkommastellen
				x2 = Math.round(x2 * 100.) / 100.;
				System.out.println("\nx1= " + x1 + " x2= " + x2);
			}

			else if (D == 0) {
				System.out.println("\n1 reele Doppellösunge");
				x1 = (-p / 2) + Math.sqrt(Math.pow((p / 2), 2) - q);
				x2 = (-p / 2) - Math.sqrt(Math.pow((p / 2), 2) - q);
				x1 = Math.round(x1 * 100.) / 100.; // runden auf 2
													// Nachkommastellen
				x2 = Math.round(x2 * 100.) / 100.;
				System.out.println("\nx1= " + x1 + " x2= " + x2);
			}

			else {
				System.out.println("\n1 komplexe Lösungen");
				x1 = (-p / 2) + Math.sqrt(Math.pow((p / 2), 2) - q);
				x2 = (-p / 2) - Math.sqrt(Math.pow((p / 2), 2) - q);
				x1 = Math.round(x1 * 100.) / 100.; // runden auf 2
													// Nachkommastellen
				x2 = Math.round(x2 * 100.) / 100.;
				System.out.println("\nx1= " + x1 + " x2= " + x2);
			} 
		}
		else {
			System.out.println(a);
		}
	}
}

 

[moormaster]

Selbst mit richtigem Syntax sehe ich in diesem Programm nirgends eine Schleife. Ich sehe nur Fallunterscheidungen.

Das Konstrukt if-Schleife ist mir ebenso unbekannt... eine Abbruchbedingung gibt es sowohl in for als auch in while Schleifen, wobei da jeweils zwischen kopf-, fußgesteuert unterschieden wird (je nachdem, ob die Abbruchbedingung vor oder nach dem Ausführen des Schleifenkörpers überprüft wird). Während die for Schleife hier eine spezielle Form einer kopfgesteuerten Schleife ist.

Deshalb macht eine Bezeichnung if-Schleife schon gar keinen Sinn, da so oder so in jeder Schleife eine Fallunterscheidung zwischen Abbruch und nochmaliger Ausführung statt findet.

Fehlt die wiederholte Ausführung ganz, bleibt höchstens nur noch die Fallunterscheidung übrig und ist somit erst recht keine Schleife.

Nur mal so als Erläuterung, wieso es keine if-Schleife geben kann...

 

[Leroy42]

Danke!

Das mußte mal gesagt werden! :toll:






( )

 

[Eldar]

Da fehlt einfach eine schliessende geschweifte Klammer for dem else und es ist eine zuviel am Ende des Programms...

 

[The_S]

Da fehlt einfach eine schliessende geschweifte Klammer for dem else und es ist eine zuviel am Ende des Programms...

Und wo unterscheidet sich jetzt deine Antwort von meiner und von meinem Beispielcode ?

 

[Anxiety]

Oh ich wusste ja nicht das meine frage in eine solche Grundsatz diskussion ausartet. aber erstmal vielen dank für die schnelle hilfe. Die semantik war mir noch nicht so richtig klar.

Nun muss ich das konstrukt noch um die ausgabe komplexer zahlen erweitern. Dazu habe ich folgende veränderung am code vorgenommen.

double re;
double im;

re=(-p/2);
im=Math.sqrt(Math.pow((p / 2), 2) - q);

:
:
:
:

else {
            System.out.println("\n1 komplexe Lösungen");
            x1 = Math.round(x1 * 100.) / 100.; // runden auf 2
            System.out.println("x1= " + re);           
            System.out.println("x2= " + im + "i"); 
         }

Nun erhalte ich die Fehlermeldung, das die variable x1 nicht initialisiert ist... obwohl das oben geschehen ist. Siehe QT erster post. Was könnte man noch machen.

 

[The_S]

Nein, du musst x beim deklarieren einen Wert zuweißen damit sie initialisiert ist.

 

[Tobias]

Die semantik war mir noch nicht so richtig klar.

<klugscheiß>Nein, dir war die Syntax nicht klar.</klugscheiß>

mpG
Tobias

 

[Eldar]

Und wo unterscheidet sich jetzt deine Antwort von meiner und von meinem Beispielcode icon_wink.gif ?

Ein Satz im Gegensatz zu ähm... vielen, vielen, vielen .. dass er ein Klammerproblem hatte wusste er ja schon.

 

[Anxiety]

@hobbit

x ist doch ein wert zugewiesen... siehe oben im qt

 

[LoN_Nemesis]

Du weist x1 zwar einen Wert zu, aber innerhalb eines if-Konstruktes. Wenn die Bedingung false ist, dann erfolgt diese Zuweisung nicht und x1 ist nicht initialisiert.

Deswegen sagte Hobbit "beim Deklarieren".

double x1 = 0;

zum Beispiel