Lesen von .txt Datei und trennen

Neeneenee. Zwischen nichts ausgewählt und 0 ausgewählt besteht ein eklatanter Unterschied.

Oben ist nichts ausgewählt, unten die 0. Dass die 0 bei Dir sowas heißt wie keine Auswahl getroffen, ist dem Programm erstmal egal. Das Programm und ich wollen wissen, wass passieren soll, wenn der obere Zustand erreicht ist, also wirklich nichts ausgewählt wurde. Wenn also SelectedItem = Nothing ist. Denn dieser Zustand wird (z.B: zu Programmstart) erreicht, sonst würde es nicht zu der genannten Fehlermeldung kommen.

MrSKY schrieb:

Ehrlich gesagt habe ich folgenden Post noch immer nicht verstanden

Ok, dann versuche ich es anders zu erklären. Ich halte mich dazu an Dein Video. Egal, was man auswählt, ob Spielertyp, Gegenstand, Upgrade oder Rarity. Immer dann, wenn man was davon auswählt, muss der Neuschreibecode aufgerufen werden. Also so in etwa:

Das kann man zwar auch zusammenfassen zu
aber das ist jetzt erstmal unwichtig. Wichtig ist wie gesagt: Sobald ein anderer RadioButton ausgewählt wird oder bei den CheckBoxen sich was ändert, wird diese CreateNewCommandText()-Methode aufgerufen. Eben damit der komplette CommandText-ErstellCode nicht an 20 Stellen steht, sondern nur an einer. Eben in dieser hier noch nicht gezeigten CreateNewCommandText()-Methode. Mehr wollte ich in dem von Dir erwähnten, schwer verständlichen Post nicht sagen.

In dieser Methode muss jetzt also der CommandText zusammengesetzt werden. Dazu muss erstmal geschaut werden, ob überhaupt alle Daten valide sind. Wenn keine Rolle für den Spieler ausgewählt ist, also keiner der Swordsman-/Archer-/Mage/…-Buttons, dann darf da gar kein CommandText entstehen. Genauso, wenn in den ComboBoxen nix (also wirklich nixnix!) ausgewählt ist. Denn bei ComboBoxen - egal in welchem Style sie ist - ist erstmal gar nichts ausgewählt. Auch wenn Du im Designer festlegst, dass es die Elemente 0, 1, 2, 3, 4, 5 geben soll. Zu Programmstart ist nichts von denen ausgewählt. Und das wird zum Problem. Denn dann kann kein CommandText erstellt werden. Also muss zu Beginn der CreateNewCommandText()-Methode nicht nur geprüft werden, ob mindestens ein RadioButton ausgewählt wurde, sondern auch, ob bei allen ComboBoxen ein Element ausgewählt wurde (und sei es nur die neutrale 0). Ansonsten muss die Methode vorzeitig abbrechen und darf nicht versuchen, einen CommandText zu erstellen. Klar soweit? Nach den Vorabüberprüfungen geht es mit dem ComanndTextAufbau wie oben genannt los.
Also ganz grob als Beispielanfang:


##########

bzgl. die Null wird nicht eingetragen:
Dann muss Du eben eine Sonderbedingung einfügen. Denn Standard schein ja zu sein: Wenn Item gewählt, aber sonst 0, dann taucht quasi nur das Item im CommandText auf. Und vielleicht die Spielerrolle, keine Ahnung. Dass eine 0 eingetragen werden soll, wenn in Rarity 0 gewählt ist, aber in Upgrade keine 0, ist ja eine Abweichung der Grundsatzregel. Also musst Du explizit das schreiben: Wenn Rarity = 0 Und Upgrade nicht 0, dann ist der CommandText Spielerrolle und Item und 0 und Upgrade

MrSKY schrieb:

Heißt damit ist nun die CreateNewCommandText() - Methode abgeschlossen

Nee, das ist nur der Anfang. Hinter dem Then muss ein Return hin. Der soll die Methode also abbrechen, wenn die Bedingungen zutreffen. In die Bedingungen gehört auch rein, dass auch die Rarity-ComboBox und die UpgradeComboBox auf SelectedIndex = -1 geprüft werden muss. Sobald das Return hinter dem Then steht, muss auch das End If weg. Das Ganze soll also heißen: Wenn nix Vernünftiges ausgewählt wurde, brich die Methode ab (denn die Daten sind nicht ausreichend).

Der Rest der CreateNewCommandText-Methode besteht aus dem, was der Methodenname sagt: CommandText zusammen bauen.
Ich geb ein anderes Beispiel, vielleicht wird es dann klar:

Am Anfang der Methode stehen die Abbruchbedingungen. Wenn was davon zutrifft, kann kein Obstsalat hergestellt werden. Dann wird der Vorgang abgebrochen. Die Anweisungzeilen zur Herstellung darunter werden erst dann ausgeführt, wenn der Vorgang bis dahin nicht abgebrochen wurde.

MrSKY schrieb:

kann man auch das besser verpacken in diese Methode?

Die Frage versteh ich nicht. Ja, nach dem If … Return … End If kommt der TextBox_EQACommand.Text = -Code, also der CommandTextErstellungscode rein. Wenn Dein bisheriger Erstellungscode läuft, nimm ihn. Wenn er klappt, poste ihn, dann können wir uns über Verkürzungen/Optimierungen Gedanken machen.
Grundsätzlich sollte es ja so laufen, wie Du es angefangen hast. Aber Moment mal. Wie stellst Du Dir das mit den Rollen vor? Du hast da jetzt EQASwordsman() drin und früher EQAAdventurer(). Das klingt nach einer seeehr langen Funktion. Aber wurscht. Wie geschrieben. Erstmal den Erstellungscode zum Laufen bringen, optimieren kann man später noch.

Das If am Anfang hast Du mit AndAlso richtig geschrieben.
Warum machst Du das danach nicht weiter?
Außerdem werden in VB.NET Strings mit & verknüpft:

MrSKY schrieb:

Aber ändere ich bei Programstart z.B. nur Upgrade dann bricht das Programm zusammen was "gefixt" wurde durch den Code darüber.

Was soll das konkret heißen? Dass Du keine Spielerrolle angewählt hast und nur was in der Upgrade-ComboBox? Wo bricht der Code mit welcher Fehlermeldung ab? In der CreateNewCommandText() wohl nicht, denn das wird ja am Anfang abgefragt.

Wie sieht denn bisher Dein Code zur Erstellung des CommandTexts im Gesamten aus? Vielleicht finden wir so das Problem.

Wenn das von Dir verlinkte Video Dein Projekt zeigt, ist der vorhandene Code ja schon fast vollständig und bedarf nur kleiner Anpassungen. Aber solange ich nicht weiß, wie der aktuelle Code aussieht, weiß ich leider auch nicht, wie ich Dir jetzt noch weiterhelfen kann. Unterstütze bitte den Lösungsfindungsprozess, indem Du den inzwischen vorhanenen Code zur Erstellung des CommandTexts postest.

Na, da biste aber trotz der zahlreichen Posts nicht weit gekommen …

Ah, eine konkrete Fehlermeldung.
Na, dann ist CommandText wohl woanders bei Dir deklariert als bei mir. Bei mir: Deklaration (= Bekanntgabe) in Zeile#4 und Endnutzung in Zeile#12. Beides im gleichen Scope (Sichtbereich für den Compiler). Und bei Dir? Hm, keine Ahnung, da sieht man nicht viel. Da solltest Du wohl besser die komplette Methode screenshotten (?). Oder besser gleich einfach den Methodencode an sich posten …

btw: Bilder solltest Du forenintern über [+ Erweiterte Antwort] -> _|Dateianhänge|_ -> [Hochladen] posten. Nicht wenige (Power)User hier klicken ungern auf ImageHosterLinks und stellen stattdessen die Hilfe hier im Thread ein bzw. fangen gar nicht erst mit Hilfe an. Deine Entscheidung.