Переход по локациям

@erampa · Регистрация: 11.06.2014

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

Здравствуйте!
Собственно в чем вопрос: сделал имитацию перехода по локациям - создал фрэйм, сделал пару кнопок с названиями локаций, все работает, но если многократно переходить туда сюда, начинаем очень долго выполняться переход.
В связи с чем это связано?

Вот фрагмент кода который делает переход по нажатию на кнопку.

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
    public void perehod(String name, int location_id)
    {
        if(location_id == 1) //Степь
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(true);
            locations[1].setVisible(true);
            
            locations[2].setVisible(true);
            locations[3].setVisible(false);
            
        }
        
        if(location_id == 2) // Водопад
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(false);
            locations[1].setVisible(false);
            
            locations[2].setVisible(true);
            locations[3].setVisible(true);
        }
        
        if(location_id == 3) // Опушка леса
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(true);
            
            locations[1].setVisible(false);
            locations[2].setVisible(false);
            
            locations[3].setVisible(true);
        }
        
        if(location_id == 4) // Хижина
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(false);
            locations[1].setVisible(false);
            locations[2].setVisible(false);
            
            locations[3].setVisible(true);
        }
        
        
        locations[0].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                
                removeAll();
                System.gc();
                perehod(locations[0].getText(), 2);
            }
            
        });
        
        locations[1].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                removeAll();
                System.gc();
                perehod(locations[1].getText(), 3);
            }
        });

Может нехватка памяти или кнопки не перерисовываются?

@Max Dark · 06.07.2015, 14:16

erampa, попробуйте перенести добавление обработчиков из функции perehod в конструктор класса

@erampa · 06.07.2015, 14:20 **[ТС]**

Спасибо,попробую!

Добавлено через 3 минуты
Я не правильно сделал?

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
Location()
    {
        locations[0].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                
                removeAll();
                System.gc();
                perehod(locations[0].getText(), 2);
            }
            
        }); // и т.д.

мой класс :

Java
1
public class Location extends JFrame

Pablito · 06.07.2015, 14:25

Java
1
System.gc();

вытри по всей программе этот бред

@erampa · 06.07.2015, 14:32 **[ТС]**

уже убрал.

Добавлено через 6 минут
Вы мне не поможете?

Pablito · 06.07.2015, 14:41

каждый раз при выполнении метода perehod() куда-то добавляется слушатель, так и задумано?
а что за Location да еще и наследованый от JFrame ?

@erampa · 06.07.2015, 15:18 **[ТС]**

Припустим я в локации А, хочу перейти в локацию B (в то время как локация C - это кнопка видимость которой я поставил false,но она станет видимой при переходе в другую локацию) при нажатии на кнопку "локация B" начинаем слушать мышь,то же самое я должен суметь сделать в каждой локации.
Location это мною созданный класс, почему наследуется от JFrame, где то видел что так делают, и решил унаследовать своему классу все свойства JFrame(я сделал глупость,уже понял).

Добавлено через 31 минуту
Уже третий час ковыряюсь(

Pablito · 06.07.2015, 16:24

а там много классов?
так что бы долго не собирать, а просто запустить и оно скомпилировалось

@erampa · 06.07.2015, 16:42 **[ТС]**

Нет,у меня всего 2 класса из которых 1 - main.
Я то запускаю,оно компилируется, ничего не ругается,но если я буду несколько раз переходить по кнопкам,в итоге начинает тормозить прорисовка нажатия на батоны, вообщем начинает тормозить.

Pablito · 06.07.2015, 16:43

выкладывай сюда оба класса

@erampa · 06.07.2015, 16:58 **[ТС]**

Вот:

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
import javax.swing.*;
 
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
public class Location 
{
    public String name; // имя локации
    public String npc; // npc в локации
    public String monster; // монстры в локации
    public String resourses; // ресурсы что можно добыть в локации
    public int location_id; // id локации, нужно для того,чтоб знать в какой мы локации и в какие перейти можем
    
    public JFrame location_frame = new JFrame(); // окно нашего приложения
    
    //кнопки локаций================
    public JButton [] locations = new JButton[4];
    
    
    public JButton Step = new JButton("Степь");
    public JButton Opyshka_lesa = new JButton("Опушка леса");
    public JButton Hijina = new JButton("Хижина");        
    
    public JLabel loc_status = new JLabel(); // Лэйбл,показывающий в какой локации мы находимся
    public JPanel loc_panel = new JPanel();
    
    public void create_frame()
    {
        
        locations[0] = new JButton("Водопад");
        locations[0].setVisible(false);
        locations[1] = new JButton("Опушка леса");
        locations[1].setVisible(false);
        locations[2] = new JButton("Степь");
        locations[2].setVisible(false);
        locations[3] = new JButton("Хижина");
        locations[3].setVisible(false);
        
        location_frame.setVisible(true);   // главное окно
        location_frame.setSize(new Dimension(600,500));
        location_frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        location_frame.add(loc_panel);
        loc_panel.add(locations[0]);
        loc_panel.add(locations[1]);
        loc_panel.add(locations[2]);
        loc_panel.add(locations[3]);
        
        loc_status.setVisible(false);
        
        loc_panel.setLayout(new BoxLayout(loc_panel, BoxLayout.Y_AXIS)); // расположение элементов в панели (вертикально)
    
    }
    
    public void perehod(String name, int location_id)
    {
        if(location_id == 1) //Степь
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(true);
            locations[1].setVisible(true);
            
            locations[2].setVisible(true);
            locations[3].setVisible(false);
            
        }
        
        if(location_id == 2) // Водопад
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(false);
            locations[1].setVisible(false);
            
            locations[2].setVisible(true);
            locations[3].setVisible(true);
        }
        
        if(location_id == 3) // Опушка леса
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(true);
            
            locations[1].setVisible(false);
            locations[2].setVisible(false);
            
            locations[3].setVisible(true);
        }
        
        if(location_id == 4) // Хижина
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(false);
            locations[1].setVisible(false);
            locations[2].setVisible(false);
            
            locations[3].setVisible(true);
        }
        
        if(location_id == 5)
        {
            location_frame.setTitle(name);
            location_frame.setVisible(true);
            locations[0].setVisible(false);
            locations[1].setVisible(true);
            locations[2].setVisible(false);
            
            locations[3].setVisible(false);
        }
    
        locations[0].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                
                perehod(locations[0].getText(), 2);
            }
            
        });
        
        locations[1].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                perehod(locations[1].getText(), 3);
            }
        });
    
        locations[2].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                perehod(locations[2].getText(), 4);
            }
        });
        
        locations[3].addActionListener(new ActionListener() 
        {
            
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                perehod(locations[3].getText(), 5);
            }
        });
        //========//
        //.КОНЕЦ.//
        //========//
    
    }
}

И main:

Java
1
2
3
4
5
6
7
8
9
10
11
public class Main 
{
 
    public static void main(String[] args)
    {
            Location position = new Location();
            position.create_frame();
            position.perehod("Степь", 1);
    }
 
}

Pablito · 06.07.2015, 17:09

да надо просто строки

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

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
locations[0].addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
 
                perehod(locations[0].getText(), 2);
            }
 
        });
 
        locations[1].addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                perehod(locations[1].getText(), 3);
            }
        });
 
        locations[2].addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                perehod(locations[2].getText(), 4);
            }
        });
 
        locations[3].addActionListener(new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                perehod(locations[3].getText(), 5);
            }
        });

перенести из метода perehod() в метод create_frame()

@erampa · 06.07.2015, 17:15 **[ТС]**

Это проще чем я мог подумать,спасибо Вам огромное!))
Действительно, фрейм обновляется в методе другом..я наверное сидя за Джавой столько времени)
Спасибо еще раз!)

Новые блоги и статьи Все статьи Все блоги /
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	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 Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии 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. . . .

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,793 Записей в блоге: 4
	06.07.2015, 14:16
	erampa, попробуйте перенести добавление обработчиков из функции perehod в конструктор класса 0

@erampa 1 / 1 / 0 Регистрация: 11.06.2014 Сообщений: 43
	06.07.2015, 14:32 [ТС]
	уже убрал. Добавлено через 6 минут Вы мне не поможете? 0

Pablito 2884 / 2296 / 769 Регистрация: 12.05.2014 Сообщений: 7,978
	06.07.2015, 14:41
	каждый раз при выполнении метода perehod() куда-то добавляется слушатель, так и задумано? а что за Location да еще и наследованый от JFrame ? 0

@erampa 1 / 1 / 0 Регистрация: 11.06.2014 Сообщений: 43
	06.07.2015, 15:18 [ТС]
	Припустим я в локации А, хочу перейти в локацию B (в то время как локация C - это кнопка видимость которой я поставил false,но она станет видимой при переходе в другую локацию) при нажатии на кнопку "локация B" начинаем слушать мышь,то же самое я должен суметь сделать в каждой локации. Location это мною созданный класс, почему наследуется от JFrame, где то видел что так делают, и решил унаследовать своему классу все свойства JFrame(я сделал глупость,уже понял). Добавлено через 31 минуту Уже третий час ковыряюсь( 0

Pablito 2884 / 2296 / 769 Регистрация: 12.05.2014 Сообщений: 7,978
	06.07.2015, 16:24
	а там много классов? так что бы долго не собирать, а просто запустить и оно скомпилировалось 0

@erampa 1 / 1 / 0 Регистрация: 11.06.2014 Сообщений: 43
	06.07.2015, 16:42 [ТС]
	Нет,у меня всего 2 класса из которых 1 - main. Я то запускаю,оно компилируется, ничего не ругается,но если я буду несколько раз переходить по кнопкам,в итоге начинает тормозить прорисовка нажатия на батоны, вообщем начинает тормозить. 0

Pablito 2884 / 2296 / 769 Регистрация: 12.05.2014 Сообщений: 7,978
	06.07.2015, 16:43
	выкладывай сюда оба класса 0

@erampa 1 / 1 / 0 Регистрация: 11.06.2014 Сообщений: 43
	06.07.2015, 17:15 [ТС]
	Это проще чем я мог подумать,спасибо Вам огромное!)) Действительно, фрейм обновляется в методе другом..я наверное сидя за Джавой столько времени) Спасибо еще раз!) 0

Переход по локациям

Решение