Opened by mark05 at 2011-07-03 17:21
User since
2006-07-10
2611 Artikel
BenutzerIn
2006-07-10
2611 Artikel
BenutzerIn
Was mir auffällt ist, neben dem exzessiven aber sinnlosen nutzen von Referenzen bei Rückgabewerten von Methoden, ist die Stelle:
In der Message schreibst du "$dir/$wrkfile" beim verschieben aber nur $wrkfile. Ich habe keine Ahnung welche Form die Rückgabe von _get_files_in_dir hat. Wenn die Angaben relativ sind, dann kann es sein, dass das Kopieren fehlschlägt, sind sie absolut ist die Angabe in der Nachricht nicht korrekt.
Du machst keinerlei Überprüfung, ob ach eine Datei gefunden wurde, die kopiert werden kann. Mit dem Modul, welches du nutzt, kenne ich mich nicht aus, und kann nicht sagen ob das fatal sein könnte. Unter Umständen verschiebst du ein Verzeichnis, nicht nur eine Datei.
Code (perl): (dl
)
1 2 3
$vdrtranscode->message({msg=>"_rename_and_move|copy $dir/$wrkfile to $tracoenv->{'outdir'}/$workfile",v=>'v',}); #copy $wrkfile,"$tracoenv->{'outdir'}/$workfile"; move $wrkfile,"$tracoenv->{'outdir'}/$workfile";
In der Message schreibst du "$dir/$wrkfile" beim verschieben aber nur $wrkfile. Ich habe keine Ahnung welche Form die Rückgabe von _get_files_in_dir hat. Wenn die Angaben relativ sind, dann kann es sein, dass das Kopieren fehlschlägt, sind sie absolut ist die Angabe in der Nachricht nicht korrekt.
Du machst keinerlei Überprüfung, ob ach eine Datei gefunden wurde, die kopiert werden kann. Mit dem Modul, welches du nutzt, kenne ich mich nicht aus, und kann nicht sagen ob das fatal sein könnte. Unter Umständen verschiebst du ein Verzeichnis, nicht nur eine Datei.