Создал змейку, но получился "шиткод", посмотрим что можно сделать?

@HelgeUA · Регистрация: 29.01.2015

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

Вот ссылка на гитхабе
https://github.com/OlegNIKUA/SnakeFirst

Это основной код

@HelgeUA · 13.04.2016, 09:38 **[ТС]**

Думаю что можно вынести в отдельный класс

@Storm23 · 13.04.2016, 10:41

HelgeUA,

1) Класс Input выглядит как костыль. Используйте

Keyboard

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
    /// <summary>
    /// Клавиатура
    /// </summary>
    public static class Keyboard
    {
        [Flags]
        private enum KeyStates
        {
            None = 0,
            Down = 1,
            Toggled = 2
        }
 
        [DllImport("user32.dll", CharSet = CharSet.Auto, ExactSpelling = true)]
        private static extern short GetKeyState(int keyCode);
 
        private static KeyStates GetKeyState(Keys key)
        {
            KeyStates state = KeyStates.None;
 
            short retVal = GetKeyState((int)key);
 
            if ((retVal & 0x8000) == 0x8000)
                state |= KeyStates.Down;
 
            if ((retVal & 1) == 1)
                state |= KeyStates.Toggled;
 
            return state;
        }
 
        public static bool IsKeyDown(Keys key)
        {
            return KeyStates.Down == (GetKeyState(key) & KeyStates.Down);
        }
 
        public static bool IsKeyToggled(Keys key)
        {
            return KeyStates.Toggled == (GetKeyState(key) & KeyStates.Toggled);
        }
    }

И кстати классом Hashtable давно уже никто не пользуется. Используйте Dictionary.

2) StreamWriter myfile = new StreamWriter("records.txt", true); - используйте using для потоков. У вас в случае ошибки записи - выводится сообщение пользователю, но файл остается открытым. Кроме того, нельзя использовать относительные пути для файлов. Нужно всегда использовать абсолютные. И кстати нехорошо хранить данные там же где и exe. Используйте Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData);.

3) Именование методов типа write() - неправильное, нужно с большой буквы. (вернее оно могло бы быть правильным, если бы оно везде было одинаково, но у вас то с большой, то с малой буквы).

4) Нет осмысленных именований контролов. Всех вот этих button1, textBox1 - не должно быть.

5) Класс SnakePart - избыточен, вместо него - обычный Point. Класс DataRecords - костыль, его нужно убрать.

6) Зачем вы везде клонируете изображения Resources.snake_graphics1.Clone ? Это лишнее.

7) Обилие волшебных чисел var recf = new Rectangle(0 * 64, 3 * 64, 64, 64); - нужно заменять на константы.

8) _ty подчеркивания для локальных переменных лично я не люблю, они просто мусорят код. Это дело вкуса конечно, но...

9) Ну и самое главное - архитектура. Ее нет.
Нужна модель данных - модель игрового поля (Field), модель змейки (Snake), модель всей игры в целом (GameState). Там же должны быть все эти методы типа Update, GenerateFoodandBomb, UpdateSnake, Draw, GameOver. В форме всем этим методам делать нечего.

10) Попробуйте сделать функции сохранения и загрузки сохраненной игры. Здесь же и повылазят все баги архитектуры.

11) Сохранение рекордов: во-первых, оно не обновляет рекорд, если имя игрока тоже самое (добавляется просто вторая строчка, а должна обновляться предыдущая). Во-вторых, DGV на форме выглядит как здрасте, некрасиво:

Кликните здесь для просмотра всего текста

Создал змейку, но получился "шиткод", посмотрим что можно сделать?

Попробуйте сделать красивый список со скроллингом в виде кастомного UserControl. Здесь же нет ввода, только отображение, сделать легко.

Новые блоги и статьи Все статьи Все блоги /
Установка Emscripten SDK (emsdk) и CMake на Windows для сборки C и C++ приложений в WebAssembly (Wasm) 8Observer8 30.01.2026 Чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. Система контроля версиями Git. . .	Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .

@HelgeUA 0 / 0 / 0 Регистрация: 29.01.2015 Сообщений: 10
	13.04.2016, 09:38 [ТС]
	Думаю что можно вынести в отдельный класс 0

Создал змейку, но получился "шиткод", посмотрим что можно сделать?

Решение