Opened 5 years ago

Closed 5 years ago

#98 closed ожидаются исправления (задача НЕ сдана)

WW #1

Reported by: nenashev.aleksey Owned by: nenashev.aleksey
Component: WW_make Version: 3.0
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to nenashev.aleksey
Type: ожидается проверкаожидаются исправления
  1. В репозитории не должно быть лишних файлов (например, домашки по алгоритмам).
  2. Сейчас у вас не перекомпилируется код, если поменять только заголовочный файл (например, от algorithm.h зависит много исходного кода — см. требования к корректности Makefile в задании). Добавьте зависимостей.
  3. Для включения заголовков из проекта (algorithm.h, io.h, util.h) используйте кавычки, а не угловые скобки.
  4. У lab1 зачем-то в зависимостях стоит папка bin (нужны файлы из неё, а не сама папка). К тому же указана неправильно.
  5. В util.c есть команды после последнего return. Они никогда не выполнятся. Это странно.
  6. Слишком большие отличия между correct и multiple: посмотрите на вывод команды diff -r correct multiple.

По стилю:

  1. Обычно rm -rf, а не rm -Rfv (одинаковый регистр ключей, подробный вывод всех удалённых файлов скорее не нужен, его вряд ли кто-то будет читать в предположении "Makefile работает правильно"). Ну или rm -rfv в крайнем случае.

comment:2 Changed 5 years ago by nenashev.aleksey

Owner: changed from nenashev.aleksey to Egor Suvorov
Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to nenashev.aleksey
Type: ожидается проверкаожидаются исправления

Всё ещё актуальны замечания 2, 3, 6.

comment:4 Changed 5 years ago by nenashev.aleksey

Owner: changed from nenashev.aleksey to Egor Suvorov
Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:5 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to nenashev.aleksey
Type: ожидается проверкаожидаются исправления

Увы, неуспех :(

Но близко — прям рекомендую досдать, пусть и на ноль баллов. В следующих домашках нужно будет примерно то же самое делать.

  1. Не совсем корректно прописаны зависимости в Makefile. Например, у util.o нет зависимостей от заголовочных файлов, хотя util.c делает #include "util.h" и #include "algorithm.h". У main.o всё прописано окей при этом.
  2. Для включения stdio.h используйте угловые скобки, а не кавычки — это стандартный заголовок.
  3. Всё ещё актуально замечание 6:
diff -r correct/Makefile multiple/Makefile
21d20
< 
diff -r correct/include/io.h multiple/include/io.h
1c1
< void logDebug(char s);
---
> void logDebug(char c) ;
diff -r correct/src/main.c multiple/src/main.c
4c4
< #include"algorithm.h"
---
> #include"algorithm.c"
diff -r correct/Makefile unresolved/Makefile
21d20
< 
diff -r correct/src/io.c unresolved/src/io.c
3c3
< void logDebug(char c) {
---
> void loDebug(char c) {
diff -r correct/src/util.c unresolved/src/util.c
5c5
< logDebug('m');
---
>   logDebug('m');
9c9
< logDebug('m');
---
>   logDebug('m');

comment:6 Changed 5 years ago by Egor Suvorov

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

Если хотите, можете досдать не на баллы онлайн (уже не через Trac, просто напишите мне, я посмотрю в SVN) или лично на паре.

Note: See TracTickets for help on using tickets.