Lesen von .txt Datei und trennen

Nä, das ergibt überhaupt keinen Sinn. In Zeile#3 schreibst Du Return. Wenn diese Zeile erreicht wird, weil eine der Bedingungen zutrifft, dann bricht der Compiler die Methodenbearbeitung ab und verlässt sie. Das ist zwar richtig. Aber darunter als Folgeanweisung zu schreiben, wie CommandText zusammengebaut werden soll, ist absolut widersinnig. Diese Zeilen werden niemals erreicht, weil eben der Compiler vorher schon die Methode verlässt. Schau nochmal meinen Methodenvorschlag an und erkenne den relevanten Unterschied. Kleiner Hinweis: Ich habe keinen If … End If Block, sondern eine If-Befehlszeile, die mit einem Then Return endet.

##########

Ok, Du hast den Fehler erkannt. Aber was meinst Du mit TextBoxen durchklicken? Welche TextBoxen?

Ehh … Du musst die Methode CreateNewCommandText schon auch aufrufen, damit der Compiler da was macht. Aber das hatte wir ja schon vor gefühlten 20 Posts, dass Du diese Methode in den CheckedChanged-EventHandlern der RadioButtons und SelectedIndexChanged-EventHandler der ComboBoxen aufrufen musst.

Hä? Die CreateNewCommandText-Methode ist doch fertig. Haste doch selber gepostet. Die muss nur aufgerufen werden. Was ist jetzt das Problem? Der Aufruf? Steht in Post#84.
Wenn Du was anderes meinst, erklär es bitte ausführlich oder poste den gesamten Code der Form1-Klasse. Sonst weiß ich echt nicht mehr, wie ich helfen kann.

Kannst Du mal bitte das Projekt ohne bin-, obj-, .vs- und .git-Ordner und gezippt über [+ Erweiterte Antwort] hochladen? Vielleicht finden wir so das Problem und eine Lösung.

Tja, dann musst Du eben Haltepunkte setzen und selber debuggen. Einen Haltepunkt da setzen, wo Du erwartest, dass der Compiler beim Programmablauf vorbei kommt und dann durch Starten und Benutzen des Programmes prüfen, ob Deine Annahmen bzgl. des Programmablaufs/der Codebearbeitung zutreffen.

Ich seh jetzt erstmal nur: Wenn der Adventurer angeklickt wird, wird die Datei geladen und die Item-ComboBox anscheinend befüllt. Die anderen ComboBoxen vermutlich sind schon befüllt. Es wird nix (also wieder nixnix) in der ItemComboBox vom Programm ausgewählt (SelectedIndex wird auf -1 gesetzt). Wenn was in der ItemComboBox ausgewählt wird, wird eine Prüfung gemacht (da ist z.B. der Ansatz, ob die Prüfung plausibel ist) und dann die CreateNewCommandText-Sub aufgerufen. Damit könntest Du in Zeile#27 nen Haltepunkt setzen und so prüfen, ob die Methode an dieser Stelle wirklich aufgerufen wird.

schaust Du hier: Debuggen, Fehler finden und beseitigen

Sowas ist beim Prüfen gemeint:


Mit Deinem Rückgabewert kann ich überhaupt nix anfangen. Ist ohne Kontext nicht bewertbar.

Ok, jetzt weißt Du schon mal, dass der Compiler die Methode aufrufen wird. Dann ist der nächste Schritt: Haltepunkt in die CreateNewCommandText-Methode setzen, und zwar bei TextBox_CommandA.Text = CommandText. Wenn der Compiler dort ankommt, ist das ein gutes Zeichen. Dann kannst Du prüfen, was in CommandText drinsteht und ob das korrekt ist.

Dann musst Du dieses Item eben noch in den CommandText einbauen. Das wird ja unverändert übernommen, oder? Ach Moment, nee, das ist irgend ne Kennziffer. Also "Der Eine Ring" hat die Nummer 666 oder sowas, richtig? Wo sind diese Wertpaare hinterlegt? Ist das das, was aus der Datei eingelesen wird? Also die Itemnamen in der ItemComboBox und die Kennzahlen in EQAAdventurer? Dann musst Du also nur CommandText aus dem Spielerrollen-Command, dem EQAAdventurer(ComboBox_EQAItem.SelectedIndex) und jenem Rest zusammensetzen, den ich in Post#97 aus den anderen beiden ComboBoxen zusammengeschustert habe.

Dann musst Du - aber das sollte offensichtlich sein - die CreateNewCommandText eben auch im ComboBox_EQAItem_SelectedIndexChanged-EventHandler aufrufen.

Und der Compiler kommt in der Zeile#8 vorbei, wenn die Items geladen sind, Du in Rarity und Upgrade was auswählst und dann das Item änderst? Zu prüfen mit einem Haltepunkt.

Ich blick's grad nicht, weil's widersprüchlich ist, was Du schreibst:

MrSKY schrieb:

Hab das nun EQAAdventurer(ComboBox_EQAItem.SelectedIndex) hinzugefügt und es klappt nun jedoch erst wenn man etwas in Rarity und Upgrade ausgewählt hat das sollte davor auch gehen also sobald man was in der CBItems auswählt soll schon in der TextBox "$ci 666" z.B. kommen wie das anfangs war.

MrSKY schrieb:

Wie gesagt das einzige was bleibt das CBItem direkt in die TextBox ausgegeben wird bevor man überhaupt Rarity und Upgrade ausgewählt hat

Soll ein CommandText entstehen, wenn bei Rarity und Upgrade nix gewählt wurde oder nicht? Die beiden Zitate widersprechen sich da m.E. und daher weiß ich nicht, was das Sollziel ist.

Und bittebittebitte. Verwende Punkt und Komma. Oder Zeilentrennung. Allein

MrSKY schrieb:

Hab das nun EQAAdventurer(ComboBox_EQAItem.SelectedIndex) hinzugefügt und es klappt nun jedoch erst wenn man etwas in Rarity und Upgrade ausgewählt hat das sollte davor auch gehen also sobald man was in der CBItems auswählt soll schon in der TextBox "$ci 666" z.B. kommen wie das anfangs war.

ist sooo anstrengend zu lesen, weil man die Satzeinheiten nicht erkennt. Das schadet dem Verständnis extrem. Und am Ende führt sowas zu inhaltlichen Missverständnissen und somit evtl. zu falschen Lösungsansätzen/-vorschlägen. Dass meine Texte manchmal ne Katastrophe, da gerne verschachtelt sind, ist mir bewusst, ich bin also auch nicht gerade ein Dauervorbild.