<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2013/12/16 Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">




Le 2013-12-15 14:36, Hugues Peccatte a écrit :<br>
<div><div>> Hi,<br>
><br>
> What would you think about using the "nesting level" sniff please?<br>
> This sniff permits to detect when there are too much nested levels like:<br>
> if (...) {<br>
>     if (...) {<br>
>         if (...) {<br>
>             if (...) {<br>
>                 if (...) {<br>
>                     ....<br>
>                 }<br>
>             }<br>
>         }<br>
>     }<br>
> }<br>
><br>
> This kind of syntax often means that a refactoring is needed or a rewrite.<br>
><br>
> I can add it myself if you agree.<br>
><br>
<br>
</div></div>Good idea but how many levels is considered "too many"?  Can you give an<br>
example of nesting levels you found?<br>
<br>
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a> | <a href="http://phpmyadmin.net" target="_blank">http://phpmyadmin.net</a><br>
<br>
------------------------------------------------------------------------------<br>
Rapidly troubleshoot problems before they affect your business. Most IT<br>
organizations don't have a clear picture of how application performance<br>
affects their revenue. With AppDynamics, you get 100% visibility into your<br>
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!<br>
<a href="http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk" target="_blank">http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk</a><br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net" target="_blank">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
</blockquote></div><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">Hi,<div><br></div><div>To add this new rule won't add to many PHPCS errors. I already test this rule and there are currently not so much this errors in the sources. (I don't have an exact count. I would say around 20-30.)</div>


<div><br></div>
<div>See: <a href="http://www.squizlabs.com/php-codesniffer/php_codesniffer-ruleset.xml-support-in-svn" target="_blank">http://www.squizlabs.com/php-codesniffer/php_codesniffer-ruleset.xml-support-in-svn</a></div><div>See: <a href="http://pear.php.net/manual/en/package.php.php-codesniffer.annotated-ruleset.php" target="_blank">http://pear.php.net/manual/en/package.php.php-codesniffer.annotated-ruleset.php</a></div>


<div><br></div><div>The rule could be:</div><div><div>  <rule ref="Generic.Metrics.NestingLevel"></div><div>    <properties></div><div>      <property name="nestingLevel" value="2" /></div>


<div>      <property name="absoluteNestingLevel" value="5" /></div><div>    </properties></div><div>  </rule></div></div><div><br></div><div>We don't need to define both properties.</div>


<div>"nestingLevel" is the number of nested level which throw a warning. (default = 5)</div><div>"absoluteNestingLevel" is the maximum of allowed nested levels. (default = 10)</div><div><br></div><div>


I think that the default values don't need to be overridden, so the rule would be:</div><div><rule ref="Generic.Metrics.NestingLevel" /><br></div><div><br></div><div>Hugues.</div></div></div>