2011-09-27 5 views
2

Je travaillais avec un l'examinateur « Outils CodePro » par google et hélé l'écoulement:Tableau initialisé de manière statique?

new Object[] { max } 

avec « tableau statiquement initialisé »

Explication:

Un initialiseur de tableau est utilisé initialiser un tableau.

Recommandation

  1. La matrice doit être dynamiquement initialisé.

Y at-il une bonne raison pour cela? ou est juste préférable d'ignorer.

ce drapeau fait partie d'une section de règles appelée "style de code".

Merci

+1

J'utilise fréquemment des initialiseurs statiques. Je crois que dans certaines situations, ils peuvent rendre les structures plus claires. – mcfinnigan

+0

Cela disparaît-il lorsque vous déclarez la variable finale? – michael667

+0

@ michael667 Je suppose ne pas regarder le doc: http://code.google.com/javadevtools/codepro/doc/features/audit/audit_rules_com.instantiations.assist.eclipse.auditGroup.codingStyle.html#com.instantiations.assist .eclipse.audit.disallowArrayInitializers –

Répondre

4

Comme toujours: cela dépend. C'est une question de style. Personnellement, je ne vois rien de mal à cela. Dans ce cas, je pense que cela ne ferait que masquer le code pour l'initialiser dynamiquement. J'utilise des tableaux initialisés statiquement tout le temps. Le style de code est très subjectif et varie d'un projet à l'autre, pas seulement d'une personne à l'autre. C'est à vous de décider si c'est une bonne chose pour votre projet.

2

Prenez tous les outils d'inspection de code automatisés avec un grain de sel. Ils font des recommandations, ne pas émettre de commandes.

Si vous avez une bonne raison d'écrire votre code de cette façon, et que vous pouvez l'articuler bien à vous-même et à d'autres, alors respectez votre code et ignorez CodePro.

Questions connexes