String.Split Methode Delimiter mehrmals vorhanden

[Dejan Vu]

Zitat von Sir Rufo:

Oder man sucht einfach nach dem ersten Vorkommen des Trennzeichens (Komma) und trennt an der Stelle den String einfach auf.

Siehe den Vorschlag von himitsu und meine preisverdächtigen und zum Patent angemeldeten Code.

[Sir Rufo]

Zitat von Dejan Vu:

Zitat von Sir Rufo:

Oder man sucht einfach nach dem ersten Vorkommen des Trennzeichens (Komma) und trennt an der Stelle den String einfach auf.

Siehe den Vorschlag von himitsu und meine preisverdächtigen und zum Patent angemeldeten Code.

Habe ich doch gesehen (und ja auch so beschrieben) ...

allerdings habe ich an deinem Code etwas auszusetzen. Eine Veränderung des Übergabewertes bei dieser Funktion kommt eher überraschend und zwingt den Entwickler dazu, den String vorher zu kopieren, wenn dieser Wert noch benötigt wird oder als const Parameter übergeben wurde.

Generell ist es besser, diese Funktion so zu schreiben (wie auch die vorhandene String.Split Funktion), dass dort ein Array zurückgegeben wird und der Übergabewert nicht verändert wird.

Und in XE4 gibt es auch diese Überladungen System.SysUtils.TStringHelper.Split (DocWiki Delphi XE4) somit ist die Diskussion ja schon fast obsolet

[himitsu]

Zum Glück hatte ich Mikkey's Vorschlag noch nicht erwähnt.

@Dejan Vu: Joar, eine passende Methode ist schon gut, aber wäre es nicht besser die anders zu nennen und nicht genauso, wie die Andere?

[Dejan Vu]

Ich verstehe die Diskussion nicht. Da will jemand einen String in zwei Teile teilen:
'A,B,C' => 'A' und 'B,C'. Das macht die fossile Funktion 'Split' aus meinem Fundus. Da wird kein Array benötigt und nichts.

Natürlich habt ihr recht (blöder Name und blöder Seiteneffekt mit 'Var'). AAAAABER!!!1!!!11!!eins!!!EINS!!ELF

Ich habe diese Methode aus den 70er Jahren des letzten Jahrhunderts nur vorgestellt, um zu zeigen, was für ein toller Superprogrammierer ich bin wie alt die Notwendigkeit für das Aufteilen von Strings ist. Als ich das geschrieben habe, konnte man keine Funktionen schreiben, die Arrays liefern (BASIC, wer kennt das noch?) und seitem habe ich das. Wozu soll ich was anderes nehmen?
Zum Thema 'einen anderen Namen wählen': Wenn sich hier einer einen anderen Namen ausdenken muss, dann ja wohl die Leute von .NET und Emba und nicht ich Als ich mein 'Split' gebaut habe, gab es die Programmierer noch gar nicht (David I vielleicht, aber sonst?).... (Und wenn jetzt noch jemand drauf eingeht, werde ich anfangen, laut zu weinen)

Die Routine ist doch super zu gebrauchen (so wie sie ist). Wenn Performance nicht so die Rolle spielt, kann ich damit sehr schön einen String zerteilen und während des Zerteilens gleich prüfen, ob alles ok ist. Oder zuweisen

Delphi-Quellcode:

// Zeit|Datum|Prüfnr|
procedure Zerteile (zeile :String);
Begin
  zeit := split(zeile,'|');
  datum := split(zeile,'|');
  Prüfnr := split(zeile,'|');
end;

Mach das mal mit einem Array. (Klar, geht). Aber nun... Hups, jetzt haben die noch die Terminalnummer vorne ran gestellt. Na und?

Delphi-Quellcode:

// TerminalNr|Zeit|Datum|Prüfnr|
procedure Zerteile (zeile :String);
Begin
  terminal := split(zeile,'|');
  zeit := split(zeile,'|');
  datum := split(zeile,'|');
  Prüfnr := split(zeile,'|');
end;

Bei Verwendung eines Arrays muss ich nun die Indexe nachpflegen, die ich natürlich als Konstanten deklariert habe (was für ein Aufwand)...
Einfacher also so gehts nicht. Oh, doch. Clean Code.

Delphi-Quellcode:

procedure Zerteile (zeile :String);
  function _naechsterTeilString:String;
  begin
    result := split(zeile,'|');
  end;
Begin
  terminal := naechsterTeilString;
  zeit := naechsterTeilString;
  datum := naechsterTeilString;
  Prüfnr := naechsterTeilString;
end;

Mach das mal mit dem Split, was Arrays liefert in weniger Zeilen, leichter verständlich und leichter erweiterbar... Kriegt man nicht hin, behaupte ich mal.

Das nur mal so zur Verteidigung gegen Angriffe des sich nicht wehren könnenden Codeschnipsels aus dem 1970ern. Man Leute, der ist grenzdebil (der Code, nicht der Coder)

Allerdings: Heute mit Delphi XY würde ich vermutlich auch anfangen, meine Stringroutinen zu entstauben und diese Split-O-Mat Routine nicht mehr verwenden. Aber darum gehts ja hier nicht, sondern....


... um das Aufteilen eines Strings an einem Trennzeichen in den Teil davor und den Teil danach... Oder?

[Stevie]

Zitat von Sir Rufo:

Und in XE4 gibt es auch diese Überladungen System.SysUtils.TStringHelper.Split (DocWiki Delphi XE4) somit ist die Diskussion ja schon fast obsolet

Also zumindest in meinem XE6 machen die nicht, was der TE möchte, sondern Count gibt hier nur an, wie viele elemente maximal man im Result haben möchte. Also würde Split([','], 1) nur ein array mit einem Element zugeben, in dem 'Hallo' stünde.

Und übrigens:

Delphi-Quellcode:

meinString := 'Hallo,ich,bin,ein,String';
kommaPos := meinString.IndexOf(',');
teil1 := meinString.Substring(0, kommaPos);
teil2 := meinString.Substring(kommaPos + 1);

[Uwe Raabe]

Zitat von Stevie:

Also zumindest in meinem XE6 machen die nicht, was der TE möchte, sondern Count gibt hier nur an, wie viele elemente maximal man im Result haben möchte. Also würde Split([','], 1) nur ein array mit einem Element zugeben, in dem 'Hallo' stünde.

Wo du recht hast...

[himitsu]

Zitat von Stevie:

Und übrigens:

Delphi-Quellcode:

meinString := 'Hallo,ich,bin,ein,String';
kommaPos := meinString.IndexOf(',');
teil1 := meinString.Substring(0, kommaPos);
teil2 := meinString.Substring(kommaPos + 1);

Sind diese Methoden auch in Win32 0-basierend?

[Dejan Vu]

Zitat von Stevie:

Und übrigens:

Knallts, wenn kein Komma vorhanden ist
Und übrigens: Die Lösung hatten wir schon hier.

Zitat von himitsu:

Jupp, Pos und dann mit Copy den gewünschten Teil rauskopieren, wäre besser.
Split macht nunmal das, wofür es gemacht wurde. -> an allen Delimitern aufteilen.
Ansonsten bräuchte man kein dynamisches Array, als Result, sondern nur 2 Strings.
...

und hier.

Zitat von Dejan Vu:

Bei Ausgrabungen habe ich das hier gefunden. ...

und hier

Zitat von Sir Rufo:

Oder man sucht einfach nach dem ersten Vorkommen des Trennzeichens (Komma) und trennt an der Stelle den String einfach auf.

Wollen wir das Thema wechseln?

[Stevie]

Zitat von Dejan Vu:

Zitat von Stevie:

Und übrigens:

Knallts, wenn kein Komma vorhanden ist

Täts auch im Code, den der TE gepostet hat.
Ich trau ihm aber zu, ein if zu schreiben, was kommaPos überprüft...

Zitat von Dejan Vu:

Und übrigens: Die Lösung hatten wir schon hier.

Das muss wohl in der ganzen Metadiskussion untergegangen sein.
Ich dachte aber, ich poste das nochmal mit Verwendung der entsprechenden TStringHelper Methoden, denn:

Zitat von himitsu:

Sind diese Methoden auch in Win32 0-basierend?

Ja

Zitat von Dejan Vu:

Wollen wir das Thema wechseln?

Genau! back to OT

[sx2008]

Zitat von Dejan Vu:

... und meine preisverdächtigen und zum Patent angemeldeten Code.

Das sieht aber schlecht für Dich aus weil folgende Funktion ist schon Open Source

Delphi-Quellcode:

function StrToken(var S: string; Separator: Char): string; // kopiert aus der JCL
var
  I: Integer;
begin
  I := Pos(Separator, S);
  if I <> 0 then
  begin
    Result := Copy(S, 1, I - 1);
    Delete(S, 1, I);
  end
  else
  begin
    Result := S;
    S := '';
  end;
end;