Problema IF-ELSE

[Jack496]

Salve a tutti, sono alle prime armi con la programmazione con Java, non riesco a venire a capo di questa situazione riesco a collegarmi al database ed ad ottenere i dati che ho bisogno ma quando faccio il confronto con le variabili inserite dall' utente mi si bugga il pg. Il bug consiste che quando il confronto è giusto mi stampa "Login effettuato con successo" come da codice,mentre se sbaglio il confronto non mi dice nulla.Vi ringrazio per l' attenzione. Salve a tutti

 
   if(user.equals(databaseUsername) && subname.equals(databaseCognome) && pw.equals(databasePassword)){
                JOptionPane.showMessageDialog(null, "Login effettuato con successo", "Info", JOptionPane.INFORMATION_MESSAGE);
                new Menu();
                break;
                }else{
                    JOptionPane.showMessageDialog(null, "Non sono stati inseriti i campi in modo corretto", "Info", JOptionPane.INFORMATION_MESSAGE);   }
            }

[francesco.muia]

Ti conviene postare tutto il metodo a occhio vedo che c'è qualche graffa in +, oltretutto controlla che non ti vengano lanciate eccezioni

[Jack496]

Ecco a te

public class Database {    Connection con = null;
    Statement stmt = null;
    ResultSet rs = null;
    PreparedStatement pst = null;
    
    String hostName="locahost";
    String dbPort = "3306";
    String databaseName="lspd";
    String dbUser="root";
    String dbPassword="ciao";
    String jdbc_driver = "com.mysql.jdbc.Driver";
    String user,pw,subname;
    public Database(String nome,String password,String cognome){
        this.user = nome; 
        this.pw = password;
        this.subname=cognome;
        connect(hostName,dbPort,databaseName,dbUser,dbPassword);
         
    }
    
    public void connect(String hostName,String dbPort,String databaseName,String dbUser,String dbPassword){
         
        try {
            Class.forName(jdbc_driver);
            //con = (Connection) DriverManager.getConnection("jdbc:mysql://"+hostName+":"
                   // + dbPort+"/"+databaseName+"?"+"user="+dbUser+"&password=" + dbPassword);
             /*con = DriverManager.getConnection("jdbc:mysql://"+hostName+":"
                    + dbPort+"/"+databaseName, dbUser, dbPassword);*/
             
             String connectionString = "jdbc:mysql://localhost/" + databaseName+ "?user=" + dbUser + "&password=" + dbPassword + "&useUnicode=true&characterEncoding=UTF-8";
            con = DriverManager.getConnection(connectionString);
            
             if(con != null)
            {
                System.out.println("Connesso");
            }
            
             
            /*String query = "SELECT * FROM `account` where nome=? and password=?";
            try{
                pst = con.prepareStatement(query);
                pst.setString(1,user);
                pst.setString(2, pw);
                rs = pst.executeQuery();
                
                if(rs.next())
                {
                    System.out.println("Trovato utente");
                }else{
                    System.out.println("non trovato");
                }
            }catch(Exception e){
                
            }*/
             String databaseUsername="",databasePassword="",databaseCognome="";
             System.out.println("user"+user+"\npw:"+pw+"\n Cog:"+subname);
             Statement stmt = con.createStatement();
            String SQL = "SELECT * FROM account WHERE nome='" + user + "' && cognome='" + subname +"' && password='" + pw + "'";


            ResultSet rs = stmt.executeQuery(SQL);


            // Check Username and Password
            while (rs.next()) {
                databaseUsername = rs.getString("nome");
                databaseCognome = rs.getString("cognome");
                databasePassword = rs.getString("password");
                if(user.equals(databaseUsername) && subname.equals(databaseCognome) && pw.equals(databasePassword)){
                JOptionPane.showMessageDialog(null, "Login effettuato con successo", "Info", JOptionPane.INFORMATION_MESSAGE);
                new Menu();
                break;
                }else{
                    JOptionPane.showMessageDialog(null, "Non sono stati inseriti i campi in modo corretto", "Info", JOptionPane.INFORMATION_MESSAGE);   }
            }
  
            
        } catch (SQLException ex) {
            Logger.getLogger(Database.class.getName()).log(Level.SEVERE, null, ex);
        } catch (ClassNotFoundException ex) {
            Logger.getLogger(Database.class.getName()).log(Level.SEVERE, null, ex);
        }
    }
}

[Alex'87]

Tag CODE...

[francesco.muia]

Bè se i dati sono sbagliati nel while non ci entra perchè giustamente non trova nulla sul db....

[LeleFT]

@Jack496: quando posti del codice, lo devi inserire all'interno dei tag CODE, non dei tag QUOTE che servono a fare citazioni.


Ho corretto io i tuoi post.


Ciao.

[Jack496]

Ma se fosse sbagliato il while non dovrebbe neanche fare l'if giusto?


PS scusate per i post e i tag

[francesco.muia]

Cerco di essere + chiaro...

ResultSet rs = stmt.executeQuery(SQL);

Eseguendo il tuo sql se i dati sono corretti allora il resulset conterrà un riga e quindi rs.next() è true e quindi entra nel while e ovviamente entra anche nell'if.

Se i dati sono sbagliati la query non tirerà fuori nessun risultato e quindi rs.next() sarà false e quindi il corpo del while non viene eseguito.

P.S
L'if dentro al while non serve a nulla infatti. Ti basta controllare se rs.next() è true allora i dati inseriti sono corretti altrimenti sicuramente uno dei dati nome,cognome,password sono sbagliati

[Jack496]

Non è che riusciresti a correggermi il codice? questo aspetto non lo avevo calcolato ... grazie mille.

[francesco.muia]

Qui non si scrive codice per gli altri....
Cmq non vedo quale sia la difficoltà nel scrivere un if al posto di un while....

   if(rs.next())
   {
       //DATI UTENTE TROVATI
   }else
   {
       //DATI UTENTE NON TROVATI
    }