Поправить игру ЗМЕЙКА

@maksmkv · Регистрация: 29.08.2012

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

Всем привет. Решил переписать код с си++ на c# в рамках обучения - не более того (делаю по данному видео https://itproger.com/course/cpp-snake ).

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
using System;
using System.Security.Cryptography.X509Certificates;
 
namespace ConsoleApp2
{
   
 
    class Program
    {
        static bool gameOver;
        const int width = 20;
        const int height = 20;
        static int x, y, fruitX, fruitY, score;
        public enum eDirection { STOP = 0, LEFT, RIGHT, UP, DOWN };
        static  eDirection dir;
 
        
        static void Main(string[] args)
        {
            Setup();
            while (!gameOver) 
            {
               Draw();
               Input();
               Logic();
            }
        }
        static void Setup()
        {
            Random rand = new Random();
 
            gameOver = false;
            dir = eDirection.STOP;
           
            x = width / 2 - 1;
            y = height / 2 - 1;
            fruitX = rand.Next() % width;
            fruitY = rand.Next() % height;
            score = 0;
        }
 
        static void Draw()
        {
            //System("cls");
            Console.Clear();
            for (int i = 0; i < width + 1; i++)
            {
                Console.Write("#");
            }
            //Console.Write(" ");
 
            for (int i = 0; i < height; i++)
            {
                for (int j = 0; j < width; j++)
                {
                    if (j == 0 || j == width -1) 
                    {
                        Console.Write("#");
                    }
                    else
                    {
                        Console.Write(" ");
                    }
 
                    if (i == y && j == x)
                    {
                        Console.Write("0");
                    }
                    else if (i == fruitY && j == fruitX)
                    {
                        Console.Write("F");
                    }
                    else
                        Console.Write(" ");
 
                }
                Console.WriteLine();
            }
 
            for (int i = 0; i < width + 1; i++)
            {
                Console.Write("#");
            }
           // Console.Write(" ");
        }
 
        static void Input()
        {
            if (Console.KeyAvailable)
            {
                var keyInfo = Console.ReadKey();
                switch (keyInfo.KeyChar)
                {
                    case 'a':
                        dir = eDirection.LEFT;
                        break;
                    case 'd':
                        dir = eDirection.RIGHT;
                        break;
                    case 'w':
                        dir = eDirection.UP;
                        break;
                    case 's':
                        dir = eDirection.DOWN;
                        break;
                    case 'x':
                        gameOver = true;
                        break;
                }
            }
 
        }
 
        static  void Logic()
        {   
            switch (dir)
            {
                case eDirection.LEFT:
                    x--;
                    break;
                case eDirection.RIGHT:
                    x++;
                    break;
                case eDirection.UP:
                    y--;
                    break;
                case eDirection.DOWN:
                    y++;
                    break;
            }
 
        }
    }
}

Код работает, но не так как надо ((( При запуске - поле сдвигается еще на 20 клеток вправо, дико мерцает, нов видео об этом сказано. При наджатии клавиш w,a,s,d - дико быстро перемещается курсор, хотя должен только на 1-у позицию. Что не так? Спасибо

@Enifan · 20.04.2020, 11:47

maksmkv,
1) 89 строку удалить
2) Цикл в Draw() рисует дважды, сперва символ в if-else потом символ if-elseif-else
3) Методы Input и Logic схлопнуть в один
4) Перерисовка консоли Console.Clear затратное занятие, это вызывает мерцание. Как вариант использовать Console.SetCursorPosition. Кода будет больше, но эффективность возрастет.
Это советы не сделают игру полноценную, но многие проблемы решит.

@maksmkv · 20.04.2020, 11:52 **[ТС]**

Спасибо. Сейчас все поправлю. Тут не нужен шедевр, просто знакомлюсь с языком и это оочень тяжело дается.

Закоментил 89 строку и все стало на свои места. Теперь другие строки чиню.

по 2-у - это так объясняется в уроке, что сначала рисуем верхнее поле, потом бока, потом нижнее поле. Пока не могу найти в чем косяк ((

по 3-у тоже пока не очень понял, где схлопнулись - проверил по скобкам вроде все ок. Сможете подсказать?

СПАСИБО

@Enifan · 20.04.2020, 12:15

Сообщение от maksmkv

Тут не нужен шедевр

дело не в шедевре, а в удобстве работы с классами. При правильном подходе и ошибок меньше, и код читается легко и модифицируется легко. В общем только плюсы. Примерные наброски по исправлению вашего варианта

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
using System;
class Program
{
    static bool gameOver;
    const int width = 20;
    const int height = 20;
    static int x, y, fruitX, fruitY, score;
 
    static void Main(string[] args)
    {
        Setup();
        while (!gameOver)
        {
            Draw();
            Input();
        }
    }
    static void Setup()
    {
        Random rand = new Random();
 
        gameOver = false;
 
        x = width / 2 - 1;
        y = height / 2 - 1;
 
        fruitX = rand.Next(1, width);
        fruitY = rand.Next(1, height);
 
        score = 0;
    }
 
    static void Draw()
    {
        Console.Clear();
        for (int i = 0; i < width; i++)
            Console.Write("#");
        Console.WriteLine();
 
        for (int i = 0; i < height; i++)
        {
            for (int j = 0; j < width; j++)
            {
                if (j == 0 || j == width - 1)
                    Console.Write("#");
                else if (i == y && j == x)
                    Console.Write("0");
                else if (i == fruitY && j == fruitX)
                    Console.Write("F");
                else
                    Console.Write(" ");
            }
            Console.WriteLine();
        }
 
        for (int i = 0; i < width; i++)
            Console.Write("#");
    }
 
    static void Input()
    {
        switch (Console.ReadKey().KeyChar)
        {
            case 'a': x--; break;
            case 'd': x++; break;
            case 'w': y--; break;
            case 's': y++; break;
            case 'x': gameOver = true; break;
        }
    }
}

@maksmkv · 20.04.2020, 12:29 **[ТС]**

А я во первых везде скобки фигурные навставлял и смотрю у вас условие изменилось

C#
1
2
3
4
 
if (j == 0 || j == width -1) 
Console.Write("#");
else if (i == y && j == x)

В вашем варианте все заработало. Спасибо большое за помощь. Вот на таких примерах делали бы книги, а то такая ерунда поанписана - сходу вообще не въехать. До этого сидел на DELPHI. Решил что-то по-новее изучить.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

Поправить игру ЗМЕЙКА

Решение

Решение

@Enifan 1848 / 1190 / 501 Регистрация: 14.10.2018 Сообщений: 3,211
	20.04.2020, 11:47
	Сообщение было отмечено maksmkv как решение Решение maksmkv, 1) 89 строку удалить 2) Цикл в Draw() рисует дважды, сперва символ в if-else потом символ if-elseif-else 3) Методы Input и Logic схлопнуть в один 4) Перерисовка консоли Console.Clear затратное занятие, это вызывает мерцание. Как вариант использовать Console.SetCursorPosition. Кода будет больше, но эффективность возрастет. Это советы не сделают игру полноценную, но многие проблемы решит. 1

@maksmkv 3 / 3 / 1 Регистрация: 29.08.2012 Сообщений: 306
	20.04.2020, 11:52 [ТС]
	Спасибо. Сейчас все поправлю. Тут не нужен шедевр, просто знакомлюсь с языком и это оочень тяжело дается. Закоментил 89 строку и все стало на свои места. Теперь другие строки чиню. по 2-у - это так объясняется в уроке, что сначала рисуем верхнее поле, потом бока, потом нижнее поле. Пока не могу найти в чем косяк (( по 3-у тоже пока не очень понял, где схлопнулись - проверил по скобкам вроде все ок. Сможете подсказать? СПАСИБО 0