Programm macht nicht was es soll

[991jo]

Hallo liebe Community

ich wollte mir aus Spaß ein Programm schreiben, dass mir Pi berechnet.
Pi will ich berechnen nach diesem System:
1:1²+1:2²+1:3².... x6 und dann die Wurzel daraus.

Auf meinem GTR hat das Problemlos geklappt, leider fehlt den etwas an Performance (nach 90 Min Unterricht nur 7 oder 8 nachkommastellen), deshalb will ich das mit mit dem PC machen, damit das etwas flotter geht.
Leider zeigt mir die Konsole in Eclipse die falschen Zahlen an (also was mit 1000+ kann ja für Pi schlecht hinkommen)
Wie schon gesagt benutzte ich Eclipse, dieses läuft auf Ubuntu 11.04.
hier mal mein Programmcode (bitte nich zu viel meckern, sondern verbesserungsvorschläge geben, das is das 1. Programm, das ich schreib, abgesehen vom HelloWorld)

public class Pi {

	/**
	 * Dieses Programm berechnet Pi
	 */
	public static void main(String[] args) {
		float Bruchzahl = 1;
		double Speicher = 0;
		boolean Eins = true;
		double Speicher2 = 0;
		double Speicher3 = 0;
		while (Eins=true){
			Speicher=1/Bruchzahl*Bruchzahl+Speicher;
			Bruchzahl=Bruchzahl+1;
			Speicher2 = Speicher*6;
			Speicher3 = Math.sqrt(Speicher2);
			System.out.println ("Pi ist gerundet " + Speicher3);
		}

	}

}

 

[my life]

Speicher=1/Bruchzahl*Bruchzahl+Speicher;

setz mal Klammern um Bruchzahl * Bruchzahl

 

[njans]

Hier mal etwas angemerkt

public class Pi {

	/**
	 * Dieses Programm berechnet Pi
	 */
	public static void main(String[] args) {
		float Bruchzahl = 1;
		double Speicher = 0;
		boolean Eins = true;
		double Speicher2 = 0;
		double Speicher3 = 0;
		while (Eins=true){ // Eins = true ist eine Zuweisung! Eins == true ist eine Bedingung. Und die kannst du auch einfach als "Eins" schreibe, das ist auch ein boolischer Ausdruck 
			Speicher=1/Bruchzahl*Bruchzahl+Speicher;
			Bruchzahl=Bruchzahl+1;
			Speicher2 = Speicher*6;
			Speicher3 = Math.sqrt(Speicher2);
			System.out.println ("Pi ist gerundet " + Speicher3);
                        // dir fehlt hier auch eine Abbruchbedingung, so geht das immer und immer weiter
		}

	}

}

 

[Fab1]

Hi,

Variablen wie Bruchzahl, Speicher, Eins etc. sollten in Java generell klein geschrieben werden.

Sowie den Titel des Threads solltest du dir nochmal überlegen.

 

[991jo]

Danke läuft, bin zu arg an meinen GTR gewöhnt, der macht Punkt vor Strich, da geht das ohne (), aber jetzt bin ich auch schlauer.

Ähm, ok, danke, werde das ändern und auch in Zukunft alle Variablen klein schreiben.


So, ich würde sagen Optimierung wäre noch ganz cool, damits schneller läuft, und wie kann ich in der Eclipse-Konsole mehr Nachkommastellen anzeigen lassen/was muss ich dazu am Programm ändern?

Das ich das logisch nachvollziehen kann: Du beziehst dich da auf die Gleichung der Riemann Zeta Funktion, deren Grenzwert (PI^2)/6 sein soll?

wenn du das da meinst, dann ja

Eins = true ist eine Zuweisung! Eins == true ist eine Bedingung. Und die kannst du auch einfach als "Eins" schreibe, das ist auch ein boolischer Ausdruck

#

Ohh danke, wenn while (true) {} auch geht, ok, und danke, wieder ein bischen schlauer A=0 ist eine Zuweisung, A==0 ist eine Bedingung

 

[njans]

Das ich das logisch nachvollziehen kann: Du beziehst dich da auf die Gleichung der Riemann Zeta Funktion, deren Grenzwert (PI^2)/6 sein soll?

 

[991jo]

@njans

Ich habs oben schon mal gepostet, ich glaube wir beide meinen das hier:

Ich weis, dass es effektivere Methoden gibt, aber den Kettenbruch versteh ich noch nicht ganz, da sich mir mit meinen Mathe-Kenntnissen das Problem stellt, dass man diesen Bruch erst in der Unendlichkeit lösen kann... vll lernt man das in der Kursstufe...
Auf jeden Fall kann man dashier relativ einfach umsetzten...