Codeprüfung/-vorschläge | Thema: MP3, ID-Infos

[Techcrawler]

Danke nochmal für alle Infos.
Sobald wieder Zeit vorhanden ist, führe ich das ganze weiter.
Bis dahin gehe ich im allgemeinen Arbeitsstreß unter *glugg*

[Muetze1]

Nochwas: Warum liest du keinen Record anstatt eines Buffers? Dann könntest du dir auch die Schleifen sparen können. Siehe Lib in der Signatur.

[OldGrumpy]

Nochmal was zum Thema lesen, schreiben und Filenamen übergeben... Ich hab mehrere Libs in Gebrauch, die einen etwas anderen Ansatz haben: Es gibt ein Objekt namens z.B. TAudioInfo, das Properties wie Interpret (Artist), Songtitel, usw. hat, und Methoden, um diese Infos in ein File zu schreiben bzw. daraus zu lesen. Eine Objektinstanz fest an ein File zu binden... naja, Ansichtssache, mir gefällt obige Idee halt einfach besser :)

BTW, warum wollt ihr Fehler immer über Exceptions "rausreichen"? Schlechte Angewohnheit? Oder "Hamwer schon immer so gemacht"? Im "Windows von heute" haben Exceptions im Vergleich zu eigenem Errorhandling enormen Overhead, Microsoft (man kann von den Jungs halten was man will, aber grossenteils können die schon was) setzt nicht umsonst auf andere Wege. Neuere Methoden (Win32 API, MFC, ATL, usw.) liefern in den allermeisten Fällen ein BOOL zurück anhand dessen man feststellen kann obs einen Fehler gab - und wenns einen gab, dann kann man über GetLastError rausfinden, was passiert ist. Ausnahmen gibts natürlich auch hier, es gibt z.B. CreateFile-Aufrufe (bei den MFC-Klassen zumindest) denen ein Pointer auf ein vorgefertigtes Exception-Objekt mitgegeben wird. Falls dann der Rückgabewert FALSE ist, schaut man halt ins Exception-Objekt rein, um rauszufinden was Sache ist.

Exceptions sollte man sich für die wirklich bösen Showstopper aufsparen, Access Violations zum Beispiel.

Hier gibts nen netten Artikel zu dem Thema...

Zitat:

Depending on your environment, just raising one exception can be from 10-100.000 times as slow as alternatively returning a simple return code from the method.

Ich denke, das sollte man beherzigen. Oder wie seht Ihr das?

[Luckie]

Wegen der Instanz und dem Dateinamen: Deswegen zwei Konstruktoren, da kann es sich jeder aussuchen. Beim Zuweisen der Datei-Property ist eine Exception gerechtfertigt, da ohne Datei das ganz keinen Sinn macht. Die Exception bei GetMp3Info ist geschmackssache.