Не могу понять почему останавливается игра Змейка

@Ruslan_Barinov · Регистрация: 07.02.2018

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

Здравствуйте!
Написал змейку, весьма каряво, но как то написал. Нет только условия, если змейка врежется сама в себя. В общем проблема такая: бегаешь, кушаешь еду, все нормально, но потом, когда змейка равна примерно 20 квадратам игра просто останавливается и все. Игра стоит, а проц начинает постепенно нагружаться. Первым делом у меня пало подозрение на то, что у меня было много изменяемых статических переменных, и я думал, что в связи с этим забивалась память. Переделал все, чтобы не было вообще меняющихся статических переменных. Не помогло. Прошу помочь мне с данной проблемой

К любой критике отношусь положительно, так что не стесняйтесь говорить прямо

Всего два класса:

Java
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
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
 =======================================           КЛАСС 1         ==========================================
 
package snake;
 
import java.awt.BorderLayout;
import java.awt.Canvas;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Toolkit;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.awt.image.BufferStrategy;
import java.util.Random;
import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.JFrame;
 
 
public class Snake extends Canvas implements Runnable{
    Draw draw = new Draw();
    
    public boolean running = true;   // раннинг для цикла while в методе run();
    public boolean food_bool = true;      //   Булевы переменные для условия рандомного распределения еды в методе
    public boolean food_bool_one = true;  //   food_random()
    public boolean food_bool_two = true;
    
    
    public static final int WIDTH = 560;  // размеры фрейма
    public static final int HEIGHT = 560;
   
    boolean move_right = true;  // Булевы перменные на нажатие клавиш
    boolean move_left = false;
    boolean move_up = false;
    boolean move_down = false;
  
    
    
    public void realised_Start_body(){ // начальне координаты тела змеи, которые записываются в список.
        draw.array_x.add(362);         // Змея в старте состоит из 5 квадратов
        draw.array_y.add(362);
        draw.array_x.add(322);
        draw.array_y.add(362);
        draw.array_x.add(282);
        draw.array_y.add(362);
        draw.array_x.add(282);
        draw.array_y.add(322);
        draw.array_x.add(282);
        draw.array_y.add(282);
    }
    
    public void array_body(){                                      // Идея такая:  каждый  следующий элемент листа запоминает
        for(int i = 0; i < draw.list_length; i ++){                // координату предшествующего, и при сьедении еды лист
            if(draw.list_length - i == 1){                         // увеличивается на единицу. По этому листу рисуется тело 
            draw.array_x.set(0, draw.vary_x);                      // змеи
            draw.array_y.set(0, draw.vary_y);
            }
            if(draw.list_length - i != 1){
                if(draw.list_length - i != 0){
            draw.array_x.set(((draw.list_length -1) - i), draw.array_x.get((draw.list_length -1) - i - 1));
            draw.array_y.set(((draw.list_length -1) - i), draw.array_y.get((draw.list_length -1) - i - 1));
            }}}
    }
    
    public void border_field(){ // Границы поля
        if(draw.vary_x > 530){
            draw.vary_x = 2;} 
        if(draw.vary_x < 0){
            draw.vary_x = 522;}
        if(draw.vary_y > 530){
            draw.vary_y = 2;}
        if(draw.vary_y < 0){
            draw.vary_y = 522;}
    }
    
   
     @Override
    public void run() { // --------- RUN()----------------------------------------
        while(running){
           
        render();
        
        try {
            Thread.sleep(100);    // остановка времени на пол секунды, возможно можно и другим способом осуществить это(Через Таймер например и метод)
        } catch (InterruptedException ex) {
            Logger.getLogger(Snake.class.getName()).log(Level.SEVERE, null, ex);
        }
        
        update();
        
        border_field();
        
        eat_food();
        array_body();
        food_random();
        }
    }
    
    public void render(){
        BufferStrategy bs = getBufferStrategy();      // buffer , чтобы создать холст
        
        if(bs == null){
            createBufferStrategy(2);
            requestFocus();
            return;
        }
        
        Graphics g = bs.getDrawGraphics();    //  Полотно для рисовки
        draw.Draw_Body(g);
        
        g.dispose();
        bs.show();
    }
    
    public void food_random(){                                       // Положение еды
           int _for = 1;                                                 // Идея такая: проверяю, не совпадают координаты еды 
           if(food_bool == true){                                    // с координатами тела змеи. Знаю бредово исполнено, но 
           for(int i = 0; i < _for; i ++){                           // как придумал. Пробал через цикл While с использованием 
           draw.food_x = new Random().nextInt(14);         // breack и булевой переменной в условии. Не получилось 
           draw.food_y = new Random().nextInt(14);         // ни так, ни так :(
           food_bool_one = false;
           food_bool_two = false;
           
           for(int j = 0; j < draw.list_length; j ++){
           if(draw.food_x * 40 + 2 == draw.array_x.get(j)){
               food_bool_one = true;
           }
           if(draw.food_y * 40 + 2 == draw.array_y.get(j)){
                  food_bool_two = true;}
           }
           
            if(food_bool_one == true){
                    _for++;
                if(food_bool_one == false){
                if(food_bool_two == true){
                   _for++; 
                }}
            }
            }
           food_bool = false;
           
            
            
           }
    }
    
    public void eat_food(){                                        // Условие, когда змея съела еду
        if(draw.vary_x == draw.food_x * 40 + 2){
            if(draw.vary_y == draw.food_y * 40 + 2){
               food_bool = true;
               draw.list_length++;
               draw.vary_x = draw.food_x * 40 + 2;
               draw.vary_y = draw.food_y * 40 + 2;
               
               draw.array_x.add(draw.array_x.get(draw.array_x.size() - 1));
               draw.array_y.add(draw.array_y.get(draw.array_y.size() - 1));
            }
        }
        
    }
    
    public void update(){       // Реализация движения головы змеи
        
        if(move_right == true){
            draw.vary_x += 40;
        }
        if(move_left == true){
            draw.vary_x -= 40;
        }
        if(move_up == true){
            draw.vary_y -= 40;
        }
        if(move_down == true){
            draw.vary_y += 40;
        }
    }
    
    
    
    public static void main(String[] args) {
        
        Dimension dim = Toolkit.getDefaultToolkit().getScreenSize();  // Тоолкит для размещения фрейма по центру экрана
        int Screen_x =  (int) (dim.getWidth()/2 - WIDTH/2);
        int Screen_y =  (int) (dim.getHeight()/2 - HEIGHT/2);
        
        Snake snake = new Snake();
        
        snake.realised_Start_body();
        
        snake.addKeyListener(new KeyAdapter(){    // Слушатель на клавиши
            public void keyReleased(KeyEvent e) {
                if(e.getKeyCode() == KeyEvent.VK_LEFT){ // Отклики флагов на клавиши
        if(snake.move_right == false){
        snake.move_left = true;
        snake.move_right = snake.move_up = snake.move_down = false;
        }
        }
      if(e.getKeyCode() == KeyEvent.VK_RIGHT){
        if( snake.move_left == false){
         snake.move_right = true;
         snake.move_left =  snake.move_up =  snake.move_down = false;
        }
      }
      if(e.getKeyCode() == KeyEvent.VK_UP){
        if( snake.move_down == false){
         snake.move_up = true;
         snake.move_left =  snake.move_right =  snake.move_down = false;
        }
      }
       if(e.getKeyCode() == KeyEvent.VK_DOWN){
         if(snake.move_up == false){
         snake.move_down = true;
         snake.move_left = snake.move_right =  snake.move_up = false;
         }
       }
            }
        });  // Конец слушателя 
        
        snake.setPreferredSize(new Dimension(WIDTH, HEIGHT));
        
        JFrame frame = new JFrame();
        
        frame.setLocation(Screen_x, Screen_y);
        frame.setLayout(new BorderLayout());
        frame.add(snake, BorderLayout.CENTER);
        
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setResizable(false);
        frame.setVisible(true);
        
        snake.run();
    } 
}

Java
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
=====================================              КЛАСС  2                 ====================================
 
 
 
package snake;
 
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.util.ArrayList;
 
public class Draw {
    
    public  ArrayList<Integer> array_x = new ArrayList<>(); //Лист для координат тела змеи по х
    public  ArrayList<Integer> array_y = new ArrayList<>(); //Лист для координат тела змеи по у
    public int list_length = 5; //длина змеи
  
    public int wh = 40;//размер квадрата поля
    public static int width_height = 37;//размер квадрата змеи
    
    public int vary_x = 362; //начальное положение головы змеи по х
    public int vary_y = 362; //начальное положение головы змеи по у
    
    public  int food_x;      //кратность координаты еды по х
    public  int food_y;      //кратность координаты еды по у
    
    public void Draw_Body(Graphics g){
        Graphics2D g2 = (Graphics2D) g;
                                  
        g2.setColor(Color.WHITE);   
        g2.fillRect(0, 0, Snake.WIDTH, Snake.HEIGHT);  //Прорисовка белого фона
        g2.setColor(Color.BLACK);
        for(int i = 0; i < 14; i ++){
            for(int j = 0; j < 14; j ++){
                  g2.drawRect(0 + wh * i, 0 + wh * j, wh, wh);}  // Прорисовка поля
        }
        
        
        g2.fillRect(vary_x, vary_y, width_height, width_height); // Прорисовка головы змеи
        
        for(int i = 1; i < list_length; i++){ 
            g2.fillRect(array_x.get(i), array_y.get(i), width_height, width_height);// Прорисовка тела змеи
        }
        g2.fillRect(wh*food_x + 2, wh*food_y + 2, width_height, width_height);  //Прорисовка еды
        
    }
}

@xoraxax · 07.02.2018, 19:00

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

не знаю причины

наверно, причина в том, что код не форматируешь

@Ruslan_Barinov · 07.02.2018, 19:08 **[ТС]**

Разве из за этого может такое быть?

@Nidl · 07.02.2018, 20:23

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

@Ruslan_Barinov · 09.02.2018, 17:38 **[ТС]**

Добавил в методе render() g.clearRect(...); Результат тот же :\

@Nidl · 09.02.2018, 21:55

думаю все здесь

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 while(running){
           
        render();
        
        try {
            Thread.sleep(100);    // остановка времени на пол секунды, возможно можно и другим способом осуществить это(Через Таймер например и метод)
        } catch (InterruptedException ex) {
            Logger.getLogger(Snake.class.getName()).log(Level.SEVERE, null, ex);
        }
        
        update();
        
        border_field();
        
        eat_food();
        array_body();
        food_random();
        }

вы постоянно в вашем интервале вызываете конструкторы.

@Ruslan_Barinov · 11.02.2018, 18:16 **[ТС]**

Поменял цикл for на while такой конструкцией и теперь не останавливается. Спасибо за ответы!

Java
1
2
3
4
5
6
7
8
9
10
      
         while(food_bool){
         draw.food_x = new Random().nextInt(14);          
         draw.food_y = new Random().nextInt(14);
         for(int i = 0; i < draw.list_length; i++){
         if((draw.food_x * 40 + 2) != draw.array_x.get(i) & (draw.food_y * 40 + 2) != draw.array_y.get(i)){
             food_bool = false; 
         }
           }
       }

Новые блоги и статьи Все статьи Все блоги /
Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это дополнительная запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:
Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .

@Ruslan_Barinov 0 / 0 / 0 Регистрация: 07.02.2018 Сообщений: 6
	07.02.2018, 19:08 [ТС]
	Разве из за этого может такое быть? 0

@Nidl 296 / 149 / 67 Регистрация: 22.02.2017 Сообщений: 592
	07.02.2018, 20:23
	Имелось ввиду что ваш код не читабелен и что либо понять невозможно, могу предположить что в потоке который вам заменяет таймер вы постоянно рисуете змею ее еду и прочее, это и съедает память 1

@Ruslan_Barinov 0 / 0 / 0 Регистрация: 07.02.2018 Сообщений: 6
	09.02.2018, 17:38 [ТС]
	Добавил в методе render() g.clearRect(...); Результат тот же :\ 0