views:

319

answers:

8

I have a small PHP web-based application that is beginning to grow moderately in size.

I'm starting to become concerned with managing my PHP code base, given PHP is a loosely/weak typed, dynamic language.

How do others manage their code based for loosely/weak typed, dynamic languages?

Do pre-parsers exist for PHP that allow me to runs checks on my code base to identity such things like below?

$var1 = 'data';
// vr1 doesn't exist, it's a typo of $var1, but PHP would allow for this and not complain
echo $vr1;

UPDATE:

The example above might not be the best example but essentially, what I'm trying to convey is that certain errors in a dynamically weak typed language would only be found when the code is run in production at RUN TIME; whereas, some of these issues would typically be found in strongly typed static languages at COMPILE time.

How can I also find these non-algorithm type of errors in PHP prior to moving my code into production without having to create an insane number of Unit Tests?

As such, does anything exist where I can run my PHP code through it, prior to moving into production, and this pre-processor parses my code to ensure I'm only using defined variables, etc. Essentially, check my code for validation for non-algorithmic type of uses. E.g. not trying perform algebra on a string, etc.

UPDATE 2

Please note, this question is still not answered because I'm looking for a way to identity these type of non-algorithmic errors in PHP at "compile" type, not RUN TIME.

+10  A: 

You can lint your PHP with php -l filename.php. This would show any syntax errors. There is IDEs out there that will lint while you write the code. Those usually can also detect issues like shown in your question in addition to linting.

Apart from this, consider writing UnitTests for your code to ensure functionality and have a look at http://phpqatools.org for a number of other tools that can assist you in increasing code quality.

Make sure you have error_reporting(-1); set during development to enable all errors, in addition to enable display_errors and display_startup_errors in php.ini. Disable the latter two on your production system to prevent exposure of server information.

Edit after update: PHP source code is compiled on-the-fly. PHP's compile time is effectively at run time. If you want compiled PHP, you have to use Facebook's HipHop.

Gordon
Writing UnitTest is simple unrealistic. Look at my example above, I'd have to write a unit test for nearly every line of code I have if I were to make a simply variable name typo.
Eric B
@Eric It's neither unrealistic nor is it uncommon to write tests for nearly every line of code. A typo will usually break something somewhere. If you have a `function add($var1, $var2) { return $vr1 + $var2; }` it will not pass a UnitTest testing that the input data adds up.
Gordon
@Eric Gordon is right. If you haven't started unit testing your PHP code, you should. Right now. Your life will be a lot easier, trust me.
NullUserException
If you find unit testing to be unrealistic for your code, then that's a problem with your code not with unit testing. The idea is to break your code into distinct functions, with the goal being ones that are referentially transparent (given input a it always returns output b). You can then test that each function does what it's supposed to do. If it does, then either there are no typos or those typos don't effect the behavior of the function.
RHSeeger
+4  A: 

PHP will definitely complain about that with either a warning or a notice if you set your error_reporting config directive appropriately.

See:

http://us2.php.net/manual/en/errorfunc.configuration.php#ini.error-reporting

beamrider9
+1: Doesn't solve all problems, but at least catches some issues like nonexistent variables or undefined indexes.
delnan
My concern though is that it catches these error at RUN TIME (while in production). These error should be catch before the code is in production. These kind of errors would typically be catch in compiled languages at compile time.
Eric B
@bearmrider9, please note - my original question is asking for a way to identity these type of non-algorithmic erros at COMPILE time and not RUNTIME. As such, you're answer does not apply.
Eric B
correct me if im wrong, but isn't php interpreted, rather than compiled like ASP.NET is?
Rob
Yeah, with php COMPILE and RUNTIME are *the same thing* because its an interpreted language.
indieinvader
A: 

That type of error would be caught if you set error reporting to the max. It would give a Notice indicating that $vr1 wasn't set.

You can set error reporting in your php.ini file, or on individual pages using the ini_set() function.

babonk
But that error would only be found at RUN TIME. How can I get PHP to inform me of that error that $vr1 is not defined at COMPILE time?
Eric B
PHP isn't compiled.
babonk
A: 

The closest thing is php's lint checker, but that's more of a syntax checker. You can run lint from a command line:

php -l path/to/file.php

You could build this into your file repository system by setting up a pre-commit check.

Ben Rowe
A: 

This is a very interesting question.

Since PHP is interpreted (hence, compiled on-the-fly at runtime), I'm not sure how you would accomplish this question BUT, it's VERY much needed.

+3  A: 
banzaimonkey
A: 

As PHP is not usually considered to go through a separate COMPILE process perhaps you could explain at what point you consider your code to be COMPILED?

trowel
A: 

Here's another SO question that focuses on PHP code analysis tools.

banzaimonkey