Opened 3 years ago
Closed 3 years ago
#247 closed ожидаются исправления (задача сдана)
HW #1 (BMP) kozlovceva.mariya hw_01
| Reported by: | Maria Kozlovtseva (kozlovceva.mariya) | Owned by: | Maria Kozlovtseva (kozlovceva.mariya) |
|---|---|---|---|
| Component: | HW #1 (BMP) | Version: | 2.0 |
| Keywords: | Cc: |
Description
Change History (7)
comment:1 Changed 3 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to Maria Kozlovtseva (kozlovceva.mariya) |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 3 years ago by
| Owner: | changed from Maria Kozlovtseva (kozlovceva.mariya) to Дмитрий Лапшин (lapshin) |
|---|
Так, ща переделаем
comment:3 Changed 3 years ago by
| Type: | ожидаются исправления → ожидается проверка |
|---|
comment:4 Changed 3 years ago by
| Version: | 1.0 → 2.0 |
|---|
Исправила.
Мне кажется, что удобно, когда выводится конкретное сообщение об ошибке, а это удобнее и нагляднее, если записано прямо рядом с местом, где может возникнуть ошибка.
+ бонус
comment:5 Changed 3 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to Maria Kozlovtseva (kozlovceva.mariya) |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
Про ошибки: это мудрые вещи! Но хочется иметь и баланс в сторону того, когда именно их сообщают.
Корректность: 20.
Бонус не завёлся. Incorrect message format.
Стиль:
else ifещё и с пустой строчкой напрягает.- Волшебные числа вместо 'x'.
- Ну и switch наверное был бы не лишним.
8/10.
Я боюсь за сегодня надо успеть поправить, если в ведомость надо запихнуть.
comment:6 Changed 3 years ago by
В своем сообщении Вы используете перенос строки? Если да, то вроде в задании сказано, что нельзя использовать этот символ.
comment:7 Changed 3 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Корректность:
10/20.
Стиль:
bmp.cне решал как именно сообщить пользователю об ошибке.main:return 0;.8/10.