Bekomme dauernd Runtimefehler - The variable is being used without being initialized...

[314159265358979]

Das ist keine Optimierung sondern Faulheit.

[pumuckl]

Lol?

double ideal = (size - 100) * (male ? male_factor : female_factor) / 100;

-> weniger zu tippen, leichter zu lesen, optimierte Faulheit

[314159265358979]

Na wenn du meinst. Ich mag ?: in Berechnungen nicht.

[volkard]

314159265358979 schrieb:

Na wenn du meinst. Ich mag ?: in Berechnungen nicht.

Geht mir auch so.

[EOutOfResources]

volkard schrieb:

Geht mir auch so.

Mir gefällts.

[DaNewGuy]

Post in falschen thread, sorry.

[DaNewGuy]

in Zeile 17 verwendest du eine variable ohne sie deklariert zu haben, darin liegt vielleicht dein Fehler.

cout << geschlecht;

so würde ich das versuchen

//geschlecht = true = 1 für weiblich und geschlecht = false = 0 für männlich 
bool geschlecht;

dann kannst du das auch in der if-Abfrage direkt integrieren ohne einen == operator.

if( geschlecht!=0 || geschlecht !=1) cout << "Fehleingabe!\n";
else if (geschlecht) idealgewichtfrau = ((körpergröße - 100) * 90) / 100;
else if (!geschlecht) idealgewichtmann = ((körpergröße - 100) * 95) / 100;

[volkard]

DarkChip schrieb:

if (geschlecht == 0)
		idealgewichtfrau = ((körpergröße - 100) * 90) / 100;
	else if (geschlecht == 1)
		idealgewichtmann = ((körpergröße - 100) * 95) / 100;

Würde ich eher machen zu

int geschlechtsfaktor;
	if (geschlecht == 0)
		geschlechtsfaktor = 90;
	else
		geschlechtsfaktor = 95;
	int idealgewichtfrau = ((körpergröße - 100) * geschlechtsfaktor) / 100;

Doppelten Code vermeiden und Variablen immer so spät wie möglich anlegen.

[CStoll]

314159265358979 schrieb:

Na wenn du meinst. Ich mag ?: in Berechnungen nicht.

Das mag Ansichtssache sein - ich habe es schon öfter vermisst. Aber auf jeden Fall ist es übersichtlicher als diese bool-Multiplikation.

[forenseeker]

Weiß einer wozu das Gewicht abgefragt wird?
Muß nicht auch die Haarfarbe abgefragt werden?

Nutzlosen Code vermeiden