22.08.2011, 02:13
Nicely explained, however, there are a few errors and or unnecessary things done I would like to point out:
- You free a result when you don't select anything (when creating the table if not exists)
- You don't free the result after selecting information (under OnPlayerConnect)
- You use ` around field names (only necessary around table names)
- You use ' around integer values (only necessary around strings)
- You check if the player is registered under OnDialogResponse in the registration dialog, even though you only show it if they are registered
- When seeing if the player is registered, you select ALL their information, which is unnecessary, and even inefficient with large amounts of fields. Just retrieve one value.