Strano comportamento di un JavaBean

[Dreamer89]

Salve a tutti, c'è una cosa che non riesco a capire. Allora, ho creato una semplice pagina JSP che dichiara e usa un semplicissimo bean con una proprietà getter che recupera il valore di un elemento di un array ogni volta che la pagina viene aggiornata:

package beans;

public class ImageBean
{
    private static int index = -1;
    private String[] images;
    
    public ImageBean()
    {
        images = new String[3];
        images[0] = "java1.jpg";
        images[1] = "pinkfloyd.jpg";
        images[2] = "acer.jpg";
    }
    
    public String getImage()
    {        
        index++;
        
        if (index > images.length - 1)
            index = 0;
        
        return images[index];
    }
}

Con questo codice funziona perfettamente! Ma perchè se la variabile index invece di dichiararla "private static int" la dichiaro solamente "private int", quando poi aggiorno la pagina l'immagine non viene più aggiornata e rimane la stessa??

[LeleFT]

Perchè c'è una serie di errori di fondo.
Hai detto che dichiari e utilizzi questa classe all'interno di una JSP. E questo è un primo errore. I Beans e tutto ciò che fa parte della Business Logic di una web application andrebbero tenuti separati dalle pagine JSP (che implementano la Presentation Logic).

Poi, dichiarando come "non static" una variabile all'interno della classe, essa viene legata alla singola istanza della classe: ciascuna invocazione della JSP crea una nuova istanza del bean, che, di conseguenza, inizializza a -1 la variabile index.

Al contrario, dichiarandola static, essa non viene legata alla singola istanza del Bean, ma alla classe, quindi tutte le istanze del bean condividono la stessa variabile "index".

Non oso immaginare quali possano essere i problemi più subdoli che si possano creare nell'utilizzare un bean non statico all'interno di una JSP...

Consiglio spassionato: separa la business logic dalla presentation logic. Lascia alle servlet il compito di gestire i Bean, e usa le JSP solo per recuperarne i valori e visualizzarli.


Ciao.

[Dreamer89]

Originariamente inviato da LeleFT
Perchè c'è una serie di errori di fondo.
Hai detto che dichiari e utilizzi questa classe all'interno di una JSP. E questo è un primo errore. I Beans e tutto ciò che fa parte della Business Logic di una web application andrebbero tenuti separati dalle pagine JSP (che implementano la Presentation Logic).

Poi, dichiarando come "non static" una variabile all'interno della classe, essa viene legata alla singola istanza della classe: ciascuna invocazione della JSP crea una nuova istanza del bean, che, di conseguenza, inizializza a -1 la variabile index.

Al contrario, dichiarandola static, essa non viene legata alla singola istanza del Bean, ma alla classe, quindi tutte le istanze del bean condividono la stessa variabile "index".

Non oso immaginare quali possano essere i problemi più subdoli che si possano creare nell'utilizzare un bean non statico all'interno di una JSP...

Consiglio spassionato: separa la business logic dalla presentation logic. Lascia alle servlet il compito di gestire i Bean, e usa le JSP solo per recuperarne i valori e visualizzarli.


Ciao.

Capito, in effetti se dichiaro il bean con scope "session", funziona anche se la variabile non è statica Lo so che è più sano attenersi alle regole del MVC, però sto approfondendo il campo dell'Enterprise Edition da un libro quindi sto seguendo ogni passo, grazie