Changes between Initial Version and Version 1 of Ticket #594, comment 1


Ignore:
Timestamp:
04/08/21 08:46:32 (3 years ago)
Author:
Антон Филатов
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #594, comment 1

    initial v1  
    1 Падает тест на загрузку "плохой" матрицы:
     1~~Падает тест на загрузку "плохой" матрицы:
    22если сделать load корректной матрицы, а потом в тот же регистр попробовать положить матрицу, у которой заявленная размерность больше, чем подано чисел, то регистр не откатывается к старому нормальному состоянию.
    33
    4 Стиль:
    5 * пробелы в конце строки выпилите
    6 * вы в странном месте кидаете bad_alloc - его надо кидать там, где он может выпасть. То есть в конструкторе. В вашем случае рискуете утекать по памяти, если bad_alloc возникнет на какой-нибудь итерации в середине цикла в конструктора.
    7 * все printf замените на cout
    8 * {{{printf("Bruh.\n"); }}} - выпилите
    9 * вам точно нужны и operator+, и оператор +=, и add? Может, что-то лишнее?
    10 * не хватает слова noexcept в методах, которые ничего не кидают
     4~~Стиль~~
     5~~ пробелы в конце строки выпилите ~~
     6~~ вы в странном месте кидаете bad_alloc - его надо кидать там, где он может выпасть. То есть в конструкторе. В вашем случае рискуете утекать по памяти, если bad_alloc возникнет на какой-нибудь итерации в середине цикла в конструктора. ~~
     7~~ все printf замените на cout ~~
     8~~ {{{printf("Bruh.\n"); }}} - выпилите ~~
     9~~ вам точно нужны и operator+, и оператор +=, и add? Может, что-то лишнее? ~~
     10~~ не хватает слова noexcept в методах, которые ничего не кидают ~~
    1111
    12 **6/10**
     12~~**6/10**~~
     13
     14а, это вектор, я сразу не понял. Комментарий сверху касается матрицы. Я сейчас скопирую его в тот тикет