[JAVA] JList, int e selezione multipla

[Sergei]

Salve ragazzi,
siccome sto male fisicamente per via della gola e della febbre, stavo pensando di impazzire completamente anche mentalmente. Allora stavo sistemando il mio editor in Java e ho un piccolo problema nella schermata del cambio di carattere. Questa è la finestra:


Il problema è con i numeri. A quanto ho capito la JList gestisce solo String. Quindi ho creato le due liste con due array. Quella per gli stili e quella per la grandezza del font:
String[] fonts = { "Arial", "Mangal", "Courier", "Comic Sans MS", "Script" };
String grandezza[] = { "8", "9", "10", "11", "12", "14", "16", "20" };

Ora il problema è che volevo fare in modo che selezionando lo stile e i punti in Esempio si vedesse l'anteprima. A dire il vero l'ho fatto e funziona pure, ma funziona BENE solo se seleziono prima la grandezza e poi lo stile, perché altrimenti inizio ad ottenere java.lang.NumberFormatException che dovrebbe essere generato dal fatto che per rendere l'array grandezza un int ho usato Integer.parseInt(x)
Veramente a funzionare, funziona anche, nel senso che se seleziono prima il tipo di carattere nell'esempio cambia solo che il carattere non viene evidenziato e nella console ho quell'errore. Se invece clicco sui punti e poi sul tipo di carattere funziona tutto bene, anche se dopo provo altri tipi di caratteri. Per fare questo ho usato un IF innestato (si dice così?)

if (e.getValueIsAdjusting() == true) {
			String ant = (String) tipoCaratteri.getSelectedValue();
			font = new Font(ant, Font.BOLD, 12);
			anteprima.setFont(font);

			if (e.getValueIsAdjusting() == true) {
				String n1 = (String) punti.getSelectedValue();
				n2 = Integer.parseInt(n1);

				font = new Font(ant, Font.BOLD, n2);
				anteprima.setFont(font);
			}
		}

Esattamente dov'è che ho sbagliato? Chiedo scusa per l'ignoranza

[ally]

n2 = Integer.parseInt(n1);
font = new Font(ant, Font.BOLD, n2);
anteprima.setFont(font);

...sospetto che se il parse non va a buon fine un eccezione viene scatenata e ti salta le righe di codice successive...annega il tutto su un blocco try/catch ed assegna un valore di default ad n2 se il parse da problemi...

[PGI-Bis]

La condizione è "getValueIsAdjusting() == false" e non "true".

Il valore di quel metodo denota se l'utente stia ancora interagendo con la lista per determinare la selezione oppure no. A te interessa gestire la situazione quando l'utente ha completato la selezione e non mentre è ancora in corso, da cui il false.

Parte 2. Se la lista punti contiene solo le stringhe dell'array "grandezza" allora l'unico caso logicamente possibile in cui la conversione di getSelectedValue risulti in una NumberFormatException è quello in cui il valore restituito da getSelectedValue() sia null.

Dunque ci va un bell'if: if(n1 != null) {parseint e triccheballacche } else { vedi tu }.

Parte 3. Nella JList puoi mettere riferimenti del tipo che preferisci, non solo stringhe. Probabilmente dovresti usare degli Integer:

Integer[] grandezze = {8, 9, 10, 11, 12, 13, 18, 24}

o dei BigDecimal se vuoi usare frazioni di punto.

Il "limite delle stringhe" in realtà è una conversione che il decoratore standard dei contenuti di una JList applica ai valori della lista: invoca toString sul valore (se il valore è diverso da null) per determinare cosa scrivere nella posizione della lista occupata da quell'elemento. Se metti dei numeri va bene lo stesso, quando usi dei componenti tipo un contatto o un libro - dati aggregati insomma - allora a volte conviene sostituirlo. Ma è solo una questione di disegno, non c'entra con ciò che una JList (il suo modello) può contenere.

[Sergei]

provo e ti faccio sapere
grazie mille!

EDIT:
allora... mi sa che mi sto incasinando tutto
è vero, bastava fare un Integer nomelista[] = {1,2,3}; e la jlist andava bene.
Allora ho fatto così, creando un array chiamato points che poi passo come parametro alla jlist.
La cosa bella è che ora quando clicco la selezione si illumina, il carattere viene cambiato, solo se guardo in console:
java.lang.NullPointerException che fa riferimento a:
n2 = (Integer) punti.getSelectedValue();

[PGI-Bis]

Come diceva Turing "ghe 'nqueicos che va no", c'è qualcosa che non va.

Nella linea che incolli l'unica parte che può generare una NullPointerException è l'accesso al metodo (o dereferenziazione, il punto insomma).

Cioè anche se il valore selezionato fosse "null", la conversione a Integer avrebbe successo:

x = (Integer) null;

è valido. E rimane solo "punti". Sicuro che la linea sia proprio quella?

[Sergei]

Originariamente inviato da PGI-Bis

Come diceva Turing "ghe 'nqueicos che va no",

chiaro accento londinese, giusto?

purtropp punta proprio a quella riga
questo è report completo della console:

java.lang.NullPointerException
	at TipoCaratteri.valueChanged(TipoCaratteri.java:126)
	at javax.swing.JList.fireSelectionValueChanged(Unknown Source)
	at javax.swing.JList$ListSelectionHandler.valueChanged(Unknown Source)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(Unknown Source)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(Unknown Source)
	at javax.swing.DefaultListSelectionModel.setValueIsAdjusting(Unknown Source)
	at javax.swing.JList.setValueIsAdjusting(Unknown Source)
	at javax.swing.plaf.basic.BasicListUI$Handler.mouseReleased(Unknown Source)
	at java.awt.AWTEventMulticaster.mouseReleased(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$000(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

dove TipoCaratteri.java:126 è: n2 = (Integer) punti.getSelectedValue();
Ho provato anche a impostare un valore a n2 ma niente, stesso risultato.
ma non è che il problema sia il ciclo IF? Nel senso, per fare una selezione multipla da JList si usa il ciclo così?

Perché ho come l'impresione che cliccando su un tipo di carattere JAVA pensi che automaticamente abbia premuto anche una grandezza e, visto che non l'ho fatto, genera quell'eccezione.

[PGI-Bis]

Eppure un intoppo nel codice ci dev'essere. Ho fatto un rapido esperimento e le liste funzionano come dovrebbero:

package javaapplication76;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.GridLayout;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextField;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;

public class FontChooserPanel extends JPanel {

    private static class FontStyle {
        final int code;
        final String name;

        public FontStyle(int code, String name) {
            this.code = code;
            this.name = name;
        }

        @Override
        public String toString() { return name; }
    }

    /**
     * Instance initializer
     */
    public FontChooserPanel() {
        super(new BorderLayout());
        final JPanel mainPanel = new JPanel(new BorderLayout());
        final JPanel listContainer = new JPanel(new GridLayout(1, 3, 10, 10));
        final JPanel familyPanel = new JPanel(new BorderLayout());
        final JPanel stylePanel = new JPanel(new BorderLayout());
        final JPanel sizePanel = new JPanel(new BorderLayout());
        final JList familyList = new JList(new String[] {"Arial", "Courier New", "Times", "Monospace"});
        final FontStyle[] fontStyles = {
            new FontStyle(Font.PLAIN, "Plain"),
            new FontStyle(Font.ITALIC, "Italic"),
            new FontStyle(Font.BOLD, "Bold"),
            new FontStyle(Font.ITALIC | Font.BOLD, "Italic-Bold")
        };
        final JList styleList = new JList(fontStyles);
        final JList sizeList = new JList(new Integer[] {7, 8, 9, 10, 11, 12, 14, 16, 18, 20, 24, 28, 40});
        final JTextField familyField = new JTextField();
        final JTextField styleField = new JTextField();
        final JTextField sizeField = new JTextField();
        final JLabel previewLabel = new JLabel("Hello World");
        final ListSelectionListener selectionListener = new ListSelectionListener() {

            @Override
            public void valueChanged(ListSelectionEvent e) {
                if(!e.getValueIsAdjusting()) {
                    FontChooserPanel.this.updateChosenFont(familyList, styleList, sizeList, familyField, styleField, sizeField, previewLabel);
                }
            }
        };
        familyList.addListSelectionListener(selectionListener);
        styleList.addListSelectionListener(selectionListener);
        sizeList.addListSelectionListener(selectionListener);
        familyPanel.add(familyField, BorderLayout.NORTH);
        familyPanel.add(new JScrollPane(familyList));
        stylePanel.add(styleField, BorderLayout.NORTH);
        stylePanel.add(new JScrollPane(styleList));
        sizePanel.add(sizeField, BorderLayout.NORTH);
        sizePanel.add(new JScrollPane(sizeList));
        previewLabel.setPreferredSize(new Dimension(300, 75));
        previewLabel.setHorizontalAlignment(JLabel.CENTER);
        listContainer.add(familyPanel);
        listContainer.add(stylePanel);
        listContainer.add(sizePanel);
        mainPanel.add(listContainer);
        mainPanel.add(previewLabel, BorderLayout.SOUTH);
        add(mainPanel);
    }

    private void updateChosenFont(JList familyList, JList styleList, JList sizeList, JTextField familyField, JTextField styleField, JTextField sizeField, JLabel previewLabel) {
        final String familyName = (String) familyList.getSelectedValue();
        final Integer size = (Integer) sizeList.getSelectedValue();
        final FontStyle style = (FontStyle) styleList.getSelectedValue();
        if(familyName != null && size != null && style != null) {
            final Font font = new Font(familyName, style.code, size);
            previewLabel.setFont(font);
        }
        familyField.setText(familyName == null ? "" : familyName);
        sizeField.setText(size == null ? "" : size.toString());
        styleField.setText(style == null ? "" : style.toString());
    }
}

Ti confermo al 100% che nella riga:

n2 = (Integer) punti.getSelectedValue();

l'unica NullPointerException che può saltar fuori è se "punti" è null.

[Sergei]

Un mistero, insomma

ma c'è un altro modo di utilizzare più jlist selezionate oltre a if?

EDIT: è proprio null:

if (e.getValueIsAdjusting() == false) {
				if (punti.getSelectedValue() != null){
				n2 = (Integer) punti.getSelectedValue();
				font = new Font(ant, Font.BOLD, n2);
				anteprima.setFont(font);
				} else
					System.out.println("E' null");
			}

In compenso non mi devo più sorbire 20000 righe rosse

[PGI-Bis]

Secondo me è sbagliata l'indicazione della stack trace. Perchè un NullPointer nel caso in cui n2 sia null (perchè getSelectedValue() restituisce null) ce l'avresti alla riga dopo:

new Font(ant, Font.BOLD, n2)

java jvm cerca di fare l'unboxing di n2 da Integer a int prima di invocare il costruttore e PAF, null pointer exception. Capita - raramente in verità - che la linea di codice indicata dalla traccia dell'eccezione manchi il bersaglio di qualche riga (credo dipenda dalla codifica delle interruzioni di linea).

Non mi viene in mente un modo diverso per usare le JList. Nota però che in un componente come quello che stai creando, che ha un testo in anteprima, dovrebbe sempre esserci un valore selezionato, anche nel caso in cui la finestra di dialogo sia mostrata senza una scelta precedente perchè l'utente vede una stringa in anteprima e deduce che il formato rappresentato dalla selezione attuale nelle liste di opzioni corrisponda a quel formato. Se presenti l'anteprima senza che ci sia una combinazione di valori selezionati "l'output" dell'interfaccia utente diventa ambiguo.

Fai in modo che le liste abbiano come valore iniziale selezionato la combinazione che rappresenta il font usato dall'etichetta di anteprima e sei a posto.

[Sergei]

giusto, hai perfettamente ragione!

Dopo imposto

Grazie mille!

EDIT: provato, funziona alla grande. Grazie mille!