[JAVA SWING]Metodo per caricare immagine velocemente

[DonDony]

Il mio programma "simula" un banale antifurto ( sensore + tastiera e 1 lucina rossa e una verde ).
Quando passo sopra il sensore con il mouse la lucina rossa si deve accendere, scattare una 50ina di foto e inviarle per email.
Il programma in sè funziona ma la luce rossa non si accende come prima cosa, si accende solo una volta completato l'invio dell'email.
L'immagine della luce la carico dentro una label con il setImageIcon.
Vorrei sapere se c'è un modo per caricare/ricaricare la foto prima di inviare l'email..

[Alex'87]

Senza vedere il codice non si può dire molto... Hai usato i thread per caso?

[DonDony]

private void pnlsensoreMouseEntered(java.awt.event.MouseEvent evt) {                                        
        if(onof==true && tastiera.lblPassword.getText().equals("PASSWORD CORRETTA!")==true && firstTime == true)
        {
            lucine.lblRossoSpento.setIcon(new ImageIcon("."+File.separatorChar+"src"+File.separatorChar+"img"+File.separatorChar+"RossoAcceso.png"));
            firstTime = false;
            Date data = new Date();
            if(j == 0){
                j = 50;
            }
            for(int i = j; i >= 0; i--){
                BufferedImage image = webcam.getImage();
                try {
                    ImageIO.write(image, "PNG", new File("."+File.separatorChar+"allarmi"+File.separatorChar+data.getDate()+"-"+(data.getMonth()+1)+"-"+(data.getYear()-100)+File.separatorChar+"Intruso"+i+".png"));
                    if(i == 0)
                    {
                        setJ();
                        invioMail.send();
                    }
                } catch (IOException ex) {
                    Logger.getLogger(CircuitoC.class.getName()).log(Level.SEVERE, null, ex);
                }
            }
            
        }
    }

Questa è la funzione dove accendo la lucina rossa.
setJ() è per non far sovrascrivere le immagini..

[andbin]

Il programma in sè funziona ma la luce rossa non si accende come prima cosa, si accende solo una volta completato l'invio dell'email.

Stai tenendo "impegnato" il EDT (Event Dispatch Thread). Swing non è thread-safe, tutto quanto (paint, gestione eventi, ecc...) viene fatto nel contesto di un unico thread che è appunto chiamato convenzionalmente EDT. E se lo tieni impegnato tu ......

-> Concurrency in Swing

[DonDony]

Ci darò un'occhiata!

[DonDony]

Grazie Mille!
Ho usato un SwingUtilities.invokeLater e adesso fa quello che deve fare !!
Grazie ancora!

[andbin]

Ho usato un SwingUtilities.invokeLater e adesso fa quello che deve fare !!

Onestamente non ho proprio idea di come/dove hai usato invokeLater ..... il punto è che invokeLater fa eseguire "qualcosa" nel contesto del EDT ma in un momento successivo, appena possibile. Mentre tutto quel tuo ciclo di scritture e invio email andrebbe fatto non nel EDT.

[DonDony]

Alla fine poi, non è che ho capito bene bene il funzionamento di tal cosa, sono andato a tentativi fin quando non mi ha caricato per prima cosa la lucina e poi mandato l'email!

private void pnlsensoreMouseEntered(java.awt.event.MouseEvent evt) {                                        
        if(onof==true && tastiera.lblPassword.getText().equals("PASSWORD CORRETTA!")==true && firstTime == true)
        {
            lucine.lblRossoSpento.setIcon(new ImageIcon("."+File.separatorChar+"src"+File.separatorChar+"img"+File.separatorChar+"RossoAcceso.png"));
            firstTime = false;
            SwingUtilities.invokeLater(new Runnable() {
                @Override
                public void run() {
                    Date data = new Date();
                    for(int i = j; i > j-50; i--){
                        BufferedImage image = webcam.getImage();
                        try {
                            ImageIO.write(image, "PNG", new File("."+File.separatorChar+"allarmi"+File.separatorChar+data.getDate()+"-"+(data.getMonth()+1)+"-"+(data.getYear()-100)+File.separatorChar+"Intruso"+i+".png"));
                        } catch (IOException ex) {
                            Logger.getLogger(CircuitoC.class.getName()).log(Level.SEVERE, null, ex);
                        }
                    }
                    invioMail.send();
                }
            });
        }
    }

Questo è il codice di adesso..

[LeleFT]

@DonDony: quando posti del codice, inseriscilo all'interno degli appositi tag CODE, altrimenti perde formattazione ed indentazione, diventando di difficile lettura.

[CODE]
codice
[/CODE]


In alternativa, entra in "Modalità Avanzata" e usa l'apposito pulsante "#".


Ciao.

[andbin]

Alla fine poi, non è che ho capito bene bene il funzionamento di tal cosa, sono andato a tentativi fin quando non mi ha caricato per prima cosa la lucina e poi mandato l'email!

Adesso l'unica cosa di diverso è che pnlsensoreMouseEntered completa praticamente immediatamente. Imposti la immagine RossoAcceso.png e poi invokeLater va a "parcheggiare" il Runnable nella coda degli eventi (il run() non viene eseguito subito!), quindi pnlsensoreMouseEntered termina subito e il controllo viene restituito al framework, che tra le prime cose che fa sicuramente va ad aggiornare la label della icona a video. Quindi solo in un secondo momento leggermente più avanti, viene eseguito il run() del tuo Runnable. Ma qui sta di fatto che tieni comunque "impegnato" il EDT per un tempo potenzialmente lungo. Ed è questo che, ti ripeto ancora una volta, non andrebbe bene (nulla di grave in sé ma la tua interfaccia utente è "congelata" in questo frangente di tempo).

Per risolvere dovresti usare un thread apposito (con tutte le attenzioni che questo comporta).