Thread: Anyone wanna review my code for me?

Originally Posted by ninja9578

Just a few notes Some people use spaces over tabs for company standard reasons (my company requires it.) It means that it will always look the same way in any editor.

Yeah, but it also means you'll spend so much more time when navigating the document using the keyboard :p
But if it's not your fault...

Originally Posted by ninja9578

/* this is a C comment */
// this is an illegal line in C

Actually C99 supports // comments. But the reason I like them is because they're the preferred C++ comments, and because if you exclusively use those in your code, commenting out larger portions while debugging is super-easy.

Originally Posted by ninja9578

?> tags are not required at the end of a file, and normally not included

Oh? I did not know that.

Originally Posted by ninja9578

I use echo as a function for consistency. Constructs can be use as functions too, and most of them require to be used that way, for instance isset.

Well yes, but I believe it's the only construct/function without a return value of any kind. Also it means I don't have to type as much.