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
comment:2 Changed 5 years ago by
Component: | HW #1 (BMP) → WW_make |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:3 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:4 Changed 5 years ago by
Что стоит поправить:
- если сборка не удалась, 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
Type: | ожидается проверка → ожидаются исправления |
---|
comment:6 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
comment:7 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Что-то в svn нет никаких файлов в директории. Их нужно добавить, чтобы я мог начать проверять.