Zeichen in Memo ersetzten und Formatierung beibehalten?!

[Uwe Raabe]

Zitat von Luckie:

Und wieder fehlt der entscheidende Code: coding.

Wieso?

Zitat von Rawsche:

Delphi-Quellcode:

procedure TForm1.Coding(var x:string);
var i:integer;
begin
  for i:=0 to listbox1.items.count-1 do
  if x=listbox1.items[i] then x:=listbox2.items[i];
end;

[Uwe Raabe]

OK, hier läuft was falsch:

Zitat von Rawsche:

Delphi-Quellcode:

      for i:=1 to Length(Memo1.lines.text) do
      begin
        x:=coding(memo1.lines.Strings[i]);
        memo2.lines.Strings[i]:=x;
      end;

Memo1.Lines.Text gibt den gesamten Text inklusive Zeilenumbrüche zurück. Memo1.lines.Strings[i] gibt aber die i-te Zeile zurück und nicht das i-te Zeichen.

Weiterhin

• erwartet coding einen variablen string, bekommt aber ein string-Property
• coding gibt keinen Rückgabewert, weswegen man es auch nicht einem x zuweisen kann

[Luckie]

Zitat von Uwe Raabe:

Zitat von Luckie:

Und wieder fehlt der entscheidende Code: coding.

Wieso?

Zitat von Rawsche:

Delphi-Quellcode:

procedure TForm1.Coding(var x:string);
var i:integer;
begin
  for i:=0 to listbox1.items.count-1 do
  if x=listbox1.items[i] then x:=listbox2.items[i];
end;

Ups. Ich hatte eine Methode gesucht, die was mit einem String "macht", eben codiert, wie auch imner, und nicht irgendwelchen Code der was mit VCL Steuerelementen macht.

[KodeZwerg]

Zitat von Rawsche:

Ich freue mich auf eure Ideen. Bitte berichtigt NICHT einfach den Code, sondern gebt mir vielleicht lieber Tipps, wie ich ihn selbst verbessern kann. Sonst lerne ich daraus leider nichts...

Ich würde, sobald Dein visuell Nachvollziehbares Produkt soweit funktioniert das Du Verstanden hast Wie & Wieso es nun funktioniert folgendes Vorschlagen:
Steige von ListBoxen für Deine Umwandlung auf zum Beispiel ein Array oder Set um. Auf diese Weise blendest Du für andere aus woraus Deine Umwandlung basiert / non-Visuell sozusagen.

[Rawsche]

Schonmal Vielen Dank

[API]

...

[Rawsche]

Hi,

bis auf ein kleines Problem habe ich Fortschritte gemacht. Ich bin auf Funktionen umgestiegen. Der Text lässt sich umwandeln und auch wieder zurückwandeln.

Ein Problem tritt nur auf. Bei mir wird in der Verschlüsselung jedes Leerzeichen durch ein ß ersetzt. Jedes ß wird zum Leerzeichen.
Bei der Entschlüsselung ist es umgekehrt. Soweit alles geplant.

Leider tritt bei jeder Umwandlung von einem ß ein Fehler auf. Das ß wird nicht durch ein Leerzeichen ersetzt, sondern es bleibt ein ß, egal ob Ver- oder Entschlüsselung. Kann es sein, dass ein ß nicht mit einem ß verglichen werden kann oder dass ein bestehendes ß nicht durch ein Leerzeichen ersetzt werden kann?

Vielen Dank

Delphi-Quellcode:

unit Unit1;

{$mode objfpc}{$H+}

interface

uses
  Classes, SysUtils, FileUtil, Forms, Controls, Graphics, Dialogs, Grids,
  StdCtrls;

type

  { TForm1 }

  TForm1 = class(TForm)
    Button1: TButton;
    Button2: TButton;
    Label1: TLabel;
    Label2: TLabel;
    ListBox1: TListBox;
    ListBox2: TListBox;
    Memo1: TMemo;
    Memo2: TMemo;
    Memo3: TMemo;
    procedure Button1Click(Sender: TObject);
    procedure Button2Click(Sender: TObject);
    procedure Button3Click(Sender: TObject);
  private
    function Coding(x: string): string;
    function Decoding(x: string): string;

  public

  end;

var
  Form1: TForm1;

implementation

{$R *.lfm}

{ TForm1 }

function TForm1.Coding(x: string): string;
var
  i: integer;
  y: string;
begin
  y := x;
  for i := 0 to listbox1.items.Count - 1 do
  begin
    if x = listbox1.items[i] then
    begin
      y := listbox2.items[i];
    end;
  end;

  Result := y;
end;

function TForm1.Decoding(x: string): string;
var
  i: integer;
  y: string;
begin
  y := x;
  for i := 0 to listbox2.items.Count - 1 do
  begin
    if x = listbox2.items[i] then
    begin
      y := listbox1.items[i];
    end;
  end;

  Result := y;
end;

procedure TForm1.Button1Click(Sender: TObject);
var
  i: integer;
  i2: integer;
  x: string;
begin
  label2.Caption := '';
  memo2.Clear;
  for i := 0 to (Memo1.Lines.Count - 1) do
  begin
    label1.Caption := memo1.Lines.strings[i];
    for  i2 := 1 to length(label1.Caption) do
    begin
      x := label1.Caption[i2];
      x := coding(x);
      label2.Caption := label2.Caption + x;

    end;
    memo2.Lines[i] := (label2.Caption);
    label2.Caption := '';
  end;
end;

procedure TForm1.Button2Click(Sender: TObject);
var
  i: integer;
  i2: integer;
  x: string;
begin
  label2.Caption := '';
  memo3.Clear;
  for i := 0 to (Memo2.Lines.Count - 1) do
  begin
    label1.Caption := memo2.Lines.strings[i];
    for  i2 := 1 to length(label1.Caption) do
    begin
      x := label1.Caption[i2];
      x := decoding(x);
      label2.Caption := label2.Caption + x;

    end;
    memo3.Lines[i] := (label2.Caption);
    label2.Caption := '';
  end;
end;

procedure TForm1.Button3Click(Sender: TObject);
begin
  if edit1.text='ß' then showmessage('');
end;

end.

[Delphi.Narium]

Zitat von himitsu:

@Delphi.Narium:
Falsch, Memo1.Lines.Text ist schneller, denn Delphi (und bestimmt auch Lazarus) leiten das direkt an das Memo und holen den kompletten Text mit Einmal ab oder weisen ihn direkt zu,
genau so wie bei Memo.Text

Der Zugriff auf Einzelzeilen (Lines[]) ist langsam, vor allem beim Ändern und wenn viel Text im Memo steht.

Ja, aber das Zusammenbauen des Inhaltes von Text dauert ggfls. sehr lange. (Zumindest bei älteren Delphis). Meine praktische Erfahrung ist, dass das zeilenweise Ändern deutlich schneller ist, als erst Text "zusamenbauenzulassen", dann dadrin zu ändern und anschließend bei der Zuweisung zu Text wieder "allesauseinandernehmenzulassen".

Wenn man sich mal die Sourcen (Delphi 7) von function TStrings.GetTextStr: string;, welches beim Lesen von Text aufgerufen wird, anschaut, so wird dort bereits eine Schleife for I := 0 to Count - 1 do aufgerufen.
Und bei SetTextStr (zum Schreiben von Text), wird der übergebene String an den Zeilenumbrüchen aufgeteilt. Vorher wird Lines (vom Typ TStrings) geleert und dann jede der getrennten Zeilen per Add an die Lines angefügt.

Dies erspare ich mir bei meiner Vorgehensweise. Und niemand kann mir ernsthaft weißmachen, dass diese Ersparnis = 0 ist. Jedenfalls sind meine Programme, die häufig mit TStrings (bzw. entsprechenden Nachfahren) und recht großen Texten arbeiten, ausnahmslos alle (nach dieser Änderung) deutlich schneller geworden.

[himitsu]

Ich sehe garkeinen Code, der irgendwo die ß übersetzt, demnach wäre es verständlich, wenn das auch nicht gemacht wird.


PS: Du gehst doch die Items "manuell" durch und vergleichst jeden Text einzeln.
Du könntest aber auch suchen lassen, mit einem Befehl ala ListBox1.Items.IndexOf und kannst bei >=0 das gewünschte Item aus der anderen Listbox auslesen.

Zitat von Delphi.Narium:

Ja, aber das Zusammenbauen des Inhaltes von Text dauert ggfls. sehr lange. (Zumindest bei älteren Delphis). Meine praktische Erfahrung ist, dass das zeilenweise Ändern deutlich schneller ist, als erst Text "zusamenbauenzulassen", dann dadrin zu ändern und anschließend bei der Zuweisung zu Text wieder "allesauseinandernehmenzulassen".

Theoretisch ja, ABER

bei der normalen StringList wird im .Text der Text aus allen Zeilen erst zusammengesetzt,
wenn das TMemoStrings (der spezielle TStrings-Nachfahre im TMemo) das ebenfalls machen würde, also alle Zeilen einzeln abfragen und zusammensetzen, dann wäre es wirklich "extrem" langsam.

Aber das MEMO verwaltet intern den ganzen Text als einen Block im Speicher, weswegen im TMemo das Lines.Text überschrieben wurde und somit das Gleiche wie Memo.Text macht, also den kompletten Text mit nur einer Abfrage ausliest und auch zuweist.

[p80286]

Delphi-Quellcode:

type

   { TForm1 }

   TForm1 = class(TForm)
     Button1: TButton;
     Button2: TButton;
     Label1: TLabel;
     Label2: TLabel;
     ListBox1: TListBox;
     ListBox2: TListBox;
     Memo1: TMemo;
     Memo2: TMemo;
     Memo3: TMemo;
     procedure Button1Click(Sender: TObject);
     procedure Button2Click(Sender: TObject);
     procedure Button3Click(Sender: TObject);
   private
     function Coding(x: string): string;
     function Decoding(x: string): string;

   public

   end;

procedure TForm1.Button3Click(Sender: TObject);
begin
   if edit1.text='ß' then showmessage('');
end;

end.

Ich hab Deinen Source mal auf das absolut notwendige zusammen gestrichen.
Das kann nicht kompilieren, geschweige denn ein "ß" in " " übersetzen.

Gruß
K-H