[AOM_Set]

Hi.
I have a method that should check the arguments that the user has given. In the database I have columns named email, and password (among others). How can I make the check if the arguments that the user has given are right? Here is what I tried, and I have a very strong feeling that this is very wrong. Can I in some way say: Select email and password from users (The table is called users)

public boolean returnValidCredentials(String emailParsed, String passwordParsed){
		try {
			rSet = statement.executeQuery("SELECT * FROM users");
			String emailRes = rSet.getString("email");
			String passwordRes = rSet.getString("password");
			
			while(rSet.next()){							
				if(!(emailRes.equalsIgnoreCase(passwordParsed) && passwordRes.equals(passwordParsed))){
					return false;
				}
			}			
		} catch (SQLException e) {
			displayMsg("Exception at method returnValidCredentials()");
		}		
		return true;
	}

I have a feeling that my code while break/return false after this first check that returns false, and it wont even check the rest. Even though, the true result might be right after it returned false

This post has been edited by AOM_Set: 02 May 2012 - 02:17 PM

[AOM_Set]

AOM_Set, on 02 May 2012 - 02:06 PM, said:

						
if(!(emailRes.equalsIgnoreCase(passwordParsed) && passwordRes.equals(passwordParsed))){	

This should obviously say:

if(!(emailRes.equalsIgnoreCase(emailParsed) && passwordRes.equals(passwordParsed)))

[IamTw_]

Instead of returning false in the while-loop, you could make the check that return true there. That will make sure that you check all the values in the columns, and return false at the end.
Something like this:

public boolean returnValidCredentials(String emailParsed, String passwordParsed){
		try {
			rSet = statement.executeQuery("SELECT email, password FROM users");
			String emailRes = rSet.getString(1); //column 1
			String passwordRes = rSet.getString(2); //column 2
			
			while(rSet.next()){							
				if((emailRes.equalsIgnoreCase(emailParsed) && passwordRes.equals(passwordParsed))){
					return true;
				}
			}			
		} catch (SQLException e) {
			displayMsg("Exception at method returnValidCredentials()");
		}		
		return false;
	}