Funktion erstellen

[Rackergen2]

Delphi-Quellcode:

function TForm1.createautorunfile(app : String; icon : String; destfile : String):boolean;
var
 sl : TStringList;
begin
  result:=true;
  try
    sl:=TStringList.create;
    sl.add('[autorun]');
    sl.add('OPEN=' + app);
    sl.add('ICON=' + icon);
    if copy(destfile,length(destfile-4),4)<>'.inf' then destfile:=destfile+'.inf';
    sl.SaveToFile(destfile);
    s1.free;
  except
    result:=false;
  end;
end;

end.

So besser?

[Matze]

if copy(destfile,length(destfile-4),4)<>'.inf' then destfile:=destfile+'.inf'; Gefällt mir so besser:

if not Pos('.inf', destfile) > 0 then destfile := destfile + '.inf'; Aber das spielt ja auch keine große Rolle.

Edit:

Unde du solltest es irgendwie so machen:

Delphi-Quellcode:

...

try

  ...

finally
  sl.Free;

[p0w3r5tr34m3r]

geht das nich auf ohne stringliste ??? inifiles ???

der aufbau is ja eigentlich der selbe oder ?

[S2B]

Klar geht das mit ini-Dateien! 8)

[Rackergen2]

Zitat von Matze:

if copy(destfile,length(destfile-4),4)<>'.inf' then destfile:=destfile+'.inf'; Gefällt mir so besser:

if not Pos('.inf', destfile) > 0 then destfile := destfile + '.inf'; Aber das spielt ja auch keine große Rolle.

Denkste! Lass mal die Datei so heißen: 'C:\test\test.inf.txt' Du musst schon dafür sorgen, dass das .inf hinten ist... höchstens so:

if Pos('.inf', destfile)<length(destfile)-4 then destfile:=destfile+'.inf';

Delphi-Quellcode:

function TForm1.createautorunfile(app : String; icon : String; destfile : String):boolean;
var
sl : TStringList;
begin
  result:=true;
  sl:=TStringList.create;
  sl.add('[autorun]');
  sl.add('OPEN=' + app);
  sl.add('ICON=' + icon);
  if copy(destfile,length(destfile-4),4)<>'.inf' then destfile:=destfile+'.inf';

  try
    sl.SaveToFile(destfile);
  except
    result:=false;
  end;

  s1.free;
end;

[Matze]

Hmm, stimmt, an so einen "Sonderfall" habe ich nicht gedacht.

Nur sollte man noch die Resourcenschutzblöcke einbauen:

Delphi-Quellcode:

function TForm1.CreateAutorunFile(app, icon, destfile: String): boolean;
var
  sl: TStringList;
begin
  result := true;
  sl := TStringList.create;
  try
    sl.add('[autorun]');
    sl.add('OPEN=' + app);
    sl.add('ICON=' + icon);
    if copy(destfile, length(destfile) - 4, 4) <> '.inf' then destfile := destfile + '.inf';
    try
      sl.SaveToFile(destfile);
    except
      result:=false; 
    end;
  finally
    sl.free;
  end;
end;

Ich hab auch noch den ein oder anderen Fehler verbessert, dein Code lies sich nicht kompilieren.

Eines darf man nicht vergessen:
Wenn ich nun den Pfad und das Icon angebe, so habe ich den kompletten Pfad beider in der Ini, was natürlich Mist ist.
Nacher befinden sich die Anwendung und das Icon (im Normalfall) auf der CD, somit stimmen die Pfade nicht mehr!
Man musste Icon und Anwendung in den Projektordner geben und dann die relativen Pfade (Anwendung + ggf. Ordnername) in die inf-Datei eintragen.

[Rackergen2]

Zitat von Matze:

Delphi-Quellcode:

...
  sl.add('[autorun]');
  sl.add('OPEN=' + app);
  sl.add('ICON=' + icon);
  if copy(destfile, length(destfile) - 4, 4) <> '.inf' then destfile := destfile + '.inf';
  ...

Warum hast du das in einer try-Klausel? Daran kann doch nix schief gehen...

Delphi-Quellcode:

function TForm1.CreateAutorunFile(app, icon, destfile: String): boolean;
var
  sl: TStringList;
begin
  result:=true;
  sl:=TStringList.create;
  sl.add('[autorun]');
  sl.add('OPEN='+app);
  sl.add('ICON='+icon);
  if copy(destfile, length(destfile)-4, 4)<>'.inf' then destfile:=destfile+'.inf';

  try
    sl.SaveToFile(destfile);
  except
    result:=false;
  end;

  sl.free;
end;

Und "den einen oder anderen Fehler" kann ich auch nicht erkennen, nur dass du da noch Leerzeichen reingeklatscht hast...

EDIT: und das -4, aber wo ist "der andere Fehler"?

PS: Wie ich diese ganze Leerzeichensetzung hasse...

[mirage228]

Hi,

wie wärs hiermit?

Delphi-Quellcode:

function TForm1.CreateAutorunFile(const App, Icon, DestFile: String): Boolean;
var
  sl: TStringList;
begin
  Result := False;
  sl := TStringList.create;
  try
    sl.add('[autorun]');
    sl.add('OPEN=' + App);
    sl.add('ICON=' + Icon);
    sl.SaveToFile(ChangeFileExt(DestFile, '.inf'));
    Result := True;
  finally
    sl.Free;
  end;
end;

mfG
mirage228

[Matze]

@Rackergen2: Jo, des war glaub der einzige Fehler.

@Mirage: Cool!

[Rackergen2]

Zitat von mirage228:

wie wärs hiermit?

Delphi-Quellcode:

function TForm1.CreateAutorunFile(const App, Icon, DestFile: String): Boolean;
var
  sl: TStringList;
begin
  Result := False;
  sl := TStringList.create;
  try
    sl.add('[autorun]');
    sl.add('OPEN=' + App);
    sl.add('ICON=' + Icon);
    sl.SaveToFile(ChangeFileExt(DestFile, '.inf'));
    Result := True;
  finally
    sl.Free;
  end;
end;

Das ist doch Schwachsinn. Was sollen die ersten 3 Zeilen in einem try? Wenn, dann so:

Delphi-Quellcode:

function TForm1.CreateAutorunFile(const App, Icon, DestFile: String): Boolean;
var
  sl: TStringList;
begin
  Result := False;
  sl := TStringList.create;
  sl.add('[autorun]');
  sl.add('OPEN=' + App);
  sl.add('ICON=' + Icon);
  try
    sl.SaveToFile(ChangeFileExt(DestFile, '.inf'));
    Result := True;
  finally
    sl.Free;
  end;
end;

Aber das käme sowas von das gleiche raus... dann doch lieber meine Funtion oben...