delete query not working java netbeans

Question

I am trying to delete a record in database where if the username i typed is not admin i can delete it however its not working I think its the query any idea thanks in advance.I am a newbie i been trying to figure it out ...

   String password = new String(oldPass.getPassword());
    String realpass = pw.getText();
    String us = userr.getText();
      user = us;

        System.out.println("ok");
        String query = "DELETE FROM user WHERE privilege = 'NOT ADMIN' +        username = '"+us+"'";
      try {
            Statement st = (Statement) con.createStatement();
            int r = st.executeUpdate(query);
            if (r != 0) {
                JOptionPane.showMessageDialog(null, "Successfully deleted!", "Delete", JOptionPane.OK_OPTION);

                login w = new login();
                w.setVisible(true);
                this.dispose();
            } else {
                JOptionPane.showMessageDialog(null, "Wait! something's wrong, please try again later.", "Ooopppss!", JOptionPane.OK_OPTION);


            }

        } catch (Exception e) {
            System.out.println(e);
        }


      // TODO add        // TODO add your handling code here:
}

Your query looks wrong in so many ways. It has a plus symbol in it, and is NOT ADMIN really an actual privilege? What database are you using? — Tim Biegeleisen
– Tim Biegeleisen, Commented Dec 6, 2016 at 0:45
im using php mysql even if i change the privilege as staff its still not deleted — lec
– lec, Commented Dec 6, 2016 at 0:49
DELETE FROM user WHERE privilege = 'NOT ADMIN' AND username = 'lec' ... use this syntax, do not use a plus symbol — Tim Biegeleisen
– Tim Biegeleisen, Commented Dec 6, 2016 at 0:50
I'd strongly recommend using a PreparedStatement to avoid an SQL injection attack. This is just a learning exercise, but better to get into good habits now. — Brad
– Brad, Commented Dec 6, 2016 at 2:23

jam · Accepted Answer · 2016-12-06 00:56:54Z

1

Maybe you are missing the AND or the Or operator depend on what are you trying to do here an example

DELETE FROM user WHERE privilege = 'NOT ADMIN' AND user='user'

Anyway I recommend you that instead use the string 'not admin' you create a new catalogue from privileges

edited Dec 6, 2016 at 0:56

answered Dec 6, 2016 at 0:51

jam

5391 gold badge5 silver badges21 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

delete query not working java netbeans

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related