Wertrückgabe aus FOR-Schleife

[lprade]

Hallo zusammen,
als Java-Anfänger bin ich auf folgendes Problem gestossen: zum Üben wollte ich ein Program (in BlueJ) schreiben, welches mir mit n Schleifendurchläufen die Zahl Pi berechnet und ausgibt. Der Quellcode sieht folgendermasen aus:

public class Pi {

    static void rechnen(int n) {
    double pi=0;
    
        for(int i=1; i==n; i++) {
        pi+=1/((4*n)-3)-1/((4*n)-1);
    }    
    pi*=4;
    System.out.println(pi);
    
}
}

aus irgendeinen Grund krieg ich aber immer nur den Wert 0 zurück, den ich ja vorher so initialisiert habe, und nicht der, der in der Schleife berechnet wurde. Kann mir irgendjemand sagen warum und wie ich das ändern kann?
Besten Dank,
Ludwig

[Edit by Beni: Codetags]

 

[Der Müde Joe]

Methoden mit void haben keinen Rückgabewert

public static int foo(){

/*.....*/

return 27;
}

return type: int
return Wert: 27

 

[outbreaker]

weil er deine for Schleife nicht ausführt
du überprüfst in der For schleife ob i==n
du brauchst aber eine Bedingung die erfüllt ist damit er in die For Schleife geht also z.b. i < n

 

[Schandro]

Außerdem wird die schleife nur einmal ausgeführt, und dass auch nur dann, wenn n 1 ist. Du müsstest als bedingung i<= n schreiben

 

[lprade]

Hallo
ich habe das Program jetzt so geändert, wie Schandro es vorgeschlagen hat, aber es hat sich immer noch nichts geändert. Gibt es denn eine Möglichkeit, wie man überprüfen kann, obe die Schleife überhaupt ausgeführt wird?
Danke nochmals,
Ludwig

 

[outbreaker]

schreibe dir eine Ausgabe in deine Schleife

so z.B.

System.out.println("drin");

 

[eldios1]

ja schreib in deine for schleife ein system.out.println und lass dir da pi ausgaben

und schreib mal das
pi*=4;
System.out.println(pi);
mit in die for schleife weil die nicht mit angesprochen werden.

 

[outbreaker]

du Rechnest in deiner Schleife mit einer int Variablen
somit gibt es auch keine Nachkommastellen und somit ist die Ausgabe auch 0

Es gibt jetzt mehrere Möglichkeit

1.) Möglichkeit

public class Pi {

	 public double rechnen(double n) {
		 double pi=0;

		 for(double i=1; i < n; i++) 
		 {
			 pi += 1/((4*n)-3)-1/((4*n)-1);
			 System.out.println("drin");
		 }
		 pi*=4;
		 System.out.println(pi);
		 return pi;
		 } 
	 
	 public static void main(String[] args) {
		System.out.println("Wert:" + new Pi().rechnen(5));
	}

}

2.) Möglichkeit

public class Pi {

	 public double rechnen(int n) {
		 double pi=0;

		 for(double i=1; i < n; i++) 
		 {
			 pi += 1/((4*(double)n)-3)-1/((4*(double)n)-1);
			 System.out.println("drin");
		 }
		 pi*=4;
		 System.out.println(pi);
		 return pi;
		 } 
	 
	 public static void main(String[] args) {
		System.out.println("Wert:" + new Pi().rechnen(5));
	}

}

3.) Möglichkeit

public class Pi {

	 public double rechnen(int n) {
		 double pi=0;

		 for(double i=1; i < n; i++) 
		 {
			 pi += 1/((4.0*n)-3)-1/((4.0*n)-1);
			 System.out.println("drin");
		 }
		 pi*=4;
		 System.out.println(pi);
		 return pi;
		 } 
	 
	 public static void main(String[] args) {
		System.out.println("Wert:" + new Pi().rechnen(5));
	}

}

 

[lprade]

irgendwas stimmt hier nicht,
wenn ich System.out.println(pi); mit in die Schleife schreibe, krieg ich einfach n-mal 0.0 raus. Er scheint wirklich nichts in die Variable pi hineinzuschreiben. Bin jetzt ziemlich ahnungslos.

 

[SnooP]

du multiplizierst ganze zahlen mit double also Fließkommazahlen... das wird dir in den meisten Fällen das Ergebnis zerhageln... - also immer gucken, dass du da 1.0 und 3.0 drinstehen hast und n sollte auch double sein... oder man castet auf double: ((double) n)

 

[lprade]

Super Danke! jetzt funktionierts. Ich habe jetzt n als double definiert.
Danke an alle

 

[outbreaker]

also bei meinen 3 Möglichkeit bekomme ich jedesmal ein Ergebnis raus :bae:

 

[outbreaker]

na dann is ja schön :wink:

 

[Gast]

for(double i=1; i < n; i++)
{
pi += 1/((4.0*n)-3)-1/((4.0*n)-1);
System.out.println("drin");
}
return pi;
}

wenn ich das so habe, dass geht nicht oder? das das return alleine hinter der for sdhleife ist?
mal abgesehen von dem sinn mir gehts nur um syntax

 

[outbreaker]

klar geht das
nur nach dem return darf keine Anweisung mehr kommen da diese ja nie Ausgeführt werden kann

 

[Gast]

hmm komisch wäre der meinung das ich das irgendwann mal so hatte und das nicht ging. kann aber auch sein das ich das gerade falsch im kopf habe. muss ich mal nach gucken wenn ich zu hause bin.

mfg

 

[outbreaker]

mach das mal :wink: