C# Anfänger, Problem bei variablenumwandlung

Du solltest natürlich schon irgendwo abfragen, ob das übergebene Gewicht überhaupt in einen Float-Wert konvertiert werden _kann_. Wenn dir irgendjemand "bla" eingibt, ist das klar, daß das kein Float ist.

Wie heißen denn die Strings in den Comboboxen? Vielleicht hast du nur Punkte mit Kommas vertauscht, oder sonst falsch formatiert.

Hallo Tensai!

Zitat von Tensai

Hi nochmal!

Die Werte in den Comboboxen haben nen punkt wie z.B 500.0
und die strings sind männlich und weiblich

Gewicht.Text liefert dir hier nur den text, der zu der combobox gehört (das ist wahrscheinlich nichts brauchbares). Was du haben möchstest ist der string, den du (wahrscheinlich) als item in der combobox abgelegt hast.

Versuche mal

Gewichtx  = Convert.ToSingle((string) Gewicht.SelectedItem);

Ich hoffe ich liege damit richtig. Wenn nicht, poste doch bitte den code, in dem du deine comboboxen mit werten füllst.

lass dir bei solchen problemen immer zuerst alle werte ausgeben, und schränke so die fehlerquelle ein. Wenn dir z.b eine exception sagt, dass einstring nicht in ein float konvertiert werden kann, dann ist der erste logische schritt, sich diesen string einmal anzusehen (zum beispiel irgendwo ausgeben lassen).

und zum schluss noch ein notwendiger hinweis auf unser FAQ.

Zitat von a9bejo

Gewicht.Text liefert dir hier nur den text, der zu der combobox gehört (das ist wahrscheinlich nichts brauchbares). Was du haben möchstest ist der string, den du (wahrscheinlich) als item in der combobox abgelegt hast.

falsch, Gewicht.Text liefert den Text zurück der gerade in der combo angezeigt wird.

Zitat von a9bejo

Code

Gewichtx  = Convert.ToSingle((string) Gewicht.SelectedItem);

entweder

Gewichtx  = Convert.ToSingle(Gewicht.SelectedItem.ToString());

oder

Gewichtx  = Convert.ToSingle(Gewicht.SelectedItem);

zum ersten poster:
schreib mal bevor du die floats deklarierst folgendes
MessageBox.Show(Gewicht.Text);
und poste was er dir anzeigt

noch was:
AlkWert = (((Mengex * 0.48F)*0.8F)/(Gewichtx * 0.7F));

(((Mengex * 0.48F)*0.8F)/(Gewichtx * 0.7F)) diesen teil könntest in ne eigene funktion ausgliedern, dann könntest dir das if/else sparen

AlkWert = Geschlechtx.Equals("männlich") ? GetAlkWert(Mengex,Gewichtx,0.7F) : GetAlkWert(Mengex,Gewichtx,0.8F);

etc

edit: habs mal kurz probiert:

this.comboBox1.Items.AddRange(new object[] {
														   "0.9",
														   "0.8",
														   "0.7",
														   "500.0"});

dann kann ich es hier locker auslesen:

float f = Convert.ToSingle(this.comboBox1.SelectedItem);

Zitat von bosie

falsch, Gewicht.Text liefert den Text zurück der gerade in der combo angezeigt wird.

hmm du hast völlig recht. danke für die korrektur.