Подскажите, пожалуйста, почему в операторе switch-case подчёркивается красным razdelHCode...

@Иван24 · Регистрация: 17.08.2017

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

Здравствуйте. Подскажите, пожалуйста, почему в операторе switch-case подчёркивается красным razdelHCode (константа, имеющая тип int).

Java
1
private final int razdelHCode = razdel.hashCode();

Среда разработки пишет: constant expression required, т. е. требуется константное выражение. Но вроде ещё "константнее"-то некуда...

Java
1
2
3
4
5
6
7
8
9
@Override
    public void itemStateChanged(ItemEvent e) {
        switch (e.getSource().hashCode()) {
            case razdelHCode:
                
                break;
            default:
        }
    }

@Alradalion · 22.09.2018, 22:25

Иван24, потому что razdelHCode у вас не константа.
Посмотрите вот тут, тут есть решения

@Иван24 · 23.09.2018, 13:00 **[ТС]**

Alradalion, при помощи Google-переводчика я прочитал. Получается в case можно подставлять лишь "явные" константы, значение которых видно. Например, 10, "А" и т. п. Мне же нужно однозначно идентифицировать объект класса JComboBox, что возможно насколько я понимаю лишь по hash-коду. При каждом запуске приложения он будет другой, следовательно, конкретное значение актуальное при следующем запуске не отловить. Получается выход состоит в том, чтобы нужному объекту явно ставить в соответствие некоторое значение? Или я чего-то не понял?

@Alradalion · 23.09.2018, 19:44

Иван24, давайте попробуем зайти сдругой стороны. Что вы хотите сделать? Что значит идентифицировать объект класса JComboBox?

@Иван24 · 24.09.2018, 11:07 **[ТС]**

У меня есть несколько комбобоксов, например, JCB1, JCB2, JCB3. Мне нужно точно знать с которым из них произошло событие. Как я понимаю, метод e.getSource() как раз возвращает искомый объект, но case опять затребует константу, когда я в качестве вариантов подставлю конкретные имена комбобоксов, то есть JCB1, JCB2, JCB3. Напоровшись на это я решил опознать конкретный комбобокс по хэш-коду, но снова не вышло. Вот я и не знаю, что надо писать в case, чтобы он не ругался.

Pablito · 24.09.2018, 11:37

ну как вариант, при создании комбо-боксов, назначать им имена

Java
1
comboBox.setName("some name");

и тогда

Java
1
2
3
4
5
6
7
8
9
10
11
12
        comboBox.addItemListener(new ItemListener() {
            @Override
            public void itemStateChanged(ItemEvent e) {
                JComboBox box = (JComboBox) e.getSource();
                switch (box.getName()) {
                    case "some name":
                        // 
                        break;
                }
 
            }
        });

@korvin_ · 24.09.2018, 12:40

Сообщение от Иван24

У меня есть несколько комбобоксов, например, JCB1, JCB2, JCB3. Мне нужно точно знать с которым из них произошло событие. Как я понимаю, метод e.getSource() как раз возвращает искомый объект, но case опять затребует константу, когда я в качестве вариантов подставлю конкретные имена комбобоксов, то есть JCB1, JCB2, JCB3. Напоровшись на это я решил опознать конкретный комбобокс по хэш-коду, но снова не вышло. Вот я и не знаю, что надо писать в case, чтобы он не ругался.

1) Нужно писать не switch-case, а if ... else if ... else
2) Нужно вешать разные обработчики на разные комбобоксы, а не один на все.

@Иван24 · 24.09.2018, 22:45 **[ТС]**

Отчасти Вы правы. Используя if ... else, я на следующий день после создания темы выкрутился, но проблема осталась. Как быть, если вдруг будет много вариантов? Можно написать кучу if ... else, да, но switch - case нагляднее, плюс благодаря ответам на мой вопрос, лучше познакомился с возможностями java.

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

Нужно вешать разные обработчики на разные комбобоксы, а не один на все.

Совет интересный. Обдумаю. Если несколько комбобоксов (пусть три), на каждый свой обработчик. Значит надо создавать вложенные классы, так как в одном классе ведь нельзя вроде создать несколько обработчиков событий, принадлежащих одному классу (интерфейсу), например, ItemListener.

@korvin_ · 25.09.2018, 09:42

Сообщение от Иван24

Значит надо создавать вложенные классы, так как в одном классе ведь нельзя вроде создать несколько обработчиков событий, принадлежащих одному классу (интерфейсу), например, ItemListener.

Чего?

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
import javax.swing.*;
import java.awt.*;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
 
final class Listeners extends JFrame {
 
    public static void main(String[] args) {
        final Listeners window = new Listeners();
        window.pack();
        window.setVisible(true);
    }
 
    private final JTextArea log = new JTextArea();
    private final JComboBox<String> comboBox1 = new JComboBox<>();
    private final JComboBox<String> comboBox2 = new JComboBox<>();
    private final JComboBox<String> comboBox3 = new JComboBox<>();
 
    private Listeners() throws HeadlessException {
        super("Listeners");
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setMinimumSize(new Dimension(800, 600));
        setLayout();
        addItems();
        addListeners();
    }
 
    private void setLayout() {
        final JPanel comboBoxPane = new JPanel(new FlowLayout());
        comboBoxPane.add(comboBox1);
        comboBoxPane.add(comboBox2);
        comboBoxPane.add(comboBox3);
 
        final JScrollPane logPane = new JScrollPane(log);
 
        final JPanel contentPane = new JPanel(new BorderLayout());
        contentPane.add(comboBoxPane, BorderLayout.NORTH);
        contentPane.add(logPane, BorderLayout.CENTER);
 
        setContentPane(contentPane);
    }
 
    private void addItems() {
        comboBox1.addItem("Foo");
        comboBox1.addItem("Bar");
        comboBox1.addItem("Gee");
 
        comboBox2.addItem("RED");
        comboBox2.addItem("GREEN");
        comboBox2.addItem("BLUE");
 
        comboBox3.addItem("Circle");
        comboBox3.addItem("Triangle");
        comboBox3.addItem("Square");
    }
 
    private void addListeners() {
        addListenersByMethodReference();
        addListenersByLambdaExpression();
        addListenersByAnonymousClass();
    }
 
    private void addListenersByMethodReference() {
        comboBox1.addItemListener(this::comboBox1ItemStateChanged);
        comboBox2.addItemListener(this::comboBox2ItemStateChanged);
        comboBox3.addItemListener(this::comboBox3ItemStateChanged);
    }
 
    private void comboBox1ItemStateChanged(ItemEvent event) {
        if (event.getStateChange() == ItemEvent.SELECTED) {
            logItemSelectedMR("combo box 1", event.getItem());
        }
    }
 
    private void comboBox2ItemStateChanged(ItemEvent event) {
        if (event.getStateChange() == ItemEvent.SELECTED) {
            logItemSelectedMR("combo box 2", event.getItem());
        }
    }
 
    private void comboBox3ItemStateChanged(ItemEvent event) {
        if (event.getStateChange() == ItemEvent.SELECTED) {
            logItemSelectedMR("combo box 3", event.getItem());
        }
    }
 
    private void addListenersByLambdaExpression() {
        comboBox1.addItemListener(event -> {
            if (event.getStateChange() == ItemEvent.SELECTED) {
                logItemSelectedLE("combo box 1", event.getItem());
            }
        });
        comboBox2.addItemListener(event -> {
            if (event.getStateChange() == ItemEvent.SELECTED) {
                logItemSelectedLE("combo box 2", event.getItem());
            }
        });
        comboBox3.addItemListener(event -> {
            if (event.getStateChange() == ItemEvent.SELECTED) {
                logItemSelectedLE("combo box 3", event.getItem());
            }
        });
    }
 
    private void addListenersByAnonymousClass() {
        comboBox1.addItemListener(new ItemListener() {
            @Override
            public void itemStateChanged(ItemEvent event) {
                if (event.getStateChange() == ItemEvent.SELECTED) {
                    logItemSelectedAC("combo box 1", event.getItem());
                }
            }
        });
        comboBox2.addItemListener(new ItemListener() {
            @Override
            public void itemStateChanged(ItemEvent event) {
                if (event.getStateChange() == ItemEvent.SELECTED) {
                    logItemSelectedAC("combo box 2", event.getItem());
                }
            }
        });
        comboBox3.addItemListener(new ItemListener() {
            @Override
            public void itemStateChanged(ItemEvent event) {
                if (event.getStateChange() == ItemEvent.SELECTED) {
                    logItemSelectedAC("combo box 3", event.getItem());
                }
            }
        });
    }
 
    private void logItemSelectedMR(String source, Object item) {
        logItemSelected(source, "by method reference: " + item);
    }
 
    private void logItemSelectedLE(String source, Object item) {
        logItemSelected(source, "by lambda expression: " + item);
    }
 
    private void logItemSelectedAC(String source, Object item) {
        logItemSelected(source, "by anonymous class: " + item);
    }
 
    private void logItemSelected(String source, String message) {
        log.append(source + " [SELECTED] " + message + System.lineSeparator());
    }
}

Pablito · 25.09.2018, 10:54

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

Нужно вешать разные обработчики на разные комбобоксы, а не один на все.

это спорно

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

Нужно писать не switch-case, а if ... else if ... else

это тоже, хотя бы потому что читается хуже

@korvin_ · 25.09.2018, 15:10

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

это спорно

В большинстве случаев, в том числе и в этом — нет.

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

это тоже, хотя бы потому что читается хуже

Когда switch/case научится проверять не только константы и сможет полностью заменить if, хотя бы как котлиновский when, например, тогда и соглашусь. А пока что единственный надёжный способ проверять объекты на идентичность — оператор «==» в if-else-if-else-…

Новые блоги и статьи Все статьи Все блоги /
Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определенном условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .	Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .
Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .

@Alradalion 132 / 126 / 85 Регистрация: 13.09.2013 Сообщений: 485
	22.09.2018, 22:25
	Иван24, потому что razdelHCode у вас не константа. Посмотрите вот тут, тут есть решения 1

@Иван24 6 / 6 / 0 Регистрация: 17.08.2017 Сообщений: 141
	23.09.2018, 13:00 [ТС]
	Alradalion, при помощи Google-переводчика я прочитал. Получается в case можно подставлять лишь "явные" константы, значение которых видно. Например, 10, "А" и т. п. Мне же нужно однозначно идентифицировать объект класса JComboBox, что возможно насколько я понимаю лишь по hash-коду. При каждом запуске приложения он будет другой, следовательно, конкретное значение актуальное при следующем запуске не отловить. Получается выход состоит в том, чтобы нужному объекту явно ставить в соответствие некоторое значение? Или я чего-то не понял? 0

@Alradalion 132 / 126 / 85 Регистрация: 13.09.2013 Сообщений: 485
	23.09.2018, 19:44
	Иван24, давайте попробуем зайти сдругой стороны. Что вы хотите сделать? Что значит идентифицировать объект класса JComboBox? 0

@Иван24 6 / 6 / 0 Регистрация: 17.08.2017 Сообщений: 141
	24.09.2018, 11:07 [ТС]
	У меня есть несколько комбобоксов, например, JCB1, JCB2, JCB3. Мне нужно точно знать с которым из них произошло событие. Как я понимаю, метод e.getSource() как раз возвращает искомый объект, но case опять затребует константу, когда я в качестве вариантов подставлю конкретные имена комбобоксов, то есть JCB1, JCB2, JCB3. Напоровшись на это я решил опознать конкретный комбобокс по хэш-коду, но снова не вышло. Вот я и не знаю, что надо писать в case, чтобы он не ругался. 0

Подскажите, пожалуйста, почему в операторе switch-case подчёркивается красным razdelHCode...

Решение