Kommentare große Funktion

imageInput.c

@@ -9,11 +9,11 @@
 
 // TODO Implementieren Sie geeignete Hilfsfunktionen für das Lesen der Bildserie aus einer Datei
 
-static int checkDimensions(GrayScaleImage imageData){
+static int checkDimensions(unsigned int width, unsigned int height){
 
     int ok = 0;
     
-    if(imageData.width == 10 && imageData.height == 10 && imageData.buffer <= BUFFER_SIZE)
+    if(width > 0 && height > 0)
         return ok = 1;
     
     else 
@@ -48,6 +48,115 @@ GrayScaleImageSeries *readImages(const char *path)
 {
     GrayScaleImageSeries *series = NULL;
 
+    FILE *fp = fopen(path, "rb"); //Datei im Binärmodus öffnen
+    if(!fp)
+        return NULL; //Fehler: Datei konnte nicht geöffnet werden 
+
+    //Buffer zum Einlesen des Headers 
+    GrayScalePixelType header[HEADER_LEN];
+    
+    //Prüfen, ob genau die HEADER_LEN Bytes eingelesen werden
+    if(fread(header, sizeof(GrayScalePixelType), HEADER_LEN, fp) != HEADER_LEN){
+        fclose(fp);
+        return NULL;
+    }
+
+    //Prüfen, ob es sich um den korrekten Header handelt mit Hilfsfunktion
+    if(!checkHeader(header)){
+        fclose(fp);
+        return NULL;
+    }
+
+    //Anzahl der Bilder, Breite und Höhe einlesen 
+    unsigned  int numImages, width, 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)
+    {
+        fclose(fp);
+        return NULL;
+    }
+
+    if (!checkDimensions(width, height))
+    {
+        fclose(fp);
+        return NULL;
+    }
+
+    //Speicher für die gesamte Bildserie reservieren (aber nur für die Struktur)
+    series = malloc(sizeof(GrayScaleImageSeries));
+    if(!series) {
+        free(series);
+        fclose(fp);
+        return NULL; //Fehler bei Speicherreservierung 
+    }
+
+    series->count = numImages;
+    //Speicher für die Pointer an sich belegen
+    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)); //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){
+
+        free(series->images); //wenn trotzdem was reingeschrieben wurde, dann freigeben
+        free(series->labels);
+        free(series);
+
+        fclose(fp);
+        return NULL;
+
+    }
+
+    //Speicher für die Pixeldaten selbst reserviern und Prüfen, ob er reserviert werden konnte 
+    for(unsigned int i = 0; i < numImages; i++){
+
+        series -> images[i].buffer = malloc(width * height); //Speicher für Pixeldaten selbst
+
+        if(series->images[i].buffer == NULL){ 
+
+            for(unsigned int j = 0; j < i; j++){
+                free(series->images[j].buffer); //wenn Speicher nicht reserviert werden konnte, dann zuerst das Array mit Pixeldaten freigeben
+            }
+
+        
+        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->labels);
+        free(series);
+    
+        fclose(fp);
+        return NULL;
+
+        
+        }
+    }
+
+   for(int i = 0; i < numImages; i++)
+   {
+        //Breite und Höhe werden gesetzt (GrayScaleImage)
+        series -> images[i].width = width;
+        series -> images[i].height = 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 NULL;
+        }
+
+        //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);
+            fclose(fp);
+            return NULL;
+        }
+    }
+
+
+
+    
+    
+    
     return series;
 }