still fixed changes related to matrix

imageInput.c

@@ -7,47 +7,71 @@
 #define FILE_HEADER_STRING "__info2_image_file_format__"
 
 // TODO Implementieren Sie geeignete Hilfsfunktionen für das Lesen der Bildserie aus einer Datei
-void setParameters(const char buffer[], unsigned int* imageCount, int* width, int* height);
+
+int checkString (FILE *file){ //Checks if String at the start of File equals expected Format (0 for wrong 1 for right)
+	if (file == NULL){ //returns 0 for empty file
+		return 0;
+		}
+	char expectedString[] = FILE_HEADER_STRING;
+	char actualString[BUFFER_SIZE];
+
+	rewind(file);
+	size_t bytesRead = fread(actualString, 1,27 , file); //Stores Bytes of start of File to actualString
+	if (bytesRead != 27){
+	return 0;} //Returns 0 if File is to short
+	actualString[27] = '\0';
+
+	if (strcmp(actualString, expectedString) != 0){
+	return 0;
+}
+	else{
+	return 1;
+}
+}
+
+void setParameters(FILE* file, unsigned int* imageCount, unsigned int* width, unsigned int* height);
 void allocateMemory(GrayScaleImageSeries *s, const int imageCount, const int width, const int height);
 // TODO Vervollständigen Sie die Funktion readImages unter Benutzung Ihrer Hilfsfunktionen
 GrayScaleImageSeries *readImages(const char *path)
 {
+	FILE * file;
+	// TODO open binary file from file name
     GrayScaleImageSeries *series = NULL;
+	if(!(checkString(file)))
+		return NULL;
 
-	// hier Niko Stuff
-    // create char array buffer with bytes containing initialization information
-    // TODO some kind of logic to get correct address in binary file
-    char initBuffer[3];
-    /*
-	int readBytes = fread(initBuffer, 1, 3, //file);
-     */
     unsigned int* imageCount = &(series->count); // sets a pointer for int variable count in struct
-    setParameters(initBuffer, imageCount, series->images->width, series->images->height);
+
-    allocateMemory(*imageCount, series->images->width, series->images->height);
+    setParameters(file, imageCount, &(series->images->width), &(series->images->height));
+    allocateMemory(series, *imageCount, series->images->width, series->images->height);
 
     return series;
 }
 
-void setParameters(const char buffer[], unsigned int* imageCount, int* width, int* height) { // sets the parameters
-	*imageCount = (int) buffer[0];
-    *width = (int) buffer[1];
-    *height = (int) buffer[2];
-}
-
-void allocateMemory(GrayScaleImageSeries s, const int imageCount, const int width, const int height) {
-	for (int i = 0; i < imageCount; i++) {
-		s->images[i].buffer = (unsigned char *) malloc(width * height * sizeof(unsigned char)); // allocates memory for every image in the series
-	}
-}
-
 // TODO Vervollständigen Sie die Funktion clearSeries, welche eine Bildserie vollständig aus dem Speicher freigibt
 void clearSeries(GrayScaleImageSeries *series)
 {
-	for(size_t i = GrayScaleImageSeries.count - 1; i >= 0; i--)
+	for(size_t i = series->count - 1; i >= 0; i--)
 	{
-		free(GrayScaleImageSeries.images+GrayScaleImageSeries.count*sizeof(GrayScaleImage)*i);
+		free(series->images+series->count*sizeof(GrayScaleImage)*i);
-		free(GrayScaleImageSeries.labels+GrayScaleImageSeries.count*sizeof(unsigned char)*i);
+		free(series->labels+series->count*sizeof(unsigned char)*i);
+	}
+	series->images = NULL;
+	series->labels = NULL;
+}
+void setParameters(FILE* file, unsigned int* imageCount, unsigned int* width, unsigned int* height) { // sets the parameters
+	char buffer[3];
+
+	fseek(file, 27, SEEK_SET);
+	fread(buffer, 1, 3, file);
+
+	*imageCount = (int) buffer[0];
+    *width = (int) buffer[1];
+    *height = (int) buffer[2];
+}
+// change this to createMatrix?
+void allocateMemory(GrayScaleImageSeries* s, const int imageCount, const int width, const int height) {
+	for (int i = 0; i < imageCount; i++) {
+		s->images[i].buffer = (unsigned char *) malloc(width * height * sizeof(unsigned char)); // allocates memory for every image in the series
 	}
-	GrayScaleImageSeries.images = NULL;
-	GrayScaleImageSeries.labels = NULL;
 }

matrix.c

@@ -5,11 +5,11 @@
 
 // TODO Matrix-Funktionen implementieren
 
-typedef struct Matrix {
+typedef struct Matrix{
     unsigned int rows;
     unsigned int cols;
     MatrixType* buffer;
-} Matrix;
+}Matrix;
 
 
 Matrix createMatrix(unsigned int rows, unsigned int cols)
@@ -23,18 +23,18 @@ Matrix createMatrix(unsigned int rows, unsigned int cols)
 
 void clearMatrix(Matrix *matrix)
 {
-    free(matrix.buffer);
+    free(matrix->buffer);
-	matrix.buffer = NULL;
+	matrix->buffer = NULL;
 }
 
 void setMatrixAt(MatrixType value, Matrix matrix, unsigned int rowIdx, unsigned int colIdx)
 {
     if (rowIdx >= matrix.rows || colIdx >= matrix.cols){
         printf("Index out of bounds\n"); //Error Message because Index Input exceeds Matrix
-}
+	}
     else{
         matrix.buffer[rowIdx * matrix.cols + colIdx] = value; //Writes Value of value variable in the selected place in Matrix
-}
+	}
 }
 
 MatrixType getMatrixAt(const Matrix matrix, unsigned int rowIdx, unsigned int colIdx)
@@ -47,16 +47,15 @@ MatrixType getMatrixAt(const Matrix matrix, unsigned int rowIdx, unsigned int co
 
        MatrixType value = matrix.buffer[rowIdx * matrix.cols + colIdx]; //Stores value of selected place of Matrix in value variable
         return value;
+	}
 }
-  }
 
 Matrix add(const Matrix matrix1, const Matrix matrix2)
 {
     Matrix MatrixErgebnis = createMatrix(matrix1.rows, matrix1.cols); //Creating Result Matrix
     if(matrix1.cols != matrix2.cols || matrix1.rows != matrix2.rows){
         printf("Matrix dimensions do not match\n"); //Error Message if dimensions of Input Matrixes are not identical
-        clearMatrix(MatrixErgebnis);
+        clearMatrix(&MatrixErgebnis);
-        MatrixErgebnis = NULL;
         return MatrixErgebnis;
     }
     else{
@@ -66,16 +65,15 @@ Matrix add(const Matrix matrix1, const Matrix matrix2)
                 //Adding Matrix Elements of same row and col index together and store in new Matrix
                 MatrixErgebnis.buffer[i * matrix1.cols + j] = matrix1.buffer[i * matrix1.cols + j] + matrix2.buffer[i * matrix1.cols + j];
 
-                }
+            }
         }
 
-  }
+	}
-return MatrixErgebnis;
+	return MatrixErgebnis;
 }
 
 Matrix multiply(const Matrix matrix1, const Matrix matrix2)
 {
-
     Matrix result;
     result.rows = matrix1.rows;
     result.cols = matrix2.cols;

matrix.h

@@ -7,7 +7,11 @@ typedef float MatrixType;
 
 // TODO Matrixtyp definieren
 
-typedef struct Matrix Matrix;
+typedef struct Matrix{
+    unsigned int rows;
+    unsigned int cols;
+    MatrixType* buffer;
+}Matrix;