VPF::Проблемы с самописной ListModel - Форум программистов

Hidrag

Дата 5.2.2007, 22:27 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 877
Регистрация: 9.4.2005
Где: JDK

Репутация: 2
Всего: 25

Похожая ситуация...
Добавляю в лист элементы - все работает, удаляю из листа - работает до последнего элемента, точнее и последний элемент уже не существует (проверяю длину ArrayListа который и хранит элементы для отображения), но все равно отображается! и репаинт и валидэйт у диста и скролпэйна вызывал не помогает....

...при добавлении нового элемента в такой ситуации он рисуется на месте "не стертого", последнего как и положено, в чем может быть причина или нужно в какой то особой последовательности выполнять валидэты и репаинты, типа сначала у фрэйма, потом у панели потом у скролпэйна и наконец у листа?

--------------------

LSD

Дата 5.2.2007, 22:57 (ссылка)

(нет голосов)

Загрузка ...

Leprechaun Software Developer

Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 38
Всего: 538

ListModel умолчальная используется или самописная?

--------------------

Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.

Hidrag

Дата 5.2.2007, 23:25 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 877
Регистрация: 9.4.2005
Где: JDK

Репутация: 2
Всего: 25

самописная

Код


public class ContactListModel extends AbstractListModel {
...
private ContactListModel clm = new ContactListModel();
...
private JList list = new JList(clm);
...

Это сообщение отредактировал(а) Hidrag - 5.2.2007, 23:26

--------------------

LSD

Дата 5.2.2007, 23:41 (ссылка)

(нет голосов)

Загрузка ...

Leprechaun Software Developer

Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 38
Всего: 538

Надо проверить, что модель правильно выдает количество элементов в ней, и корректно оповещает всех слушателей.

--------------------

Hidrag

Дата 6.2.2007, 17:20 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 877
Регистрация: 9.4.2005
Где: JDK

Репутация: 2
Всего: 25

да вроде все правильно, поскольку после удаления последнего элемента он остается "нарисованным" в листе но я не могу его выделить мышью.
И потом если после этого удалить лист из контейнера и вернуть назад - этого "глюка"(последнего элемента) не будет.

После удаления и добавления элементов в список вызываю list.validate() и все бы хорошо, пока не удаляю последний элемент...

Добавление и удаление у меня работает так: Я формирую массив элементов и передаю его модели листа в моделе листа его обрабатываю (не все элементы подлежат выводу) вывожу и вконце вызываю метод

Код


fireIntervalAdded("",0,size);

где size - количество элементов для отображения

смысл fireIntervalAdded мне пока не понятен, но без него ничего на листе не обновляется.
Удаление работает анологично - опять же передается массив по которому строится лист. Если в массиве нет ничего для отображения, то и выводиться не должно ничего..

Может еще что нибудь где нибудь вызвать нужно, слушателей в модели не определял, в листе тоже

--------------------

LSD

Дата 6.2.2007, 23:16 (ссылка)

(нет голосов)

Загрузка ...

Leprechaun Software Developer

Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 38
Всего: 538

Выложи код твоей модели листа.

--------------------

Hidrag

Дата 6.2.2007, 23:43 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 877
Регистрация: 9.4.2005
Где: JDK

Репутация: 2
Всего: 25

Ок, вот код.
Немного поясню:
Метод setDataSource получает ArrayList типа Contact, в зависимости от данных в полях каждого Контакта строится строка, которая и добавляется в ArrayList <String> data, элементы которого и выводятся.

Код


public class Contact {
    
    private String nickname;    //ник или название группы
    private boolean onLine;     //статус он лайн/оф лайн
    private int status=-1;         //статус контакта -1 для группы
    private int dopStatus=-1;      //дополнительный статус контакта
    private boolean group;      //показатель группы
    private String groupName;   //имя группы владельца
   ....

А вот и сама модель:

Код


package gui.models;

import javax.swing.AbstractListModel;
import java.util.ArrayList;
import gui.Contact;

public class ContactListModel extends AbstractListModel{
    
    private ArrayList <String> data = new ArrayList <String> ();
    private String path; // путь к jar
    
    public ContactListModel(String s) {
        path=s;
    }
    
    public void setDataSource(ArrayList <Contact> contacts) {
        data.clear();
        int size=0;
        for (Contact c: contacts) {
            if (!c.isGroup()) {
                size++;
                String s="<html><font size=3 ";
                if (c.isOnLine()) {
                    s+="color = blue>";
                    if (c.getStatus()==0) s+="<img src=jar:file:"+path+"!/skin/FFC.gif><html>";
                    if (c.getStatus()==1) s+="<img src=jar:file:"+path+"!/skin/wicked.gif><html>";
                    if (c.getStatus()==2) s+="<img src=jar:file:"+path+"!/skin/depression.gif><html>";
                    if (c.getStatus()==3) s+="<img src=jar:file:"+path+"!/skin/athome.gif><html>";
                    if (c.getStatus()==4) s+="<img src=jar:file:"+path+"!/skin/atwork.gif><html>";
                    if (c.getStatus()==5) s+="<img src=jar:file:"+path+"!/skin/food.gif><html>";
                    if (c.getStatus()==6) s+="<img src=jar:file:"+path+"!/skin/away.gif><html>";
                    if (c.getStatus()==7) s+="<img src=jar:file:"+path+"!/skin/NA.gif><html>";
                    if (c.getStatus()==8) s+="<img src=jar:file:"+path+"!/skin/busy.gif><html>";
                    if (c.getStatus()==9) s+="<img src=jar:file:"+path+"!/skin/DND.gif><html>";
                    if (c.getStatus()==10) s+="<img src=jar:file:"+path+"!/skin/online.gif><html>";
                    if (c.getStatus()==11) s+="<img src=jar:file:"+path+"!/skin/invisible.gif><html>";
                    if (c.getStatus()==12) s+="<img src=jar:file:"+path+"!/skin/fullinvisible.gif><html>";

                    if (c.getDopStatus()==0) s+="<html>";
                    if (c.getDopStatus()==1) s+="<html><img src=jar:file:"+path+"!/skin/custom/thinking.gif><html>";
                    if (c.getDopStatus()==2) s+="<html><img src=jar:file:"+path+"!/skin/custom/business.gif><html>";
                    if (c.getDopStatus()==3) s+="<html><img src=jar:file:"+path+"!/skin/custom/sale.gif><html>";
                    if (c.getDopStatus()==4) s+="<html><img src=jar:file:"+path+"!/skin/custom/print.gif><html>";
                    if (c.getDopStatus()==5) s+="<html><img src=jar:file:"+path+"!/skin/custom/q.gif><html>";
                    if (c.getDopStatus()==6) s+="<html><img src=jar:file:"+path+"!/skin/custom/ugly.gif><html>";
                    if (c.getDopStatus()==7) s+="<html><img src=jar:file:"+path+"!/skin/custom/eat.gif><html>";
                    if (c.getDopStatus()==8) s+="<html><img src=jar:file:"+path+"!/skin/custom/film.gif><html>";
                    if (c.getDopStatus()==9) s+="<html><img src=jar:file:"+path+"!/skin/custom/ill.gif><html>";
                    if (c.getDopStatus()==10) s+="<html><img src=jar:file:"+path+"!/skin/custom/sh.gif><html>";
                    if (c.getDopStatus()==11) s+="<html><img src=jar:file:"+path+"!/skin/custom/utg.gif><html>";
                    if (c.getDopStatus()==12) s+="<html><img src=jar:file:"+path+"!/skin/custom/bath.gif><html>";
                    if (c.getDopStatus()==13) s+="<html><img src=jar:file:"+path+"!/skin/custom/tv.gif><html>";
                    if (c.getDopStatus()==14) s+="<html><img src=jar:file:"+path+"!/skin/custom/lng.gif><html>";
                    if (c.getDopStatus()==15) s+="<html><img src=jar:file:"+path+"!/skin/custom/sleeping.gif><html>";
                    if (c.getDopStatus()==16) s+="<html><img src=jar:file:"+path+"!/skin/custom/heart.gif><html>";
                    if (c.getDopStatus()==17) s+="<html><img src=jar:file:"+path+"!/skin/custom/tired.gif><html>";
                    if (c.getDopStatus()==18) s+="<html><img src=jar:file:"+path+"!/skin/custom/friends.gif><html>";
                    if (c.getDopStatus()==19) s+="<html><img src=jar:file:"+path+"!/skin/custom/sea.gif><html>";
                    if (c.getDopStatus()==20) s+="<html><img src=jar:file:"+path+"!/skin/custom/tetris.gif><html>";
                    if (c.getDopStatus()==21) s+="<html><img src=jar:file:"+path+"!/skin/custom/party.gif><html>";
                    if (c.getDopStatus()==22) s+="<html><img src=jar:file:"+path+"!/skin/custom/coffee.gif><html>";
                    if (c.getDopStatus()==23) s+="<html><img src=jar:file:"+path+"!/skin/custom/games.gif><html>";
                    if (c.getDopStatus()==24) s+="<html><img src=jar:file:"+path+"!/skin/custom/inet.gif><html>";
                    if (c.getDopStatus()==25) s+="<html><img src=jar:file:"+path+"!/skin/custom/zzz.gif><html>";
                    if (c.getDopStatus()==26) s+="<html><img src=jar:file:"+path+"!/skin/custom/beer.gif><html>";
                    if (c.getDopStatus()==27) s+="<html><img src=jar:file:"+path+"!/skin/custom/music.gif><html>";
                    if (c.getDopStatus()==28) s+="<html><img src=jar:file:"+path+"!/skin/custom/learning.gif><html>";
                    if (c.getDopStatus()==29) s+="<html><img src=jar:file:"+path+"!/skin/custom/ingenering.gif><html>";
                    if (c.getDopStatus()==30) s+="<html><img src=jar:file:"+path+"!/skin/custom/sortiere.gif><html>";
                }
                if (!c.isOnLine()) {
                    s+="color = red>";
                    s+="<img src=jar:file:"+path+"!/skin/offline.gif><html>";
                } else if ((c.getStatus()==13)&&(c.isOnLine())) {
                    s+="<img src=jar:file:"+path+"!/skin/online.gif><html>";
                }
                s+=c.getNickName();
                data.add(s);
                //fireIntervalAdded("",0,size);
            }
        }
        fireIntervalAdded("",0,size);
        fireIntervalRemoved("",0,size);
        fireContentsChanged("",0,size);
    }
    
    public int getSize() {
        return data.size();
    }
    
    public void setElementAt(Contact value) {
        data.add(value.getNickName());
    }
    
    public boolean containsElement(Contact value) {
        return data.contains(value.getNickName());
    }
    
    public Object getElementAt(int index) {
        return data.get(index);
    }
    
    public int getElementIndex(Contact value) {
        return data.indexOf(value.getNickName());
    }
    
    public void deleteElement(Contact value) {
        data.remove(value.getNickName());
    }
    
    public void deleteElement(int index) {
        data.remove(index);
    }
    
    public void clear() {
        data.clear();
    }
}

Если будут советы по оптимизации кода или способ написать класс более грамотно но с тем же функционалом - с удовольствием выслушаю.

--------------------

LSD

Дата 7.2.2007, 11:54 (ссылка)

(нет голосов)

Загрузка ...

Leprechaun Software Developer

Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 38
Всего: 538

Вот исправленная модель (единственное, я удалил оттуда гигантские if-ы):

Код

class ContactListModel extends AbstractListModel
{
  private ArrayList<String> data = new ArrayList<String>();
  private String path; // путь к jar

  public ContactListModel(String s)
  {
    path = s;
  }

  public void setDataSource(ArrayList<Contact> contacts)
  {
    int oldSize = data.size();
    data.clear();
    for(Contact c : contacts)
    {
      if(!c.isGroup())
      {
        String s = "<html><font size=3 ";
        if(c.isOnLine())
        {
          //....
        }
        if(!c.isOnLine())
        {
          //...
        }
        else if((c.getStatus() == 13) && (c.isOnLine()))
        {
          //...
        }
        s += c.getNickName();
        data.add(s);
      }
    }
    fireIntervalRemoved(this, 0, oldSize - 1);
    fireIntervalAdded(this, 0, data.size() - 1);
  }

  public int getSize()
  {
    return data.size();
  }

  public void setElementAt(Contact value)
  {
    data.add(value.getNickName());
    fireContentsChanged(this, data.size() - 1, data.size() - 1);
  }

  public boolean containsElement(Contact value)
  {
    return data.contains(value.getNickName());
  }

  public Object getElementAt(int index)
  {
    return data.get(index);
  }

  public int getElementIndex(Contact value)
  {
    return data.indexOf(value.getNickName());
  }

  public void deleteElement(Contact value)
  {
    int index = data.indexOf(value.getNickName());
    if(index != -1)
    {
      data.remove(index);
      fireIntervalRemoved(this, index, index);
    }
  }

  public void deleteElement(int index)
  {
    data.remove(index);
    fireIntervalRemoved(this, index, index);
  }

  public void clear()
  {
    int size = data.size();
    data.clear();
    fireIntervalRemoved(this, 0, size - 1);
  }
}

Теперь замечания по коду:
1. if лучше заменить на case, или хотя бы на else if, потому как сейчас идет проверка всех условий, а не до первого совпадения.
2. Статусы контактов лучше забить константами, а если есть возможность требовать JDK 1.5 и выше, то вообще - Enum. Имена файлов с картинками тоже. Менять только путь к файлам.
3. Не стоит хранить в модели имена контактов, и рендерить HTML. Лучше хранить в модели Contact и написать свой рендерер, который будет его отображать. А иконки подгружать естественно заранее, можно их даже в Enum засунуть.

--------------------

Hidrag

Дата 7.2.2007, 12:13 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 877
Регистрация: 9.4.2005
Где: JDK

Репутация: 2
Всего: 25

LSD, Огромное спасибо!
...буду с рендерером разбираться... smile

--------------------

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Java: GUI и Java FX приложения \| Следующая тема »