Opened 5 years ago

Last modified 5 years ago

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

WW #1

Reported by: kalugin.evgeniy Owned by: Vasily Alferov
Component: WW_make Version: 1.0
Keywords: Cc:

Description


Change History (2)

comment:1 Changed 5 years ago by Vasily Alferov

Спасибо, что отправил недоделанную версию, как и было предложено. Пожалуйста, не забудь доделать.

  1. Структура папок: папка cpp2019 лишняя, убери. Просто в корень личного репозитория положи lab_01.
  1. Упс, не собирается :(
platypus@platypus-pc:~/hse/cpp/cpp19/kalugin.evgeniy/cpp2019/lab_01/correct$ make
mkdir bin
gcc ./src/io.c -Iinclude -c -Wall -Wextra -Werror -o ./bin/io.o
gcc ./src/algorithm.c -Iinclude -c -Wall -Wextra -Werror -o ./bin/algorithm.o
gcc ./src/util.c -Iinclude -c -Wall -Wextra -Werror -o ./bin/util.o
./src/util.c: In function ‘max’:
./src/util.c:9:6: error: suggest parentheses around assignment used as truth value [-Werror=parentheses]
  if (a=b)
      ^
./src/util.c:15:1: error: control reaches end of non-void function [-Werror=return-type]
 }
 ^
cc1: all warnings being treated as errors
make: *** [Makefile:19: bin/util.o] Error 1

Обязательно нужно поправить. Вкратце про твои ошибки:

  • Сравнение на равенство — оператор ==, а не =.
  • Бывает так, что ни одно из условий a > b, a < b, a == b не выполнено. Это связано с особенностями вещественных чисел. В таком случае функция max ничего не вернёт, и случится undefined behaviour. Чтобы этого не случилось, можно, например, убрать последний if. Если мы дошли до той строчки, давай всегда будем возвращать b.
  • Стиль. Он должен быть. Посмотри, например, первые две с половиной страницы этого документа.

Дальше проверять не вижу смысла, пока домашка не компилируется.

comment:2 Changed 5 years ago by Vasily Alferov

Type: ожидается проверкаожидаются исправления
Note: See TracTickets for help on using tickets.