Opened 3 years ago

Closed 3 years ago

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

WW figures Maksim Martynov lab_09

Reported by: martynov.maksim Owned by: Антон Филатов
Component: WW figures Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Антон Филатов

Type: ожидается проверкаожидаются исправления
make
mkdir -p obj
g++ -O2 -Wall -Werror -std=c++11 -Iinclude -c -MMD -o obj/Scheme.o src/Scheme.cpp
src/Scheme.cpp: In member function ‘void Scheme::remove_figure(int)’:
src/Scheme.cpp:30:21: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     for (; position < size_; position++)
            ~~~~~~~~~^~~~~~~
cc1plus: all warnings being treated as errors
Makefile:17: recipe for target 'obj/Scheme.o' failed
make: *** [obj/Scheme.o] Error 1

В большинстве своём тесты проходят. но в конструкторе Circle вы зря не копируете честно строку, а просто перебрасываете указатели. Это приводит к неправильному поведению.

Стиль:

  • в функции is_inside у прямоугольника уместно ли деление? Ваше счастье, что нет теста, проверяющего граничные значения на п/у нечётной длины и ширины

8,5/10

comment:2 Changed 3 years ago by martynov.maksim

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

comment:3 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

10/10

Note: See TracTickets for help on using tickets.