WW #1

[Vasily Alferov]

Неплохо, но есть замечания.

I. correct

Замечания по корректности:

1. bin не является зависимостью всех объектных файлов. Значит, make может сломаться, если я захочу собрать только один объектный файл (не io.o). Нужно прописать его всем. Кроме того, такая последовательность команд ведёт себя странно:

make
make
make

Сначала она собирает всё, потом только io.o и lab1, потом ничего. Ожидаемое поведение: первый раз собирает всё, второй и третий — ничего. Чтобы этого добиться, нужно сделать bin order-only зависимостью всех объектных файлов (см. ​описание домашки).

2. В C-файлах вида src/io.c обычно подключают include/io.h в первой строчке. И так для всех файлов. Почему — вам раскажут на лекции; если успею, повторю на практике. Ну и если тебе всё равно исправлять, исправишь ещё и это?

Замечание по стилю:

Стиль должен быть. Программы с произвольно расставленными отступами очень трудно читать и мне их очень трудно проверять. Можно выбрать любой популярный стиль для C. Лично мой любимый — Java-like, это вариация ​1TBS с более-менее единственным отличием: открывающей фигурной скобкой для функции на той же строке, что и название функции. Это достаточно популярный стиль. Я приму любой конкретный.

В частности, должно выполняться следующее:

1. Во всех файлах ширина отступов в пробелах совпадает.

2. Отдельные функции отделены от инклюдов и между собой пустыми строками.

Я с уверенностью могу сказать, что эти элементы есть во всех популярных стилях.

II. multiple

Ок, но замечания из пункта I сюда тоже относятся.

III. unresolved

Ок, но замечания из пункта I сюда тоже относятся.