VB.NET Text sortieren

[Temx Keys]

Hallo,

ich habe eine Frage und zwar habe ich eine RichTextBox1 mit Inhalt von Text:

Herr Christian Wolf
Herr Markus Stein

Wie kann ich den Vornamen also Christian & Markus löschen, sodass am Ende nur der Nachname da steht?

[IceTrailer]

Mit String.Split solltest du den String (RichTextBox1.Text) splitten und entsprechend einen neuen String bauen, ohne den Vornamen.

[xShizoidx]

1. Du musst alle Wörter pro Zeile einzeln haben. Dies kannst du z.B. machen, indem du alles in einer Zeile bei jeder Leertaste mit der Split Funktion trennst.
2. Du entfernst in jeder Zeile einfach jedes zweite Wort, welches du zuvor ja mit der Split Funktion ermittelt hast

Ergebnis:

        Dim result As String

        For Each line As String In RichTextBox1.Lines
            Dim parts = line.Trim().Split(" ")
            result += line.Replace(parts(1), "") + Environment.NewLine
        Next

        RichTextBox1.Text = result

[ℓ℮ዞni™]

Falls du es noch nicht alleine hinbekommst kann ich dir gerne per TV helfen

[Onkelmat]

RichTextBox1.Text.Replace("Christian","").Replace( "Markus","");

so wäre es bei C#, sollte aber auch bei dir klappen.

[Ludder231]

Originally Posted by Onkelmat

RichTextBox1.Text.Replace("Christian","").Replace( "Markus","");

so wäre es bei C#, sollte aber auch bei dir klappen.

Schlechte Lösung. Ich denke er hätte es gerne dynamisch wie die oberen beiden Vorschläge.

[Onkelmat]

Originally Posted by Ludder231

Schlechte Lösung. Ich denke er hätte es gerne dynamisch wie die oberen beiden Vorschläge.

Naja also von der Codelänge wesentlich besser
und dynamisch wäre ja wohl auch kein Problem:

private voidLöscheVornamen(string Vorname1, string Vorname2)
{
RichTextBox1.Text.Replace(Vorname1,"").Replace( Vorname2,"");
}

er könnte natürlich auch ein Objekt mit den zu entfernenden Namen übergeben und dann mit einer Schleife alle löschen.

[Serraniel]

Originally Posted by Onkelmat

Naja also von der Codelänge wesentlich besser
und dynamisch wäre ja wohl auch kein Problem:

private voidLöscheVornamen(string Vorname1, string Vorname2)
{
RichTextBox1.Text.Replace(Vorname1,"").Replace( Vorname2,"");
}

er könnte natürlich auch ein Objekt mit den zu entfernenden Namen übergeben und dann mit einer Schleife alle löschen.

Welchen Vorteil bringt dann bitte diese Funktion? Dann kannst du auch direkt das Replace aufrufen wo du es gerade brauchst...

[xShizoidx]

Originally Posted by Onkelmat

Naja also von der Codelänge wesentlich besser
und dynamisch wäre ja wohl auch kein Problem:

private voidLöscheVornamen(string Vorname1, string Vorname2)
{
RichTextBox1.Text.Replace(Vorname1,"").Replace( Vorname2,"");
}

er könnte natürlich auch ein Objekt mit den zu entfernenden Namen übergeben und dann mit einer Schleife alle löschen.

Deine Methode macht es eher unübersichtlicher, weil du eine void Methode hast, welche einfach irgendwelche Werte verändert, welche du nicht die Methode übergeben hast, sodass jemand nicht direkt aus der Ferne sieht, dass sich dort Werte ändern. Zudem macht es nicht wirklich Sinn für einen einfachen Replace eine Methode zu schreiben.

[Onkelmat]

Originally Posted by xShizoidx

Deine Methode macht es eher unübersichtlicher, weil du eine void Methode hast, welche einfach irgendwelche Werte verändert, welche du nicht die Methode übergeben hast, sodass jemand nicht direkt aus der Ferne sieht, dass sich dort Werte ändern. Zudem macht es nicht wirklich Sinn für einen einfachen Replace eine Methode zu schreiben.

Naja also eine Zeile ist wohl übersichtlicher als die for Geschichte von xShizoidx. Mit mehr Hintergrund könnte man auch meine Methode entsprechend anpassen. Ich fands als Methode geschrieben verständlicher was ich machen würde.


Ich weiß ja nicht genau was er haben will - das sind Möglichkeiten. Er kann wie Serraniel es sagt auch direkt die Replace Methode verwenden. Ich weiß nicht ob er einfach einmalig die zwei Namen löschen will oder ob er das dynamisch machen will.
Wenn man sich damit länger beschäftigt geht alles besser und kürzer.

[xShizoidx]

Originally Posted by Onkelmat

Naja also eine Zeile ist wohl übersichtlicher als die for Geschichte von xShizoidx. Mit mehr Hintergrund könnte man auch meine Methode entsprechend anpassen. Ich fands als Methode geschrieben verständlicher was ich machen würde.


Ich weiß ja nicht genau was er haben will - das sind Möglichkeiten. Er kann wie Serraniel es sagt auch direkt die Replace Methode verwenden. Ich weiß nicht ob er einfach einmalig die zwei Namen löschen will oder ob er das dynamisch machen will.
Wenn man sich damit länger beschäftigt geht alles besser und kürzer.

Bei mir ist das ein Lösungsvorschlag und nicht direkt um eine Methode, welche ich für ihn erstellen will, Methoden kann er sich immer noch selber zusammenbasteln, falls er welche benötigt, dadurch lernt er dann auch noch etwas. Wie gesagt sorgt dein Vorschlag, eine Replace Methode in eine seperate von dir erstellte void Methode zu packen eher für Unübersichtlichkeit. Wenn du eine Methode ohne Rückgabewert ausführst, dann erwartest du auch nicht direkt, dass diese etwas verändert, was du nicht übergeben hast. Schau dir mal die Stringfunktionen von VB.Net an. Replace gibt ja auch etwas zurück und zwar einen String, darum geht es mir, wenn es um Übersichtlichkeit geht.

[Onkelmat]

Ja, auf jeden Fall haben wir de Frage wohl gelöst. Er kann sich ja jetzt das zusammensuchen was er braucht. Eine Methode mit Rückgabe wäre ok, jeder wie er´s will

[th0rex]

Originally Posted by xShizoidx

Wenn du eine Methode ohne Rückgabewert ausführst, dann erwartest du auch nicht direkt, dass diese etwas verändert, was du nicht übergeben hast. Schau dir mal die Stringfunktionen von VB.Net an. Replace gibt ja auch etwas zurück und zwar einen String, darum geht es mir, wenn es um Übersichtlichkeit geht.

Was? Wo lebst du denn. Wir sind hier in einer objekt orientierten Sprache, da ändern nicht konstante Funktionen (die C# bzw. VB.net afaik ja nichtmal hat) doch andauernd irgendeinen State, und das erwartet man auch.