Domanda su MouseListener

[mike1991]

Buongiorno a tutti, io, sto facendo un progetto di un giochino per scuola, sono alle prime armi e ho riscontrato un problema che spero di risolvere con i vostri consigli.
Ho creato una classe contenente un frame con diversi bottoni etc.
Su questo frame devo poter stampare una lista di pannelli, creata in runtime, non statica. Per creare la lista di pannelli ho utilizzato questo metodo nella mia classe

public void CreaLista(){
    pannelli = new ArrayList<JPanel>();
    for (int i = 0; i < 8; i++) {
           pannelli.add(new JPanel());
          pannelli.get(i).setBackground(Color.BLACK);
    }
}

successivamente li ho stampati nel frame tramite un GridLayout. Fino a qui tutto bene, tutto funziona.
Ora però ho bisogno che questi pannelli, durante la pressione del mouse cambino colore e che quando la pressione del mouse termina ritornino del colore originale. Ho visto che tra gli event handler di un Panel è presente MousePressed e MouseRelased. Ho provato a implementare questi due metodi su un pannello solo e sono riuscito a fargli cambiare colore alla pressione

        JPanel panel = new JPanel();
        panel.addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(MouseEvent e) {
                panel.setBackground(Color.RED);
            }
            @Override
            public void mouseReleased(MouseEvent e) {
                panel.setBackground(Color.YELLOW);
            }
        });
        panel.setBounds(107, 52, 80, 71);
        frame.add(panel);

il problema è che non riesco nella maniera più assoluta ad applicare questi due metodi a ogni singolo pannello contenuto nell'ArrayList. Qualcuno sa aiutarmi? Grazie mille in anticipo

[andbin]

        JPanel panel = new JPanel();
        panel.addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(MouseEvent e) {
                panel.setBackground(Color.RED);
            }
            @Override
            public void mouseReleased(MouseEvent e) {
                panel.setBackground(Color.YELLOW);
            }
        });
        panel.setBounds(107, 52, 80, 71);
        frame.add(panel);

il problema è che non riesco nella maniera più assoluta ad applicare questi due metodi a ogni singolo pannello contenuto nell'ArrayList. Qualcuno sa aiutarmi? Grazie mille in anticipo

Se questo codice che ho quotato lo metti (o l'hai già messo) nel for che si vede prima (quel for (int i = 0; i < 8; i++)), non ci sono problemi.
Perché ad ogni ciclo hai un nuovo JPanel e ciascuno ha agganciato la sua implementazione del MouseListener, che fa riferimento al panel relativo poiché quel "panel" nei due metodi si riferisce alla variabile JPanel che hai definito prima del addMouseListener, appunto che è distinta per ciascun ciclo.

E questo tra l'altro vuol dire che stai usando Java 8, perché in questa release le anonymous inner-class possono fare riferimento alle variabili locali del metodo anche se non sono fisicamente marcate final, purché siano "effectively final".


P.S. in generale non postare mai codice "colorato", è un casino quotarlo per intero o peggio parzialmente.

[mike1991]

Se questo codice che ho quotato lo metti (o l'hai già messo) nel for che si vede prima (quel for (int i = 0; i < 8; i++))

ciao avevo già provato a fare cosi e purtroppo non sono riuscito a farlo funzionare perché mi da un errore riguardante qualcosa che tu hai accennato sotto. Facendo cosi

public void AggiungiPannelli(){
    for (int i = 0; i < 8; i++) {
        pannelli.add(new JPanel());
        pannelli.get(i).setBackground(Color.BLACK);
        pannelli.get(i).addMouseListener(new MouseAdapter() {
        @Override
         public void mousePressed(MouseEvent e) {
               pannelli.get(i).setBackground(Color.RED);
         }
        @Override
         public void mouseReleased(MouseEvent e) {
               pannelli.get(i).setBackground(Color.BLACK);
         }
    });
}

dentro nei due metodi mousePressed e Relased quando faccio la get per aggiornare il colore di background mi da errore sulla i dicendo "Local variable i defined in an enclosing scope must be final or effectively
final".
Ho provato a dichiarare la i come final globale ma non cambia nulla. E se metto la i come final all'interno del metodo nel ciclo poi non me la fa più incrementare perché mi dice che la variabile finale i non può essere assegnata. Sono bloccato a questo.
Grazie mille davvero per l'aiuto che mi stai dando.

ps mi scuso per i colori ma pur incollando senza formattazione il codice poi quando lo pubblico i colori rimangono...

[andbin]

ciao avevo già provato a fare cosi e purtroppo non sono riuscito a farlo funzionare perché mi da un errore riguardante qualcosa che tu hai accennato sotto. Facendo cosi

public void AggiungiPannelli(){
    for (int i = 0; i < 8; i++) {
        pannelli.add(new JPanel());
        pannelli.get(i).setBackground(Color.BLACK);
        pannelli.get(i).addMouseListener(new MouseAdapter() {
        @Override
         public void mousePressed(MouseEvent e) {
               pannelli.get(i).setBackground(Color.RED);
         }
        @Override
         public void mouseReleased(MouseEvent e) {
               pannelli.get(i).setBackground(Color.BLACK);
         }
    });
}

No, perché qui il problema è il riferimento alla variabile 'i'.

Più semplicemente (e più pulito):

for (int i = 0; i < 8; i++) {
    JPanel panel = new JPanel();
    // altri setup su panel
    
    panel.addMouseListener(new MouseAdapter() {
        // i metodi qui POSSONO fare riferimento a panel, che è quel JPanel distinto per ciascun ciclo
    });

    // .. altro (aggiungi panel dove vuoi)
}

Da Java 8 panel può non essere final, prima di Java 8 DEVE essere final.

[mike1991]

No, perché qui il problema è il riferimento alla variabile 'i'.

Più semplicemente (e più pulito):

for (int i = 0; i < 8; i++) {
    JPanel panel = new JPanel();
    // altri setup su panel
    
    panel.addMouseListener(new MouseAdapter() {
        // i metodi qui POSSONO fare riferimento a panel, che è quel JPanel distinto per ciascun ciclo
    });

    // .. altro (aggiungi panel dove vuoi)
}

Da Java 8 panel può non essere final, prima di Java 8 DEVE essere final.

e ma facendo cosi creo una serie di pannelli che non sono nell'ArrayList di pannelli. Io ho bisogno che siano in una lista perché una volta creati ho bisogno di poterli spostare etc. Tramite un ArrayList non si può fare?

[mike1991]

No, perché qui il problema è il riferimento alla variabile 'i'.

Più semplicemente (e più pulito):

for (int i = 0; i < 8; i++) {
    JPanel panel = new JPanel();
    // altri setup su panel
    
    panel.addMouseListener(new MouseAdapter() {
        // i metodi qui POSSONO fare riferimento a panel, che è quel JPanel distinto per ciascun ciclo
    });

    // .. altro (aggiungi panel dove vuoi)
}

Da Java 8 panel può non essere final, prima di Java 8 DEVE essere final.

Ho provato cosi

            public void AggiungiPannelli(){
        for (int i = 0; i < 8; i++) {
            JPanel panel = new JPanel();
            panel.setBackground(Color.BLACK);
            panel.addMouseListener(new MouseAdapter() {
@Override
                public void mousePressed(MouseEvent e) {
                    panel.setBackground(Color.RED);
                }
@Override
                public void mouseReleased(MouseEvent e) {
                    panel.setBackground(Color.BLACK);
                }
            });
            pannelli.get(i).add(panel);
        }

in questo modo non mi da righe con errori quando lo avvio, ma poi non parte più non si apre nemmeno il frame e la console mi dice:

java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.ArrayList.rangeCheck(ArrayList.java:653)
at java.util.ArrayList.get(ArrayList.java:429)
at showgame.Playground.AggiungiPannelli(Playground.java:443)
at showgame.Playground.DisegnaPiramideIniziale(Playground.java:421)
at showgame.Playground.initialize(Playground.java:814)
at showgame.Playground.<init>(Playground.java:78)
at showgame.Playground$1.run(Playground.java:68)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:311)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:756)
at java.awt.EventQueue.access$500(EventQueue.java:97)
at java.awt.EventQueue$3.run(EventQueue.java:709)
at java.awt.EventQueue$3.run(EventQueue.java:703)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessI mpl.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:726)
at java.awt.EventDispatchThread.pumpOneEventForFilter s(EventDispatchThread.java:201)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
at java.awt.EventDispatchThread.pumpEventsForHierarch y(EventDispatchThread.java:105)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

[andbin]

            pannelli.get(i).add(panel);

Semplicemente

pannelli.add(panel);


P.S. Per cortesia NON postare codice "colorato"! Copialo prima in Notepad/Notepad++/ecc.. poi da lì lo ricopi e solo dopo lo incolli qui sul forum.

[mike1991]

Semplicemente

pannelli.add(panel);


P.S. Per cortesia NON postare codice "colorato"! Copialo prima in Notepad/Notepad++/ecc.. poi da lì lo ricopi e solo dopo lo incolli qui sul forum.

Hai ragione funziona cosi, che stupido sono stato... grazie mille veramente...