Không kiểm tra được username và password

[boykhonghut]

Anh em xem giúp em với, kiểm tra Username và Password trùng với database. Làm như thế này đã đúng chưa? nếu sai sửa giúp em cảm ơn all

private void btnloginActionPerformed(java.awt.event.ActionEvent evt) {    try {        boolean check;        username = txtuser.getText();        password = txtpassword.getText();        stmt = con.createStatement();         rs = stmt.executeQuery("select Username,Password from Admin ");        while(rs.next())        {            if(rs.getString("Username") == username && rs.getString("Password")== password)             {                check = true;                frmMain fm = new frmMain();                fm.setVisible(true);                this.setVisible(false);                break;            }            else            {                JOptionPane.showMessageDialog(this, "Erorr username or password");                txtuser.grabFocus();                return;            }        }    } catch (SQLException ex) {        ex.printStackTrace();    }}

View more random threads:

• Học java như thế nào cho hiệu quả????
• Làm thế nào để hiển thị nội dung chat lên JTextArea1
• sự khác nhau giữa phương thức setSize và phương thức setpreferredsize trong java
• Tạo 2 JFrame riêng biệt nhưng thông tin lại trùng nhau
• Eclipse Birt Report trong Jsp
• Cách đọc dữ liệu từ file .dat đó và đổ vào JList để hiển thị?
• Hàm boolean cụ thể ở đây là ý nghĩa làm gì các anh nhỉ
• Lớp Ticker nằm trong package nào?
• Sử dụng 1 thư viện được lập trình trên C++ trong Java như thế nào ?
• [Hỏi] Đóng Frame được gọi từ Frame cha?

[phongkhamkt1]

Thứ nhất là bạn nên đưa code vào tag code nâng cao như mình đã sửa, như thế mới thu hút người trả lời và cũng là tôn trọng người trả lời hỗ trợ bạn.

Đối với vấn đề của bạn, bạn đã tư duy sai (chính xác là bạn không hiểu về vòng lặp). Trong câu code trên của bạn chỉ đúng nếu người dùng là tài khoản đầu tiên trong danh sách. Bạn chuyển đoạn code ở else block ra ngoài while, thay break ở if block thành return và thử lại.

[luattrihung]

Thứ nhất là bạn nên đưa code vào tag code nâng cao như mình đã sửa, như thế mới thu hút người trả lời và cũng là tôn trọng người trả lời hỗ trợ bạn.

Đối với vấn đề của bạn, bạn đã tư duy sai (chính xác là bạn không hiểu về vòng lặp). Trong câu code trên của bạn chỉ đúng nếu người dùng là tài khoản đầu tiên trong danh sách. Bạn chuyển đoạn code ở else block ra ngoài while, thay break ở if block thành return và thử lại.

Trc hết e sẽ sữa cái sai.
Cảm ơn anh về vấn đề vòng lặp, có lẽ do trường e quá rặp khuôn 1 + 1 = 2, giờ = 0 em lại bối rối, e cũng gỡ khúc mắc được vấn đề truy xuất = cách sử dụng equal().
Thanks a

[mituot]

Trong lập trình chuyên nghiệp, người ta thường hạn chế sử dụng if else tới mức thấp nhất. Việc hạn chế if else sẽ làm cho code được ngắn gọn và dễ hiểu hơn rất nhiều. Đây là điều mà trường học thật sự ít khi nói đến, và bản chất tư duy của nhiều giáo viên vẫn 'cồng kềnh, phức tạp' nhưng hiệu quả thực tế lại không cao. Vì thế, hầu hết các sinh viên bướng bỉnh, năng động, cãi giáo viên nhem nhẻm lại có hiệu quả công việc cao hơn các sinh viên chịu khó, miệt mài.

[hungosp]

@Kevin Hoang cho mình hỏi là tại sao hạn chế sử dụng if else đến mức thấp nhất ? Vậy khi bắt điều kiện thường dùng như thế nào ?

Bạn có thể nói rõ hơn 1 xíu được ko ?

[trothinhthienduc]

@Kevin Hoang cho mình hỏi là tại sao hạn chế sử dụng if else đến mức thấp nhất ? Vậy khi bắt điều kiện thường dùng như thế nào ?

Bạn có thể nói rõ hơn 1 xíu được ko ?

Chắc là bắt exception thay vì soi error code.

[ngocha]

@Kevin Hoang cho mình hỏi là tại sao hạn chế sử dụng if else đến mức thấp nhất ? Vậy khi bắt điều kiện thường dùng như thế nào ?

Bạn có thể nói rõ hơn 1 xíu được ko ?

Ví dụ thế này:

public bool IsGood(int V) {    if (V > 5) return true;    else return false;}

Và thế này:

public bool IsGood(int V) {    return V > 5;}

Hai đoạn mã đó hoàn toàn trả về giá trị giống nhau.

[bacsiquaidi1]

Ví dụ thế này:

public bool IsGood(int V) {    if (V > 5) return true;    else return false;}

Và thế này:

public bool IsGood(int V) {    return V > 5;}

Hai đoạn mã đó hoàn toàn trả về giá trị giống nhau.

Tks bạn đã trả lời, nhưng mình nghĩ đó chẳng qua là cách viết khác nhau thôi, thay vì if else thì .. " ? : " ! Hoặc như bạn nói .v.v.!
Còn bạn @Kevin Hoàng bảo là viết code sẽ đơn giản và dễ hiểu hơn nên mình không rõ lắm.

[anhhungan]

Mình không hiểu lắm là như code của cariboga thì nếu không sử dụng if else thì còn cách nào để tối giản code như anh Kevin nói. Bạn nào tối giản được code thì chỉ dùm mình với. Tks