Opened 5 years ago

Closed 5 years ago

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

WW #1

Reported by: Vavilov Mark Owned by: Sokolov Viacheslav
Component: WW_make Version: 3.0
Keywords: Cc:

Description


Change History (7)

comment:1 Changed 5 years ago by Sokolov Viacheslav

Что-то в svn нет никаких файлов в директории. Их нужно добавить, чтобы я мог начать проверять.

comment:2 Changed 5 years ago by Sokolov Viacheslav

Component: HW #1 (BMP)WW_make
Type: ожидается проверкаожидаются исправления

comment:3 Changed 5 years ago by Vavilov Mark

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

comment:4 Changed 5 years ago by Sokolov Viacheslav

Что стоит поправить:

  • если сборка не удалась, make clean завершается с ошибкой
  • повторный запуск make запускает пересборку, чтобы этого не происходило, имя цели должно совпадать с результатом запуска цели (не main.o, а bin/main.o)
  • bin как order-only зависимость (после символа pipe)
  • цели, не соответствующие файлам, пометить как .PHONY:
  • добавить #pragma once в заголовочные файлы
  • убать #include <stdio.h> : смысл заголовочных файлов в минималистичности. Конечно, сценарии могут быть разные, но в данном случае предлагаю считать, что от включения этого файла ожидается декларация функции logDebug и больше ничего
  • обращаю внимание на codestyle: не стоит смешивать пробелы и табуляции в рамках одного проекта. Вместо
      if(a>b)
          return a;
      return b;
    

лучше написать return (a > b) ? a : b; либо

if(a>b) {

return a;

} else {

return b;

}

потому что (субъективно) такие варианты обладают лучшей читаемостью, их проще редактировать и труднее допустить ошибку.

  • Обращаю внимание на вызов функции int a = doComputation(5, 7.0); : он не согласован с сигнатурой int doComputation(int a, int b);

comment:5 Changed 5 years ago by Sokolov Viacheslav

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

comment:6 Changed 5 years ago by Vavilov Mark

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

comment:7 Changed 5 years ago by Sokolov Viacheslav

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