Opened 3 years ago

Closed 3 years ago

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

WW_exceptions (lab_12) Korkin

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

Description

Pizza

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

comment:2 Changed 3 years ago by korkin.igor

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

Я если честно не понял, в чем проблема теста на загрузку плохой матрицы. Я попрбовал потестить, и вроде все нормально было. Возможно, оно само починилось, ибо я немного перелопатил код, но это не точно.
P.S.: проверьти вектора пожалуйста.

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

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

Про плохую матрицу вот такой тест падает:

load $0 smoke_2x3
print $0
load $0 truncated_2x3
print $0

при этом

hexdump -C misc/smoke_2x3
00000000  32 20 33 0a 31 20 32 20  33 0a 34 20 35 20 36 0a  |2 3.1 2 3.4 5 6.|
00000010

0a - это символ 10 - перевод строки

hexdump -C misc/truncated_2x3
00000000  32 20 33 0a 31 31 20 31  32 20 31 33 0a 31 34 20  |2 3.11 12 13.14 |
00000010  31 35 0a                                          |15.|
00000013

9/10

Note: See TracTickets for help on using tickets.