Assigment 3 Image Rotation Kolesnikov Nikita P33131

solution/src/image.c

+#include "image.h"
+#include "malloc.h"
+#include "stdint.h"

solution/src/imageEditor.c

+#include "imageEditor.h"
+
+int rotateImage(struct image src, struct image *out);
+
+int rotateBMPImage(char *source, char *dest) {
+    FILE *input = {0};
+    if (readF(source, &input))
+        return 1;
+    struct image image;
+    if (fromBMP(input, &image)){
+        destroyImage(&image);

solution/include/BMP.h

+#ifndef IMAGE_TRANSFORMER_BMP_H
+#define IMAGE_TRANSFORMER_BMP_H
+#include "errorUtils.h"
+#include "fileUtils.h"
+#include "image.h"
+#include <inttypes.h>
+#include <stdio.h>
+
+int fromBMP(FILE* to, struct image* img);
+
+int toBMP(FILE* out, struct image img);
+
+#endif //IMAGE_TRANSFORMER_BMP_H

solution/include/errorUtils.h

+#ifndef IMAGE_TRANSFORMER_ERRORUTILS_H
+#define IMAGE_TRANSFORMER_ERRORUTILS_H
+#include <stdio.h>
+
+
+// FILE ERRORS : 0x
+// MEM ERRORS : 1x
+// POINTERS ERRORS: 2x
+// ARGS ERRORS: 3x
+// IMAGE ROTATION ERRORS: 4x
+enum error {
+    OPEN_FILE_ERROR = 00,
+    WRITE_FILE_ERROR = 01,
+    READ_FILE_ERROR = 02,
+    CLOSE_FILE_ERROR = 03,
+    READ_NO_MEM = 10,
+    READ_INVALID_BITS = 11,
+    WRONG_POINTER_ERROR = 12,
+    WRONG_ARG_ERROR = 30,
+    IMAGE_ROTATION_ERROR = 40,
+    WRONG_BMP_HEADER_ERROR = 41,
+    WRONG_BMP_SIGNATURE_ERROR = 42,
+    INVALID_IMAGE_ERROR = 43
+};
+
+void logErrorDescription(enum error err);
+
+void logErrorDescriptionWithLabel(enum error err, char* debugLabel);
+
+#endif //IMAGE_TRANSFORMER_ERRORUTILS_H

solution/include/fileUtils.h

+#ifndef IMAGE_TRANSFORMER_FILEUTILS_H
+#define IMAGE_TRANSFORMER_FILEUTILS_H
+
+#include "errorUtils.h"
+#include <stdio.h>
+
+int readF(const char *pathname, FILE **file);
+
+int writeF(const char *pathname, FILE **file);
+
+int closeF(FILE *file);
+
+#endif //IMAGE_TRANSFORMER_FILEUTILS_H

solution/include/image.h

+#ifndef IMAGE_TRANSFORMER_IMAGE_H
+#define IMAGE_TRANSFORMER_IMAGE_H
+#include <stdbool.h>
+#include <stdint.h>
+struct pixel {
+    uint8_t b;
+    uint8_t g;
+    uint8_t r;
+};
+
+struct image {
+    uint64_t width;
+    uint64_t height;
+    struct pixel* data;
+};
+
+extern const struct image wrongImage;
+
+struct image initImage(uint64_t width, uint64_t height);
+
+struct pixel* target(struct image img, uint64_t x, uint64_t y);
+
+bool destroyImage(struct image *image);
+#endif //IMAGE_TRANSFORMER_IMAGE_H

solution/include/imageEditor.h

+#ifndef IMAGE_TRANSFORMER_IMAGEEDITOR_H
+#define IMAGE_TRANSFORMER_IMAGEEDITOR_H
+#include "BMP.h"
+#include <stdbool.h>
+#include <stdio.h>
+
+int rotateBMPImage(char* source, char* dest);
+
+#endif //IMAGE_TRANSFORMER_IMAGEEDITOR_H

[Oleg Pashchenko]

сейчас проревьювю

[Oleg Pashchenko]

Нет четкого code style, но в C рекомендуется писать на snake case

[Oleg Pashchenko]

Согласно правилам стиля написания программ на C следует включать заголовочные файлы в следующем порядке:

file.h (соответствующий ему заголовочный файл). Файлы из стандартной библиотеки. Другие заголовочные файлы.

Также <> обычно используются для заголовочных файлов, встроенных в компилятор, а "" - для пользовательских заголовочных файлов. Поэтому лучше написать include <stdio.h>, но include "image.h"

[Oleg Pashchenko]

Код файла выглядит слабо читаемым, на мой взгляд невооруженным взглядом сложно понять что тут происходит. Лучше добавить разбиение.

[Oleg Pashchenko]

Идея с логированием классная, думаю в следующих проектов буду использовать ее благодаря вам ;)

[Oleg Pashchenko]

@andrew review готово