Wo ist der fehler C++

[~*Breakdown*~]

Hey eine frage ich soll hier mal für ein kollegen nachfragen wo hierdrin der fehler ist weil er den irgentwie nicht findet
bzw es fehlt irgentwas das programm startet nicht oder irgent wie sowas

#include<stdio.h>

void main (void)

{
float pi=1;
int nenner=3;
int nennerMax;
int methode=1; //um zu entscheiden ob "+" oder "-"


printf("geben sie den nenner ein");
scanf("%i",& nennerMax);
fflush(stdin);
do
{

do
{

pi = pi-1/nenner;

nenner=nenner+2;

methode=0;
}
while(methode=1);
do
{

pi = pi+1/nenner;

nenner=nenner+2;

methode=1;
}
while(methode=0);
}
while(nenner<=nennerMax); // solange bis nenner genauso gróß ist wie nennermax

pi=pi*4;

printf("nach der leibniz reihe lautet das ergebnis %0.8f",pi);

getchar();
}

[ms​]

Erst einmal vergleicht man zwei Werte mit dem Operator ==. In zwei while-Teilen wird jeweils der Variable methode eine Zahl zugeordnet. Außerdem sind die beiden Schleifen dann Endlosschleifen, weil methode nie diesen Wert annimmt. Wahrscheinlich wollte er eigentlich if-Anweisungen benutzen.

[~*Breakdown*~]

mein kollege hat geschrieben:

es sind keine endlosschleifen, da er immer min. ein mal durch eine do-schleife geht und in ihr der wert so geändert wird, dass er eben das abbruchkriterium erfüllt.

[Dr. Coxxy]

wenn er in der bedingung (methode = 0) steht wird das nichts.
er muss den vergleichsoperator '==' verwenden und nicht den zuweisungsoperator '='.
sag deinem freund:

1. er soll sich ein ordentliches buch anschaffen und sorgfältig lesen.
2. sich selber nen account hier machen, wenn er fragen hat.

außerdem bist du hier in der falschen section, c/c++ hat eine eigene...

[ms​]

Originally Posted by ~*Breakdown*~

mein kollege hat geschrieben:

es sind keine endlosschleifen, da er immer min. ein mal durch eine do-schleife geht und in ihr der wert so geändert wird, dass er eben das abbruchkriterium erfüllt.

Er soll sich meinen Post nochmal durchlesen und erst einmal die Grundlagen lernen. Eine Zuweisung wird zu dem neuen Wert evaluiert, (methode = 1) wird also immer zu 1 evaluiert.

[vwap]

Warum float pi = 1;?
"1" ist ein Integer, kein Float

[xNopex]

Das ist kein C++.

[.BritainAndy]

erinnert mich eher an C

[jacky919]

Erinnert nicht nur an C, sondern ist C. Ansonsten sollte man die Aus- und Eingabeklassen aus nutzen.

[MrSm!th]

Originally Posted by Headpuster

Warum float pi = 1;?
"1" ist ein Integer, kein Float

das wird automatisch konvertiert