schnelleres StringReplace und MultiStringReplace

[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ß.

[youuu]

Hi, ich verwende deine Stringreplace Routine, allerdings erhalte ich einen Fehler hier:

Delphi-Quellcode:

  Function StringReplaceX(Const S: String; Const OldPattern, NewPattern: Array of String; Flags: TReplaceFlags = []): String;
    Function GetNewPatt(i: Integer): String;
      Begin
        If i < Length(NewPattern) Then Result := NewPattern[i]
        Else If Length(NewPattern) = 1 Then Result := NewPattern[1]
        Else Result := '';
      End;

Fehlermeldung: Erste Gelegenheit für Exception bei $75F89617. Exception-Klasse EAccessViolation mit Meldung 'Zugriffsverletzung bei Adresse 75F89F11 in Modul 'KERNELBASE.dll'. Lesen von Adresse 00C81000'.

Wenn ich die Exception abfange, rennt er mit OutOfMemory

[mkinzler]

In welcher Zeile?
Was hast du als OldPattern bzw. NewPattern übergeben?

[youuu]

   Else If Length(NewPattern) = 1 Then Result := NewPattern[1] Ist rot markiert, was genau übergeben wurde, werde ich nun nach schauen

[himitsu]

joar, da stimmt was nicht.
tausch mal die 1 gegen eine 0 aus Result := NewPattern[0] [edit]
oder besser so ... ich denk mal das wäre logischer, wenn der Letzte, anstatt der erste Eintrag genutzt würde.

Delphi-Quellcode:

Function GetNewPatt(i: Integer): String;
  Begin
    If i < Length(NewPattern) Then Result := NewPattern[i]
    Else If Length(NewPattern) > 0 Then Result := NewPattern[High(NewPattern)]
    Else Result := '';
  End;

ich tausche die Anhänge auch gleich aus

[youuu]

Himitsu, das wars !

[himitsu]

ach mißt, das läßt sich ja garnicht editieren
(und das fehlende [PRE]-Tag fällt auch auf ... also bezüglich der nicht mehr geparsten Format-Tags )

@Mods: Arbeit für euch > Anhänge dieses Post im ersten Post hochladen

[edit]
Anhänge wurden nun nach Beitrag #1 kopiert, drum lösch ich sie hier mal wieder

[freejay]

Hi himitsu,

danke für diese schnelle StringReplace-Version!

Ich konnte die Dauer in einem Fall von 32 Minuten auf eine knappe Sekunde drücken! Das ist ca. Faktor 2000!

Danke & Gruß

Freejay

[p80286]

@Himitsu

Ich hab eine 25MB große Stringlist.
Ansireplacestring braucht irgendwas um die 30 Minuten, etwas selbstgebasteltes ca 12 und Himitsus irgendwas im 1-5 Sekunden Bereich.

Vielen Dank!

K-H