matrix.c

@@ -1,6 +1,5 @@
 #include <stdlib.h>
 #include <string.h>
-#include <stdio.h>
 #include "matrix.h"
 
 // TODO Matrix-Funktionen implementieren
@@ -8,13 +7,6 @@
 Matrix createMatrix(unsigned int rows, unsigned int cols)
 {
     Matrix matrix;
-    Matrix empty = {0, 0, NULL};
-
-    if(rows == 0 || cols == 0)
-    {
-        //print("Fehler: Dimensionen muessen >= 1 sein!");
-        return empty;
-    }
 
     matrix.rows = rows;
     matrix.cols = cols;
@@ -22,7 +14,7 @@ Matrix createMatrix(unsigned int rows, unsigned int cols)
 
     if(matrix.buffer == NULL)
     {
-        //printf("Fehler bei der Speicherreservierung! Keine Matrix erstellt!");
+        printf("Fehler bei der Speicherreservierung! Keine Matrix erstellt!");
         matrix.rows = 0;
         matrix.cols = 0;
     }
@@ -32,41 +24,41 @@ Matrix createMatrix(unsigned int rows, unsigned int cols)
 
 void clearMatrix(Matrix *matrix)
 {
-    free(matrix->buffer);
+    free(matrix.buffer);
 
     matrix->buffer = NULL;
     matrix->rows = 0;
     matrix->cols = 0;
 }
 
-void setMatrixAt(MatrixType value, Matrix matrix, unsigned int rowIdx, unsigned int colIdx)
+void setMatrixAt(MatrixType value, Matrix *matrix, unsigned int rowIdx, unsigned int colIdx)
 {
-    if(matrix.buffer == NULL)
+    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)
+    if(rowIdx >= matrix->rows || colIdx >= matrix->cols)
     {
-        //printf("Ungueltige Indizes beim Setzen!\n");
+        printf("Ungueltige Indizes beim Setzen!\n");
         return;
     }
 
-    matrix.buffer[rowIdx * matrix.cols + colIdx] = value;
+    matrix->buffer[rowIdx * matrix->cols + colIdx] = value;
 }
 
 MatrixType getMatrixAt(const Matrix matrix, unsigned int rowIdx, unsigned int colIdx)
 {
     if(matrix.buffer == NULL)
     {
-        //printf("Fehler beim Lesen! Matrix nicht initialisiert");
+        printf("Fehler beim Lesen! Matrix nicht initialisiert");
         return 0;
     }
 
     if(rowIdx >= matrix.rows || colIdx >= matrix.cols)
     {
-        //printf("Ungueltige Indizes beim Lesen!\n");
+        printf("Ungueltige Indizes beim Lesen!\n");
         return 0;
     }
 
@@ -77,7 +69,7 @@ Matrix add(const Matrix matrix1, const Matrix matrix2)
 {
     if((matrix1.rows != matrix2.rows) || (matrix1.cols != matrix2.cols))
     {
-        //printf("Fehler bei Addition: Matrix Dimensionen passen nicht ueberein!\n");
+        printf("Fehler bei Addition: Matrix Dimensionen passen nicht ueberein!\n");
         Matrix empty = {0, 0, NULL};
         return empty;
     }
@@ -96,7 +88,7 @@ Matrix add(const Matrix matrix1, const Matrix matrix2)
             matrix2Wert = getMatrixAt(matrix2, row, col);
             summe = matrix1Wert + matrix2Wert;
 
-            setMatrixAt(summe, ergebnisMatrix, row, col);
+            setMatrixAt(summe, &ergebnisMatrix, row, col);
         }
     }
 
@@ -107,7 +99,7 @@ Matrix multiply(const Matrix matrix1, const Matrix matrix2)
 {
     if(matrix1.cols != matrix2.rows)
     {
-        //printf("Fehler bei Multiplikation: Matrix Dimensionen passen nicht ueberein!\n");
+        printf("Fehler bei Multiplikation: Matrix Dimensionen passen nicht ueberein!\n");
         Matrix empty = {0, 0, NULL};
         return empty;
     }
@@ -127,7 +119,7 @@ Matrix multiply(const Matrix matrix1, const Matrix matrix2)
                 erg += getMatrixAt(matrix1, row, k) * getMatrixAt(matrix2, k, col);
             }
 
-            setMatrixAt(erg, ergebnisMatrix, row, col);
+            setMatrixAt(erg, &ergebnisMatrix, row, col);
         }
     }
 

matrixTests.c

@@ -164,7 +164,7 @@ void test_setMatrixAtFailsOnIndicesOutOfRange(void)
     Matrix matrixToTest = {.rows=2, .cols=3, .buffer=buffer};
 
     setMatrixAt(-1, matrixToTest, 2, 3);
-    TEST_ASSERT_EQUAL_FLOAT_ARRAY(expectedResults, matrixToTest.buffer, sizeof(buffer)/sizeof(MatrixType));
+    TEST_ASSERT_EQUAL_FLOAT_ARRAY(expectedResults, matrixToTest.buffer, matrixToTest.cols * matrixToTest.rows);
 }
 
 void setUp(void) {