schöne Version

imageInput.c

@@ -6,17 +6,95 @@
 #define BUFFER_SIZE 100
 #define FILE_HEADER_STRING "__info2_image_file_format__"
 
-// TODO Implementieren Sie geeignete Hilfsfunktionen für das Lesen der Bildserie aus einer Datei
 
 // TODO Vervollständigen Sie die Funktion readImages unter Benutzung Ihrer Hilfsfunktionen
 GrayScaleImageSeries *readImages(const char *path)
 {
-    GrayScaleImageSeries *series = NULL;
+    // Initialisiert einen Zeiger zur struct und reserviert Speicherplatz
+    GrayScaleImageSeries *series = malloc(sizeof(GrayScaleImageSeries));    
+    if(series == NULL){
+        printf("Es ist nicht genügend Speicher übrig");
+        return NULL;
+    }
 
+    FILE * data = fopen(path, "rb");
+    if (data == NULL){
+        printf("Die Datei konnte nicht gelesen werden");
+        return NULL;
+    }
+
+    // Überprüfung, ob die Datei einen Header hat
+    char header[BUFFER_SIZE];
+    fread(header, strlen(FILE_HEADER_STRING), 1, data);
+    header[strlen(FILE_HEADER_STRING)] ='\0';
+    if(strncmp(header, FILE_HEADER_STRING, strlen(FILE_HEADER_STRING) )!= 0){
+        printf("Die Datei hat keinen Header");
+        fclose(data);
+        return NULL;
+    }
+    //liest die Anzahl der Bilder aus
+    fread(&series->count, sizeof(unsigned short),1, data);
+    series->images = malloc(series->count * sizeof(GrayScaleImage));
+    if (series->images == NULL){
+        printf("Es ist nicht genügend Speicher übrig");
+        fclose(data);
+        return NULL;
+    }
+
+    //liest die Höhe und Breite der Bilder aus
+    unsigned short height = 0, width = 0;
+    fread(&width, sizeof(unsigned short), 1, data);
+    fread(&height, sizeof(unsigned short), 1, data);
+    
+    //reserviert Speicher für die Labels, die aber erst nach jedem Bild eingelesen werden
+    series->labels = malloc(sizeof(unsigned char) * series->count);
+    if (series->labels == NULL){
+        printf("Es ist nicht genügend Speicher übrig");
+        free(series->images);
+        fclose(data);
+        return NULL;
+    }
+    
+    //liest jedes Bild einzeln aus und speichert es in images
+    for(int counter_picture = 0 ; counter_picture < series->count; counter_picture++){
+        
+        // für jedes Bild muss vorher eine Größe festgelegt werden, die jedoch in diesem Fall immer gleich ist
+        series->images[counter_picture].width = width;
+        series->images[counter_picture].height =height;
+        unsigned int size_picture = height * width;
+
+        //reservieren des Speichers für Buffer, der die einzelnen Pixels speichert
+        series->images[counter_picture].buffer = malloc(size_picture* sizeof(GrayScalePixelType));
+        if (series->images[counter_picture].buffer == NULL){
+            printf("Es ist nicht genügend Speicher übrig");
+            free(series->images);
+            free(series);
+            fclose(data);
+            return NULL;
+        }
+
+        //einlesen der einzelnen Pixel in buffer
+        for(int counter_pixels = 0; counter_pixels < size_picture; counter_pixels++){
+            fread(&series->images[counter_picture].buffer[counter_pixels], sizeof(unsigned char), 1, data);
+        } 
+        
+        //einlesen der Labels
+        fread(&series->labels[counter_picture], sizeof(unsigned char), 1, data);        
+        
+    }
+
+    fclose(data);
     return series;
 }
 
 // TODO Vervollständigen Sie die Funktion clearSeries, welche eine Bildserie vollständig aus dem Speicher freigibt
 void clearSeries(GrayScaleImageSeries *series)
 {
+    //erst den Speicherplatz der Pixel freigeben
+    for(int number= 0; number < series->count; number++){
+        free(series->images[number].buffer);
+    }
+    // dann die Bilder freigeben
+    free(series-> images);
+    free(series);
 }

imageInput.h

@@ -12,7 +12,7 @@ typedef struct
 
 typedef struct
 {
-    GrayScaleImage *images;
+    GrayScaleImage *images; //in sich verschachtelte Struktur
     unsigned char *labels;
     unsigned int count;
 } GrayScaleImageSeries;

matrix.c

@@ -2,8 +2,6 @@
 #include <string.h>
 #include "matrix.h"
 
-// TODO Matrix-Funktionen implementieren
-
 Matrix createMatrix(unsigned int rows, unsigned int cols)
 {
     Matrix m = {NULL, 0, 0};
@@ -32,7 +30,7 @@ void clearMatrix(Matrix *matrix)
     }
 }
 
-void setMatrixAt(MatrixType value, Matrix matrix, unsigned int rowIdx, unsigned int colIdx)     //Problematisch Call-by-Value --> Call-by-Pointer
+void setMatrixAt(MatrixType value, Matrix matrix, unsigned int rowIdx, unsigned int colIdx)
 // Matrix matrix zu Matrix *matrix, empfehlung
 {
     if (rowIdx < matrix.rows && colIdx < matrix.cols && matrix.buffer != NULL)
@@ -102,13 +100,11 @@ Matrix multiply(const Matrix matrix1, const Matrix matrix2)
 {
     // Needed: rows/Zeilen, collums/Spalten
     MatrixType buffer_add;
-    
+    // Probe ob Spalten1 = Zeilen2
-    if (!matrix1.buffer || !matrix2.buffer) // Probe ob leere Matrize vorliegt 
+    if (matrix1.cols != matrix2.rows)
-        return createMatrix(0, 0);
-    if (matrix1.cols != matrix2.rows) // Probe ob Spalten1 = Zeilen2
         return createMatrix(0, 0);
 
-    Matrix result_mul = createMatrix(matrix1.rows, matrix2.cols);
+    Matrix result_mul = createMatrix(matrix1.rows, matrix2.cols); // ""
 
     for (unsigned int index = 0; index < matrix1.rows; index++)
     {
@@ -121,7 +117,7 @@ Matrix multiply(const Matrix matrix1, const Matrix matrix2)
                 buffer_add += getMatrixAt(matrix1, index, skalar) * getMatrixAt(matrix2, skalar, shift);
             }
             // matrix_mul[index][shift] = buffer_add;
-            setMatrixAt(buffer_add, result_mul, index, shift);
+            setMatrixAt(buffer_add, result_mul, index, shift); // result als Pointer, also mit &result
         }
     }
     return result_mul;

matrix.h

@@ -7,10 +7,10 @@ typedef float MatrixType;
 
 // TODO Matrixtyp definieren
 typedef struct {
-    MatrixType *buffer;
+    MatrixType *matrix;
     unsigned int rows;
     unsigned int cols;
-} Matrix;
+}Matrix;
 
 
 Matrix createMatrix(unsigned int rows, unsigned int cols);