[Java+MySql] Estrarre dati da Jlist

[andbin]

Ma se facessi così a quel punto il metodo fillJlistMenu() non mi servirebbe più o dovrei solo modificarlo?

Servirebbe ancora, ovviamente (li devi estrarre questi menù da qualche parte, no). Io però non lo chiamerei fillJlistMenu, perché con JList non ha nulla direttamente a che fare. E non userei Vector ma List (a meno che ci siano ragioni ben valide).

Insomma, qualcosa del tipo:

public List<Menu> findAllMenus()

(Menu, classe specifica per modellare una voce di menù)

perché i (nomi dei) menu inseriti sono univoci

Allora ok!

[Jamie04]

Servirebbe ancora, ovviamente (li devi estrarre questi menù da qualche parte, no). Io però non lo chiamerei fillJlistMenu, perché con JList non ha nulla direttamente a che fare. E non userei Vector ma List (a meno che ci siano ragioni ben valide).

Perché non usare Vector?
Non ho nessuna dimestichezza con le liste :-|
Comunque continuando il discorso coi vettori, se creo un vettore di oggetti menu per riempirlo con i dati presi dal db come devo fare?
Perché se faccio:
temp.addElement(rs.getString("menuName"));
dove temp è un vector di Menu, mi dice che non può convertire una stringa in menu :-|

edit: questo l'ho risolto così:
temp.addElement((Menu)rs.getObject("menuName"));
ri-edit: no, mi da errore a run-time
java.lang.String cannot be cast to Menu

[andbin]

Perché non usare Vector?
Non ho nessuna dimestichezza con le liste :-|

Innanzitutto Vector a partire da Java 1.2 implementa List ... quindi di fatto è una "lista". Comunque Vector è la "vecchia" collezione legacy, che oltretutto è synchronized e non sempre serve una collezione che sia tale perché magari, ad esempio, è sempre acceduta nel contesto dello stesso thread.

Infine se usi Vector (ma anche es. ArrayList) stai codificando verso una implementazione, mentre invece (sempre se non ci sono ragioni ben valide) sarebbe meglio codificare verso una astrazione, ovvero nel caso delle collezioni verso una interfaccia come List.
In questo modo hai più flessibilità nel cambiare in futuro il tipo di collezione che viene realmente istanziato e restituito dal metodo. All'inizio il "findAllMenus" potrebbe creare un ArrayList ma se in futuro volessi passare a LinkedList puoi farlo, senza che il resto all'esterno debba cambiare. Se volessi un giorno restituire un List "immutabile" puoi farlo, grazie a Collections.unmodifiableList.
Insomma, hai più possibilità di cambiare .... mentre se usi Vector hai abbastanza le mani legate o comunque diventa più complicato poter cambiare "idea".

Comunque continuando il discorso coi vettori, se creo un vettore di oggetti menu per riempirlo con i dati presi dal db come devo fare?
Perché se faccio:
temp.addElement(rs.getString("menuName"));

dove temp è un vector di Menu, mi dice che non può convertire una stringa in menu :-|

Ovviamente così no.

Invece es.:

Menu menu = new Menu();
menu.setName(rs.getString("menuName"));
menu.setXXXX(rs.getYYYY("xxxxxx"));
.....
lista.add(menu);

[Jamie04]

All'inizio il "findAllMenus" potrebbe creare un ArrayList ma se in futuro volessi passare a LinkedList puoi farlo, senza che il resto all'esterno debba cambiare. Se volessi un giorno restituire un List "immutabile" puoi farlo, grazie a Collections.unmodifiableList.
Insomma, hai più possibilità di cambiare .... mentre se usi Vector hai abbastanza le mani legate o comunque diventa più complicato poter cambiare "idea".

Sì avevo letto che Vector è obsoleta, però guarda, in questo corso non ci hanno mai parlato delle liste, per cui non è nemmeno supposto che io le debba usare. Vector per i miei scopi va bene.

Invece es.:

Menu menu = new Menu();
menu.setName(rs.getString("menuName"));
menu.setXXXX(rs.getYYYY("xxxxxx"));
.....
lista.add(menu);

Ok, supponendo che continuo a usare il vettore (quindi sarà vettore.add(menu)), come posso risolvere il fatto che l'id del menu non deve venir mostrato nella jlist? Se evito di aggiungerlo al vettore poi però non posso recuperarlo in seguito senza fare tutta la manfrina che ho postato all'inizio, o mi sbaglio?
Grazie infinite.


Edit: ora però ho un altro problema. Nella jlist adesso mi mostra sempre lo stesso menu, l'ultimo inserito, ripetutamente (per un numero di volte pari alla dimensione del vector). Questo era il codice che riempie la jlist:

menuList = new javax.swing.JList(model);
for (int i=0; i<menuVoices.size(); i++) {
model.addElement(menuVoices.get(i));
}

dove menuVoices è il vettore dei menu creato col metodo fillJlistMenu di cui sopra.

ri-edit: ho risolto, perché l'istruzione Menu menu=new Menu(); doveva stare all'interno del ciclo while, io l'avevo messa fuori e così praticamente lavorava sempre sullo stesso oggetto. E, tra l'altro, ho notato che il menuId non viene comunque mostrato nella jlist.
Il costruttore di menu non l'ho creato. Dovrei farlo secondo voi? Non saprei come inizializzare l'oggetto menu.

[andbin]

per cui non è nemmeno supposto che io le debba usare.

Vabbè ... ok.

Ok, supponendo che continuo a usare il vettore (quindi sarà vettore.add(menu)), come posso risolvere il fatto che l'id del menu non deve venir mostrato nella jlist?

Il renderer predefinito di JList invoca toString() su ogni elemento del list model. Quindi in Menu basta definire un toString() che restituisce il "menuName". Stop, tutto qui.

Se evito di aggiungerlo al vettore poi però non posso recuperarlo in seguito senza fare tutta la manfrina che ho postato all'inizio, o mi sbaglio?

Non ho capito ... quale è il dubbio? Avrai oggetti di tipo Menu ... il list model conterrà oggetti Menu e puoi prenderli dal list model in qualunque momento (es. l'elemento "selezionato"). Se non usi DefaultListModel parametrizzato con i generics da Java 7, esso fornisce Object ma tu "sai" che sono oggetti Menu. Quindi basta un cast e poi es. getId().

Nella jlist adesso mi mostra sempre lo stesso menu (l'ultimo inserito credo).

Il new Menu(); è da fare chiaramente per ogni record. Non devi istanziarlo es. una volta sola prima del classico ciclo while (rs.next()) !

menuList = new javax.swing.JList(model);
for (int i=0; i<menuVoices.size(); i++) {
model.addElement(menuVoices.get(i));
}

Questo è ok ma ovviamente è da un'altra parte (dove crei la GUI, immagino).

[Jamie04]

Non ho capito ... quale è il dubbio? Avrai oggetti di tipo Menu ... il list model conterrà oggetti Menu e puoi prenderli dal list model in qualunque momento (es. l'elemento "selezionato"). Se non usi DefaultListModel parametrizzato con i generics da Java 7, esso fornisce Object ma tu "sai" che sono oggetti Menu. Quindi basta un cast e poi es. getId().

Scusami, ho editato così tante volte il messaggio che ti ho fatto perdere tempo a rispondere a cose già risolte (cioè il problema dei dati duplicati nella jlist e dell'assenza del menuId)!
Adesso devo gestire le selezioni sulla lista, così p come era prima più o meno:

    private void menuListValueChanged(javax.swing.event.ListSelectionEvent evt) {                                      
        //model.addListDataListener(null);
        int index = menuList.getSelectedIndex();
        int size = model.getSize();
        if (index >= 0) { 
            bDeleteMenu.setEnabled(true);
        } else { 
            bDeleteMenu.setEnabled(false);
            }
        String field = (String)menuList.getSelectedValue();
        menuName.setText(field);
        //int menuId=DBConnection.getMenuId(field);
        //System.out.println(menuId);
    }

Ma ovviamente ora mi da errore a runtime di casting sulla riga String field = menuList.getSelectedValue();
Il mio listModel contiene il vettore di oggetti menu.

[andbin]

String field = (String)menuList.getSelectedValue();

Menu selectedMenu = (Menu) menuList.getSelectedValue();