[organizedKaoS]

Originally Posted by Xanimos TBH, I'm just messing with you. I don't doubt that I did it. I was just pointing out that you can't possible prove it in anyway.

LOL

[JFeldon]

Good work Ryan! Very helpful.

[OneMoreLevel]

This might fit, I usually use this habit.



-- Using the ! with statements.

Generally using code with ! like this:

PHP Code:

 /* Example plugin */
#include <amxmodx>
#include <amxmisc>
#define PLUGIN "Example plugin"
#define VERSION "1.1"
#define AUTHOR "OneMoreLevel"
public plugin_init() {
 
 register_plugin(PLUGIN,VERSION,AUTHOR)
 
 register_clcmd( "say /alive","Example_code")
}
public Example_code (id) {
 
 // Is the user alive if not, end the plugin.
 if (!is_user_alive(id))
 return PLUGIN_HANDLED;
 
 // We dont need to use an else if here because is_user_alive only has two options, 1 or 0. 
 else
 {
  // Send the message "You are currently alive!"
  client_print(id, print_chat,"You are currently alive!")
 }
 return PLUGIN_CONTINUE;
 
} 


Is better than using code like this:

PHP Code:

/* Example plugin */
#include <amxmodx>
#include <amxmisc>
#define PLUGIN "Example plugin"
#define VERSION "1.1"
#define AUTHOR "OneMoreLevel"
public plugin_init() {
 
 register_plugin(PLUGIN,VERSION,AUTHOR)
 
 register_clcmd( "say /alive","Example_code")
}
public Example_code (id) {
 
 // Is the user alive if not, end the plugin.
 if (is_user_alive(id) == 0)
 return PLUGIN_HANDLED;
 
 // We dont need to use an else if here because is_user_alive only has two options, 1 or 0. 
 else
 {
  // Send the message "You are currently alive!"
  client_print(id, print_chat,"You are currently alive!")
 }
 return PLUGIN_CONTINUE;
 
} 


This isn't required by the AMXModX include, nor the pawn language, its just a good habit to get into if you want to learn future languages, and it makes your code look cleaner.

[fysiks]

Actually, I think it would be better to use:

PHP Code:

if( is_user_alive(id) )
{
    // User is Alive
}
else
{
    // User is Dead
} 


This is much more readable if you need both cases. Otherwise you can do the following if you only need one or the other:

PHP Code:

if( is_user_alive(id) )
{
    // User is Alive
} 


OR

PHP Code:

if( !is_user_alive(id) )
{
    // User id Dead
} 


So, basically don't use the '!' (not) if you need both a dead and alive case.

Disclaimer: This is my opinion and I believe that it is much easier to read.

[Hawk552]

Boolean inversion is faster than direct comparison anyway. It's not a matter of style; you're just outright wrong otherwise.

[drekes]

Originally Posted by GHW_Chronic NOTEPAD!!! W00T! --Oldschool boi

Do you really code such great plugins in notepad,
I'm sorry but i can't believe that.
I tried it once and i though i was gonna throw my pc out the window because the nerves i got doing it.

No number of lines, ....

[Xanimos]

Originally Posted by drekes Do you really code such great plugins in notepad, I'm sorry but i can't believe that. I tried it once and i though i was gonna throw my pc out the window because the nerves i got doing it. No number of lines, ....

...I coded all of my plugins in notepad....in fact I did a lot of things in notepad.
Also to get the line number use Ctrl+G. That's also a nifty way to skip around based on line numbers.

[ConnorMcLeod]

Notepad++ is also fine for coding.