View by Classes

Findings - Overview

code-style InnerAssignment (Checkstyle)

Inner assignments should be avoided.

Further info on the wiki.

Checks for assignments in subexpressions, such as in String s = Integer.toString(i = 2);.

Rationale: With the exception of for iterators, all assignments should occur in their own toplevel statement to increase readability. With inner assignments like the above it is difficult to see all places where a variable is set.

6org.jcoderz.phoenix.sqlparser.SqlParser
 [199:21],  [433:21],  [943:21],  [1006:24],  [1023:24],  [1106:21]
5org.jcoderz.commons.logging.StackTraceElementParser
 [105:19],  [109:24],  [113:24],  [117:24],  [121:24]
1org.jcoderz.phoenix.sqlparser.SqlTransformer
 [174:25]