spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

1. Warum lieferst du das img-Array als Rückgabewert zurück? Das wäre nur dann sinnvoll, wenn die Funktion eine Kopie des Arrays erzeugen und auf dieser Kopie arbeiten würde. Sie verändert das Array aber "in-place", daher ist der Rückgabewert irreführend und unnötig.
2. Das else nach dem throw brauchst du nicht. Wenn die Exception geworfen wird, wird die Funktion automatisch verlassen.
3. Das Setzen der Bits scheint auf den ersten Blick OK zu sein. Schreib dir am besten ein paar Test-Cases, die das überprüfen.

Eine Idee zum Testen: Erzeuge dir jeweils ein data-Array mit rein zufälligen Daten, angefangen bei 1 Byte Größe bis meinetwegen 1 KiB Größe. Benutze deine Funktion, um die Daten in ein Bild zu schreiben und danach deine noch zu implementierende Funktion, um die Daten wieder herauszuholen. Dann vergleichst du das Herausgeholte mit dem Original und stellst sicher, dass alles gleich ist.

Noch etwas, das mir vorhin nicht aufgefallen war: Ich bin ziemlich sicher, dass data[imgIterator] falsch ist. Erstens weil dort der Offset nicht berücksichtigt wird (imgIterator - offset? - oder wie ist offset zu verstehen?), zweitens weil du von jedem Daten-Byte nur jeweils ein einziges Bit nimmst und dann schon weiter zum nächsten Daten-Byte huschst. Ich würde vorschlagen, zwei Iteratoren zu benutzen: einen für das Bild, einen für die Daten. So wie es jetzt ist, schreibst du nur das 1. Bit vom 1. Daten-Byte, nur das 2. Bit vom 2. Daten-Byte, ..., nur das 1. Bit vom 9. Daten-Byte usw.

Und noch etwas: Deine Größenabfrage (img.length < data.length + offset) ist falsch, da du hier nicht berücksichtigst, dass du zum Unterbringen von 1 Daten-Byte nicht 1, sondern 8 Bild-Bytes brauchst.