Написал крестики-нолики. Сделайте ревью кода

@dimaws · Регистрация: 22.09.2014

Студворк — интернет-сервис помощи студентам

Добрый день.
В качестве изучения C# написал крестики-нолики на основе MVC,
но, пока без использования событий и делегатов, а просто обращаясь напрямую к экземпляру Представления, контроллера или модели.
Может ли кто-нибудь отревьюить код?
Прикладываю архив с проектом в VS.
Могу дать ссылку на гитхаб, если Вам это поможет отревьюить код.
Заранее благодарю.

@Anklav · 22.09.2014, 23:29

1. Ужасное форматирование кода (Это важно!!!)
2. Почему сетка это лист в листе? (List<List<Cell>>) очевидно же Cell[,] как минимум. А вообще отдельно класс Grid.
3. Метод checkNeighbor ужасен, я уверен что для этого дела есть алгоритмы.
4. Много не именованных констант в коде.

Отсортировано по важности, это мои личные 5 копеек.

@dimaws · 23.09.2014, 08:25 **[ТС]**

Anklav,
1. Согласен, это важно. Но мне кажется, форматирование нормальное.
Приведи пример из моего кода как не надо, и как надо, чтобы я понял.

2. Ок. Переделаю.

3. Согласен, повторяющиеся проверки if выглядят очень плохо, и надо с этим методом что-то делать, но лучше я не придумал.
Искал алгоритмы перебора пространства Мура второго порядка, но не нашел чего-то компактного и красивого.
Смотрел код других крестиков-ноликов. Везде пишут для поля 3на3 и хардкодят выигрышные комбинации. Я не хочу привязываться к размеру поля, поэтому ищу выигрышные комбинации относительно текущей клетки и если их нет, то игра продолжается.
Я подумаю еще над поиском победителя, но если есть мысли, как это написать красиво, буду рад их услышать.

4. Они есть в checkNeighbor() и одна в Program.cs. Поправлю.

Люди,
помогите красиво реализовать проверку на выигрыш для безразмерного поля.

@Anklav · 23.09.2014, 09:55

Во первых названия методов, полей, то с маленькой то с большой буквы начинается.

Почему не свойство?:

C#
1
2
3
4
5
6
7
8
        public void setState(ECellType value)
        {
            this.state = value;
        }
        public ECellType getState()
        {
            return this.state;
        }

Отступы в методе checkNeighbor вообще страшные.

Ну и в changeCurrentPlayer, мне сразу бросилась в глаза фигурная скобка которая начата не с новой строки, хотя все остальные методы и вообще скобки - с новой.

И по игровой логике замечание, вы никак не обрабатываете что введут 100 100.

И для установки значений то просто поля выставлены наружу, то 2 метода (get, set), а иногда и свойство.

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .

@Anklav 447 / 305 / 47 Регистрация: 23.01.2013 Сообщений: 661
	22.09.2014, 23:29
	1. Ужасное форматирование кода (Это важно!!!) 2. Почему сетка это лист в листе? (List<List<Cell>>) очевидно же Cell[,] как минимум. А вообще отдельно класс Grid. 3. Метод checkNeighbor ужасен, я уверен что для этого дела есть алгоритмы. 4. Много не именованных констант в коде. Отсортировано по важности, это мои личные 5 копеек. 0

@dimaws 0 / 0 / 0 Регистрация: 22.09.2014 Сообщений: 2
	23.09.2014, 08:25 [ТС]
	Anklav, 1. Согласен, это важно. Но мне кажется, форматирование нормальное. Приведи пример из моего кода как не надо, и как надо, чтобы я понял. 2. Ок. Переделаю. 3. Согласен, повторяющиеся проверки if выглядят очень плохо, и надо с этим методом что-то делать, но лучше я не придумал. Искал алгоритмы перебора пространства Мура второго порядка, но не нашел чего-то компактного и красивого. Смотрел код других крестиков-ноликов. Везде пишут для поля 3на3 и хардкодят выигрышные комбинации. Я не хочу привязываться к размеру поля, поэтому ищу выигрышные комбинации относительно текущей клетки и если их нет, то игра продолжается. Я подумаю еще над поиском победителя, но если есть мысли, как это написать красиво, буду рад их услышать. 4. Они есть в checkNeighbor() и одна в Program.cs. Поправлю. Люди, помогите красиво реализовать проверку на выигрыш для безразмерного поля. 0