diff --git a/imageInput.c b/imageInput.c index dbbbfc6..dd0f223 100644 --- a/imageInput.c +++ b/imageInput.c @@ -26,14 +26,14 @@ static int ReadCheckHeader(FILE *fp, GrayScalePixelType *header) return 1; } -static int checkHeader(const GrayScalePixelType *buffer) +static int checkHeader(const GrayScalePixelType *buffer) // mit in die readcheckheader funkiton rein, feste buffergröße, char statt grayscalepixeltype weil ja nur ein string überprüft wird und dann mit strncpy arbeiten { int ok = 0; char header[HEADER_LEN + 1]; //Array mit Platz für Header Länge + \0 Character -> String // Jeder Byte in String kopieren - for(int i = 0; i < HEADER_LEN; i++){ + for(int i = 0; i < HEADER_LEN; i++){ //geht aber auch memcpy header[i] = buffer[i]; @@ -86,7 +86,7 @@ static int AllocateCheckImageAndLabels(GrayScaleImageSeries *series, unsigned in 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 + series->labels = malloc(numImages * sizeof(unsigned char)); //Speicher für alle Labels, diese beiden zeilen in die funktion drüber mit rein //Prüfen, ob Speicher für die Arrays reserviert werden konnte, wenn nicht -> Freigabe if(series->images == NULL || series->labels == NULL){ @@ -155,20 +155,20 @@ static int ReadImagesAndLabels(FILE *fp, GrayScaleImageSeries *series, unsigned // TODO Vervollständigen Sie die Funktion readImages unter Benutzung Ihrer Hilfsfunktionen -GrayScaleImageSeries *readImages(const char *path) +GrayScaleImageSeries *readImages(const char *path)//funktionsnamen immer klein anfangen { GrayScaleImageSeries *series = NULL; //Datei im Binärmodus öffnen - FILE *fp = openImageFile(path); + FILE *fp = openImageFile(path); //funktion weglassen weil sie keine zusätzlichen infos bringt if(!fp) return NULL; //Buffer zum Einlesen des Headers - GrayScalePixelType header[HEADER_LEN]; + GrayScalePixelType header[HEADER_LEN]; //nicht grayscalepixeltype sonder char //Prüfen, ob genau die HEADER_LEN Bytes eingelesen werden - if(!ReadCheckHeader(fp, header)){ + if(!ReadCheckHeader(fp, header)){ fclose(fp); return NULL; @@ -176,7 +176,7 @@ GrayScaleImageSeries *readImages(const char *path) } //Prüfen, ob es sich um den korrekten Header handelt mit Hilfsfunktion - if(!checkHeader(header)){ + if(!checkHeader(header)){ //mit in readCheckHeader fclose(fp); return NULL; @@ -233,7 +233,7 @@ GrayScaleImageSeries *readImages(const char *path) fclose(fp); return NULL; - } + } // funktionen anders aufteilen: erst für alles speicher holen und dann bild und label einlesen aber ist geschmackssache, ziel ist es ohne kommentare auskommen zu können fclose(fp); return series; diff --git a/matrix.c b/matrix.c index 8f5c294..2c24013 100644 --- a/matrix.c +++ b/matrix.c @@ -5,7 +5,7 @@ // TODO Matrix-Funktionen implementieren -enum addModes{SAMEDIMENSIONS, COLVEC, ROWVEC}; +enum addModes{SAMEDIMENSIONS, COLVEC, ROWVEC}; // mit typedef arbeiten Matrix createMatrix(unsigned int rows, unsigned int cols) @@ -84,7 +84,7 @@ MatrixType getMatrixAt(const Matrix matrix, unsigned int rowIdx, unsigned int co } -static int get_add_mode(Matrix matrix1, Matrix matrix2) +static int get_add_mode(Matrix matrix1, Matrix matrix2) //int stimmt zwar, aber man kann auch statt int den enum als typedef und dann als rückgabetyp nehmen { int get_add_mode = -1; @@ -114,7 +114,7 @@ static int get_add_mode(Matrix matrix1, Matrix matrix2) -Matrix addSameDim(Matrix matrix1, Matrix matrix2) +Matrix addSameDim(Matrix matrix1, Matrix matrix2) //static funktionen!!! { Matrix matrix_erg = createMatrix(matrix1.rows, matrix1.cols); @@ -197,7 +197,7 @@ Matrix addRowVec(Matrix matrix1, Matrix matrix2) Matrix add(const Matrix matrix1, const Matrix matrix2) { - int ok = get_add_mode(matrix1,matrix2); + int ok = get_add_mode(matrix1,matrix2); //auch wieder enum als typ statt int (wenn oben typedef dann kann man sich auch das enum sparen) Matrix matrix_erg = createMatrix(0, 0); switch(ok)