在java中嵌套if語句

Question

我正在處理用戶登錄頁面。我希望管理員有權訪問管理員部分，但不是常規用戶。以下代碼的工作原理除了如果除管理員以外的用戶登錄，他們會得到錯誤的密碼錯誤併發送到正確的頁面。如果允許否定答案和正答案，我認爲嵌套有問題。我非常感謝在這個問題上的任何幫助。

 try { 
     String sql="Select * from Users1"; 
     String host ="jdbc:derby://localhost:1527/Nash"; 
     String uName = "CON"; 
     String uPass = "smokes"; 
     Connection con = DriverManager.getConnection(host, uName, uPass); 
     Statement stmt=con.createStatement(); 
     ResultSet rs = stmt.executeQuery(sql); 
     String user= userName.getText(); 
     String pwd= password.getText(); 
     while(rs.next()) { 
     String uname=rs.getString("User_name"); 
     String pass=rs.getString("Password"); 
     String admin1=rs.getString("admin"); 

if ((user.equals(uname)) && (pwd.equals(pass))) 
    { 
    mainPanel.setVisible(true); 
    blankContent.setVisible(true); 
    adminButton.setEnabled(false); 
    receiptContent.setVisible(false); 
    memberContent.setVisible(false); 
    securityPanel.setVisible(false); 
    } 

     if ((user.equals(uname)) && (pwd.equals(pass))&&(admin1.equals("y"))) 
     { 
      mainPanel.setVisible(true); 
      blankContent.setVisible(true); 
      adminButton.setEnabled(true); 
      receiptContent.setVisible(false); 
      memberContent.setVisible(false); 
      securityPanel.setVisible(false); 
     } 
else 
    { 
      JOptionPane.showMessageDialog(null, "User name and password do" 
        + " not match!","ALERT!", JOptionPane.ERROR_MESSAGE); 

      } 
} } catch (SQLException ex) { 
     Logger.getLogger(program.class.getName()).log(Level.SEVERE, null, ex); 
    } 

編輯 以下是編輯的代碼鏈接---> New revised code link

Answer 1

除非你缺少一個小故障導致此問題(From your pastebin code in your comment)你正在做的一切是正確的。

你的while語句在這裏是個大問題。它將繼續檢查數據庫中的所有記錄。

使用break一旦你得到了所需的用戶。

boolean isPass = false; 
while(rs.next()){ 
    // Your user authentication 
    if ((user.equals(uname)) && (pwd.equals(pass))) 
    { 
     if (admin1.equals("y")) { 
      // do your admin operation 
      break; 
     } else { 
      // do your non admin operation 
      break; 
     } 
    } else { 
     // if authentication fails 
     isPass = true 
    } 
} 
// Outside of while block 
if(isPass){ 
    // if authentication fails show error message 
    JOptionPane.showMessageDialog(null, "User name and password do" 
      + " not match!","ALERT!", JOptionPane.ERROR_MESSAGE); 
} 

我發現這個代碼片段展示瞭如何利用突破並意識到，你也應該讓這個break爲else part其他明智它會繼續顯示爲所有那些成爲虛假認證記錄錯誤信息在if部分。

我會建議從另一種方法獲取您的數據庫值，並在另一種方法中進行身份驗證。

Answer 2

您需要將您的else了一個級別。代碼格式化使得很難說明發生了什麼，但其他內容在admin if聲明中。就像這樣：

if ((user.equals(uname)) && (pwd.equals(pass))) 
{ 
    if (admin1.equals("y")) 
    { 
     // ... admin 
    } 
    else 
    { 
     // ... regular user 
    } 
} 
else 
{ 
    JOptionPane.showMessageDialog(null, "User name and password do" 
      + " not match!","ALERT!", JOptionPane.ERROR_MESSAGE); 
} 

Answer 3

做到像

if ((user.equals(uname)) && (pwd.equals(pass))) 
    { 
    mainPanel.setVisible(true); 
    blankContent.setVisible(true); 
    receiptContent.setVisible(false); 
    memberContent.setVisible(false); 
    securityPanel.setVisible(false); 
    adminButton.setEnabled(false); 
      if(admin1.equals("y")) 
       adminButton.setEnabled(true); 

    } 


else 
    { 
      JOptionPane.showMessageDialog(null, "User name and password do" 
        + " not match!","ALERT!", JOptionPane.ERROR_MESSAGE); 

      }