WW #1

[Артур Гулецкий (huletski)]

correct

зачтено (+0.5 баллов)

Замечания (учтите на будущее):

• io.c - в отсутствуют отступы в теле функции;
• util.c:7 - нет отступов, использование тернарного оператора сделало бы код лаконичнее: return (l < k) ? k : l; ;
• algorithm.h:1 - лучше выбирать идентификатор для guard'a, основываясь на имени файла/общем содержимом, а не на имени функции (при помощи #define можно так же определить макрофункцию, имя которой может случайно совпасть с 'id'). Пример идентификатора получше - ALGORITHM_H_INCLUDED. Аналогично для других заголовочных файлов;
• Makefile:3 - зависимости от заголовочных файлов лишние, так как заголовочные файлы не участвуют в линковке. Если заголовочный файл изменится, должен изменится соотвествующий объектный файл, что приведет к перелинковке;
• Makefile:4 - заголовочные файлы лишние, при линковке не нужны;
• Makefile:7 - зависимость от bin лучше поместить во все цели, которые требуют наличия папки bin для успешного выполнения, так как работоспособность текущего решения зависит от порядка перечисления зависимостей в цели lab1 (если поменять местами зависимости bin/main.o и bin/io.o решение перестанет работать). Это свойство делает решение менее удобным для поддержки в дальнейшем.

multiple

зачтено (+0.25)

unresolved

В задании требуется добиться ошибок линковки без модификации Makefile'a. В решении файл изменился:

{lab_01}[1662]$ svn up
Updating '.':
At revision 270.
{lab_01}[1663]$ pwd
/home/hfx/dvl/cpp19/kirillova.evgeniya/lab_01
{lab_01}[1664]$ diff correct/Makefile unresolved/Makefile 
4c4
< 	gcc bin/main.o bin/io.o bin/util.o bin/algorithm.o include/algorithm.h include/io.h include/util.h -o lab1 -I./include -Wall -Werror -Wextra
---
> 	gcc bin/main.o include/algorithm.h include/io.h include/util.h -o lab1 -I./include -Wall -Werror -Wextra

Итог: 0.75/1. Исправляйте.

[Артур Гулецкий (huletski)]

unresolved

зачтено +0.25