Warum funktioniert mein Programm nicht ?

[Wasserranzen]

Hallo,

Ich bräuchte wieder Hilfe. Ich war grade dabei ein Programm zu schreiben das quadratische Gleichungen der Form ax^2+bx+c=0 lösen soll. Leider will es nicht laufen.
Zwei Probleme habe ich aktuell:

1) if Else
2) Es kommt ein falsches Ergebnis raus.

Ich hoffe jemand kann mich aufklären.

Danke !

// ABC Formel = -b + - w(b^2-4ac)/2a = w(4) = 4*0,5 = 2
// ax^2 + bx + c = 0 I  1) ax^2 + bx && ax^2+c
// Nullstellen
//----------------------------------------------------------------
import java.util.Scanner;

public class ABC{
public static void main(String[]args){
//----------------------------------------------------------------
Scanner x = new Scanner(System.in);

double a;
double b;
double p;
double c;
double ABC;
double ABC2;
double w;
//----------------------------------------------------------------
a = 0;
b = 0;
p = 0;
c = 0;
w =0;
//----------------------------------------------------------------
System.out.println("Funktionsgleichung erwartet(b=p positiv)");
System.out.println( "0"+"="+"ax²+"+"bx"+"+c");
a = x.nextDouble();
System.out.print(a+"x²");
b = x.nextDouble();
System.out.println(b + "x");
c = x.nextDouble();
System.out.println(c);
p = x.nextDouble();
//----------------------------------------------------------------
if ( b < 0 && a >=-100000 && b >=-100000 && c >=-100000)
{

w = 0.5;
ABC = b*(-1)+w*(p*p-4*(a)*(c))/(2*a);
ABC2 = b*(-1)-w*(p*p-4*(a)*(c))/(2*a);
System.out.println("Die Lösungsmenge der Funktionsgleichung" + a +"x²" + b+"x"+c +"lautet");
System.out.println("x1"+" " +ABC);
System.out.println("x2"+" " +ABC2);
}

//----------------------------------------------------------------

{
else if ( b < 0 && a >= -100000 && b >=-100000)
w = 0.5;
ABC = b*(-1)+w*(p*p-4*(a)*(c))/(2*a);
ABC2 = b*(-1)-w*(p*p-4*(a)*(c))/(2*a);
System.out.println("Die Lösungsmenge der Funktionsgleichung" +  b+"x"+c +"lautet");
System.out.println("x1"+" " +ABC);
System.out.println("x2"+" " +ABC2);
}
//----------------------------------------------------------------
{
else if ( b<0 && a >= -100000 && c >= -100000)
w = 0.5;
ABC = b*(-1)+w*(p*p-4*(a)*(c))/(2*a);
ABC2 = b*(-1)-w*(p*p-4*(a)*(c))/(2*a);
System.out.println("Die Lösungsmenge der Funktionsgleichung" + a +"x²" +c +"lautet");
System.out.println("x1"+" " +ABC);
System.out.println("x2"+" " +ABC2);
}
//----------------------------------------------------------------

//Andere Verzeigung

else if ( b > 0 && a >= -100000 && b >= -100000 &&c >= -100000)
{
w = 0.5;
ABC = b+w*(p*p-4*(a)*(c))/(2*a);
ABC2 = b-w*(p*p-4*(a)*(c))/(2*a);
System.out.println("Die Lösungsmenge der Funktionsgleichung" + a +"x²" +b+"x"+c +"lautet");


System.out.println("x1"+" "+ABC);
System.out.println("x2"+" "+ABC2);
}
//----------------------------------------------------------------
//else if ( b >0 || a>= -100000 || b>= -100000 )




}
  }

 

[fhoffmann]

1) if Else
2) Es kommt ein falsches Ergebnis raus.

Das ist keine sehr genaue Fehlerbeschreibung.

 

[truesoul]

Hallo. Na ja kein wunder warum es nicht laufen will.

if else sollte so aussehen:

if(eine bedingung) {
   // CODE
} else if (noch eine bedingung) {
   // CODE
} else if(und noch eine bedingung) {
   // CODE
}

Dein if else sieht zum Teil so aus:

if(eine bedingung) {
   // CODE
} { else if (noch eine bedingung)
   // CODE
} { else if(und noch eine bedingung)
   // CODE
}

Ansonsten wäre eine bessere Fehlerbeschreibung wünschenswert.

Grüße

 

[Wasserranzen]

Danke, dass mit dem Klammern war das Problem. Leider habe ich noch eins. Die ABC Formel spuckt mir falsche Ergebnisse aus (Block 5.). Ich musste übrigens auch b zweimal in die Konsole abtippen, weil ich -b als Fall abfangen musste.

 

[truesoul]

Hallo.

Erstmal solltest du dein boilerplate code entfernen.
Alle Berechnung die du hast sind identisch.

Die Bedingung für dein If Statement kannst du auch in einer Methode auslagern.

Darfst du die Math Klasse verwenden?

Grüße

 

[Wasserranzen]

Ich bin leider ein Anfänger in Java und weiß nicht wie das geht. Ich musste die Berechnungen identisch machen, weil ich alle Vorzeichen Wechsel abfangen musste.

 

[truesoul]

Diese Abfragen kannst du in solch Methode auslagern.

boolean isValid1(double b, double a){
   return b < 0 && a >= -100000 && b >=-100000;
}

boolean isValid2(double b, double a){
   return // zweiter Statement;
}

usw  ...

if(isValid1(b, a) && isValid2(b, a) usw) {
 // Deine Berechnungen
}

Oder

boolean isValid(double b, double a, double c){
   return (b < 0 && a >= -100000 && b >=-100000) && (zweite Bedingung) && (dritte ....);
}

Und dann:

if(isValid(b, a, c)) {
 // Deine Berechnungen
}

Grüße

 

[Wasserranzen]

Verstehe ich leider nicht. Dieses Kapitel hatte ich noch nicht. Gibt es denn keine Möglichkeit mein Problem mit dem Vorzeichen auf eine andere Weise zu lösen? Ansich geht ja alles, außer das die Berechnungen falsch sind.

 

[Robat]

Für den Anfang kannst du das ruhig erstmal so lassen.
Es ist zwar viel Boilerplate-Code dabei aber bring erstmal das Programm zum laufen.

Du sagtest oben was von Block 5.
Was meinst du damit? Welcher Teil funktioniert nicht - funktioniert überhaupt eine Berechnung?

 

[truesoul]

Wie lautet die Aufgabenstellung?

Ich frage weil es für die Berechnung eine Rolle spielt.
Na ja mit der Klasse Math wäre die Berechnung einfacher. Ansonsten muss du ein wenig mehr Code schreiben.

Grüße

 

[Wasserranzen]

Ja, funktionieren schon, aber die Ergebnisse sind leider falsch. Ich musste auch b^2 = p setzen, da ich nicht weiß ob meine Variable b mehr als ein Wert annehmen kann. Schätze nicht.

Es ist eigentlich nur für mich gedacht. Als Übung quasi.

 

[Robat]

Wo hast du die Formel her? Für meine Begriffe fehlt da eine Wurzel.
x1 = (- b + sqrt(b2 - 4ac)) / (2a)
x2 = (- b - sqrt(b2 - 4ac)) / (2a)

 

[truesoul]

        double a = 2;
        double b = 3;
        double c = 1;

        double d = b * b - 4 * a * c;

        double positive = (-b + sqrt(d)) / (2 * a);
        double negative = (-b - sqrt(d)) / (2 * a);