AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren

Hilfe zu Funktion "DeleteChars"

Ein Thema von Pseudemys Nelsoni · begonnen am 17. Aug 2005 · letzter Beitrag vom 17. Aug 2005
Antwort Antwort
Benutzerbild von Pseudemys Nelsoni
Pseudemys Nelsoni

Registriert seit: 24. Dez 2002
Ort: Hamburg-Harburg
3.551 Beiträge
 
#1

Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 13:02
Moin,

Delphi-Quellcode:
function DeleteChars(const S: string; CharSet: TCharSet; Delete: TDelete): string;
var
  PResIdx: PChar;
  PSIdx: PChar;
  ResLen: Integer;
begin
  SetLength(Result, Length(S));
  PResIdx := @Result[1];
  PSIdx := @S[1];
  ResLen := 0;
  while PSIdx^ <> #0 do
  begin
    if ((Delete = deExcept) and (PSIdx^ in CharSet)) or ((Delete = deRemove) and not (PSIdx^ in CharSet)) then
    begin
      PResIdx^ := PSIdx^;
      Inc(ResLen);
    end;
    Inc(PSIdx);
    Inc(PResIdx);
  end;
  SetLength(Result, ResLen);
end;
Kann mir jemand sagen was hier falsch ist?Die Funktion soll mir wenn "Delete" = "deRemove" ist alle Zeichen die im CharSet enthalten sind, aus dem String "S" löschen, wenn "Delete" = "deExcept" ist, sollten alle Zeichen gelöscht werden, die nicht im CharSet sind.

Wenn ich die Funktion nun so aufrufe:

Zitat:
showmessage(DeleteChars('abc.def', ['.'], deremove));
Kriege ich als Ergebnis "abccde", eshätte aber "abcdef" sein müssen.

Jemand ne Idee?


EDIT: Hab schon, das "Inc(PResIdx);" musste woanders hin.

Aber ist die Funktion ansonten richtig? Ist meine Erste FUnktion mit Pointern.
Mario
  Mit Zitat antworten Zitat
Robert Marquardt
(Gast)

n/a Beiträge
 
#2

Re: Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 13:11
Unbedingt S = '' abfangen.
Das "Inc(PResIdx);" gehoert in den if-Block.
  Mit Zitat antworten Zitat
Benutzerbild von himitsu
himitsu
Online

Registriert seit: 11. Okt 2003
Ort: Elbflorenz
44.020 Beiträge
 
Delphi 12 Athens
 
#3

Re: Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 13:13
Delphi-Quellcode:
    begin
      PResIdx^ := PSIdx^;
      Inc(ResLen);
    end;
Der Fehler ist, dass nur das Zeichen, welches dem zu löschenden Zeichen folgt um eines nach vorne verschoben wird.
Aber es müßn alle nachvolgenden Zeichen verschoben werden.

In deinem Beispiel soll der Punkt gelöscht werden und es wird deinem Code entsprechenden das c um ein Beit nach vorne kopiert.

Du solltes also statt PResIdx^ := PSIdx^; eine Funktion wie Move, oder MemCopy verwenden, um alle nachfolgenden Zeichen, bis zum Stringende zu verschieben.


[add]
Oh, hab mich wohl verlesen ... dachte dass due den Resultstring vorher schonmal gesetzt hättest und in der Schleife die unerwünschten Zeichen wieder entfernst.

Aber versuch es mal so
Delphi-Quellcode:
if S <> 'then
  while PSIdx <> #0 do begin
    if ((Delete = deExcept) and (PSIdx^ in CharSet)) or
      ((Delete = dremove) and not (PSIdx^ in CharSet)) hen begin
      PResIdx^ := PSIdx^;
      Inc(ResLen);
      Inc(PResIdx);
    end;
    Inc(PSIdx);
  end;
Also Robert schein recht zu haben.

PS: Als guter Programmierer könntest du ja ResLen weglasen ... die entsprechende Länge kann man ja leicht aus der Ausgansgposition (@Result[1]) und der Position des letzten ResultZeichens (PResIdx) errechnen
Garbage Collector ... Delphianer erzeugen keinen Müll, also brauchen sie auch keinen Müllsucher.
my Delphi wish list : BugReports/FeatureRequests
  Mit Zitat antworten Zitat
Robert Marquardt
(Gast)

n/a Beiträge
 
#4

Re: Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 13:52
Die Logik im if ist falsch herum.
  Mit Zitat antworten Zitat
Benutzerbild von Pseudemys Nelsoni
Pseudemys Nelsoni

Registriert seit: 24. Dez 2002
Ort: Hamburg-Harburg
3.551 Beiträge
 
#5

Re: Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 21:45
Moin,

danke für die Hilfe, habs alles soweit abgeändert .

Noch eine Frage zu einer anderen Funktion und zwar:

Delphi-Quellcode:
function StripCharsLeft(const S: string; CharSet: TCharSet): string;
var
  PResIdx: PChar;
  PSIdx: PChar;
  Idx: Integer;
  SLen: Integer;
begin
  SetLength(Result, Length(S));
  if S <> 'then
  begin
    PResIdx := @Result[1];
    PSIdx := @S[1];
    Idx := 0;
    SLen := Length(S);
    while (PSIdx^ <> #0) and (PSIdx^ in CharSet) do
    begin
      Inc(PSIdx);
      Inc(Idx);
    end;
    if PSIdx^ <> #0 then
    begin
      MoveMemory(@Result[1], PSIdx, SLen-Idx);
      SetLength(Result, SLen-Idx);
    end;
  end;
end;
Und zwar soll die Funktion einzelne Zeichen am Anfang des Strings entfernen, solange bis ein Zeichen kommt, das nicht im CharSet vorkommt.

Hab ich hier alles richtig gemacht?

Also der Aufruf von "showmessage(StripCharsLeft('hallo', ['h', 'a']));" scheint mit "llo" zu stimmen, nur trotzdem wüsst ich gern ob alles soweit richtig ist? Z.B MoveMemory? Könnte es nicht Probleme gebe wenn ich einige Zeichen einfach verschiebe? Z.B wenn Delphi den String von dem ich verschoben habe, freigibt?
Mario
  Mit Zitat antworten Zitat
Benutzerbild von himitsu
himitsu
Online

Registriert seit: 11. Okt 2003
Ort: Elbflorenz
44.020 Beiträge
 
Delphi 12 Athens
 
#6

Re: Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 21:55
Auf den ersten Blick kann ich auch nichts daran entdecken.


Und wenn Zeichen innerhalb eines Strings verschiebst, dann gibt es keine Probleme.
Es ist natürlich auch möglich Zeichen nach oder von woanders zu kopieren ... du mußt halt nur aufpassen, dass die die Daten nur innerhalb von Bereichen abspeicherst, welche auch freigegeben werden ... und die Strings/Arrays werden ja automatisch am Funktions-/Prozedurende freigegeben.
Garbage Collector ... Delphianer erzeugen keinen Müll, also brauchen sie auch keinen Müllsucher.
my Delphi wish list : BugReports/FeatureRequests
  Mit Zitat antworten Zitat
Benutzerbild von Pseudemys Nelsoni
Pseudemys Nelsoni

Registriert seit: 24. Dez 2002
Ort: Hamburg-Harburg
3.551 Beiträge
 
#7

Re: Hilfe zu Funktion "DeleteChars"

  Alt 17. Aug 2005, 23:50
Jop, das meine ich ja, kann Delphi nicht Probleme haben wenn ich einfach ein paar Zeichen des Original-Strings in einen Anderem verschiebe, beim Free'en des Selben?
Mario
  Mit Zitat antworten Zitat
Themen-Optionen Thema durchsuchen
Thema durchsuchen:

Erweiterte Suche
Ansicht

Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 14:43 Uhr.
Powered by vBulletin® Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024 by Thomas Breitkreuz