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

[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);

[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;

[himitsu]

Gibt es im Delphi uch, nur hat man das seit Jahrzehnten vergessen öffentlich zur Verfügung zu stellen.
z.B. siehe die Behandlung von ParamStr

Man stelle sich mal vor die würden wichtige Dinnge offenlegen, dann würden z.B. alle Programme, welche einen Pascal-Parser benötigen plötzlich endlich mal enheitlich abeiten können und würden nicht jedesmal abrauchen, wenn Emba etwas neues in die Sprache einführt.


DokuInsight raucht bei den kleinsten Problemen ab.
Und CnPack ist auch grauenhaft (vorallem in einem langsamen RDP).

[Dejan Vu]

Zitat von sx2008:

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

Das ist ein drittklassiges Plagiat, alleine schon wegen der dilettantischen Verwendung von 'Delete', und dann diese Codekomposition (oh Graus), und dann auch nur ein 'Char' als Trenner. Nee, also. Nee. (Wieso gibt es kein Smiley, welches sich den rechten Handrücken an die Stirn hält und den Kopf nach links oben dreht?)

Ich melde zudem prior art an. Ich kann belegen, das dieser Code im Jahr 1979 in kommerziellen Programmen Verwendung gefunden hat. Es wäre nicht das erste Mal, das sich die Opensource-Gemeinde schadlos an nicht freigegebenen Code gehalten hat. Ich sage nur: 'Schöpfungshöhe' U..u..und Embarcadero wird auch in Grund und Boden geklagt.

Reich! Reich! Ich werde reich!

[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?