WW #1

[Sokolov Viacheslav]

Что стоит поправить:

• если сборка не удалась, make clean завершается с ошибкой

• поместить цели, не соответствующие файлам на диске, в секцию .PHONY:

• в algorithm.c не хватает #include "algorithm.h", в других файлах тоже не хватает включений заголовочных файлов

• не знаю, намеренно ли так сделано, но в языке СИ у функций по типу int doComputation(); другой смысл, чем в языке C++. Лучше использовать int doComputation(void);, если ожидается 0 аргументов. Почему - видно на текущем примере. Функции же с произвольным числом аргументов, надеюсь, в СИ нам не понадобятся.

• обращаю внимание на codestyle: не стоит смешивать пробелы и табуляции в рамках одного проекта. Вместо

    if(a>b)return a;
    return b;
  

лучше написать return (a > b) ? a : b; либо

  if(a>b) {
    return a;
  } else {
    return b;
  }

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

[Alexander Morozov]

Вроде бы все исправил. Там есть, возможно, странное форматирование функции, но это не я, это все clang-format.

[Sokolov Viacheslav]

Про форматирование понял, думаю, на паре обсудим этот момент.

В остальном порядок, за исключением того, что не собирается:

algorithm.c:(.text+0x22): undefined reference to `round'