[Eoussama]

Quote:

Originally Posted by AndySedeyn While I usually don't complain about the excessive use of unnecessary else if statements, I do get annoyed by it. A lot of people find it reasonable to use them in a situation where the given condition can only be maximum two states and thus making an else if statement pointless and may even become "ineffective": PHP Code: if(IsPlayerConnected(playerid)) {     // Some code ... } else {     if(!IsPlayerConnected(playerid))     {         // Some other code ...     } }  Given that, it seems like the OP has a bad habit of doing just that. I see the following reoccur more than a few times throughout the tutorial: PHP Code: if(IsPlayerConnected(playerid)) {     // Some code ...     return 1; } else if(!IsPlayerConnected(playerid)) {     // Some other code ...     return 1; }  A player is either connected or not. In this case, a simple else statement is the best way to go. To drop the whole else statement altogether, the function would have to return a value in the if statement above that: PHP Code: if(condition) {     // Some code...     return someValue; } // Code under this if statement is automatically the else statement, that is without explicitely using the else keyword. return someOtherValue;  Not explicitly using the else statement may reduce readability for programmers who are used to using it in such cases. I tend to not use it, but I'm fine with either. Here's another mistake I came across: EDIT: Your boolean variable 'isCuffed' will only work when nobody is cuffed. Once 1 player on the server is cuffed, nobody else can get cuffed.

OMG thank you, I actually admire replies like that, where people criticise my work so I can improve,

thanks all for the feedbacks, this is my first tuto, so It's normal