Opened 5 years ago

Closed 5 years ago

#14 closed ожидается проверка (задача сдана)

WW #1

Reported by: Brilliantov Kirill Owned by: Sokolov Viacheslav
Component: WW_make Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 5 years ago by Sokolov Viacheslav

В целом задача выполнена, но есть, что еще сделать лучше.

  • команда make clean завершается с ошибкой в случаях, когда сборка не удалась (например, в multiple). Это не очень хорошо: задача make clean - откатить изменения на файловой системе, произошедшие в результате запуска сборки, независимо от успешности выполнения сборки.
  • all тоже поместить в .PHONY
  • не знаю, намеренно ли так сделано, но в языке СИ у функций по типу int doComputation(); другой смысл, чем в языке C++. Лучше использовать int doComputation(void);, если ожидается 0 аргументов. Почему - видно на текущем примере. Функции же с произвольным числом аргументов, надеюсь, в СИ нам не понадобятся. От самой функции хотелось бы, чтобы она возвращала результат, зависящий от аргументов.
  • обращаю внимание на codestyle: не стоит смешивать в рамках одного проекта разные стили. Лучше оформить

void logDebug(char c) { printf("%c\n", c); } как

void logDebug(char c) {
  printf("%c\n", c); 
}

для единообразия.

comment:2 Changed 5 years ago by Sokolov Viacheslav

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

comment:3 Changed 5 years ago by Brilliantov Kirill

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

comment:4 Changed 5 years ago by Sokolov Viacheslav

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.