Opened 5 years ago

Closed 5 years ago

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

WW_1

Reported by: Roman Venediktov Owned by: Sokolov Viacheslav
Component: WW_make Version: 2.0
Keywords: Cc: Roman Venediktov

Description


Change History (5)

comment:1 Changed 5 years ago by Sokolov Viacheslav

Резюме: по сути все верно, но есть несколько огрех разной степени важности. Они перечислены ниже (не по порядку важности).

0) Бинарные файлы, включая lab1, не стоит держать в репозитории (они занимают место, для них diff не имеет смысла, легко получаются с помощью запуска компилятора).

1) В текущем состоянии сборка не работает

╰─>$ make
mkdir bin
mkdir: cannot create directory ‘bin’: File exists
Makefile:9: recipe for target 'makeBin' failed
make: *** [makeBin] Error 1

2) стандартная конвенция состоит в том, что цель называется make clean

3) можно разбить компиляцию исполняемого файла на две цели: создание объектного файла main.o и последующую компоновку всех объектных файлов в исполняемый

4) в таком случа стоит сменить именование с runMain (это не очень хорошее название, потому что можно более точно назвать происходящий процесс) на, например, linkMain

И то, что не играет роли в этом задании, но будет играть значение дальше:
в файле util.c имеется неоднородность стиля кода (расстановка отступов до / после фигурных скобок; заключение очередного scope в фигурные скобки). Кроме того, я бы рекомендовал придерживаться стиля кода, в котором фигурные скобки ставятся на каждый возникающий scope. Да, кода становится чуть-чуть больше, но есть много других преимуществ (труднее ошибиться; проще портировать код; проще читать; .....)

comment:2 Changed 5 years ago by Sokolov Viacheslav

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

comment:3 Changed 5 years ago by Sokolov Viacheslav

Да, и что там со второй частью?

comment:4 Changed 5 years ago by Roman Venediktov

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

comment:5 Changed 5 years ago by Sokolov Viacheslav

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