kleine Änderung

imageInput.c

@@ -4,21 +4,22 @@
 #include "imageInput.h"
 
 #define BUFFER_SIZE 100
-#define FILE_HEADER_STRING "mnist_model.info2"
+#define FILE_HEADER_STRING "__info2_image_file_format__"
-
-// TODO Implementieren Sie geeignete Hilfsfunktionen für das Lesen der Bildserie aus einer Datei
-
 
+// Hilfsfunktion: File Header überprüfen
 static int checkFileHeader(FILE* f) {
- char buffer[BUFFER_SIZE]; 
+    char buffer[BUFFER_SIZE]; 
     size_t len = strlen(FILE_HEADER_STRING);
+    
     if (fread(buffer, 1, len, f) != len) {
         return -1; // Fehler beim Lesen
     }
+    
     buffer[len] = '\0';
     return strcmp(buffer, FILE_HEADER_STRING) == 0 ? 0 : -1;
 }
 
+// Hilfsfunktion: Metadaten lesen (Anzahl, Breite, Höhe)
 static int readImageMeta(FILE* f, unsigned short* count, unsigned short* width, unsigned short* height) {
     if (fread(count, sizeof(unsigned short), 1, f) != 1) return -1;
     if (fread(width, sizeof(unsigned short), 1, f) != 1) return -1;
@@ -26,6 +27,7 @@ static int readImageMeta(FILE* f, unsigned short* count, unsigned short* width,
     return 0;
 }
 
+// Hilfsfunktion: Ein einzelnes Bild einlesen (NUR Pixel, OHNE Label)
 static GrayScaleImage readSingleImage(FILE* f, unsigned short width, unsigned short height) {
     GrayScaleImage img;
 
@@ -43,45 +45,20 @@ static GrayScaleImage readSingleImage(FILE* f, unsigned short width, unsigned sh
         img.height = 0;
         return img;
     }
-}
 
-
+    // Pixeldaten einlesen
-
+    if (fread(img.buffer, sizeof(GrayScalePixelType), pixelCount, f) != pixelCount) {
-static GrayScaleImageSeries readImageSeries(FILE* f, unsigned short count, unsigned short width, unsigned short height) {
+        // Fehler beim Lesen: Speicher freigeben
-    GrayScaleImageSeries series;
+        free(img.buffer);
-
+        img.buffer = NULL;
-    // Anzahl der Bilder speichern
+        img.width = 0;
-    series.count = count;
+        img.height = 0;
-
-    // Speicher für Bilder und Labels reservieren
-    series.images = malloc(count * sizeof(GrayScaleImage));
-    series.labels = malloc(count * sizeof(unsigned char));
-
-    if (series.images == NULL || series.labels == NULL) {
-        // Fehlerfall: kein Speicher
-        series.count = 0;
-        series.images = NULL;
-        series.labels = NULL;
-        return series;
     }
 
-    // Alle Bilder und Labels einlesen
+    return img;
-    for (unsigned short i = 0; i < count; i++) {
-        series.images[i] = readSingleImage(f, width, height);
-
-        // Label separat lesen
-        if (fread(&series.labels[i], sizeof(unsigned char), 1, f) != 1) {
-            series.labels[i] = 0; // Fehlerfall: Default-Label
-        }
-    }
-
-    return series;
 }
 
-
+// Hauptfunktion: Bildserie aus Datei lesen
-// Test
-// TODO Vervollständigen Sie die Funktion readImages unter Benutzung Ihrer Hilfsfunktionen
-
 GrayScaleImageSeries *readImages(const char *path)
 {
     FILE *f = fopen(path, "rb");
@@ -123,8 +100,18 @@ GrayScaleImageSeries *readImages(const char *path)
 
     // 4. Bilder + Labels einlesen
     for (unsigned short i = 0; i < count; i++) {
+        // Bild einlesen
         series->images[i] = readSingleImage(f, width, height);
         
+        // Prüfen ob Bild erfolgreich gelesen wurde
+        if (series->images[i].buffer == NULL) {
+            // Fehler beim Bild-Lesen → Serie freigeben
+            clearSeries(series);
+            fclose(f);
+            return NULL;
+        }
+
+        // Label einlesen
         if (fread(&series->labels[i], sizeof(unsigned char), 1, f) != 1) {
             // Fehler beim Label-Lesen → Serie freigeben
             clearSeries(series);
@@ -137,8 +124,31 @@ GrayScaleImageSeries *readImages(const char *path)
     return series;
 }
 
-
+// Funktion: Bildserie vollständig aus dem Speicher freigeben
-// TODO Vervollständigen Sie die Funktion clearSeries, welche eine Bildserie vollständig aus dem Speicher freigibt
 void clearSeries(GrayScaleImageSeries *series)
 {
+    if (series == NULL) {
+        return;
+    }
+
+    // Alle einzelnen Bilder freigeben
+    if (series->images != NULL) {
+        for (unsigned int i = 0; i < series->count; i++) {
+            if (series->images[i].buffer != NULL) {
+                free(series->images[i].buffer);
+                series->images[i].buffer = NULL;
+            }
+        }
+        free(series->images);
+        series->images = NULL;
+    }
+
+    // Labels-Array freigeben
+    if (series->labels != NULL) {
+        free(series->labels);
+        series->labels = NULL;
+    }
+
+    // Die Serie selbst freigeben
+    free(series);
 }