Opened 3 years ago

Closed 3 years ago

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

WW_vector (lab_11) Korkin

Reported by: korkin.igor Owned by: Антон Филатов
Component: WW_vector Version: 1.0
Keywords: Cc:

Description


Change History (3)

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

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

Падает тест на загрузку "плохой" матрицы:
если сделать load корректной матрицы, а потом в тот же регистр попробовать положить матрицу, у которой заявленная размерность больше, чем подано чисел, то регистр не откатывается к старому нормальному состоянию.

Стиль
пробелы в конце строки выпилите
вы в странном месте кидаете bad_alloc - его надо кидать там, где он может выпасть. То есть в конструкторе. В вашем случае рискуете утекать по памяти, если bad_alloc возникнет на какой-нибудь итерации в середине цикла в конструктора.
все printf замените на cout
printf("Bruh.\n"); - выпилите
вам точно нужны и operator+, и оператор +=, и add? Может, что-то лишнее?
не хватает слова noexcept в методах, которые ничего не кидают

6/10

а, это вектор, я сразу не понял. Комментарий сверху касается матрицы. Я сейчас скопирую его в тот тикет

Last edited 3 years ago by Антон Филатов (previous) (diff)

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

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

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

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

Падает тест на копирование (capacity у копии и у исходного вектора не совападают),
а также беда с capacity на reserve.

Стиль:

  • не хватает своп трика в операторе=
  • array_destroyers хотелось бы выразить через pop_back
  • реализацию хотелось бы тоже положить в namespace containers

6.5/10

Note: See TracTickets for help on using tickets.