String umdrehen

[Aphton]

Zitat von p80286:

Zitat von Aphton:

Delphi-Quellcode:

function ReverseSentence(const S: String; const Seperator: Char = ' '): String;
var
  i, Len,
  p: Integer;
  Word: String;
begin

das "Word" halte ich für nicht so gelungen, da es auch den Typ Word gibt(...)

Da bin ich anderer Meinung!

@Himitsu - ja, da hast du Recht! Das kann zu Problemen führen.
Was das "Move" angeht - ich finde, so dürfte das schneller sein?!

Edit: Das sollte separator heißen

[rollstuhlfahrer]

Zitat von Aphton:

@Himitsu - ja, da hast du Recht! Das kann zu Problemen führen.
Was das "Move" angeht - ich finde, so dürfte das schneller sein?!

Ab D2009 (mit Unicode) führt das zu Problemen. Man kann diese Version aber etwas abändern und dann wird sie in allen weiteren Delphi-Versionen funktionieren (auch mit UTF-32):
Move( Word[1], Result[i], p * SizeOf(Char)); .
Das SizeOf(Char) ist der Multiplikator. Vor D2009 ist der 1 (weil 1 Zeichen -> 1 Byte), danach ist der 2 (weil 1 Zeichen -> 2 Bytes oder so), auf jeden Fall ist der dann zu der Delphi-Version passend und es sollte nicht zu Problemen kommen.

Bernhard

[himitsu]

Allerdings muß man nicht immer alles extremst optimieren,
denn soo oft wird diese Funktion bestimmt nicht aufgerufen, als daß da 'ne Microsekunde wirklich auffällt ... in soeinem Fall lieber einfachen Code verwenden, da weniger fehleranfällig und übersichtlicher.

[-187-]

Tja grade wenn er eine Aufgabe für die Schule bearbeitet wird er glaube ich mit Move nicht glücklich werden

Das sind einfach keine Basics mehr...

[roboter202]

Hallo,

umdrehen geht auch so

Delphi-Quellcode:

for i := 1 to length(input) do
  begin
  output := output + input[length(input)-(i-1)]
  end;

Warum immer so Kompliziert

[Aphton]

Das ist sehr unperformant, denn du fügst einem String einen weiteren String hinzu, was zu einem stetigen Erweitern und somit Speicher neu allozieren führt, das ja bekannterweise langsam ist!

[roboter202]

Hallo,

Zitat von Aphton:

Das ist sehr unperformant, denn du fügst einem String einen weiteren String hinzu, was zu einem stetigen Erweitern und somit Speicher neu allozieren führt, das ja bekannterweise langsam ist!

Das mag sein allerdings würde es funktionieren und wenn man nicht gerade von einem string mit millionen von zeichen ausgeht würde es nicht merklich länger dauern

[Aphton]

Klar, ich habe ja die Funktionalität gar nicht bezweifel
Aber wissen sollte man das schon, deshalb habe ich es auch erwähnt.