Шкуренко Никита R3237 (Image Rotation)

solution/src/main.c

     // Trying to open source-image file.
     if(!(file_in = fopen(argv[1], "rb"))){
         printf("Opening file error.");
+        fclose(file_in);
         return 0;
     }
     //Allocating memory for image.
     struct image* image;
     if(!(image = malloc(sizeof(struct image)))){
         printf("Allocating memory for image error\n");
+        fclose(file_in);
         return 0;
     }
     // Init reading from bmp function.
     FILE* file_out;
     if(!(file_out = fopen(argv[2], "wb"))){
         printf("Output file opening error\n");
+        fclose(file_out);
         return 0;
     }
 

[Дарья Рогачёва P3213]

@sayon ревью сделано

[Nikita Shkurenko]

@bowl-of-noodles замечания исправил

[Igor Zhirkov]

Распространённые ошибки, некоторые из которых тут точно есть:

• Сделай функцию-конструктор для image (и деструктор) — будет проще. И польза будет если потребуется добавить поля в image.
• Обработка ошибок fread, fwrite, fseek не везде есть
• При обработке ошибок чтения файла надо не забывать деинициализировать картинку -- там есть указатель на выделенную память в куче
• в struct pixel нужно использовать pragma pack или attribute aligned; или использовать массив из трёх элементов внутри структуры.
• Пользователю трансформаций не хочется вообще знать про заголовки bmp файлов, ему это не нужно. Он хочет просто из картинки сделать .bmp файл. Поэтому всё про заголовок надо скрыть от него внутри .c файла.
• в main есть потенциал для рефакторинга. Например, выделить ещё обёртки над from_bmp и to_bmp, которые принимали бы имя файла, а не FILE*.
• Разумно сделать функцию, которая будет считать адрес пикселя по его координатам.
• размер файла вам знать не нужно, и ftell нельзя использовать (например, на stdin он не сработает)
• Статический массив строчек в заголовочном файле -- не очень хорошо. 1) Размер файла будет пухнуть потому что потенциально любое включение .h файла в другой .c файл это может быть копия всего массива строчек. 2) с точки зрения расширяемости лучше спрятать массив в .с файл и наружу выставить функцию.
• typedef + структуры -- плохая идея кроме упаковки одного значения в структуру для создания нового типа (см. .