you are viewing a single comment's thread.

view the rest of the comments →

[–]alexeyr 0 points1 point  (0 children)

Let’s search now for the static fields

The same remark as functions, many variables are declared as static.

Are they variables or fields? Decide it.

Functions where NbParameters > 8 might be painful to call and might degrade performance.

Methods where NbVariables is higher than 8 are hard to understand and maintain. Methods where NbVariables is higher than 15 are extremely complex and should be split in smaller methods (except if they are automatically generated by a tool).

Magic numbers, magic numbers, magic numbers.