Непрерывный ListInterator выдает повторные значения, если использовать previous после next и наоборот

@homelleon · Регистрация: 15.12.2012

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

Здравствуйте, делаю игровой движок и при разработке игрового интерфейса возникла необходимость переключаться нажатием клавиш между кнопками. Данные кнопки являются объектами, которые передаются сначала в объект "меню". И при нажатии клавиши "вниз" и "вверх", происходит переход от одной кнопки до другой. Однако, появилась проблема - почему-то когда я использую buttonIterator.previous(), после buttonIterator.Next() выводится одно и то же число. Т.е. для 3-х кнопок я прогоняю сначала метод selectNextButton() - получаю "1 2 3 1 2 3 1 2 и т.д." selectPreviousButton() - "1 3 2 1 3 2 1 ..."
а если использовать сначала один метод, потом другой, получается "1 1 1 1" или "2 2 2 2" или next: " 1 2 3 1 !2!" previous: "!2! 1 3 2 !1!" next: "!1! 2 3". Как сделать, чтобы не повторялись одни и те же элементы?

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
package object.gui.pattern.menu;
 
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.ListIterator;
import java.util.Map;
 
import object.gui.pattern.button.GUIButtonInterface;
import object.gui.pattern.object.GUIObject;
 
/**
 * Represents GUI menu for comfortable access to GUI objects
 * with ability to store buttons.
 *  
 * @author homelleon
 * @see GUIMenuInterface
 *
 */
public class GUIMenu extends GUIObject implements GUIMenuInterface {
    
    private boolean hasButtons = false;
    private Map<String, GUIObject> objects = new HashMap<String, GUIObject>();
    private List<GUIButtonInterface> buttons = new ArrayList<GUIButtonInterface>();
    private ListIterator<GUIButtonInterface> buttonIterator;
    
    @Override
    public void add(GUIObject object) {
        this.objects.put(object.getName(), object);
    }
 
    @Override
    public void addButton(GUIButtonInterface button) {
        buttons.add(button);
        this.hasButtons = true;
        this.updateIterator();
        this.add((GUIObject) button);
    }
    
    @Override
    public GUIObject get(String name) {
        return this.objects.get(name);
    }
    
    @Override
    public void selectNextButton() {
        if(checkIfHassButtons()) {
            this.deselectAllButtons();
            if(this.buttonIterator.hasNext()) {
                this.buttonIterator.next().select();
            } else {
                while(this.buttonIterator.hasPrevious()) {
                    this.buttonIterator.previous();
                }
                this.buttonIterator.next().select();
            }
        }
    }
    
    @Override
    public void selectPreviousButton() {
        if(checkIfHassButtons()) {
            this.deselectAllButtons();
            if(this.buttonIterator.hasPrevious()) {
                this.buttonIterator.previous().select();
            } else {
                while(this.buttonIterator.hasNext()) {
                    this.buttonIterator.next();
                }
                this.buttonIterator.previous().select();
            }
        }
    }
    
    @Override
    public boolean getHasButtons() {
        return this.hasButtons;
    }
 
    @Override
    public void clean() {
        this.objects.clear();
        this.buttonIterator = null;
        this.buttons.clear();
        this.hasButtons = false;
    }   
    
    @Override
    public void show() {
        super.show();
        objects.values().forEach(object -> object.show());
    }
    
    @Override
    public void hide() {
        super.hide();
        objects.values().forEach(object -> object.hide());
    }
    
    private void updateIterator() {
        this.buttonIterator = buttons.listIterator();       
    }
    
    private void deselectAllButtons() {
        this.buttons.forEach(button -> button.deselect());
    }
    
    private boolean checkIfHassButtons() {
        boolean passed = false;
        if(hasButtons) {
            passed = true;
        } else {
            throw new ArrayIndexOutOfBoundsException ("There is no buttons in " + this.getName() + " menu!");
        }
        return passed;
    }
 
 
}

в Main методе:

Java
1
2
3
4
5
6
7
if(EngineMain.getIsEnginePaused()) {
            if(KeyboardGame.isKeyPressed(Keyboard.KEY_UP)) {
                menu.selectNextButton();                
            } else if(KeyboardGame.isKeyPressed(Keyboard.KEY_DOWN)) {
                menu.selectPreviousButton();
            }
        }

@homelleon · 31.07.2017, 23:05 **[ТС]**

Исправил кодом:

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
    private boolean wasNext = false;
    private boolean wasPrevious = false;
 
@Override
    public void selectNextButton() {
        if(checkIfHassButtons()) {
            this.deselectAllButtons();
            if(this.buttonIterator.hasNext()) {
                if(!wasNext) {
                    wasNext = true;
                }
                if(wasPrevious) {
                    this.buttonIterator.next();
                    wasPrevious = false;
                    selectNextButton();
                } else {
                    this.buttonIterator.next().select();
                }
            } else {
                while(this.buttonIterator.hasPrevious()) {
                    this.buttonIterator.previous();
                }
                this.buttonIterator.next().select();
            }
        }
    }
    
    @Override
    public void selectPreviousButton() {
        if(checkIfHassButtons()) {
            this.deselectAllButtons();
            if(this.buttonIterator.hasPrevious()) {
                if(!wasPrevious) {
                    wasPrevious = true;
                }
                if(wasNext) {
                    this.buttonIterator.previous();
                    wasNext = false;
                    selectPreviousButton();
                } else {
                    this.buttonIterator.previous().select();
                }
            } else {
                while(this.buttonIterator.hasNext()) {
                    this.buttonIterator.next();
                }
                this.buttonIterator.previous().select();
            }
        }
    }

Новые блоги и статьи Все статьи Все блоги /
Отображение реквизитов в документе по условию и контроль их заполнения Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .	Фото всей Земли с борта корабля Orion миссии Artemis II kumehtar 04.04.2026 Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .	Вывод диалогового окна перед закрытием, если документ не проведён Maks 04.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .	Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита "ПричинаСписания". . .
wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .	Программное заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .