corrected else case in imageInput.c line 56

imageInput.c

@@ -18,7 +18,7 @@ GrayScaleImageSeries *readImages(const char *path)
 {
     GrayScaleImageSeries    *series                                     = NULL;
     FILE                    *readSource                                 = 0;
-    const unsigned int      sizeOfStausInfoElementsInBytes              = 2;
+    const unsigned int      sizeOfStausInfoElementsInBytes              = sizeof(unsigned short);
     const unsigned int      amountOfStatusInfoToRead                    = 1;
     unsigned int            numberOfBytesToRead                         = 0;
     unsigned int            expectedHeader                              = 0;
@@ -29,43 +29,39 @@ GrayScaleImageSeries *readImages(const char *path)
     
     if (readSource != NULL)
     {
-        series                  = calloc(sizeof(unsigned int) + 3* sizeof(headerString),  amountOfStatusInfoToRead);
+        series              = calloc            (amountOfStatusInfoToRead, sizeof(GrayScaleImageSeries));
-        series->images          = calloc(2*sizeof(unsigned int) + sizeof(headerString),  amountOfStatusInfoToRead);
+        series->images      = calloc            (amountOfStatusInfoToRead, sizeof(GrayScaleImage));
 
-        numberOfBytesToRead     = readStatusInfo(readSource, series, headerString, sizeOfStausInfoElementsInBytes, amountOfStatusInfoToRead);
+        numberOfBytesToRead = readStatusInfo    (readSource, series, headerString, sizeOfStausInfoElementsInBytes, amountOfStatusInfoToRead);
+        expectedHeader      = checkHeaderString (headerString);
 
-        expectedHeader          = checkHeaderString(headerString);
+        series->images      = realloc           (series->images,    series->count * sizeof(GrayScaleImage));
-
+        series->labels      = calloc            (series->count,     sizeof(&(series->labels)));
-        series->images->buffer  = calloc((series->count) * numberOfBytesToRead, sizeof(GrayScalePixelType));
-        series->labels          = calloc((series->count),                       sizeof(&(series->labels)));
         
         if (expectedHeader)
         {
-            printf("expected Header!\n");
+            for (int i = 0; i < series->count; i++)
-            // reallocate memory so that each image width can be saved seperately
+            {
-            series->images      = realloc(series->images, series->count * 2 * sizeof(unsigned int) + sizeof(headerString));
+                series->images[i].buffer    = calloc(numberOfBytesToRead, sizeof(unsigned char));
+            }
             
-            printf("count:  %d", series->count);
-            series->images[2].width = series->images->width;
-            series->images[2].height = series->images->height;
-            /*
             for (int i = 0; i < series->count; i++)
             {
                 series->images[i].width     = series->images->width;
                 series->images[i].height    = series->images->height;
             }
-                */
+            
-            printf("afer for Loop! \n");
             readImagedata(readSource, series, numberOfBytesToRead);
         }
         else
         {
-            series = NULL;
+            fclose(readSource);
+            return NULL;
         }
-        printf("before read Source\n");
+        
         fclose(readSource);
     }
-    printf("finished reading!\n");
+    
     
     return series;
 }
@@ -75,35 +71,36 @@ GrayScaleImageSeries *readImages(const char *path)
 void clearSeries(GrayScaleImageSeries * series)
 {
     int i = 0;
+    int j = 0;
 
 
-    // Write NULL into all memory spaces
+    for (i = 0; i < series->count; i++)
-    for (i = 0; i < ((series->count) * (series->images->width) * (series->images->height) / 4); i++)
     {
-        *(series->images->buffer + i * 4 * (series->images->width) * (series->images->height)) = '\0';
+        if (i >= 0)
+        {
+            for (j = 0; j < series->images[i].width * series->images[i].height; j++)
+            {
+                series->images[i].buffer[j] = 0;
+            }
         }
         
-    for (i = 0; i < (series->count); i++)
+        series->labels[i]           = 0;
-    {
+        series->images[i].width     = 0;
-        *(series->labels + i) = '\0';
+        series->images[i].height    = 0;  
     }
     
-    series->count           = 0;
+    for (i = 0; i < series->count; i++)
-    series->images->width   = 0;
+    {
-    series->images->height  = 0;
+        free(series->images[i].buffer);
+        series->images[i].buffer = NULL;
+    }
     
-    // Closse all allocated memory
-    // AND write NULL into every pointer
-    // so they can't be accessed
-    free(series->images->buffer);
-    series->images->buffer  = NULL;
     free(series->labels);
     series->labels  = NULL;
     free(series->images);
     series->images  = NULL;
     free(series);
     series          = NULL;
-
 }
 
 
@@ -126,15 +123,15 @@ unsigned int readStatusInfo(FILE *const source, GrayScaleImageSeries *const seri
 
 
 //  reads the imagebytes and the label of all images
-void readImagedata(FILE *const source, GrayScaleImageSeries *const series,  int const amountOfBytes)
+void readImagedata(FILE *const source, GrayScaleImageSeries *const series, int const amountToRead)
 {
     int i           = 0;
 
 
     for (i = 0; i < series->count ; i++)
     {       
-        fread(&(series->images->buffer[i]), sizeof(*series->images->buffer), amountOfBytes,             source);
+        fread(&series->images[i].buffer[0], sizeof(*series->images->buffer), amountToRead,              source);
-        fread(&(series->labels[i]),         sizeof(*series->images->buffer), sizeof(*series->labels),   source);
+        fread(&series->labels[i],           sizeof(*series->images->buffer), sizeof(*series->labels),   source);
     }
 }
 

matrix.c

@@ -41,19 +41,21 @@ void clearMatrix(Matrix *matrix)
 
 void setMatrixAt(MatrixType value, Matrix matrix, unsigned int rowIdx, unsigned int colIdx)
 {
+    // printf("hier0\n");
     if(matrix.buffer == NULL)
     {
-        //printf("Fehler beim Setzen! Matrix nicht initialisiert");
+        // printf("Fehler beim Setzen! Matrix nicht initialisiert");
         return;
     }
     
     if(rowIdx >= matrix.rows || colIdx >= matrix.cols)
     {
-        //printf("Ungueltige Indizes beim Setzen!\n");
+        // printf("Ungueltige Indizes beim Setzen!\n");
         return;
     }
-
+    // printf("hier1\n");
     matrix.buffer[rowIdx * matrix.cols + colIdx] = value;
+    // printf("hier2\n");
 }
 
 MatrixType getMatrixAt(const Matrix matrix, unsigned int rowIdx, unsigned int colIdx)

neuralNetwork.c

@@ -170,7 +170,7 @@ NeuralNetwork loadModel(const char *path)
 
 static Matrix imageBatchToMatrixOfImageVectors(const GrayScaleImage images[], unsigned int count)
 {
-    Matrix matrix = {NULL, 0, 0};
+    Matrix matrix = {/*NULL,*/ 0, 0, NULL};
 
     if(count > 0 && images != NULL)
     {
@@ -182,6 +182,7 @@ static Matrix imageBatchToMatrixOfImageVectors(const GrayScaleImage images[], un
             {
                 for(int j = 0; j < images[i].width * images[i].height; j++)
                 {
+                    // printf("i %d, j %d \n", i,j);
                     setMatrixAt((MatrixType)images[i].buffer[j], matrix, j, i);
                 }
             }