errore "Array index out of range: 0"

[cri43]

ciao,
questa catena di metodi mi restituisce l'errore "Array index out of range: 0".

Una servlet riceve da un form username e password e chiama il metodo esisteAccount per vedere se è memorizzato sul database; in questo caso effettua il login:

                 String username=request.getParameter("username");
        String password=request.getParameter("password");


        HttpSession session=request.getSession();
                    
        AccountDAO account=new AccountDAO();
        
Account a=new Account(username,password);
Account b=account.esisteAccount(a);
if(username.equals(b.getNome()) && password.equals(b.getPassword()))
{            
    session.setAttribute("aut", true);
    Cookie cookie= new Cookie("nome",username);
    cookie.setMaxAge(10);    
    response.addCookie(cookie);
    
    response.sendRedirect("riservata.jsp");
    
}else
{
    session.invalidate();


Cookie cookie= new Cookie("nome","");
cookie.setMaxAge(0);
response.addCookie(cookie);
    
    response.sendRedirect("errore.html");
}

Il metodo esisteAccount sta nella classe AccountDAO, ed effettua la query:

public Account esisteAccount(Account account)    {
        String query = "SELECT * FROM Account WHERE nome = "+ account.getNome();
        Vector<Object> v =db.executeSelect(query, "Account");
        account = (Account)v.get(0);
        return account;
        
    }

Il metodo executeSelect sta nella classe DBManager:

public Vector<Object> executeSelect(String sql, String type)
    {
        startConnection();
        Vector<Object> v = new Vector<Object>();
        Statement st;
        try 
        {
            st = conn.createStatement();
        
            ResultSet rs = st.executeQuery(sql);        
            while( rs.next() )
            {
                if ( type.equals("Account") )
                {
                    Account u;
                    u = new Account();
                    u.setNome( rs.getString("Nome") );
                    u.setPassword( rs.getString("Password") );
                    v.add(u);
                }
            }
        } 
        catch (SQLException e) 
        {
            e.printStackTrace();
        }
        
        closeConnection();
        return v;
    }

So che l'errore sta nella servlet nelle righe

Account a=new Account(username,password);
Account b=account.esisteAccount(a);

ma non capisco il perchè: il risultato della query è di tipo Account e lo sto assegnando a un oggetto Account...

[andbin]

chiama il metodo esisteAccount per vedere se è memorizzato sul database

Vector<Object> v =db.executeSelect(query, "Account");

E se non è memorizzato su db .... mi sembra abbastanza comprensibile che il Vector possa essere "vuoto" (0 elementi) e quindi v.get(0) non lo puoi fare.


P.S. Ci sarebbero parecchie altre cose da farti notare dubbie/inappropriate .... ma stasera purtroppo non avrei tempo di approfondire.

[bstefano79]

da un metodo che si chiama esisteAccount mi aspetterei che tornasse un booleano

public boolean esisteAccount(Account account)    {
        String query = "SELECT * FROM Account WHERE nome = "+ account.getNome();
        Vector<Object> v =db.executeSelect(query, "Account");
       
        return v.size()>0;
        
    }

[cri43]

E se non è memorizzato su db .... mi sembra abbastanza comprensibile che il Vector possa essere "vuoto" (0 elementi) e quindi v.get(0) non lo puoi fare.

Non pretenderei tanto, ovviamente la tabella account sul db è popolata

[LeleFT]

Sì, ma quella query con buona probabilità non restituisce alcunchè... almeno per le stringhe, tutti i DBMS pretendono che queste siano inserite tra apici.

Quindi, almeno, l'istruzione SQL dovrebbe essere così:

String query = "SELECT * FROM Account WHERE nome = '" + account.getNome() + "'";

Sempre ammesso che il DBMS non faccia differenza fra maiuscole e minuscole, altrimenti ti devi anche preoccupare di normalizzare o di trovare una corrispondenza esatta.

Senza contare che query secche in quel modo si cerca sempre di evitarle per problemi di conversione fra tipi (caratteri contenenti apici che vanno quotati, date che vanno espresse con formati diversi a seconda del DBMS usato, valori numerici che possono o meno essere racchiusi tra apici, ecc). Si tende ad usare le PreparedStatements, che si occupano da sole della corretta formattazione delle istruzioni SQL.


Ciao.