Opened 4 years ago
Closed 4 years ago
#736 closed ожидается проверка (задача сдана)
HW #2
| Reported by: | Анна Михайлова | Owned by: | Дмитрий Свиридкин |
|---|---|---|---|
| Component: | HW #2 (X0) | Version: | 3.0 |
| Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 4 years ago by
| Version: | 1.0 → 2.0 |
|---|
comment:2 Changed 4 years ago by
comment:3 Changed 4 years ago by
| Owner: | changed from Дмитрий Свиридкин to Анна Михайлова |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
comment:4 Changed 4 years ago by
| Owner: | changed from Анна Михайлова to Дмитрий Свиридкин |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
| Version: | 2.0 → 3.0 |
Исправила, кажется, все, кроме ввода. Ввод уже не могу, и так в прошлый раз с ним измучилась(
comment:5 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
.......O.. ......O... .....O.... ....O..... ........X. ..O..X.X.. .OX...X... O......... ..X.X..... ..........
Ходят нолики, заполняют диагональ -- программа упала (в проверке вторую диагональ забыли)
checkCell(x, y, 1, 0) || checkCell(x, y, -1, 0) поскольку вы перебираете все возможные стартовые точки, проверять в обратном направлении (-1, 0) не нужно.
Аналогично для остальных направлений.
17 + 9 + 0
Note: See
TracTickets for help on using
tickets.
Перепутаны вертикаль и горизонталь.
Неправильно обрабатывается победа при победном ходе в последнюю пустую клетку.
Попробуйте переделать ввод.
Всякие экзотические ходы "5 5-3" обработаются не так, как нужно.
Магические константы размеров стоит сделать именованными
Константы-заполнители стоит объединить в enum и использовать его вместо int.
Программа, запускающая тесты, должна возвращать ненулевой код, если что-то пошло не так. У вас всегда ноль.
В классах стоит сначала перечислять публичные, а затем приватные поля. Пользователям ваших модулей прежде всего нужен интерфейс, а не не детали реализации.
Ровно один тест... "Remind yourself that overconfidence is a slow and insidious killer."
13 + 5 + 0