große Funktion in einzelne Teilfunktionen separiert -> Übersichtlichkeit

imageInput.c

@@ -43,14 +43,16 @@ static int checkHeader(const GrayScalePixelType *buffer){
 
 }
 
+
 static FILE *openImageFile(const char *path)
 {
-    return fopen(path, "rb");
+    return fopen(path, "rb"); 
+    
 }
 
-static int readHeaderFromFile(FILE *fp, GrayScalePixelType *headerBuffer)
+static int ReadCheckHeader(FILE *fp, GrayScalePixelType *header)
 {
-    if (fread(headerBuffer, sizeof(GrayScalePixelType),HEADER_LEN, fp) != HEADER_LEN)
+    if(fread(header, sizeof(GrayScalePixelType), HEADER_LEN, fp) != HEADER_LEN)
     {
         return 0;
     }
@@ -59,11 +61,11 @@ static int readHeaderFromFile(FILE *fp, GrayScalePixelType *headerBuffer)
 
 static int readMetaData(FILE *fp, unsigned int *numImages, unsigned int *width, unsigned int *height)
 {
-    if (fread(numImages, sizeof(unsigned int), 1, fp) != 1 || fread(width, sizeof(unsigned int), 1, fp) != 1 || fread(height, sizeof(unsigned int), 1, fp) != 1)
+    if(fread(numImages, sizeof(unsigned int), 1, fp) != 1 || fread(width, sizeof(unsigned int), 1, fp) != 1 || fread(height, sizeof(unsigned int), 1, fp)!= 1)
-
     {
-    return 0;
+        return 0;
     }
+
     return 1;
 }
 
@@ -73,136 +75,154 @@ static GrayScaleImageSeries *allocateSeriesStruct(void)
     return series;
 }
 
-static int allocateImageAndLabelArrays(GrayScaleImageSeries *series, unsigned int numImages, FILE *fp)
+static int AllocateCheckImageAndLabels(GrayScaleImageSeries *series, unsigned int numImages)
 {
     series->count = numImages;
     //Speicher für die Pointer an sich belegen
-    series->images = malloc(numImages * sizeof(GrayScaleImage));
+    series->images = malloc(numImages * sizeof(GrayScaleImage)); //Speicher für alle Bild-Strukturen, nicht aber für die Pixel selbst (denn image ist nochmal ein Array selbst)
-    series->labels = malloc(numImages * sizeof(unsigned char));
+    series->labels = malloc(numImages * sizeof(unsigned char)); //Speicher für alle Labels
+
+    //Prüfen, ob Speicher für die Arrays reserviert werden konnte, wenn nicht -> Freigabe
+    if(series->images == NULL || series->labels == NULL){
 
-    if (!series->images || !series->labels)
-    {
         free(series->images); //wenn trotzdem was reingeschrieben wurde, dann freigeben
         free(series->labels);
         free(series);
-
-        fclose(fp);
         return 0;
+
     }
 
     return 1;
 }
 
-static int allocateBufferForImages(GrayScaleImageSeries *series, unsigned int numImages, unsigned int width, unsigned int height, FILE *fp)
+static int AllocateCheckPixelData(GrayScaleImageSeries *series, unsigned int numImages, unsigned int width, unsigned int height)
 {
-    for (unsigned int i = 0; i < numImages; i++)
+    for(unsigned int i = 0; i < numImages; i++){
-    {
+
-        series ->images[i].buffer = malloc(width * height);
+        series -> images[i].buffer = malloc(width * height); //Speicher für Pixeldaten selbst
-        if (series ->images[i].buffer == NULL) 
+
-        {
+        if(series->images[i].buffer == NULL){ 
-            for (unsigned int j = 0; j < i; j++)
+
-            {
+            for(unsigned int j = 0; j < i; j++){
-                free(series->images[j].buffer);
+                free(series->images[j].buffer); //wenn Speicher nicht reserviert werden konnte, dann zuerst das Array mit Pixeldaten freigeben
             }
 
-            free(series->images);
+        
-            free(series->labels);
+        free(series->images); //anschließend auch die Speicher für Bildstrukturen und Label-Arrays freigeben, weil zuvor für diese schon Speicher reserviert wurde
-            free(series);
+        free(series->labels);
-
+        free(series);
-            fclose(fp);
+    
-            return 0;
+        return 0;       
         }
     }
+
     return 1;
 }
 
-static int readImagesAndLabels(FILE *fp, GrayScaleImageSeries *series, unsigned int numImages, unsigned int width, unsigned int height)
+static int ReadImagesAndLabels(FILE *fp, GrayScaleImageSeries *series, unsigned int numImages, unsigned int width, unsigned int height)
 {
-    for (unsigned int i = 0; i < numImages; i++)
+    for(int i = 0; i < numImages; i++)
-    {
+   {
-        series->images[i].width = width;
+        //Breite und Höhe werden gesetzt (GrayScaleImage)
-        series->images[i].height = height;
+        series -> images[i].width = width;
+        series -> images[i].height = height;
 
-        if (fread(series->images[i].buffer, 1, width *height, fp) != (width * height))
+        
+        //Pixeldaten einlesen und Prüfen ob alle essentiellen Pixel eingelesen wurden
+        if(fread(series -> images[i].buffer, 1, width * height, fp) != (width * height))
         {
             clearSeries(series);
-            fclose(fp);
             return 0;
         }
 
-        fread(&series->labels[i], 1, 1, fp);
+        //Jedes Bild hat ein Label, was direkt danach eingelesen wird und auf Größe geprüft wird
+        if(fread(&series -> labels[i], 1, 1, fp) != 1){
+            clearSeries(series);
+            return 0;
+        }
     }
 
     return 1;
 }
 
 
+
+
+
+
+
+
+
 // TODO Vervollständigen Sie die Funktion readImages unter Benutzung Ihrer Hilfsfunktionen
 GrayScaleImageSeries *readImages(const char *path)
 {
     GrayScaleImageSeries *series = NULL;
 
+    //Datei im Binärmodus öffnen
     FILE *fp = openImageFile(path);
-    if (!fp)
+    if(!fp)
-    return NULL;
+        return NULL;
 
+    //Buffer zum Einlesen des Headers 
     GrayScalePixelType header[HEADER_LEN];
-
+    
-    if (!readHeaderFromFile(fp, header))
+    //Prüfen, ob genau die HEADER_LEN Bytes eingelesen werden 
-    {
+    if(!ReadCheckHeader(fp, header)){
         fclose(fp);
         return NULL;
     }
 
-    if (!checkHeader(header))
+    //Prüfen, ob es sich um den korrekten Header handelt mit Hilfsfunktion
-    {
+    if(!checkHeader(header)){
         fclose(fp);
         return NULL;
     }
 
-    unsigned int numImages, width, height;
+    //Anzahl der Bilder, Breite und Höhe einlesen 
-    if (!readMetaData(fp, &numImages, &width, &height))
+    unsigned  int numImages, width, height; //muss man das als Zeiger initialisieren?????????
-    {
+    if(!readMetaData(fp, &numImages, &width, &height)){
-        fclos(fp);
-        return NULL;
-    }
-
-    if (!checkDimensions(width, height))
-    {
         fclose(fp);
         return NULL;
     }
 
+
+    //Prüfen, ob gültige Länge oder Breite
+    if(!checkDimensions(width, height)){
+        fclose(fp);
+        return NULL;
+    }
+
+    //Speicher für die gesamte Bildserie reservieren (aber nur für die Struktur)
     series = allocateSeriesStruct();
-    if (!series)
+    if(!series) {
-    {
+        fclose(fp);
-        free(series);
+        return NULL; //Fehler bei Speicherreservierung 
+    }
+
+    if(!AllocateCheckImageAndLabels(series, numImages)){
         fclose(fp);
         return NULL;
     }
+    
 
-    if (!allocateImageAndLabelArrays(series, numImages, fp))
+    //Speicher für die Pixeldaten selbst reserviern und Prüfen, ob er reserviert werden konnte 
-    {
+    if(!AllocateCheckPixelData(series, numImages, width, height)){
+        
+        fclose(fp);
         return NULL;
-    }
 
-    if (!allocateImageAndLabelImages(series, numImages, width, height, fp))
+    }
-    {
+    
+
+   if(!ReadImagesAndLabels(fp, series, numImages, width, height)){
+        fclose(fp);
         return NULL;
-    }
+   }
-
+   
-    if (!readImagesAndLabels(fp, series, numImages, width, height))
-    {
-        return NULL;
-    }
-
     return series;
-
+}
-}   
-
 
 // TODO Vervollständigen Sie die Funktion clearSeries, welche eine Bildserie vollständig aus dem Speicher freigibt
 void clearSeries(GrayScaleImageSeries *series)
 {
-
+}
-}