WW #9

[Святослав Власов]

1. Не прошел тест is_inside для Rectangle
2. Выделять буфер константного размера для надписи -- плохая идея, потому что если надпись короткая ты попоусту тратишь память, а если она больше 100 символов -- твоя программа отправляется в сегфолт.
3. std::map в этом задании нельзя было пользоваться, тем более при удалении фигуры ты пишешь mp_[id] = 0; и не удаляешь элемент из мапы, тем самым создаешь ситуацию схожую с утечкой: если предположить сценарий активного использования класса Scheme с активным добавлением и удалением фигур из неё с разными айдишниками, то твоя мапа будет неограниченно расти.
4. У перегруженных функций не хватает override
5. Не хватает ассертов в критичных местах

Если бы ты прислала работу раньше, ты бы легко эти замечания исправила. Но сейчас дедлайн прошел и больше чем 6/10 я тебе поставить не могу.

Присылайте работы на проверку хотя бы до первого дедлайна!

[Святослав Власов]

Upd: хочу добавить, твоя идея с std::map на самом деле хорошая, ты молодец что про этот контейнер знаешь и заюзала его здесь совершенно правильно с т.з. идеи (если бы не косяк с удалением), но конкретно в этом задании нужно было написать реализацию без него.