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 Vasily Alferov

Type: ожидается проверкаожидаются исправления

Всё очень хорошо, мне нравится. У меня есть небольшие замечания по стилю. После их исправления работа станет зачтённой.

  1. В C-файлах вида src/io.c обычно подключают include/io.h в первой строчке. И так для всех файлов. Почему — вам раскажут на лекции; если успею, повторю на практике. Ну и если тебе всё равно исправлять, исправишь ещё и это?
  1. Не мешай табы с пробелами — либо табы, либо пробелы. Например, в doComputation у тебя пробелы, а в logDebug — табуляция. А в max — сразу и то, и то.
  1. После вызова функции printf пробелов ставить не надо, она ничем не отличается от функций max и logDebug. Ну или тогда уж можно ставить пробелы вообще везде. Но если ты имел ввиду страуструповский стиль (смотрю по else на отдельной строчке), то там этого пробела нет. Вообще, я почти уверен, что ни в одном популярном стиле нет.

comment:2 Changed 5 years ago by lebedev.egor

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 5 years ago by Vasily Alferov

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 Vasily Alferov

Для истории: я считаю, что тут пока не больше, чем 0.7, так как в последней версии код не компилируется.

comment:5 Changed 5 years ago by lebedev.egor

Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:6 Changed 5 years ago by Vasily Alferov

Resolution: задача сдана
Status: assignedclosed

Зачтено.

Note: See TracTickets for help on using tickets.