Почему не заполняется jList

@richel · Регистрация: 17.04.2012

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

При смене Item-а в jComboBox я хочу, чтобы менялся список JList в зависимости от параметра запроса. Код такой:

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
  private void jComboBox1ActionPerformed(java.awt.event.ActionEvent evt) {
        // Add your handling code here:
        String project = jComboBox1.getSelectedItem().toString();
          try{  
    Statement ValuesSt = conn.createStatement();
    if (project != ""){
    ResultSet rsValues = ValuesSt.executeQuery("SELECT  *  FROM Loading_DocSector WHERE Project = '"+project+"'  ORDER BY proj");
 
    while (rsValues.next()) {
        DefaultListModel dlm = (DefaultListModel) jList1.getModel();
 
            dlm.addElement(rsValues.getString("proj"));
 
    }
    jList1.setSelectedIndex(0);
    jList1.setVisibleRowCount(-1);
    rsValues.close();
    ValuesSt.close();
 
    
    }
    }catch(SQLException ex) {
   JOptionPane.showMessageDialog(this, "SQLException: " + ex, "Error", JOptionPane.ERROR_MESSAGE);
     }
    }

Ошибок нет, сам JComboBox выдает переменную "project", а список не заполняется. Что я сделала не так? Подскажите, пожалуйста.

24.04.2012, 19:16

Model vi izmenili no teper zhe nado izmenit list - dat emu izmenennuyu Model

Java
1
JList.setModel(dlm);

Vtoroy variant - ne ispolzovat Model voobshe - zapihnut vse dannie iz ResultSet v Vector i updatnut sushestvuyushiy - jList.setListData(Vector listData) ili sozdat noviy JList

Java
1
JList newJList = new JList(Vector listData);

@eklmn · 25.04.2012, 10:19

Мне кажется, правильнее обновить модель. Т.е.:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
...
while (rsValues.next()) {
DefaultListModel dlm = (DefaultListModel) jList1.getModel();
  dlm.addElement(rsValues.getString("proj"));
}
// ------------------------ add-in: start
jList1.setModel(dlm);
jList1.validate();
// jList1.updateUI(); // раскоментировать, если после добавления модели 
                      // не происходит перерисовки view.
// ------------------------ add-in: end
jList1.setSelectedIndex(0);
...

@richel · 25.04.2012, 12:13 **[ТС]**

Сделала через вектор таким образом:

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
   private void jComboBox1ActionPerformed(java.awt.event.ActionEvent evt) {
        // Add your handling code here:
         project = jComboBox1.getSelectedItem().toString();
        
        System.out.println(project);   
     //  JOptionPane.showMessageDialog(this, project, "", JOptionPane.OK_OPTION); 
          try{  
    Statement ValuesSt = conn.createStatement();
    if (project != ""){
    ResultSet rsValues = ValuesSt.executeQuery("SELECT  *  FROM Loading_DocSector WHERE Project = '"+project+"' ORDER BY proj");
 
    Vector Data = new Vector();
    while (rsValues.next()) {
 
            System.out.println(rsValues.getString("proj")); 
            Data.add(rsValues.getString("proj")); 
 
    }
    jList1.setListData(Data);
    jList1.setSelectedIndex(0);
    rsValues.close();
    ValuesSt.close();
 
    
    }
    }catch(SQLException ex) {
   JOptionPane.showMessageDialog(this, "SQLException: " + ex, "Error", JOptionPane.ERROR_MESSAGE);
     }
    }

Работает. Огромное спасибо за помощь.
P.S. После 6-го Билдера многое не понятно. Если не против, буду спрашивать...

@Dederer · 28.04.2012, 15:22

Код работы с DB не очень аккуратен. Можно получить memory leack. Лучше так:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
Statement ValuesSt = null;
ResultSet rsValues = null;
try {
  ValuesSt = conn.createStatement();
  rsValues = ValuesSt.executeQuery("...");
  // работаем с выборкой
 
} catch (SQLException ex) {
  // отрабатываем исключение
 
} finally {
  if (rsValues != null) { try { rsValues.close(); } catch (Exception ex) { }
  if (ValuesSt != null) { try { ValuesSt.close(); } catch (Exception ex) { }
}

Ну и по мелочи - не зачем брать Statement если project == null. Лучше так:

Java
1
2
3
4
if (project != null && !project.equals("")){
  ValueSt = conn.createStatement();
  // ...
} // if

Вместо сборки в ручную SQL-запроса лучше использовать PrepareStatement и нужные параметры биндить (bind) - этим убережешься от SQL-injection

Ах. Да! Vection в данном случае совершенно не имеет смысл использовать, т.к. он является тредо-безопасным и из-за этого работает медленней обычного ArrayList.

Ну и на последок: тяжелые вещи вроде выборки из базы и прочую тяжелую логику лучше делать в отдельном треде, а не в том-же где произошло событие (ActionEvent) это позволит сделать пользовательский интерфейс (GUI) более отзывчивым, без зависаний.

Смешивать обновление данных GUI (listModel) лучше отдельно от кода работы с БД. Что-то типа этого:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
boolean hasError = false;
Statement st = ...;
try {
  // process and aggregate SQL result
 
} catch (Exception ex) {
 // process Exception user dialog
 hasError = true;
} finally {
 // close all opened resourses;
}
 
if ( ! hasError) {
  // process some GUI logic like update ListModel
}

Удачи!

28.04.2012, 18:53

Дяденька а в API нету конструкторов и методов для JList чтоб работать с ArrayList.Про Вектор мы как то и сами знаем но малость ограничены API так что не обессудьте если чё....

@richel · 29.04.2012, 09:49 **[ТС]**

Спасибо за поправку кода. Сосет дельный. Буду учиться.

@Dederer · 29.04.2012, 10:43

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

Дяденька а в API нету конструкторов и методов для JList чтоб работать с ArrayList.Про Вектор мы как то и сами знаем но малость ограничены API так что не обессудьте если чё....

На этот случай всегда есть интерфейс ListModel ну или переопределить AbstractListModel и сделать всё так как надо. Поверьте, в SWING чаще проще переопределить абстрактный класс или интерфейс и получить большую производительность чем подстраиваться под имеющиеся реализации.

29.04.2012, 19:42

Вы б почитали посты человека.Она только учится а вы хотите чтоб она занималась переопределнием объектов...

@Dederer · 29.04.2012, 21:05

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

Вы б почитали посты человека.Она только учится а вы хотите чтоб она занималась переопределнием объектов...

Я не хочу, просто предлагаю более эффективные варианты решения проблемы. А если писать как получается, лишь бы быстрее то получится фигня. Так и появляются мифы о тормознутом SWING и прочее.

@eklmn · 01.05.2012, 10:11

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

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

Вы б почитали посты человека.Она только учится а вы хотите чтоб она занималась переопределнием объектов...

Я не хочу, просто предлагаю более эффективные варианты решения проблемы. А если писать как получается, лишь бы быстрее то получится фигня. Так и появляются мифы о тормознутом SWING и прочее.

Доброго времени суток!

Dederer, в Вашем профайле я не нашел контактной информации. Пожалуйста, сообщите Ваш e-mail или скиньте пустое письмо на grant.slkuni@gmail.com. Я ищу опытного специалиста в области SWING. Подробности при переписке.

Новые блоги и статьи Все статьи Все блоги /
Программный контроль заполнения реквизита табличной части документа 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, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем

mishgun
	28.04.2012, 18:53
	Дяденька а в API нету конструкторов и методов для JList чтоб работать с ArrayList.Про Вектор мы как то и сами знаем но малость ограничены API так что не обессудьте если чё....

@richel 0 / 0 / 0 Регистрация: 17.04.2012 Сообщений: 28
	29.04.2012, 09:49 [ТС]
	Спасибо за поправку кода. Сосет дельный. Буду учиться. 0

mishgun
	29.04.2012, 19:42
	Вы б почитали посты человека.Она только учится а вы хотите чтоб она занималась переопределнием объектов...