Opened 5 years ago

Closed 5 years ago

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

WW #1

Reported by: lopatin.mikhail Owned by: Sokolov Viacheslav
Component: WW_make Version: 1.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 5 years ago by Sokolov Viacheslav

Основная цель задания выполнена. Несколько комментариев (в этот раз можно не исправлять, но стоит учесть в дальнейшем):

  • из main вызывается функция doComputation с аргументами не таких типов, как ожидается в декларации функции, так лучше не делать
  • стоит быть внимательнее с стилем кода, где-то есть пробел после запятой, где-то нет; мое субъективное мнение - стиль if-else с однострочными возвратами и дополнительной табуляцией не очень удачный. Я бы рекомендовал оформить, например, так:
        if (a > b) {
            return a;
        } else {
            return b;
        }
    

это, конечно, дело вкуса, но на мой взгляд так проще читать, труднее ошибиться, проще в дальнейшем работать с этим кодом (что перекрывает аргумент про количество символов).

  • для увелечения в том числе надежности (вижу .PHONY) стоит цели бить на соответствующие файлам (тем самым не main.o, а bin/main.o) и остальные (их записывать в .PHONY).

comment:2 Changed 5 years ago by Sokolov Viacheslav

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