.url öffnen geht nicht

[McRobins]

Hey,
also ich bin neu hier im Forum und auch nicht erfahrenste.
Aber zur Zeit hab ich ein Problem und zwar, ich will per Doppelklick in einer Listbox eine bestimmte Url öffnen per .url Datei.

Delphi-Quellcode:

procedure TForm1.ListBox1DblClick(Sender: TObject);
var filename: string;
begin
  Ini:=TIniFile.Create('ListBoxItems\ListBoxItems.ini');
  ini.ReadSection('ListboxItems',Memo1.Lines);
  filename := (ExtractFilePath(ParamStr(0)) + ChangeFileExt('Html\' + Memo1.Text, '.url'));

  case (ListBox1.ItemIndex) of
   0: ShellExecute(Handle, nil, PChar(filename) , nil, nil, SW_SHOW);
end;
end;

Wenn ich Doppelklick mache, passiert da nichts.
Aber wenn ich anstatt 'filename' den richtigen Pfad angebe, dann macht er das.
Ich hab auch mal als Test den ShowMessage Befehl genommen um zu gucken ob der Pfad auch richtig ist und das ist er :s
An was liegt das ? :s
Ich hoffe ihr versteht was ich meine

Mfg
McRobins

[sx2008]

Dein "filename" ist sicher nicht korrekt, denn wenn man Memo1.Text abruft sind immer CR/LF enthalten (ausser das Memofeld wäre leer).
Wenn du prüfst, ob ein Dateiname in Ordnung ist, dann darfst du dich nicht auf dein Gehirn verlassen
(denk an den Tomaten-Auf-Den-Augen-Effekt)
sondern solltest das im Programm überprüfen:

Delphi-Quellcode:

if not FileExists(filename) then
  raise Exception.CreateFmt('Datei %d existiert nicht', [filename];

Hier erst einmal kleine Hilfsfunktion

Delphi-Quellcode:

function OpenURL(const url: string): Boolean;
begin
   Result := ShellExecute(0, 'open', PChar(url), nil, nil, SW_SHOW) > 32;
end;

// Beispielaufrufe
OpenURL('http://www.delphipraxis.net');
OpenURL('C:\MeineFavoriten\test.url');

[McRobins]

Danke für die Antwort
Auf jedenfall hat es anfangs noch nicht funktioniert.
Aber ich hab aus dem 'Memo1.Text' einfach 'Edit1.Text' gemacht und dann hat es auch funktioniert. Bloß ist das alles noch ein bisschen durcheinander und muss noch ausgebessert werden, aber das wichtigste funktioniert
Außer das mit der Fehlermeldung klappt noch nicht ganz, also wenn die Datei nicht existiert :/ Da zeigt er an: (siehe Anhang)

Aber danke sx2008 für deine Hilfe

Hier der nochmal der Code:

Delphi-Quellcode:

function TForm1.OpenURL(const url: string): Boolean;
begin
   Result := ShellExecute(0, 'open', PChar(url), nil, nil, SW_SHOW) > 32;
end;

procedure TForm1.ListBox1DblClick(Sender: TObject);
var filename: string;
begin
   Ini:=TIniFile.Create('ListBoxItems\ListBoxItems.ini');
   ini.ReadSection('ListboxItems',Memo1.Lines);

    filename := (ExtractFilePath(ParamStr(0)) + ChangeFileExt('Html\' + Edit1.Text, '.url'));

case (ListBox1.ItemIndex) of
 0: begin  OpenURL(filename);
     if not FileExists(filename) then
     raise Exception.CreateFmt('Datei %d existiert nicht', [filename]);
    end;
 1: begin  OpenURL(filename);
     if not FileExists(filename) then
     raise Exception.CreateFmt('Datei %d existiert nicht', [filename]);
    end;
  end;
 end;

Dann die Function in private einfügen:

Delphi-Quellcode:

  private
     function OpenURL(const url: string): Boolean;
   { Private declarations }

Mfg
McRobins

[DeddyH]

%d steht für ganze Zahl, für Strings brauchst Du %s. Allerdings steige ich persönlich bei Deinem Code nicht durch, da gibt es eine Listbox, ein Memo und ein Edit. Was ist denn was? Außerdem erzeugst Du die TIniFile immer wieder neu, gibst sie aber nicht frei -> Speicherleck.

[McRobins]

Ok danke auch für deine Antwort.
Jap, also mit %s klappt das schonmal.
Ja jetzt gibt es nur noch eine Listbox und ein edit.
Das ist jetzt alles weg, dass sieht jetzt so aus:

Delphi-Quellcode:

procedure TForm1.ListBox1DblClick(Sender: TObject);
begin
case (ListBox1.ItemIndex) of
 0: begin
     filename := (ExtractFilePath(ParamStr(0)) + ChangeFileExt('Html\' + Listbox1.Items[ListBox1.Itemindex] , '.url'));
     OpenURL(filename);
     if not FileExists(filename) then
     raise Exception.CreateFmt('Datei %s existiert nicht', [filename]);
    end;
 1: begin
     filename := (ExtractFilePath(ParamStr(0)) + ChangeFileExt('Html\' + Listbox1.Items[ListBox1.Itemindex] , '.url'));
     OpenURL(filename);
     if not FileExists(filename) then
     raise Exception.CreateFmt('Datei %s existiert nicht', [filename]);
    end;
 2: begin
     filename := (ExtractFilePath(ParamStr(0)) + ChangeFileExt('Html\' + Listbox1.Items[ListBox1.Itemindex] , '.url'));
     OpenURL(filename);
     if not FileExists(filename) then
     raise Exception.CreateFmt('Datei %s existiert nicht', [filename]);
    end;
...

Also ich muss jetzt nur noch feinarbeiten machen und dann klappt das
Danke für eure Hilfe !

Mfg
McRobins

[DeddyH]

DRY

Delphi-Quellcode:

procedure TForm1.ListBox1DblClick(Sender: TObject);
begin
  if ListBox1.ItemIndex > -1 then
    begin
      filename := (ExtractFilePath(ParamStr(0)) + ChangeFileExt('Html\' + Listbox1.Items[ListBox1.Itemindex] , '.url'));
      if not FileExists(filename) then
        raise Exception.CreateFmt('Datei %s existiert nicht', [filename]);
      OpenURL(filename);
    end;
end;

[McRobins]

Ah gut dankeschön
So wollte ich auch das haben, aber wie das machen musste wusste ich nicht :/
Aber wieder was dazu gelernt

Mfg
McRobins