WW #1

[Sokolov Viacheslav]

1) Обращаю внимание на стиль написания кода:

• не стоит смешивать использование пробелов и табуляций, это приводит к тому, что форматирование зависит от способа (настроек и программы) просмотра. Я бы рекомендовал использовать только пробелы, тогда выглядеть будет всегда одинаково.
• лишние пустые строки в начале файла не помогают читать код
• отступы не соблюдены - например, перед main лишний пробел
• я бы рекомендовал не экономить на количестве строк и использовать стиль

     if (condition) {
     }
     else {
     }
  

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

2) Плохо, когда в программе есть строчки кода, которые ничего не делают. int c = max(3, 4); - здесь вычисление максимума не имеет никакого смысла, потому что следующим шагом c получит другое значение.

3) В doComputation результат вычисления a+b имеет тип double, а записывается в тип int. В языке Си это валидно, но не очень безопасно (легко ошибиться, а понять, ошибка или так задумано из кода может быть непросто), поэтому я бы рекомендовал в таких случаях явно приводить тип: c = (int)(a + b);

4) make; make запустит повторную сборку. Как я рассказывал на последней паре, проблема в цели bin, нужно для нее использовать синтаксис | bin.

[Sokolov Viacheslav]

Также цели all, clean лучше пометить как .PHONY