Opened 3 years ago
Closed 3 years ago
#625 closed ожидается проверка (задача сдана)
WW_exceptions lab_12 rachkin.vladimir
| Reported by: | Владимир Рачкин (rachkin.vladimir) | Owned by: | Дмитрий Лапшин (lapshin) |
|---|---|---|---|
| Component: | WW_exceptions | Version: | 3.0 |
| Keywords: | Cc: |
Description
Change History (7)
comment:1 Changed 3 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to Владимир Рачкин (rachkin.vladimir) |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
| Version: | 1.0 |
comment:2 Changed 3 years ago by
Я неправильно понял RAII-класс. Я подумал, что это означает зависимость от файла.
comment:3 Changed 3 years ago by
| Owner: | changed from Владимир Рачкин (rachkin.vladimir) to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | → 2.0 |
comment:4 Changed 3 years ago by
| Component: | WW Matrix → WW_exceptions |
|---|---|
| Summary: | WW_matrix lab_12 rachkin.vladimir → WW_exceptions lab_12 rachkin.vladimir |
comment:5 Changed 3 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to Владимир Рачкин (rachkin.vladimir) |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
Ловить исключения по значению не стоит (особенно учитывая что там виртуальные методы). Формат вывода по пробелам страдает. Сравниваешь знаковые и беззнаковые числа.
Я бы перед передачей размеров в конструктор вектора проверил что они прочитались, а то использование неинициализированного значения и UB.
8.
comment:6 Changed 3 years ago by
| Owner: | changed from Владимир Рачкин (rachkin.vladimir) to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 2.0 → 3.0 |
Note: See
TracTickets for help on using
tickets.
Имхо было бы логичнее ловить исключения один раз, а внутри разбирать команды. А то много дублируется. Тем более ловить по значению... Ну и лучше чтобы ошибка была самодостаточной и сообщала о причине сама.
Зачем матрица хранит имя файла? О_о. ЗАЧЕМ ТЫ ФАЙЛЫ-ТО ПЕРЕЗАПИСЫВАЕШЬ?
эээ мне это не потестить чтобы каждый раз файл восстанавливать.