Re: [Phpmyadmin-devel] New PHPCS rule

19 Dec 2013


      Hi

Dne Sun, 15 Dec 2013 20:36:04 +0100
Hugues Peccatte <hugues.peccatte@gmail.com> napsal(a):
...
What would you think about using the "nesting level" sniff please?
This sniff permits to detect when there are too much nested levels like:
if (...) {
    if (...) {
        if (...) {
            if (...) {
                if (...) {
                    ....
                }
            }
        }
    }
}
This kind of syntax often means that a refactoring is needed or a rewrite.
Lot of things which need refactoring are already detected by phpmd
(though it's only executed nightly as it takes quite a lot of time):

http://ci.phpmyadmin.net/job/phpMyAdmin/891/pmdResult/

-- 
	Michal Čihař | http://cihar.com | http://phpmyadmin.net