kommentare zu verbesserungen hinzugefügt

This commit is contained in:
AD005\z0046beb 2025-11-20 10:12:11 +01:00
parent 952555f162
commit 98e0fb1ec5
2 changed files with 13 additions and 13 deletions

View File

@ -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;

View File

@ -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)