Opened 5 years ago

Last modified 5 years ago

#70 assigned ожидаются исправления

WW#1

Reported by: asatullaev.maruf Owned by: Vasily Alferov
Component: WW_make Version: 2.0
Keywords: alferov Cc:

Description


Change History (3)

comment:1 Changed 5 years ago by Vasily Alferov

Owner: changed from Evgeny Linsky to Vasily Alferov
Type: ожидается проверкаожидаются исправления

Всё очень хорошо, но есть несколько замечаний.

На будущее, пожалуйста, пиши мой логин не в keywords, а вешай (assign) тикет на меня.

Корректность

В multiple имелась ввиду другая ошибка. У тебя ошибка компиляции (на этапе создания объектного файла), а должна быть ошибка линковки. И она выглядит именно так, как в условии.

В остальном никаких замечаний по корректности у меня нет.

Стиль

Эти замечания нужно исправить во всех трёх папках.

  1. В Makefile на этапе линковки можно не писать флаг -Iinclude.
  1. Стиль кода. Он должен быть. Можешь воспользоваться любым популярным стилем для C-подобных языков. Я, например, предпочитаю какой-то вариант 1TBS, похожий на Java: единственное отличие в том, что и перед {, открывающей тело функции, переноса строки не будет.

Конкретно у тебя: отступы в разных файлах разные, а где-то их вообще нет. Зачем-то иногда стоят пробелы перед ; — никогда такого не видел. Зато нет пробелов вокруг операторов — тоже не хорошо, у нас на вики выложены гайдлайны, и всё это им противоречит.

  1. Не забудь подключить util.h в util.c.
  1. Давай уже использовать guards или #pragma once, раз они уже рассказаны.

comment:2 Changed 5 years ago by asatullaev.maruf

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

comment:3 Changed 5 years ago by Vasily Alferov

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

С корректностью теперь всё ок, но со стилем всё ещё беда.
Пока не могу поставить больше, чем 0.5.

Я вижу попытку сделать какой-то стиль, но, кажется, тебе категорически не хватает примеров. Почитай вот этот документ, тут их много.

Отступы в каких-то файлах есть, а в каких-то нет. Пробелы: есть _перед_ запятой в перечислении аргументов у функции, нет вокруг операторов. Это, конечно, никуда не годится.

Note: See TracTickets for help on using tickets.