schnelleres StringReplace und MultiStringReplace

[himitsu]

Hmmm, hast du mal einen TestString parat?
Mir ist grade aufgefallen, daß in meinem Testcode das Pattern auch schon am Ende des Strings liegt.

und ebenso hier tritt kein Fehler auf.

Delphi-Quellcode:

S := '123Wort';
S2 := StringReplace(S, 'Wort', 'xxx', [rfReplaceAll]);

[GPRSNerd]

Steht doch oben als Delphi Code!

Ich benutze Delphi 2010 und du?

[GPRSNerd]

Dein Beispiel knallt bei mir auch!

exception number : 1
exception class : ERangeError
exception message : Range check error.

main thread ($17c4):
005376bc +03dc STGCommonUnitTest.exe StrRep 132 +37 StringReplace

[himitsu]

Das hatte ich vorhin auch zum Testen verwendet.

Welche Version nutzt du denn?
Die _300 aus Beitrag #1 sollte die aktuellere Version 1.1 vom 20.11.2009 22°° sein.
(is'n bissl blöd, das mit den nun fehlenden Attachment-Kommentaren und den veränderten Dateinamen)



Ich teste gleich nochmal 'n bissl rum.
(bin grade auf die Idee gekommen mal wieder die Bereichsprüfung zu aktiveren )

[GPRSNerd]

Ich benutze "natürlich" die v1.1.

Wenn ich die Move-Zeile mit einer If-Abfrage auf i4<>0 ändere, geht alles gut:

if i4<>0 then Move(S[i], Result[i2], i4 * SizeOf(Char));

[himitsu]

Jupp, genau sowas hatte ich grade verbaut.

Hach ja, der Fehler ist in meinem aktiven Code nie aufgefallen,
aber dazu sei vielleicht erwähnt, daß der Originalcode eine andere Stringverwaltung via PChar besitzt.

ähhhh ... ich würde es ja gerne Anhängen, aber das geht nicht.

Also bitte vor alle "Move" das "if i4 <> 0 then " einfügen.

[GPRSNerd]

Bin gerade mal Step-by-Step da durchgegangen mit deinem Beispiel mit 123Wort:

i=8
Length(S)=7
i2=7
i4=0

Der Knall kommt dann vom S[i]=S[8] bei einem String mit 7 Zeichen.

[himitsu]

Jupp, wenn dieses der letzte Fund ist, dann wird versuchst den restlichen nachfolgenden String in das Result zu kopieren.
Der selbe Fehler sollte also auch auftreten, wenn der Pattern mehrfacht direkt hintereinander liegt.

Ohne Bereichsprüfung passiert da eigentlich auch nichts, da Move mit einer Länge von 0 aufgerufen würde,
aber mit Bereichsprüfung bekommt diese natürlich mit, daß der Zeichenindex nach dem Fund natürlich außerhalb des Strings liegt.
Daß letztendlich aber dann doch kein Zugriff auf diesen Index passiert, Aufgrund der Länge=0, bekommt dieser Prüfcode natürlich nicht mit.

Im Originalcode hatte ich die Prüfung von MoveMemory ausgenutzt und daher keine Eigene verbaut,
was allerdings bei dem direkten Zugriff auf den String (via S[i]) nicht mehr so gut enden muß.