[Konstantinos]

The better option would be to select all the data once and assign them if there are rows. In the login, you already got the password so compare it and you're done.

Few notes:
- Reset variables on connect.
- Instead of cache_get_data and storing the rows and fields to 2 variables, you could use directly cache_num_rows function:

pawn Code:

if (cache_num_rows()) // if (rows)

Also keep in mind about race condition attacks with threaded queries.

Quote:

Originally Posted by Lordz™ Nice, well explained. But in case if there's too much fields, it's better to use functions like cache_get_field_content, cache_get_field_content_int, cache_get_field_content_float, etc. So that the user who uses won't get confused in knowing the field ID.

Using the field's index instead of the name is faster, but yes - it depends on the user!