Opened 5 years ago
Closed 5 years ago
#62 closed ожидается проверка (задача сдана)
WW #1
Reported by: | lebedev.egor | Owned by: | Vasily Alferov |
---|---|---|---|
Component: | WW_make | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (6)
comment:1 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Упс, не компилируется. Очевидно, ты забыл -Iinclude
. Нужно поправить.
$ make mkdir bin gcc -Wall -Wextra -Werror -c -Iinclude src/main.c -o bin/main.o gcc -Wall -Wextra -Werror -c src/io.c -o bin/io.o src/io.c:2:10: fatal error: io.h: No such file or directory #include <io.h> ^~~~~~ compilation terminated. make: *** [Makefile:13: bin/io.o] Error 1
comment:4 Changed 5 years ago by
Для истории: я считаю, что тут пока не больше, чем 0.7, так как в последней версии код не компилируется.
comment:5 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
Note: See
TracTickets for help on using
tickets.
Всё очень хорошо, мне нравится. У меня есть небольшие замечания по стилю. После их исправления работа станет зачтённой.
src/io.c
обычно подключаютinclude/io.h
в первой строчке. И так для всех файлов. Почему — вам раскажут на лекции; если успею, повторю на практике. Ну и если тебе всё равно исправлять, исправишь ещё и это?doComputation
у тебя пробелы, а вlogDebug
— табуляция. А вmax
— сразу и то, и то.printf
пробелов ставить не надо, она ничем не отличается от функцийmax
иlogDebug
. Ну или тогда уж можно ставить пробелы вообще везде. Но если ты имел ввиду страуструповский стиль (смотрю поelse
на отдельной строчке), то там этого пробела нет. Вообще, я почти уверен, что ни в одном популярном стиле нет.