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
Schlechte Lösung. Ich denke er hätte es gerne dynamisch wie die oberen beiden Vorschläge. :)Quote:
RichTextBox1.Text.Replace("Christian","").Replace( "Markus","");
so wäre es bei C#, sollte aber auch bei dir klappen.
Naja also von der Codelänge wesentlich besser :)Quote:
Welchen Vorteil bringt dann bitte diese Funktion? Dann kannst du auch direkt das Replace aufrufen wo du es gerade brauchst...Quote:
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.Quote:
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.
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.Quote:
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.Quote:
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.
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.Quote: