Hallo,
ich habe da ein kleines Problem.

Folgender Code:




Ich möchte lediglich das er ab 5000 Euro in 3000 Schritten bis 35000 Euro die Rechnung in der i steht ausführt und dann am Ende ausgibt in den Label der in der Form Anwendung drin ist.

Also sollten von 5000 bis 35000 mehrere Ausgaben stattfinden in der jeweiligen Rechnung.

Habe auch eine Select Case versucht, vergeblich.

Wäre super wenn mir da jemand behilflich wäre, da ich nicht weiter komme. Danke!

*Topic verschoben*

Wenn i>=20000 ist, dann ist auch i>=12000

Deine ElseIf Abfragen stimmen nicht, da z.B. wenn es 25000 sind, dies auch größer als 12000 ist, und deshalb das genommen wird.

Wie könnte ich das anpassen das ich nur sage von diesem Wert bis diesen Wert?

Dreh einfach die Reihenfolge deine ElseIf-Abfragen um.

Oder einen Bereich wählen, also von bis. >=12000 And <20000

Habe es nun so:



Also umgedreht die Reihenfolge?
Das Problem ist aber das die Forms anwendung beim drücken des Buttons einfach sich aufhängt, ich vermut endlosschleife?





EDIT:

Nächste Lösung:


Die Form hängt aber und er zeigt nichts an.

Das Problem ist, dass du i innerhalb der Schleife veränderst, so dass du nie auf 35000 kommst.
Speichere das Ergebnis in einer Variablen zwischen und nimm nicht den Schleifenzähler dafür.

Wenn du die Schleife im Debugger durchgesteppt hättest, wäre das sicher ganz schnell aufgefallen.

Außerdem wird das Label immer nur den letzten Wert anzeigen.
Schreib einfach mal in Prosa nieder, was du erreichen möchtest und setze es dann erst in Code um.

Wie soll ich das außerhalb der Schleife deklarieren?

Ich komme nicht mehr mit...
Ich möchte lediglich das ein paar Zeilen ausgegeben werden, wo einfach die If verzweigungen und das jeweilige ergebnis der rechnung angezeigt wird.
muss ich mehrere labels machen?

kann mir da mal einer helfen...

Ich versteh gar nicht was das Problem ist.

Gib bitte ein Beispiel, was rauskommen sollte deiner Meinung nach und was rauskommt.

Schau, ich möchte beispielsweise das die For Schleife von 5000 bis 35000 in 3000er Schritten hochzählt und folgende Faktoren berücksichtigt bei der Ausgabe.

Sobald die Schleife bei kleiner 12000 ist, soll sie den Wert der schleife mit 0,12 multiplizieren und dies ausgeben in der Form
Dann nochmal... Sobald die Schleifenzahl größer als 12000 ist bis 20000 dann soll sie die Schleifenzahl mit 0.15 multiplizieren und ausgeben

Und dann nochmal das gleiche... Wenn die Schleifenzahl größer als 20000 ist aber nicht größer als 30000 soll sie mit 0,2 mutliplizieren und diese ausgeben...

Ich denke es ist leicht umsetzen und ich habe einen Logikfehler drin...

Er gibt mir zurzeit gar nix aus, die Formanwendung hängt einfach wenn ich auf Berechnen drücke.. Da muss ein Fehler drin sein...
Wie würdet ihr das mit der For Schleife lösen?

Was heißt es hängt? Hat sich die Form wirklich aufgehangen, oder wird einfach nur nichts im Label angezeigt?

Also zumindest beim ersten durchlaufen der Schleife sollte etwas im Label stehen.

Wenn ich das Programm genau so ausführe wie ich es oben im 2. Lösungsvorschlag habe, zeigt mir das Label nichts an und dann hängt es und ich kann die Form nicht weg drücken oder dergl

Ich gebe dir mal zwei Hinweise:

1. deine Zahl, also i, wird immer kleiner
2. lblAusgabe.Text erwartet einen String, du übergibst aber ein Int


Schmeiss mal den Debugger an, dann siehste das sofort.

@Parmaster 2. stimmt zwar, aber int sollte problemlos in string umgewandelt werden

du solltest es einfach so machen wie angesprochen, außerhalb deiner Schleife eine variable deklarieren und diese dann in der schleife setzt:

Jo schon, soll man aber nicht machen, da Option Strict Off.

Und in deinem Code ist auch noch der logische Fehler, dass die Variable variable immer kleiner wird anstatt größer.


EDIT:

hier mal mein Code. Das Sleep ist nur drin um zu sehen welche Werte zwischenzeitlich angenommen werden.

Sollte jetzt aber glaube ich erkenntlich sein, was wir meinen!

@Parmaster: wieso weist du auf Option Strict On hin und willst dann noch im selben Post eine Gleitkommazahl in einen Integer schreiben?

@programmofficer: das Label ist das falsche Control, um mehrere Ergebnisse anzuzeigen, denn es kann nur einen String anzeigen und der sollte designmäßig auch nicht zu groß und zeilenübergreifend werden. Dein Ansatz führt ja nur dazu, dass das Ergebnis des letzten Schleifendurchlaufes sofort überschrieben wird und du am Ende nur das letzte Ergebnis im Label hast.

Der richtige Ansatz sollte also sein, die Ergebnisse jedes Durchlaufes in einer Auflistung (Array, List, IEnumerable) zu speichern und diese am Ende im richtigen Control ausgeben zu lassen, zB. im Datagridview.

CStr(), Convert.ToString() seriously?
wenn ich nicht irre haben selbst die primitiven Datentypen in .Net eine Funktion namens .ToString().
etwas wie 1.ToString() sollte demnach problemlos Funktionieren.

Parmaster schrieb:

Und in deinem Code ist auch noch der logische Fehler, dass die Variable variable immer kleiner wird anstatt größer.

Du und @Goldwing Studios machen exakt das selbe. Nur dass du einen Schritt mehr drin hast.
ebenso wird sich der User freuen wenn seine Form im dauerfreeze ist, da du sie mehrfach für 2 sek schlafen lässt.

Interessant wäre in diesem Fall sicherlich auch eine ListBox die per Items.Add() dann die werte untereinander auflistet.

@Counterbug

da hast du recht, war natürlich blöd von mir.

@EaranMaleasi

Ist doch egal ob man es so oder so macht.

Und wo machen wir beide exakt das selbe?

Und dass das Label nicht dafür geeignet ist, ist wohl klar, und genau deshalb ist das Sleep drin, nur um zu zeigen das sich zwischenzeitlich auch die Werte ändern. Steht aber auch exakt so oben.