BinToHex

[AlBo55]

Hallo,
ich bin am verzweifeln:
Warum funktioniert dies:

Delphi-Quellcode:

procedure TForm1.Button1Click(Sender: TObject);
var
  x,y :PAnsiChar;

begin
  x:=PAnsiChar(Edit1.Text);
//  showmessage(IntToStr(length(x)));
  Label1.Caption:=String(x);
  BinToHex(x,y,5);
  showmessage(IntToStr(length(y)));
  Label1.Caption:=Copy(String(y),1,length(Edit1.Text)*2);
end;

aber das nicht:

Delphi-Quellcode:

procedure TForm1.Button1Click(Sender: TObject);
var
  x,y :PAnsiChar;

begin
  x:=PAnsiChar(Edit1.Text);
  showmessage(IntToStr(length(x)));
  Label1.Caption:=String(x);
  BinToHex(x,y,5);
  showmessage(IntToStr(length(y)));
  Label1.Caption:=Copy(String(y),1,length(Edit1.Text)*2);
end;

In Edit gebe ich 'abcde' ein
Im zweiten Fall Fehlermeldung: Zugriffsverletzung bei Adresse ... im Modul ...

Ich versteh's einfach nicht.

[Klaus01]

x ist ein PAnsiChar, also ein Pointer auf eine #0 terminierende Zeichenkette.
Wenn Du die Länge der Zeichenkette bestimmen willst solltest Du sie wieder in ein
String wandeln -oder gleich die Zeichenlänge von Edit1.text nehmen.

Grüße
Klaus

[himitsu]

PAnsiChar(Edit1.Text) gibt keinen Zeiger auf den Inhalt des Edits, sondern einen Zeiger auf eine temporäre und von Delphi verwaltete Kopie des Inhalts.

Tja und dieser temporäre String kann schnell mal überschrieben, gelöscht oder für was Anderes wiederverrwendet werden

eine Lösung wäre jetzt, daß du selber eine Kopie anlegst.

Delphi-Quellcode:

var
  x: PAnsiChar;
  S: AnsiString;

begin
  S := Edit1.Text;
  x := PAnsiChar(S);

und bitte ließ nochmal bei BinToHex genau nach,
denn Y wurde nicht definiert

soll heißen, daß DU selber für genug Speicher darin sorgen mußt.

[sirius]

Du hast Y keinen Speicher reserviert, dadurch entstehen zufällige Fehler (die teilweise vom Programmcode abhängen)
Mache es doch über strings:

Delphi-Quellcode:

var x,y:String;
begin
  x:=Edit1.Text;
  setlength(y,length(x)*2);
  BinToHex(PChar(x),PChar(y),length(x));
  Edit2.Text:=y;
end;

[AlBo55]

danke an alle
jetzt hab ich's verstanden.

Bei meinem Ansatz ging es wohl wirklich sehr zufäälig zu.

[himitsu]

Zitat:

Delphi-Quellcode:

x:=PAnsiChar(Edit1.Text);
showmessage(IntToStr(length(x)));
Label1.Caption:=String(x);
BinToHex(x,y,5);
showmessage(IntToStr(length(y)));
Label1.Caption:=Copy(String(y),1,length(Edit1.Text)*2);

so wirklich zufällig vielleicht nicht ... vermutlich würde es dank der Code-Optimierung etwa so aussehn

Delphi-Quellcode:

temp := Edit1.GetText;
x := PAnsiChar(temp);
temp := String(x);
temp2 := IntToStr(length(temp));
showmessage(temp2);
temp2 := String(temp)
Label1.SetCaption(temp2);
BinToHex(x, y, 5);
temp := y;
temp2 := IntToStr(length(temp));
showmessage(temp2);
temp := String(y);
temp2 := Edit1.GetText;
temp3 := Copy(temp, 1, length(temp2) * 2);
Label1.SetCaption(temp3);

Wenn die Optimierung hier nicht zufällig einige "Temp"-Variablen mehrfach nutzen würde, dann hätte es womöglich geklappt ... also abgesehn von BinToHex+Y, weil in Y ja der Speicher noch fehlte.


und wenn man da jetzt das temp ab
x := PAnsiChar(temp); verfolgt, dann wird man wohl schnell merken, daß dieses temp, auf welches X nun zeigt sich schnell wieder ändert.

[AlBo55]

@himitsu
Ich meinte zufällig nich im engeren Sinn, sondern durch Nachlässigkeit schlecht zu überblickenden Unordnung erzeugt
MfG