spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Moin

Meine Image-Loader für bmp, jpg und png sind jetzt endlich fertig.
Es ist möglich die Datein aus einer Datei zu laden oder auch direkt vom RAM aus.
Naja mehr gibt es auch nicht zu sagen
Sind halt Image-Loader.

Mal zusammengefasst:

Formate: bmp, jpg, png
Entwicklungzeit: ca. 1 Monat (wenn jmd. interessiert )
Librarys: libjpeg, libpng/libzip

Ich habe mal ein Testprogramm geschrieben, dass alle 3 Formate lädt und darstellt (hoffe ich xD).
Den Sourcecode und das Programm gibts hier.
Kritik ist sogar erwünscht

Zuguterletzt will ich david_pb, dot und idontknow danken, dass sie mir beim Bitmapladen geholfen haben.

Danke

Zitat von »Jonathan_Klein«

Hm, ist das ein Lernprojetk oder ernsthafte Konkurrenz für DevIL?

Das ist ein Lernprojekt, wobei ich es dann auch in meinem Framework verwenden werde.
Gegen DevIL kommt man ja als normaler Programmierer eh nicht an

Zitat von »Jonathan_Klein«

Aber ich hätte echt die LoadFromMemory Funktion kleiner gemacht, also 2-10 Hilfsfunktionen geschrieben. Bei 2 verschachtelten switchs verliert man echt den Überblick, würde jeder case Zweig nur die passende Funktion aufrufen, wäre alles viel übersichtlicher.

Ja da hast du recht. Bei den Bitmaps ist das etwas größer geworden als ich erwartet habe.
Werde das auch ändern.

Zitat von »Jonathan_Klein«

Man könnte auch eine gemeinsame Oberklasse haben, denn soweit ich das sehe, speichern alle 3 Klassen die größe und ein Pixel Vector. In der Oberklasse könnte dann auch gleich die LoadFromFile Funktion stehen, die ja eh jedesmal nur die LoadFromMemory Funktion aufruft, die dann ja jeweils passend überladen sein könnte.

Stimmt, daran habe ich garnicht gedacht.
Wird auch gemacht

Zitat von »Jonathan_Klein«

Viele Kommentare sind zwar löblich, aber du hast zu viele

Gut da hab ich wohl etwas übertrieben sogar jedes 'if' zu kommentieren
Ich versuch mal etwas größere Blöck mit Kommentaren zu versehen.

Danke fürs Anschauen und die hilfreiche Kritik

Tag,

Hab jetzt endlich mal Zeit gehabt die Dinge etwas zu verbessern.
Das Design wurde jetzt komplett umgekrempelt.

Es gibt jetzt nur noch eine Klasse für die Bilddaten ('Image') anstelle von jeweils einer Klasse für jedes Format.
Das heißt, dass man nur noch 'Image::LoadFromFile' oder 'Image::LoadFromMemory' aufrufen muss und die Methoden
erkennen selbst um welches Format es sich handelt.
Das Laden übernehmen jetzt 3 Klassen ('ImageLoaderBmp/Jpg/Png'), die von der Klasse 'ImageLoader' abgeleitet wurden.
Allerdings kann nur 'Image' die ctor der ImageLoader aufrufen.

Kommentare hab ich jetzt ein paar entfernt, sollte jetzt ok sein.
Die Lade-Methoden wurden jetzt auch in mehere kleine Methoden aufgeteilt.

Weiterhin ist Kritik erwünscht

Einen Punkt finde ich jetzt schon nicht so schön am Code:
In den Lade-Methoden von 'Image' habe ich jeweils eine statische Instanz der ImageLoader,
was ich jetzt nicht so optimal finde. Hättet ihr da einen besseren Vorschlag?

Hier ist der Link.

Thx für die Antworten.


P.S.: Sry für Doppelpost

EDIT: Hab da doch glatt vergessen zu beschreiben was ich verbessert habe, sry. Steht da jetzt.