imageInput.c

@@ -11,20 +11,23 @@
 // TODO Vervollständigen Sie die Funktion readImages unter Benutzung Ihrer Hilfsfunktionen
 GrayScaleImageSeries *readImages(const char *path)
 {
+    // GrayScaleImageSeries *series = NULL;
+    
+    // return series;
+
     FILE *file = fopen(path, "rb");
-    if (file == NULL)
+    if (file == NULL) {
-    {
+        return NULL;  // Datei existiert nicht oder konnte nicht geöffnet werden
-        return NULL;
     }
 
-    // Überprüfe den Header
+    // Überprüfe den Dateitag
-    char fileTag[strlen(FILE_HEADER_STRING)];
+    char fileTag[25];
-    fread(fileTag, sizeof(fileTag[0]), strlen(FILE_HEADER_STRING), file);
+    fread(fileTag, sizeof(fileTag[0]), 24, file);
+    fileTag[24] = '\0';  // Null-Terminierung des Strings
 
-    if (strcmp(fileTag, FILE_HEADER_STRING) != 0)
+    if (strcmp(fileTag, "__info2_image_file_format__") != 0) {
-    {
         fclose(file);
-        return NULL;
+        return NULL;  // Ungültiges Dateiformat
     }
 
     // Lese die Metadaten: Anzahl der Bilder, Breite und Höhe
@@ -33,34 +36,32 @@ GrayScaleImageSeries *readImages(const char *path)
     fread(&width, sizeof(width), 1, file);
     fread(&height, sizeof(height), 1, file);
 
+    // Speicher für die Bildserie und die Bilddaten allozieren
     GrayScaleImageSeries *series = (GrayScaleImageSeries *)malloc(sizeof(GrayScaleImageSeries));
-    if (series == NULL)
+    if (series == NULL) {
-    {
         fclose(file);
-        return NULL;
+        return NULL;  // Speicher konnte nicht allokiert werden
     }
 
     series->count = numberOfImages;
     series->images = (GrayScaleImage *)malloc(numberOfImages * sizeof(GrayScaleImage));
     series->labels = (unsigned char *)malloc(numberOfImages * sizeof(unsigned char));
 
-    if (series->images == NULL || series->labels == NULL)
+    if (series->images == NULL || series->labels == NULL) {
-    {
         free(series);
         fclose(file);
-        return NULL;
+        return NULL;  // Speicher konnte nicht allokiert werden
     }
 
-    for (int i = 0; i < numberOfImages; i++)
+    // Lese die Bilddaten und die Labels
-    {
+    for (int i = 0; i < numberOfImages; i++) {
         series->images[i].width = width;
         series->images[i].height = height;
         series->images[i].buffer = (GrayScalePixelType *)malloc(width * height * sizeof(GrayScalePixelType));
 
-        if (series->images[i].buffer == NULL)
+        if (series->images[i].buffer == NULL) {
-        {
+            // Fehlerbehandlung: Speicher freigeben, wenn Allocation fehlschlägt
-            // Fehlerbehandlung: Speicher freigeben, wenn malloc fehlschlägt
+            for (int j = 0; j < i; j++) 
-            for (int j = 0; j < i; j++)
             {
                 free(series->images[j].buffer);
             }
@@ -83,15 +84,13 @@ GrayScaleImageSeries *readImages(const char *path)
 // TODO Vervollständigen Sie die Funktion clearSeries, welche eine Bildserie vollständig aus dem Speicher freigibt
 void clearSeries(GrayScaleImageSeries *series)
 {
-    if (series == NULL)
+    if (series == NULL) return;
-        return;
 
-    for (int i = 0; i < series->count; i++)
+    for (int i = 0; i < series->count; i++) {
-    {
+        free(series->images[i].buffer);  // Speicher für das Bild freigeben
-        free(series->images[i].buffer); // Speicher für das Bild freigeben
     }
 
-    free(series->images); // Speicher für die Bild-Array freigeben
+    free(series->images);   // Speicher für die Bild-Array freigeben
-    free(series->labels); // Speicher für die Labels freigeben
+    free(series->labels);   // Speicher für die Labels freigeben
-    free(series);
+    free(series); 
 }