| # | File | Line | Problem |
| 1 |
regenstrief\db2xml\Workaround.java |
1 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 2 |
regenstrief\db2xml\Workaround.java |
5 |
Avoid unused imports such as 'java.sql.Timestamp' |
| 3 |
regenstrief\db2xml\Workaround.java |
11 |
Use explicit scoping instead of the default package private level |
| 4 |
regenstrief\db2xml\Workaround.java |
12 |
Use explicit scoping instead of the default package private level |
| 5 |
regenstrief\db2xml\Workaround.java |
39 |
Avoid using if...else statements without curly braces |
| 6 |
regenstrief\db2xml\Workaround.java |
41 |
Avoid using if...else statements without curly braces |
| 7 |
regenstrief\db2xml\Workaround.java |
42 |
A method should have only one exit point, and that should be the last statement in the method |
| 8 |
regenstrief\db2xml\Workaround.java |
43 |
Use equals() to compare object references. |
| 9 |
regenstrief\db2xml\Workaround.java |
44 |
A method should have only one exit point, and that should be the last statement in the method |
| 10 |
regenstrief\db2xml\Workaround.java |
44 |
Avoid using if...else statements without curly braces |
| 11 |
regenstrief\db2xml\Workaround.java |
46 |
Avoid using if...else statements without curly braces |
| 12 |
regenstrief\db2xml\Workaround.java |
56 |
Avoid using if...else statements without curly braces |
| 13 |
regenstrief\db2xml\Workaround.java |
58 |
Avoid using if...else statements without curly braces |
| 14 |
regenstrief\db2xml\Workaround.java |
59 |
A method should have only one exit point, and that should be the last statement in the method |
| 15 |
regenstrief\db2xml\Workaround.java |
60 |
Use equals() to compare object references. |
| 16 |
regenstrief\db2xml\Workaround.java |
61 |
A method should have only one exit point, and that should be the last statement in the method |
| 17 |
regenstrief\db2xml\Workaround.java |
61 |
Avoid using if...else statements without curly braces |
| 18 |
regenstrief\db2xml\Workaround.java |
63 |
Avoid using if...else statements without curly braces |
| 19 |
regenstrief\db2xml\Workaround.java |
68 |
Caught exception is rethrown, original stack trace may be lost |
| 20 |
regenstrief\db2xml\Workaround.java |
72 |
Avoid variables with short names like i |
| 21 |
regenstrief\db2xml\Workaround.java |
81 |
Avoid variables with short names like i |
| 22 |
regenstrief\db2xml\Workaround.java |
90 |
Avoid variables with short names like i |
| 23 |
regenstrief\debug\EmailDump.java |
20 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 24 |
regenstrief\debug\EmailDump.java |
33 |
Use explicit scoping instead of the default package private level |
| 25 |
regenstrief\debug\EmailDump.java |
35 |
Use explicit scoping instead of the default package private level |
| 26 |
regenstrief\debug\EmailDump.java |
37 |
Avoid variables with short names like ps |
| 27 |
regenstrief\debug\EmailDump.java |
37 |
Use explicit scoping instead of the default package private level |
| 28 |
regenstrief\mw\MWApplicationInitializer.java |
26 |
Avoid excessively long variable names like applicationRuntime |
| 29 |
regenstrief\mw\MWApplicationInitializer.java |
27 |
Consider simply returning the value vs storing it in local variable 'applicationRuntime' |
| 30 |
regenstrief\mw\MWApplicationInitializer.java |
30 |
Avoid excessively long variable names like applicationContext |
| 31 |
regenstrief\mw\NotificationChannel.java |
28 |
Avoid modifiers which are implied by the context |
| 32 |
regenstrief\mw\NotificationChannel.java |
30 |
Avoid modifiers which are implied by the context |
| 33 |
regenstrief\mw\NotificationChannel.java |
32 |
Avoid modifiers which are implied by the context |
| 34 |
regenstrief\mw\NotificationChannel.java |
34 |
Avoid modifiers which are implied by the context |
| 35 |
regenstrief\mw\StartApplication.java |
20 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 36 |
regenstrief\mw\StartApplication.java |
65 |
System.exit() should not be used in J2EE/JEE apps |
| 37 |
regenstrief\mw\StartApplication.java |
68 |
Avoid variables with short names like c |
| 38 |
regenstrief\mw\StartApplication.java |
68 |
Ensure that resources like this Connection object are closed after use |
| 39 |
regenstrief\mw\StartApplication.java |
68 |
Found 'DD'-anomaly for variable 'c' (lines '68'-'74'). |
| 40 |
regenstrief\mw\StartApplication.java |
79 |
System.exit() should not be used in J2EE/JEE apps |
| 41 |
regenstrief\mw\StartApplication.java |
82 |
Avoid if (x != y) ..; else ..; |
| 42 |
regenstrief\mw\StartApplication.java |
86 |
Ensure that resources like this Statement object are closed after use |
| 43 |
regenstrief\mw\StartApplication.java |
90 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 44 |
regenstrief\mw\StartApplication.java |
90 |
Ensure that resources like this ResultSet object are closed after use |
| 45 |
regenstrief\mw\StartApplication.java |
99 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 46 |
regenstrief\mw\StartApplication.java |
99 |
Ensure that resources like this ResultSet object are closed after use |
| 47 |
regenstrief\mw\Test.java |
14 |
Avoid variables with short names like a |
| 48 |
regenstrief\mw\Test.java |
15 |
Avoid variables with short names like b |
| 49 |
regenstrief\mw\Test.java |
19 |
Avoid variables with short names like aa |
| 50 |
regenstrief\mw\Test.java |
20 |
Avoid variables with short names like bb |
| 51 |
regenstrief\mw\Test.java |
33 |
Use explicit scoping instead of the default package private level |
| 52 |
regenstrief\mw\Test.java |
39 |
Use explicit scoping instead of the default package private level |
| 53 |
regenstrief\mw\ViewPort.java |
27 |
Avoid modifiers which are implied by the context |
| 54 |
regenstrief\mw\ViewPort.java |
30 |
Avoid modifiers which are implied by the context |
| 55 |
regenstrief\mw\ViewPort.java |
33 |
Avoid modifiers which are implied by the context |
| 56 |
regenstrief\mw\ViewPort.java |
36 |
Avoid modifiers which are implied by the context |
| 57 |
regenstrief\mw\ViewPort.java |
42 |
Avoid modifiers which are implied by the context |
| 58 |
regenstrief\mw\allergy\AllergyListController.java |
51 |
Avoid reassigning parameters such as 'view' |
| 59 |
regenstrief\mw\allergy\AllergyListController.java |
54 |
The String literal "item" appears 6 times in this file; the first occurrence is on line 54 |
| 60 |
regenstrief\mw\allergy\AllergyListController.java |
63 |
The String literal "list" appears 6 times in this file; the first occurrence is on line 63 |
| 61 |
regenstrief\mw\allergy\AllergyListController.java |
92 |
A method should have only one exit point, and that should be the last statement in the method |
| 62 |
regenstrief\mw\allergy\AllergyListController.java |
95 |
A method should have only one exit point, and that should be the last statement in the method |
| 63 |
regenstrief\mw\allergy\AllergyListController.java |
102 |
Private field '_code' could be made final; it is only initialized in the declaration or constructor. |
| 64 |
regenstrief\mw\allergy\AllergyListController.java |
129 |
Avoid using if...else statements without curly braces |
| 65 |
regenstrief\mw\allergy\AllergyListController.java |
173 |
Document empty method |
| 66 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
33 |
Avoid duplicate imports such as 'org.hl7.rim.Participation' |
| 67 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
39 |
Avoid unused imports such as 'org.hibernate.Criteria' |
| 68 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
50 |
Avoid reassigning parameters such as 'modelObject' |
| 69 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
53 |
Position literals first in String comparisons |
| 70 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
59 |
Avoid using if statements without curly braces |
| 71 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
60 |
Avoid using for statements without curly braces |
| 72 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
61 |
Avoid using if statements without curly braces |
| 73 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
61 |
Deeply nested if..then statements are hard to read |
| 74 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
68 |
Avoid variables with short names like a |
| 75 |
regenstrief\mw\allergy\AllergyNodeViewAdapter.java |
75 |
Avoid using final local variables, turn them into fields |
| 76 |
regenstrief\mw\allergy\ChangeAllergyStatusCommand.java |
20 |
Avoid using implementation types like 'ArrayList'; use the interface instead |
| 77 |
regenstrief\mw\allergy\ChangeAllergyStatusCommand.java |
25 |
Document empty constructor |
| 78 |
regenstrief\mw\allergy\ChangeAllergyStatusCommand.java |
25 |
It is a good practice to call super() in a constructor |
| 79 |
regenstrief\mw\allergy\ChangeAllergyStatusCommand.java |
36 |
A switch statement does not contain a break |
| 80 |
regenstrief\mw\allergy\ChangeAllergyStatusCommand.java |
38 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 81 |
regenstrief\mw\allergy\ChangeAllergyStatusCommand.java |
39 |
A method should have only one exit point, and that should be the last statement in the method |
| 82 |
regenstrief\mw\allergy\EditAllergyCommand.java |
27 |
Avoid unused imports such as 'org.regenstrief.mw.ApplicationAct' |
| 83 |
regenstrief\mw\allergy\EditAllergyCommand.java |
35 |
The field name indicates a constant but its modifiers do not |
| 84 |
regenstrief\mw\allergy\EditAllergyCommand.java |
39 |
Document empty constructor |
| 85 |
regenstrief\mw\allergy\EditAllergyCommand.java |
39 |
It is a good practice to call super() in a constructor |
| 86 |
regenstrief\mw\allergy\EditAllergyCommand.java |
52 |
Avoid if (x != y) ..; else ..; |
| 87 |
regenstrief\mw\allergy\EditAllergyCommand.java |
77 |
Avoid unused local variables such as 'o'. |
| 88 |
regenstrief\mw\allergy\EditAllergyCommand.java |
77 |
Avoid variables with short names like o |
| 89 |
regenstrief\mw\allergy\EditAllergyCommand.java |
77 |
Found 'DU'-anomaly for variable 'o' (lines '77'-'84'). |
| 90 |
regenstrief\mw\allergy\EditAllergyCommand.java |
80 |
A method should have only one exit point, and that should be the last statement in the method |
| 91 |
regenstrief\mw\allergy\EditAllergyCommand.java |
80 |
Avoid using if...else statements without curly braces |
| 92 |
regenstrief\mw\allergy\EditAllergyCommand.java |
82 |
Avoid using if...else statements without curly braces |
| 93 |
regenstrief\mw\allergy\EditAllergyCommand.java |
86 |
Overriding method merely calls super |
| 94 |
regenstrief\mw\app\AbstractSessionImpl.java |
47 |
Avoid excessively long variable names like _sessionPerspective |
| 95 |
regenstrief\mw\app\AbstractSessionImpl.java |
49 |
Document empty constructor |
| 96 |
regenstrief\mw\app\AbstractSessionImpl.java |
49 |
It is a good practice to call super() in a constructor |
| 97 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
71 |
Avoid unnecessary constructors - the compiler will generate these for you |
| 98 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
71 |
Document empty constructor |
| 99 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
71 |
It is a good practice to call super() in a constructor |
| 100 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
142 |
Avoid unused private fields such as '_specializations'. |
| 101 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
142 |
Private field '_specializations' could be made final; it is only initialized in the declaration or constructor. |
| 102 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
165 |
Deeply nested if..then statements are hard to read |
| 103 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
180 |
Use explicit scoping instead of the default package private level |
| 104 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
192 |
Found 'DD'-anomaly for variable 'def' (lines '192'-'198'). |
| 105 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
196 |
Avoid if (x != y) ..; else ..; |
| 106 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
207 |
Avoid if (x != y) ..; else ..; |
| 107 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
237 |
A method should have only one exit point, and that should be the last statement in the method |
| 108 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
251 |
Caught exception is rethrown, original stack trace may be lost |
| 109 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
262 |
Found 'DU'-anomaly for variable 'def' (lines '262'-'293'). |
| 110 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
267 |
Avoid if (x != y) ..; else ..; |
| 111 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
269 |
A method should have only one exit point, and that should be the last statement in the method |
| 112 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
274 |
Avoid if (x != y) ..; else ..; |
| 113 |
regenstrief\mw\app\ApplicationActDefinitionImpl.java |
302 |
A method should have only one exit point, and that should be the last statement in the method |
| 114 |
regenstrief\mw\app\ApplicationActImpl.java |
20 |
A high number of imports can indicate a high degree of coupling within an object. |
| 115 |
regenstrief\mw\app\ApplicationActImpl.java |
20 |
High amount of different objects as members denotes a high coupling |
| 116 |
regenstrief\mw\app\ApplicationActImpl.java |
20 |
This class has a bunch of public methods and attributes |
| 117 |
regenstrief\mw\app\ApplicationActImpl.java |
73 |
Abstract classes should be named AbstractXXX |
| 118 |
regenstrief\mw\app\ApplicationActImpl.java |
73 |
Too many fields |
| 119 |
regenstrief\mw\app\ApplicationActImpl.java |
77 |
Avoid excessively long variable names like _perspectiveViewport |
| 120 |
regenstrief\mw\app\ApplicationActImpl.java |
81 |
Avoid excessively long variable names like _activeApplicationActs |
| 121 |
regenstrief\mw\app\ApplicationActImpl.java |
92 |
Avoid excessively long variable names like _transitionListeners |
| 122 |
regenstrief\mw\app\ApplicationActImpl.java |
101 |
Avoid excessively long variable names like _subjectParticipation |
| 123 |
regenstrief\mw\app\ApplicationActImpl.java |
103 |
Avoid excessively long variable names like _subjectActRelationship |
| 124 |
regenstrief\mw\app\ApplicationActImpl.java |
107 |
Avoid excessively long variable names like _outboundRelationship |
| 125 |
regenstrief\mw\app\ApplicationActImpl.java |
109 |
Avoid excessively long variable names like _inboundRelationship |
| 126 |
regenstrief\mw\app\ApplicationActImpl.java |
111 |
Avoid excessively long variable names like _notificationChannel |
| 127 |
regenstrief\mw\app\ApplicationActImpl.java |
113 |
Avoid excessively long variable names like _componentSequenceNumber |
| 128 |
regenstrief\mw\app\ApplicationActImpl.java |
115 |
Private field '_components' could be made final; it is only initialized in the declaration or constructor. |
| 129 |
regenstrief\mw\app\ApplicationActImpl.java |
117 |
Avoid excessively long variable names like _assimilatedParent |
| 130 |
regenstrief\mw\app\ApplicationActImpl.java |
129 |
Document empty constructor |
| 131 |
regenstrief\mw\app\ApplicationActImpl.java |
129 |
It is a good practice to call super() in a constructor |
| 132 |
regenstrief\mw\app\ApplicationActImpl.java |
142 |
It is a good practice to call super() in a constructor |
| 133 |
regenstrief\mw\app\ApplicationActImpl.java |
151 |
Avoid if (x != y) ..; else ..; |
| 134 |
regenstrief\mw\app\ApplicationActImpl.java |
153 |
Overridable method 'getActiveSelf' called during object construction |
| 135 |
regenstrief\mw\app\ApplicationActImpl.java |
168 |
An empty method in an abstract class should be abstract instead |
| 136 |
regenstrief\mw\app\ApplicationActImpl.java |
174 |
An empty method in an abstract class should be abstract instead |
| 137 |
regenstrief\mw\app\ApplicationActImpl.java |
193 |
An empty method in an abstract class should be abstract instead |
| 138 |
regenstrief\mw\app\ApplicationActImpl.java |
201 |
An empty method in an abstract class should be abstract instead |
| 139 |
regenstrief\mw\app\ApplicationActImpl.java |
207 |
An empty method in an abstract class should be abstract instead |
| 140 |
regenstrief\mw\app\ApplicationActImpl.java |
213 |
An empty method in an abstract class should be abstract instead |
| 141 |
regenstrief\mw\app\ApplicationActImpl.java |
224 |
An empty method in an abstract class should be abstract instead |
| 142 |
regenstrief\mw\app\ApplicationActImpl.java |
236 |
An empty method in an abstract class should be abstract instead |
| 143 |
regenstrief\mw\app\ApplicationActImpl.java |
247 |
An empty method in an abstract class should be abstract instead |
| 144 |
regenstrief\mw\app\ApplicationActImpl.java |
273 |
An empty method in an abstract class should be abstract instead |
| 145 |
regenstrief\mw\app\ApplicationActImpl.java |
290 |
Avoid if (x != y) ..; else ..; |
| 146 |
regenstrief\mw\app\ApplicationActImpl.java |
311 |
Avoid if (x != y) ..; else ..; |
| 147 |
regenstrief\mw\app\ApplicationActImpl.java |
332 |
Avoid if (x != y) ..; else ..; |
| 148 |
regenstrief\mw\app\ApplicationActImpl.java |
353 |
Avoid if (x != y) ..; else ..; |
| 149 |
regenstrief\mw\app\ApplicationActImpl.java |
374 |
Avoid if (x != y) ..; else ..; |
| 150 |
regenstrief\mw\app\ApplicationActImpl.java |
376 |
Avoid if (x != y) ..; else ..; |
| 151 |
regenstrief\mw\app\ApplicationActImpl.java |
412 |
Found 'DU'-anomaly for variable 'event' (lines '412'-'422'). |
| 152 |
regenstrief\mw\app\ApplicationActImpl.java |
441 |
An empty method in an abstract class should be abstract instead |
| 153 |
regenstrief\mw\app\ApplicationActImpl.java |
464 |
Avoid reassigning parameters such as 'authorRole' |
| 154 |
regenstrief\mw\app\ApplicationActImpl.java |
494 |
An empty method in an abstract class should be abstract instead |
| 155 |
regenstrief\mw\app\ApplicationActImpl.java |
517 |
Avoid excessively long variable names like subjectParticipation |
| 156 |
regenstrief\mw\app\ApplicationActImpl.java |
524 |
Avoid excessively long variable names like subjectParticipation |
| 157 |
regenstrief\mw\app\ApplicationActImpl.java |
546 |
An empty method in an abstract class should be abstract instead |
| 158 |
regenstrief\mw\app\ApplicationActImpl.java |
550 |
Avoid excessively long variable names like outboundRelationships |
| 159 |
regenstrief\mw\app\ApplicationActImpl.java |
568 |
Avoid excessively long variable names like subjectActRelationship |
| 160 |
regenstrief\mw\app\ApplicationActImpl.java |
575 |
Avoid excessively long variable names like subjectActRelationship |
| 161 |
regenstrief\mw\app\ApplicationActImpl.java |
597 |
An empty method in an abstract class should be abstract instead |
| 162 |
regenstrief\mw\app\ApplicationActImpl.java |
602 |
Avoid excessively long variable names like COMPONENT_CONSTRUCTOR_ARGUMENTS |
| 163 |
regenstrief\mw\app\ApplicationActImpl.java |
604 |
An empty method in an abstract class should be abstract instead |
| 164 |
regenstrief\mw\app\ApplicationActImpl.java |
606 |
Returning 'COMPONENT_CONSTRUCTOR_ARGUMENTS' may expose an internal array. |
| 165 |
regenstrief\mw\app\ApplicationActImpl.java |
609 |
An empty method in an abstract class should be abstract instead |
| 166 |
regenstrief\mw\app\ApplicationActImpl.java |
614 |
An empty method in an abstract class should be abstract instead |
| 167 |
regenstrief\mw\app\ApplicationActImpl.java |
638 |
An empty method in an abstract class should be abstract instead |
| 168 |
regenstrief\mw\app\ApplicationActImpl.java |
643 |
An empty method in an abstract class should be abstract instead |
| 169 |
regenstrief\mw\app\ApplicationActImpl.java |
648 |
An empty method in an abstract class should be abstract instead |
| 170 |
regenstrief\mw\app\ApplicationActImpl.java |
667 |
Avoid excessively long variable names like authorParticipation |
| 171 |
regenstrief\mw\app\ApplicationActImpl.java |
679 |
Avoid excessively long variable names like componentRelationship |
| 172 |
regenstrief\mw\app\ApplicationActImpl.java |
732 |
An empty method in an abstract class should be abstract instead |
| 173 |
regenstrief\mw\app\ApplicationActImpl.java |
736 |
Avoid variables with short names like n |
| 174 |
regenstrief\mw\app\ApplicationActImpl.java |
736 |
Found 'DD'-anomaly for variable 'n' (lines '736'-'745'). |
| 175 |
regenstrief\mw\app\ApplicationActImpl.java |
745 |
Found 'DD'-anomaly for variable 'n' (lines '745'-'745'). |
| 176 |
regenstrief\mw\app\ApplicationActImpl.java |
755 |
An empty method in an abstract class should be abstract instead |
| 177 |
regenstrief\mw\app\ApplicationActImpl.java |
778 |
An empty method in an abstract class should be abstract instead |
| 178 |
regenstrief\mw\app\ApplicationActImpl.java |
802 |
An empty method in an abstract class should be abstract instead |
| 179 |
regenstrief\mw\app\ApplicationActImpl.java |
804 |
Found 'DD'-anomaly for variable 'parent' (lines '804'-'805'). |
| 180 |
regenstrief\mw\app\ApplicationActImpl.java |
805 |
Avoid assignments in operands |
| 181 |
regenstrief\mw\app\ApplicationActImpl.java |
805 |
Found 'DU'-anomaly for variable 'parent' (lines '805'-'810'). |
| 182 |
regenstrief\mw\app\ApplicationActImpl.java |
819 |
An empty method in an abstract class should be abstract instead |
| 183 |
regenstrief\mw\app\ApplicationActImpl.java |
831 |
Avoid excessively long variable names like outboundRelationship |
| 184 |
regenstrief\mw\app\ApplicationActImpl.java |
842 |
Avoid excessively long variable names like outboundRelationship |
| 185 |
regenstrief\mw\app\ApplicationActImpl.java |
861 |
An empty method in an abstract class should be abstract instead |
| 186 |
regenstrief\mw\app\ApplicationActImpl.java |
873 |
Avoid excessively long variable names like inboundRelationship |
| 187 |
regenstrief\mw\app\ApplicationActImpl.java |
884 |
Avoid excessively long variable names like inboundRelationship |
| 188 |
regenstrief\mw\app\ApplicationActImpl.java |
903 |
An empty method in an abstract class should be abstract instead |
| 189 |
regenstrief\mw\app\ApplicationActImpl.java |
939 |
An empty method in an abstract class should be abstract instead |
| 190 |
regenstrief\mw\app\ApplicationActImpl.java |
944 |
Avoid if (x != y) ..; else ..; |
| 191 |
regenstrief\mw\app\ApplicationActImpl.java |
951 |
Avoid if (x != y) ..; else ..; |
| 192 |
regenstrief\mw\app\ApplicationActImpl.java |
964 |
An empty method in an abstract class should be abstract instead |
| 193 |
regenstrief\mw\app\ApplicationActImpl.java |
973 |
An empty method in an abstract class should be abstract instead |
| 194 |
regenstrief\mw\app\ApplicationActImpl.java |
978 |
Avoid if (x != y) ..; else ..; |
| 195 |
regenstrief\mw\app\ApplicationActImpl.java |
985 |
Avoid if (x != y) ..; else ..; |
| 196 |
regenstrief\mw\app\ApplicationActImpl.java |
1003 |
An empty method in an abstract class should be abstract instead |
| 197 |
regenstrief\mw\app\ApplicationActImpl.java |
1008 |
Avoid if (x != y) ..; else ..; |
| 198 |
regenstrief\mw\app\ApplicationActImpl.java |
1015 |
Avoid if (x != y) ..; else ..; |
| 199 |
regenstrief\mw\app\ApplicationActImpl.java |
1035 |
An empty method in an abstract class should be abstract instead |
| 200 |
regenstrief\mw\app\ApplicationActImpl.java |
1047 |
An empty method in an abstract class should be abstract instead |
| 201 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
37 |
Avoid excessively long variable names like _applicationInstance |
| 202 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
40 |
Avoid excessively long variable names like COMPONENT_CONSTRUCTOR_SIGNATURE |
| 203 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
42 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 204 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
46 |
Document empty constructor |
| 205 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
46 |
It is a good practice to call super() in a constructor |
| 206 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
71 |
Avoid excessively long variable names like applicationInstanceOid |
| 207 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
86 |
Returning 'COMPONENT_CONSTRUCTOR_SIGNATURE' may expose an internal array. |
| 208 |
regenstrief\mw\app\ApplicationRuntimeImpl.java |
107 |
Overriding method merely calls super |
| 209 |
regenstrief\mw\app\CommandImpl.java |
53 |
Abstract classes should be named AbstractXXX |
| 210 |
regenstrief\mw\app\CommandImpl.java |
56 |
Document empty constructor |
| 211 |
regenstrief\mw\app\CommandImpl.java |
56 |
It is a good practice to call super() in a constructor |
| 212 |
regenstrief\mw\app\ExceptionNotification.java |
26 |
Private field '_exception' could be made final; it is only initialized in the declaration or constructor. |
| 213 |
regenstrief\mw\app\ExceptionNotification.java |
28 |
Avoid variables with short names like ex |
| 214 |
regenstrief\mw\app\PatientSessionImpl.java |
45 |
Avoid excessively long variable names like _newDialogViewport |
| 215 |
regenstrief\mw\app\PatientSessionImpl.java |
51 |
Avoid excessively long variable names like COMPONENT_CONSTRUCTOR_SIGNATURE |
| 216 |
regenstrief\mw\app\PatientSessionImpl.java |
55 |
Document empty constructor |
| 217 |
regenstrief\mw\app\PatientSessionImpl.java |
55 |
It is a good practice to call super() in a constructor |
| 218 |
regenstrief\mw\app\PatientSessionImpl.java |
81 |
Returning 'COMPONENT_CONSTRUCTOR_SIGNATURE' may expose an internal array. |
| 219 |
regenstrief\mw\app\PatientSessionImpl.java |
95 |
Deeply nested if..then statements are hard to read |
| 220 |
regenstrief\mw\app\PatientSessionImpl.java |
116 |
Found 'DU'-anomaly for variable 'activeSelf' (lines '116'-'124'). |
| 221 |
regenstrief\mw\app\PatientSessionImpl.java |
119 |
Avoid if (x != y) ..; else ..; |
| 222 |
regenstrief\mw\app\PatientSessionImpl.java |
138 |
Avoid reassigning parameters such as 'patientRole' |
| 223 |
regenstrief\mw\app\SimpleNotificationQueue.java |
34 |
Use explicit scoping instead of the default package private level |
| 224 |
regenstrief\mw\app\SimpleNotificationQueue.java |
77 |
Private field '_msg' could be made final; it is only initialized in the declaration or constructor. |
| 225 |
regenstrief\mw\app\SimpleNotificationQueue.java |
103 |
Private field '_msg' could be made final; it is only initialized in the declaration or constructor. |
| 226 |
regenstrief\mw\app\SimpleViewPort.java |
33 |
Use explicit scoping instead of the default package private level |
| 227 |
regenstrief\mw\app\SimpleViewPort.java |
45 |
A method should have only one exit point, and that should be the last statement in the method |
| 228 |
regenstrief\mw\app\SimpleViewPort.java |
61 |
Avoid empty finally blocks |
| 229 |
regenstrief\mw\app\TaskListController.java |
39 |
Found 'DD'-anomaly for variable 'command' (lines '39'-'43'). |
| 230 |
regenstrief\mw\app\TaskListController.java |
39 |
Found 'DD'-anomaly for variable 'command' (lines '39'-'43'). |
| 231 |
regenstrief\mw\app\UserSessionImpl.java |
43 |
Avoid excessively long variable names like _newDialogViewport |
| 232 |
regenstrief\mw\app\UserSessionImpl.java |
45 |
Avoid excessively long variable names like COMPONENT_CONSTRUCTOR_SIGNATURE |
| 233 |
regenstrief\mw\app\UserSessionImpl.java |
51 |
Document empty constructor |
| 234 |
regenstrief\mw\app\UserSessionImpl.java |
51 |
It is a good practice to call super() in a constructor |
| 235 |
regenstrief\mw\app\UserSessionImpl.java |
79 |
Returning 'COMPONENT_CONSTRUCTOR_SIGNATURE' may expose an internal array. |
| 236 |
regenstrief\mw\app\UserSessionImpl.java |
95 |
Found 'DU'-anomaly for variable 'activeSelf' (lines '95'-'103'). |
| 237 |
regenstrief\mw\app\UserSessionImpl.java |
98 |
Avoid if (x != y) ..; else ..; |
| 238 |
regenstrief\mw\commands\ApplicationExitCommand.java |
34 |
Document empty constructor |
| 239 |
regenstrief\mw\commands\ApplicationExitCommand.java |
34 |
It is a good practice to call super() in a constructor |
| 240 |
regenstrief\mw\commands\ApplicationExitCommand.java |
50 |
Document empty method |
| 241 |
regenstrief\mw\commands\CloseOpenTabsCommand.java |
33 |
Document empty constructor |
| 242 |
regenstrief\mw\commands\CloseOpenTabsCommand.java |
33 |
It is a good practice to call super() in a constructor |
| 243 |
regenstrief\mw\commands\EditActCommand.java |
46 |
Abstract classes should be named AbstractXXX |
| 244 |
regenstrief\mw\commands\EditActCommand.java |
54 |
Document empty constructor |
| 245 |
regenstrief\mw\commands\EditActCommand.java |
54 |
It is a good practice to call super() in a constructor |
| 246 |
regenstrief\mw\commands\EditActCommand.java |
58 |
Avoid excessively long variable names like parentApplicationAct |
| 247 |
regenstrief\mw\commands\EditActCommand.java |
84 |
An empty method in an abstract class should be abstract instead |
| 248 |
regenstrief\mw\commands\EditActCommand.java |
114 |
An empty method in an abstract class should be abstract instead |
| 249 |
regenstrief\mw\commands\EditActCommand.java |
123 |
An empty method in an abstract class should be abstract instead |
| 250 |
regenstrief\mw\commands\EditActCommand.java |
173 |
Avoid if (x != y) ..; else ..; |
| 251 |
regenstrief\mw\commands\EditActCommand.java |
173 |
Use equals() to compare object references. |
| 252 |
regenstrief\mw\commands\EditActCommand.java |
175 |
Avoid if (x != y) ..; else ..; |
| 253 |
regenstrief\mw\commands\EditActCommand.java |
185 |
Avoid using if statements without curly braces |
| 254 |
regenstrief\mw\commands\EditActCommand.java |
189 |
Avoid excessively long variable names like newUpdateRelationship |
| 255 |
regenstrief\mw\commands\EditActCommand.java |
243 |
An empty method in an abstract class should be abstract instead |
| 256 |
regenstrief\mw\commands\EditActCommand.java |
244 |
Document empty method |
| 257 |
regenstrief\mw\commands\EditCommand.java |
41 |
Abstract classes should be named AbstractXXX |
| 258 |
regenstrief\mw\commands\EditCommand.java |
53 |
Document empty constructor |
| 259 |
regenstrief\mw\commands\EditCommand.java |
53 |
It is a good practice to call super() in a constructor |
| 260 |
regenstrief\mw\commands\EditCommand.java |
95 |
An empty method in an abstract class should be abstract instead |
| 261 |
regenstrief\mw\commands\EditCommand.java |
123 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 262 |
regenstrief\mw\commands\EditCommand.java |
141 |
An empty method in an abstract class should be abstract instead |
| 263 |
regenstrief\mw\commands\EditRimObjectCommand.java |
13 |
Abstract classes should be named AbstractXXX |
| 264 |
regenstrief\mw\commands\EditRimObjectCommand.java |
19 |
Avoid excessively long variable names like _modelObjectVariable |
| 265 |
regenstrief\mw\commands\EditRimObjectCommand.java |
21 |
Avoid excessively long variable names like _modelObjectViewArgument |
| 266 |
regenstrief\mw\commands\EditRimObjectCommand.java |
23 |
Avoid excessively long variable names like MODEL_OBJECT_VARIABLE_NAME |
| 267 |
regenstrief\mw\commands\EditRimObjectCommand.java |
27 |
Document empty constructor |
| 268 |
regenstrief\mw\commands\EditRimObjectCommand.java |
27 |
It is a good practice to call super() in a constructor |
| 269 |
regenstrief\mw\commands\EditRimObjectCommand.java |
31 |
Avoid excessively long variable names like parentApplicationAct |
| 270 |
regenstrief\mw\commands\EditRimObjectCommand.java |
36 |
An empty method in an abstract class should be abstract instead |
| 271 |
regenstrief\mw\commands\EditRimObjectCommand.java |
53 |
Avoid if (x != y) ..; else ..; |
| 272 |
regenstrief\mw\commands\EditRimObjectCommand.java |
69 |
An empty method in an abstract class should be abstract instead |
| 273 |
regenstrief\mw\commands\EditRoleCommand.java |
42 |
Abstract classes should be named AbstractXXX |
| 274 |
regenstrief\mw\commands\EditRoleCommand.java |
46 |
Document empty constructor |
| 275 |
regenstrief\mw\commands\EditRoleCommand.java |
46 |
It is a good practice to call super() in a constructor |
| 276 |
regenstrief\mw\commands\EditRoleCommand.java |
50 |
Avoid excessively long variable names like parentApplicationAct |
| 277 |
regenstrief\mw\commands\PrintCommand.java |
50 |
Abstract classes should be named AbstractXXX |
| 278 |
regenstrief\mw\commands\PrintCommand.java |
52 |
Avoid excessively long variable names like PRINT_TRANSFORM_URI |
| 279 |
regenstrief\mw\commands\PrintCommand.java |
54 |
Avoid excessively long variable names like _transformerFactory |
| 280 |
regenstrief\mw\commands\PrintCommand.java |
55 |
The field name indicates a constant but its modifiers do not |
| 281 |
regenstrief\mw\commands\PrintCommand.java |
67 |
Document empty constructor |
| 282 |
regenstrief\mw\commands\PrintCommand.java |
67 |
It is a good practice to call super() in a constructor |
| 283 |
regenstrief\mw\commands\PrintCommand.java |
80 |
An empty method in an abstract class should be abstract instead |
| 284 |
regenstrief\mw\commands\PrintCommand.java |
86 |
An empty method in an abstract class should be abstract instead |
| 285 |
regenstrief\mw\commands\PrintCommand.java |
116 |
An empty method in an abstract class should be abstract instead |
| 286 |
regenstrief\mw\commands\PrintCommand.java |
125 |
An empty method in an abstract class should be abstract instead |
| 287 |
regenstrief\mw\commands\PrintCommand.java |
139 |
An empty method in an abstract class should be abstract instead |
| 288 |
regenstrief\mw\commands\PrintCommand.java |
139 |
Avoid excessively long variable names like applicationContext |
| 289 |
regenstrief\mw\commands\PrintCommand.java |
143 |
Avoid excessively long variable names like str_WithSpeciialChar |
| 290 |
regenstrief\mw\commands\PrintCommand.java |
143 |
Avoid unused local variables such as 'str_WithSpeciialChar'. |
| 291 |
regenstrief\mw\commands\PrintCommand.java |
143 |
Found 'DU'-anomaly for variable 'str_WithSpeciialChar' (lines '143'-'172'). |
| 292 |
regenstrief\mw\controllers\ActCommandController.java |
35 |
Abstract classes should be named AbstractXXX |
| 293 |
regenstrief\mw\controllers\ActCommandController.java |
40 |
An empty method in an abstract class should be abstract instead |
| 294 |
regenstrief\mw\controllers\ActCommandController.java |
45 |
Avoid excessively long variable names like _relationshipViewAdapter |
| 295 |
regenstrief\mw\controllers\ActCommandController.java |
45 |
Private field '_relationshipViewAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 296 |
regenstrief\mw\controllers\ActCommandController.java |
47 |
Avoid excessively long variable names like _targetViewAdapter |
| 297 |
regenstrief\mw\controllers\ActCommandController.java |
47 |
Private field '_targetViewAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 298 |
regenstrief\mw\controllers\ActCommandController.java |
52 |
Overridable method 'makeRelationshipViewAdapter' called during object construction |
| 299 |
regenstrief\mw\controllers\ActCommandController.java |
57 |
An empty method in an abstract class should be abstract instead |
| 300 |
regenstrief\mw\controllers\ActCommandController.java |
59 |
Switch statements should have a default label |
| 301 |
regenstrief\mw\controllers\ActCommandController.java |
62 |
A method should have only one exit point, and that should be the last statement in the method |
| 302 |
regenstrief\mw\controllers\ActCommandController.java |
64 |
A method should have only one exit point, and that should be the last statement in the method |
| 303 |
regenstrief\mw\controllers\ApplicationActionController.java |
31 |
Avoid unused imports such as 'net.sf.xwsf.server.util.ViewManipulator' |
| 304 |
regenstrief\mw\controllers\ApplicationActionController.java |
50 |
The class 'ApplicationActionController' has a Cyclomatic Complexity of 4 (Highest = 18). |
| 305 |
regenstrief\mw\controllers\ApplicationActionController.java |
52 |
Private field '_view' could be made final; it is only initialized in the declaration or constructor. |
| 306 |
regenstrief\mw\controllers\ApplicationActionController.java |
69 |
The method 'getActionFor' has a Cyclomatic Complexity of 18. |
| 307 |
regenstrief\mw\controllers\ApplicationActionController.java |
72 |
Switch statements should have a default label |
| 308 |
regenstrief\mw\controllers\ApplicationActionController.java |
75 |
A method should have only one exit point, and that should be the last statement in the method |
| 309 |
regenstrief\mw\controllers\ApplicationActionController.java |
77 |
A method should have only one exit point, and that should be the last statement in the method |
| 310 |
regenstrief\mw\controllers\ApplicationActionController.java |
79 |
A method should have only one exit point, and that should be the last statement in the method |
| 311 |
regenstrief\mw\controllers\ApplicationActionController.java |
81 |
A method should have only one exit point, and that should be the last statement in the method |
| 312 |
regenstrief\mw\controllers\ApplicationActionController.java |
83 |
A method should have only one exit point, and that should be the last statement in the method |
| 313 |
regenstrief\mw\controllers\ApplicationActionController.java |
85 |
A method should have only one exit point, and that should be the last statement in the method |
| 314 |
regenstrief\mw\controllers\ApplicationActionController.java |
87 |
A method should have only one exit point, and that should be the last statement in the method |
| 315 |
regenstrief\mw\controllers\ApplicationActionController.java |
89 |
A method should have only one exit point, and that should be the last statement in the method |
| 316 |
regenstrief\mw\controllers\ApplicationActionController.java |
91 |
A method should have only one exit point, and that should be the last statement in the method |
| 317 |
regenstrief\mw\controllers\ApplicationActionController.java |
93 |
A method should have only one exit point, and that should be the last statement in the method |
| 318 |
regenstrief\mw\controllers\ApplicationActionController.java |
95 |
A method should have only one exit point, and that should be the last statement in the method |
| 319 |
regenstrief\mw\controllers\ApplicationActionController.java |
98 |
A method should have only one exit point, and that should be the last statement in the method |
| 320 |
regenstrief\mw\controllers\ApplicationActionController.java |
100 |
A method should have only one exit point, and that should be the last statement in the method |
| 321 |
regenstrief\mw\controllers\ApplicationActionController.java |
102 |
A method should have only one exit point, and that should be the last statement in the method |
| 322 |
regenstrief\mw\controllers\ApplicationActionController.java |
104 |
A method should have only one exit point, and that should be the last statement in the method |
| 323 |
regenstrief\mw\controllers\ApplicationActionController.java |
106 |
A method should have only one exit point, and that should be the last statement in the method |
| 324 |
regenstrief\mw\controllers\ApplicationActionController.java |
202 |
Use explicit scoping instead of the default package private level |
| 325 |
regenstrief\mw\controllers\ApplicationActionController.java |
223 |
Found 'DD'-anomaly for variable 'arr' (lines '223'-'224'). |
| 326 |
regenstrief\mw\controllers\ApplicationActionController.java |
252 |
Avoid using final local variables, turn them into fields |
| 327 |
regenstrief\mw\controllers\CommandController.java |
61 |
A method should have only one exit point, and that should be the last statement in the method |
| 328 |
regenstrief\mw\controllers\CommandController.java |
63 |
A method should have only one exit point, and that should be the last statement in the method |
| 329 |
regenstrief\mw\controllers\CommandController.java |
65 |
A method should have only one exit point, and that should be the last statement in the method |
| 330 |
regenstrief\mw\controllers\CommandController.java |
67 |
A method should have only one exit point, and that should be the last statement in the method |
| 331 |
regenstrief\mw\controllers\CommandController.java |
69 |
A method should have only one exit point, and that should be the last statement in the method |
| 332 |
regenstrief\mw\controllers\CommandController.java |
71 |
A method should have only one exit point, and that should be the last statement in the method |
| 333 |
regenstrief\mw\controllers\CommandController.java |
73 |
A method should have only one exit point, and that should be the last statement in the method |
| 334 |
regenstrief\mw\controllers\CommandController.java |
120 |
Avoid using final local variables, turn them into fields |
| 335 |
regenstrief\mw\controllers\CommandController.java |
156 |
Avoid using final local variables, turn them into fields |
| 336 |
regenstrief\mw\controllers\CommandController.java |
156 |
Found 'DU'-anomaly for variable 'history' (lines '156'-'180'). |
| 337 |
regenstrief\mw\controllers\CommandController.java |
159 |
Found 'DU'-anomaly for variable 'command' (lines '159'-'180'). |
| 338 |
regenstrief\mw\controllers\CommandController.java |
208 |
Use explicit scoping instead of the default package private level |
| 339 |
regenstrief\mw\controllers\CommandController.java |
210 |
Avoid variables with short names like o |
| 340 |
regenstrief\mw\controllers\CommandController.java |
219 |
Avoid using final local variables, turn them into fields |
| 341 |
regenstrief\mw\controllers\CommandController.java |
260 |
Use explicit scoping instead of the default package private level |
| 342 |
regenstrief\mw\controllers\CommandController.java |
262 |
Avoid variables with short names like o |
| 343 |
regenstrief\mw\controllers\CommandController.java |
269 |
Avoid using final local variables, turn them into fields |
| 344 |
regenstrief\mw\controllers\CommandController.java |
329 |
Abstract classes should be named AbstractXXX |
| 345 |
regenstrief\mw\controllers\ListActionController.java |
43 |
Abstract classes should be named AbstractXXX |
| 346 |
regenstrief\mw\controllers\ListActionController.java |
45 |
Avoid excessively long variable names like _openApplicationActsOnItems |
| 347 |
regenstrief\mw\controllers\ListActionController.java |
46 |
Avoid excessively long variable names like _openApplicationActs |
| 348 |
regenstrief\mw\controllers\ListActionController.java |
48 |
Avoid unused private fields such as '_dragView'. |
| 349 |
regenstrief\mw\controllers\ListActionController.java |
50 |
Avoid unused private fields such as '_dragObject'. |
| 350 |
regenstrief\mw\controllers\ListActionController.java |
62 |
A method should have only one exit point, and that should be the last statement in the method |
| 351 |
regenstrief\mw\controllers\ListActionController.java |
64 |
A method should have only one exit point, and that should be the last statement in the method |
| 352 |
regenstrief\mw\controllers\ListActionController.java |
66 |
A method should have only one exit point, and that should be the last statement in the method |
| 353 |
regenstrief\mw\controllers\ListActionController.java |
68 |
A method should have only one exit point, and that should be the last statement in the method |
| 354 |
regenstrief\mw\controllers\ListActionController.java |
75 |
An empty method in an abstract class should be abstract instead |
| 355 |
regenstrief\mw\controllers\ListActionController.java |
97 |
Avoid using final local variables, turn them into fields |
| 356 |
regenstrief\mw\controllers\ListActionController.java |
155 |
Avoid using if statements without curly braces |
| 357 |
regenstrief\mw\controllers\ListActionController.java |
196 |
An empty method in an abstract class should be abstract instead |
| 358 |
regenstrief\mw\encounter\AppointmentListController.java |
51 |
A method should have only one exit point, and that should be the last statement in the method |
| 359 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
44 |
The class 'AppointmentViewAdapter' has a Cyclomatic Complexity of 5 (Highest = 20). |
| 360 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
53 |
Avoid unused private fields such as 'EXPR_END_TIME'. |
| 361 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
59 |
Private field '_personAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 362 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
68 |
Use equals() to compare strings instead of '==' or '!=' |
| 363 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
69 |
Use equals() to compare strings instead of '==' or '!=' |
| 364 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
71 |
A method should have only one exit point, and that should be the last statement in the method |
| 365 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
83 |
A method should have only one exit point, and that should be the last statement in the method |
| 366 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
88 |
A method should have only one exit point, and that should be the last statement in the method |
| 367 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
93 |
A method should have only one exit point, and that should be the last statement in the method |
| 368 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
103 |
Avoid really long methods. |
| 369 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
103 |
The method 'setProperty' has a Cyclomatic Complexity of 20. |
| 370 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
103 |
The method setProperty() has an NPath complexity of 7782 |
| 371 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
107 |
Use equals() to compare object references. |
| 372 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
112 |
Avoid excessively long variable names like effectiveTimeisNull |
| 373 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
116 |
A method should have only one exit point, and that should be the last statement in the method |
| 374 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
121 |
Static DateFormatter objects should be accessed in a synchronized manner |
| 375 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
130 |
A method should have only one exit point, and that should be the last statement in the method |
| 376 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
136 |
A method should have only one exit point, and that should be the last statement in the method |
| 377 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
142 |
A method should have only one exit point, and that should be the last statement in the method |
| 378 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
149 |
A method should have only one exit point, and that should be the last statement in the method |
| 379 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
152 |
Static DateFormatter objects should be accessed in a synchronized manner |
| 380 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
160 |
Avoid assignments in operands |
| 381 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
160 |
Found 'DD'-anomaly for variable 'newTime' (lines '160'-'160'). |
| 382 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
160 |
Found 'DU'-anomaly for variable 'newTime' (lines '160'-'205'). |
| 383 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
166 |
A method should have only one exit point, and that should be the last statement in the method |
| 384 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
170 |
A method should have only one exit point, and that should be the last statement in the method |
| 385 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
174 |
Use equals() to compare object references. |
| 386 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
178 |
Avoid excessively long variable names like effectiveTimeisNull |
| 387 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
183 |
Avoid assignments in operands |
| 388 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
183 |
Found 'DU'-anomaly for variable 'effectiveTime' (lines '183'-'205'). |
| 389 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
183 |
Found 'DU'-anomaly for variable 'width' (lines '183'-'205'). |
| 390 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
188 |
A method should have only one exit point, and that should be the last statement in the method |
| 391 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
194 |
A method should have only one exit point, and that should be the last statement in the method |
| 392 |
regenstrief\mw\encounter\AppointmentViewAdapter.java |
198 |
A method should have only one exit point, and that should be the last statement in the method |
| 393 |
regenstrief\mw\encounter\EditAppointmentCommand.java |
54 |
Document empty constructor |
| 394 |
regenstrief\mw\encounter\EditAppointmentCommand.java |
54 |
It is a good practice to call super() in a constructor |
| 395 |
regenstrief\mw\encounter\EditAppointmentCommand.java |
71 |
Avoid unnecessary comparisons in boolean expressions |
| 396 |
regenstrief\mw\encounter\EditAppointmentCommand.java |
71 |
These nested if statements could be combined |
| 397 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
26 |
Document empty constructor |
| 398 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
26 |
It is a good practice to call super() in a constructor |
| 399 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
37 |
A switch statement does not contain a break |
| 400 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
40 |
A method should have only one exit point, and that should be the last statement in the method |
| 401 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
44 |
A method should have only one exit point, and that should be the last statement in the method |
| 402 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
48 |
A method should have only one exit point, and that should be the last statement in the method |
| 403 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
51 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 404 |
regenstrief\mw\medication\ChangeMedicationStatusCommand.java |
52 |
A method should have only one exit point, and that should be the last statement in the method |
| 405 |
regenstrief\mw\medication\CurrentListViewController.java |
51 |
Avoid reassigning parameters such as 'view' |
| 406 |
regenstrief\mw\medication\CurrentListViewController.java |
55 |
Avoid using if statements without curly braces |
| 407 |
regenstrief\mw\medication\CurrentListViewController.java |
55 |
The String literal "item" appears 10 times in this file; the first occurrence is on line 55 |
| 408 |
regenstrief\mw\medication\CurrentListViewController.java |
56 |
These nested if statements could be combined |
| 409 |
regenstrief\mw\medication\CurrentListViewController.java |
62 |
Avoid using if statements without curly braces |
| 410 |
regenstrief\mw\medication\CurrentListViewController.java |
64 |
The String literal "list" appears 11 times in this file; the first occurrence is on line 64 |
| 411 |
regenstrief\mw\medication\CurrentListViewController.java |
66 |
Avoid using if statements without curly braces |
| 412 |
regenstrief\mw\medication\CurrentListViewController.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 413 |
regenstrief\mw\medication\CurrentListViewController.java |
80 |
A method should have only one exit point, and that should be the last statement in the method |
| 414 |
regenstrief\mw\medication\CurrentListViewController.java |
82 |
A method should have only one exit point, and that should be the last statement in the method |
| 415 |
regenstrief\mw\medication\CurrentListViewController.java |
85 |
A method should have only one exit point, and that should be the last statement in the method |
| 416 |
regenstrief\mw\medication\CurrentListViewController.java |
88 |
A method should have only one exit point, and that should be the last statement in the method |
| 417 |
regenstrief\mw\medication\CurrentListViewController.java |
91 |
A method should have only one exit point, and that should be the last statement in the method |
| 418 |
regenstrief\mw\medication\CurrentListViewController.java |
94 |
A method should have only one exit point, and that should be the last statement in the method |
| 419 |
regenstrief\mw\medication\CurrentListViewController.java |
115 |
Found 'DU'-anomaly for variable 'parentTask' (lines '115'-'154'). |
| 420 |
regenstrief\mw\medication\CurrentListViewController.java |
124 |
Found 'DD'-anomaly for variable 'command' (lines '124'-'124'). |
| 421 |
regenstrief\mw\medication\CurrentListViewController.java |
124 |
Found 'DU'-anomaly for variable 'command' (lines '124'-'154'). |
| 422 |
regenstrief\mw\medication\CurrentListViewController.java |
128 |
Found 'DD'-anomaly for variable 'sbadm' (lines '128'-'128'). |
| 423 |
regenstrief\mw\medication\CurrentListViewController.java |
128 |
Found 'DU'-anomaly for variable 'sbadm' (lines '128'-'154'). |
| 424 |
regenstrief\mw\medication\CurrentListViewController.java |
129 |
Found 'DD'-anomaly for variable 'keepgoing' (lines '129'-'135'). |
| 425 |
regenstrief\mw\medication\CurrentListViewController.java |
167 |
Found 'DD'-anomaly for variable 'command' (lines '167'-'168'). |
| 426 |
regenstrief\mw\medication\CurrentListViewController.java |
215 |
Private field '_code' could be made final; it is only initialized in the declaration or constructor. |
| 427 |
regenstrief\mw\medication\CurrentListViewController.java |
235 |
Avoid using if...else statements without curly braces |
| 428 |
regenstrief\mw\medication\CurrentListViewController.java |
253 |
Document empty method |
| 429 |
regenstrief\mw\medication\DoseComposition.java |
22 |
A high number of imports can indicate a high degree of coupling within an object. |
| 430 |
regenstrief\mw\medication\DoseComposition.java |
63 |
The class 'DoseComposition' has a Cyclomatic Complexity of 3 (Highest = 22). |
| 431 |
regenstrief\mw\medication\DoseComposition.java |
63 |
Too many fields |
| 432 |
regenstrief\mw\medication\DoseComposition.java |
65 |
Avoid excessively long variable names like applicationContext |
| 433 |
regenstrief\mw\medication\DoseComposition.java |
86 |
Avoid excessively long variable names like COOL_NUMBER_WITH_SEPERATOR |
| 434 |
regenstrief\mw\medication\DoseComposition.java |
89 |
Avoid excessively long variable names like COOL_NUMBER_WITH_SEPERATOR_AND_UNIT |
| 435 |
regenstrief\mw\medication\DoseComposition.java |
94 |
Avoid excessively long variable names like COOL_NUMBER_AND_PHYSICAL_UNIT |
| 436 |
regenstrief\mw\medication\DoseComposition.java |
97 |
The String literal "\\s(" appears 4 times in this file; the first occurrence is on line 97 |
| 437 |
regenstrief\mw\medication\DoseComposition.java |
100 |
Avoid excessively long variable names like COOL_NUMBER_AND_COUNT_NOUN |
| 438 |
regenstrief\mw\medication\DoseComposition.java |
104 |
Avoid excessively long variable names like COOL_NUMBER_DOSE_UNIT_COUNT_NOUN |
| 439 |
regenstrief\mw\medication\DoseComposition.java |
107 |
Avoid unused private fields such as 'VFACTORY'. |
| 440 |
regenstrief\mw\medication\DoseComposition.java |
117 |
Avoid unused private methods such as 'getCountNouns()'. |
| 441 |
regenstrief\mw\medication\DoseComposition.java |
122 |
Found 'DU'-anomaly for variable 'isFresh' (lines '122'-'142'). |
| 442 |
regenstrief\mw\medication\DoseComposition.java |
127 |
Avoid using if...else statements without curly braces |
| 443 |
regenstrief\mw\medication\DoseComposition.java |
127 |
Found 'DU'-anomaly for variable 'isFresh' (lines '127'-'142'). |
| 444 |
regenstrief\mw\medication\DoseComposition.java |
129 |
Avoid appending characters as strings in StringBuffer.append. |
| 445 |
regenstrief\mw\medication\DoseComposition.java |
129 |
Avoid using if...else statements without curly braces |
| 446 |
regenstrief\mw\medication\DoseComposition.java |
134 |
Avoid using if...else statements without curly braces |
| 447 |
regenstrief\mw\medication\DoseComposition.java |
134 |
Found 'DU'-anomaly for variable 'isFresh' (lines '134'-'142'). |
| 448 |
regenstrief\mw\medication\DoseComposition.java |
136 |
Avoid appending characters as strings in StringBuffer.append. |
| 449 |
regenstrief\mw\medication\DoseComposition.java |
136 |
Avoid using if...else statements without curly braces |
| 450 |
regenstrief\mw\medication\DoseComposition.java |
148 |
Avoid variables with short names like sa |
| 451 |
regenstrief\mw\medication\DoseComposition.java |
148 |
The method 'setObject' has a Cyclomatic Complexity of 22. |
| 452 |
regenstrief\mw\medication\DoseComposition.java |
148 |
The method setObject() has an NPath complexity of 1720 |
| 453 |
regenstrief\mw\medication\DoseComposition.java |
150 |
Found 'DU'-anomaly for variable 'oldValue' (lines '150'-'239'). |
| 454 |
regenstrief\mw\medication\DoseComposition.java |
154 |
These nested if statements could be combined |
| 455 |
regenstrief\mw\medication\DoseComposition.java |
164 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 456 |
regenstrief\mw\medication\DoseComposition.java |
169 |
Found 'DU'-anomaly for variable 'doseQuantity' (lines '169'-'239'). |
| 457 |
regenstrief\mw\medication\DoseComposition.java |
172 |
Found 'DD'-anomaly for variable 'activeIngredient' (lines '172'-'178'). |
| 458 |
regenstrief\mw\medication\DoseComposition.java |
172 |
Found 'DD'-anomaly for variable 'activeIngredient' (lines '172'-'189'). |
| 459 |
regenstrief\mw\medication\DoseComposition.java |
172 |
Found 'DU'-anomaly for variable 'activeIngredient' (lines '172'-'239'). |
| 460 |
regenstrief\mw\medication\DoseComposition.java |
174 |
Found 'DD'-anomaly for variable 'formulation' (lines '174'-'180'). |
| 461 |
regenstrief\mw\medication\DoseComposition.java |
174 |
Found 'DD'-anomaly for variable 'formulation' (lines '174'-'182'). |
| 462 |
regenstrief\mw\medication\DoseComposition.java |
175 |
Avoid excessively long variable names like ingredientQuantity |
| 463 |
regenstrief\mw\medication\DoseComposition.java |
175 |
Found 'DD'-anomaly for variable 'ingredientQuantity' (lines '175'-'179'). |
| 464 |
regenstrief\mw\medication\DoseComposition.java |
175 |
Found 'DD'-anomaly for variable 'ingredientQuantity' (lines '175'-'190'). |
| 465 |
regenstrief\mw\medication\DoseComposition.java |
178 |
Found 'DU'-anomaly for variable 'activeIngredient' (lines '178'-'239'). |
| 466 |
regenstrief\mw\medication\DoseComposition.java |
180 |
Found 'DU'-anomaly for variable 'formulation' (lines '180'-'239'). |
| 467 |
regenstrief\mw\medication\DoseComposition.java |
189 |
Found 'DD'-anomaly for variable 'activeIngredient' (lines '189'-'189'). |
| 468 |
regenstrief\mw\medication\DoseComposition.java |
189 |
Found 'DU'-anomaly for variable 'activeIngredient' (lines '189'-'239'). |
| 469 |
regenstrief\mw\medication\DoseComposition.java |
191 |
Avoid using if statements without curly braces |
| 470 |
regenstrief\mw\medication\DoseComposition.java |
200 |
Avoid empty if statements |
| 471 |
regenstrief\mw\medication\DoseComposition.java |
208 |
Avoid using if statements without curly braces |
| 472 |
regenstrief\mw\medication\DoseComposition.java |
216 |
Avoid using if statements without curly braces |
| 473 |
regenstrief\mw\medication\DoseComposition.java |
241 |
Avoid variables with short names like sa |
| 474 |
regenstrief\mw\medication\DoseComposition.java |
241 |
The method 'format' has a Cyclomatic Complexity of 14. |
| 475 |
regenstrief\mw\medication\DoseComposition.java |
242 |
Avoid using if statements without curly braces |
| 476 |
regenstrief\mw\medication\DoseComposition.java |
243 |
A method should have only one exit point, and that should be the last statement in the method |
| 477 |
regenstrief\mw\medication\DoseComposition.java |
254 |
Avoid using if...else statements without curly braces |
| 478 |
regenstrief\mw\medication\DoseComposition.java |
256 |
Avoid concatenating nonliterals in a StringBuffer constructor or append(). |
| 479 |
regenstrief\mw\medication\DoseComposition.java |
256 |
Avoid using if...else statements without curly braces |
| 480 |
regenstrief\mw\medication\DoseComposition.java |
259 |
Avoid using if...else statements without curly braces |
| 481 |
regenstrief\mw\medication\DoseComposition.java |
260 |
Avoid using if statements without curly braces |
| 482 |
regenstrief\mw\medication\DoseComposition.java |
263 |
Found 'DD'-anomaly for variable 'set' (lines '263'-'265'). |
| 483 |
regenstrief\mw\medication\DoseComposition.java |
270 |
Found 'DD'-anomaly for variable 'noun' (lines '270'-'272'). |
| 484 |
regenstrief\mw\medication\DoseComposition.java |
273 |
Avoid using if statements without curly braces |
| 485 |
regenstrief\mw\medication\DoseComposition.java |
276 |
Avoid appending characters as strings in StringBuffer.append. |
| 486 |
regenstrief\mw\medication\DoseComposition.java |
280 |
Avoid using if...else statements without curly braces |
| 487 |
regenstrief\mw\medication\DoseComposition.java |
282 |
Avoid appending characters as strings in StringBuffer.append. |
| 488 |
regenstrief\mw\medication\DoseComposition.java |
296 |
The field name indicates a constant but its modifiers do not |
| 489 |
regenstrief\mw\medication\DoseComposition.java |
309 |
String.indexOf(char) is faster than String.indexOf(String). |
| 490 |
regenstrief\mw\medication\DoseComposition.java |
313 |
Avoid if (x != y) ..; else ..; |
| 491 |
regenstrief\mw\medication\DoseComposition.java |
313 |
Found 'DD'-anomaly for variable 'value' (lines '313'-'316'). |
| 492 |
regenstrief\mw\medication\DoseComposition.java |
313 |
Found 'DD'-anomaly for variable 'value' (lines '313'-'318'). |
| 493 |
regenstrief\mw\medication\DoseComposition.java |
313 |
Found 'DD'-anomaly for variable 'value' (lines '313'-'320'). |
| 494 |
regenstrief\mw\medication\DoseComposition.java |
315 |
Position literals first in String comparisons |
| 495 |
regenstrief\mw\medication\DoseComposition.java |
316 |
Avoid using if...else statements without curly braces |
| 496 |
regenstrief\mw\medication\DoseComposition.java |
317 |
Position literals first in String comparisons |
| 497 |
regenstrief\mw\medication\DoseComposition.java |
318 |
Avoid using if...else statements without curly braces |
| 498 |
regenstrief\mw\medication\DoseComposition.java |
319 |
Avoid using if statements without curly braces |
| 499 |
regenstrief\mw\medication\DoseComposition.java |
319 |
Position literals first in String comparisons |
| 500 |
regenstrief\mw\medication\DoseComposition.java |
321 |
Avoid excessively long variable names like doseQuantityString |
| 501 |
regenstrief\mw\medication\DoseComposition.java |
336 |
Avoid using if...else statements without curly braces |
| 502 |
regenstrief\mw\medication\DoseComposition.java |
337 |
The String literal "Dose entry has already been entered." appears 4 times in this file; the first occurrence is on line 337 |
| 503 |
regenstrief\mw\medication\DoseComposition.java |
367 |
Avoid using if...else statements without curly braces |
| 504 |
regenstrief\mw\medication\DoseComposition.java |
381 |
This final field could be made static |
| 505 |
regenstrief\mw\medication\DoseComposition.java |
387 |
Found 'DU'-anomaly for variable 'quantity' (lines '387'-'441'). |
| 506 |
regenstrief\mw\medication\DoseComposition.java |
388 |
Found 'DU'-anomaly for variable 'count_noun' (lines '388'-'441'). |
| 507 |
regenstrief\mw\medication\DoseComposition.java |
392 |
The String literal "2.16.840.1.113883.3.26.1.1" appears 4 times in this file; the first occurrence is on line 392 |
| 508 |
regenstrief\mw\medication\DoseComposition.java |
392 |
The String literal "C42998" appears 4 times in this file; the first occurrence is on line 392 |
| 509 |
regenstrief\mw\medication\DoseComposition.java |
408 |
Found 'DU'-anomaly for variable 'low' (lines '408'-'441'). |
| 510 |
regenstrief\mw\medication\DoseComposition.java |
409 |
Found 'DU'-anomaly for variable 'high' (lines '409'-'441'). |
| 511 |
regenstrief\mw\medication\DoseComposition.java |
410 |
Found 'DU'-anomaly for variable 'count_noun' (lines '410'-'441'). |
| 512 |
regenstrief\mw\medication\DoseComposition.java |
453 |
This final field could be made static |
| 513 |
regenstrief\mw\medication\DoseComposition.java |
459 |
Avoid if (x != y) ..; else ..; |
| 514 |
regenstrief\mw\medication\DoseComposition.java |
475 |
Avoid excessively long variable names like doseQuantityString |
| 515 |
regenstrief\mw\medication\DoseComposition.java |
497 |
Avoid using if...else statements without curly braces |
| 516 |
regenstrief\mw\medication\DoseComposition.java |
511 |
This final field could be made static |
| 517 |
regenstrief\mw\medication\DoseComposition.java |
520 |
Avoid if (x != y) ..; else ..; |
| 518 |
regenstrief\mw\medication\DoseComposition.java |
528 |
Avoid if (x != y) ..; else ..; |
| 519 |
regenstrief\mw\medication\DoseComposition.java |
562 |
Avoid excessively long variable names like doseQuantityString |
| 520 |
regenstrief\mw\medication\DoseComposition.java |
584 |
Avoid using if...else statements without curly braces |
| 521 |
regenstrief\mw\medication\DoseComposition.java |
596 |
This final field could be made static |
| 522 |
regenstrief\mw\medication\DoseComposition.java |
605 |
Avoid if (x != y) ..; else ..; |
| 523 |
regenstrief\mw\medication\DoseComposition.java |
614 |
Avoid if (x != y) ..; else ..; |
| 524 |
regenstrief\mw\medication\DoseComposition.java |
648 |
Avoid excessively long variable names like doseQuantityString |
| 525 |
regenstrief\mw\medication\DoseComposition.java |
670 |
Avoid using if...else statements without curly braces |
| 526 |
regenstrief\mw\medication\DoseComposition.java |
683 |
Avoid unused private methods such as 'makePQ(String,String)'. |
| 527 |
regenstrief\mw\medication\DoseComposition.java |
687 |
Avoid unused private methods such as 'makeGenericIVL(PQ,PQ)'. |
| 528 |
regenstrief\mw\medication\DoseComposition.java |
692 |
Avoid unused private methods such as 'makeTN(String)'. |
| 529 |
regenstrief\mw\medication\DoseComposition.java |
692 |
Avoid variables with short names like tn |
| 530 |
regenstrief\mw\medication\DoseComposition.java |
697 |
Avoid using if statements without curly braces |
| 531 |
regenstrief\mw\medication\DoseComposition.java |
698 |
Avoid using if statements without curly braces |
| 532 |
regenstrief\mw\medication\DoseComposition.java |
698 |
These nested if statements could be combined |
| 533 |
regenstrief\mw\medication\DoseComposition.java |
699 |
A method should have only one exit point, and that should be the last statement in the method |
| 534 |
regenstrief\mw\medication\DoseComposition.java |
711 |
Avoid unused private methods such as 'getStrength(SubstanceAdministration)'. |
| 535 |
regenstrief\mw\medication\DoseComposition.java |
711 |
Avoid variables with short names like sa |
| 536 |
regenstrief\mw\medication\DoseComposition.java |
712 |
Avoid variables with short names like ps |
| 537 |
regenstrief\mw\medication\DoseComposition.java |
713 |
Avoid using if statements without curly braces |
| 538 |
regenstrief\mw\medication\DoseComposition.java |
715 |
Avoid variables with short names like e |
| 539 |
regenstrief\mw\medication\DoseComposition.java |
721 |
Avoid using for statements without curly braces |
| 540 |
regenstrief\mw\medication\DoseComposition.java |
722 |
Avoid using if statements without curly braces |
| 541 |
regenstrief\mw\medication\DoseComposition.java |
723 |
Avoid using if statements without curly braces |
| 542 |
regenstrief\mw\medication\DoseComposition.java |
723 |
These nested if statements could be combined |
| 543 |
regenstrief\mw\medication\DoseComposition.java |
724 |
A method should have only one exit point, and that should be the last statement in the method |
| 544 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
51 |
Avoid excessively long variable names like _problem_node_adapter |
| 545 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
51 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 546 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
61 |
Document empty constructor |
| 547 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
61 |
It is a good practice to call super() in a constructor |
| 548 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
75 |
Avoid if (x != y) ..; else ..; |
| 549 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
87 |
Avoid reassigning parameters such as 'indicationNode' |
| 550 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
90 |
Avoid excessively long variable names like parentArgumentCopy |
| 551 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
91 |
Found 'DD'-anomaly for variable 'needRelationship' (lines '91'-'100'). |
| 552 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
118 |
Found 'DU'-anomaly for variable 'composition' (lines '118'-'132'). |
| 553 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
126 |
A method should have only one exit point, and that should be the last statement in the method |
| 554 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
128 |
A method should have only one exit point, and that should be the last statement in the method |
| 555 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
148 |
Avoid reassigning parameters such as 'modelObject' |
| 556 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
160 |
A method should have only one exit point, and that should be the last statement in the method |
| 557 |
regenstrief\mw\medication\EditSubstanceAdministrationCommand.java |
165 |
A method should have only one exit point, and that should be the last statement in the method |
| 558 |
regenstrief\mw\medication\HistoryListViewController.java |
67 |
Document empty method |
| 559 |
regenstrief\mw\medication\ObservationReasonComposition.java |
54 |
Avoid excessively long variable names like _observationAdapter |
| 560 |
regenstrief\mw\medication\ObservationReasonComposition.java |
56 |
Avoid excessively long variable names like _problemNodeAdapter |
| 561 |
regenstrief\mw\medication\ObservationReasonComposition.java |
59 |
Avoid excessively long variable names like applicationContext |
| 562 |
regenstrief\mw\medication\ObservationReasonComposition.java |
67 |
Private field 'nonClearables' could be made final; it is only initialized in the declaration or constructor. |
| 563 |
regenstrief\mw\medication\ObservationReasonComposition.java |
77 |
Avoid variables with short names like sa |
| 564 |
regenstrief\mw\medication\ObservationReasonComposition.java |
79 |
A method should have only one exit point, and that should be the last statement in the method |
| 565 |
regenstrief\mw\medication\ObservationReasonComposition.java |
79 |
Avoid using if statements without curly braces |
| 566 |
regenstrief\mw\medication\ObservationReasonComposition.java |
83 |
A method should have only one exit point, and that should be the last statement in the method |
| 567 |
regenstrief\mw\medication\ObservationReasonComposition.java |
83 |
Avoid using if statements without curly braces |
| 568 |
regenstrief\mw\medication\ObservationReasonComposition.java |
88 |
Avoid using if statements without curly braces |
| 569 |
regenstrief\mw\medication\ObservationReasonComposition.java |
98 |
Avoid if (x != y) ..; else ..; |
| 570 |
regenstrief\mw\medication\ObservationReasonComposition.java |
99 |
Avoid using if...else statements without curly braces |
| 571 |
regenstrief\mw\medication\ObservationReasonComposition.java |
101 |
Avoid using if...else statements without curly braces |
| 572 |
regenstrief\mw\medication\ObservationReasonComposition.java |
104 |
Avoid using for statements without curly braces |
| 573 |
regenstrief\mw\medication\ObservationReasonComposition.java |
116 |
Avoid if (x != y) ..; else ..; |
| 574 |
regenstrief\mw\medication\ObservationReasonComposition.java |
117 |
A method should have only one exit point, and that should be the last statement in the method |
| 575 |
regenstrief\mw\medication\ObservationReasonComposition.java |
117 |
Avoid using if...else statements without curly braces |
| 576 |
regenstrief\mw\medication\ObservationReasonComposition.java |
121 |
Avoid using if...else statements without curly braces |
| 577 |
regenstrief\mw\medication\ObservationReasonComposition.java |
128 |
A method should have only one exit point, and that should be the last statement in the method |
| 578 |
regenstrief\mw\medication\ObservationReasonComposition.java |
128 |
Avoid using if...else statements without curly braces |
| 579 |
regenstrief\mw\medication\ObservationReasonComposition.java |
130 |
Avoid using if...else statements without curly braces |
| 580 |
regenstrief\mw\medication\ObservationReasonComposition.java |
139 |
This final field could be made static |
| 581 |
regenstrief\mw\medication\ObservationReasonComposition.java |
140 |
This final field could be made static |
| 582 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
3 |
Avoid unused imports such as 'java.io.OutputStream' |
| 583 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
4 |
Avoid unused imports such as 'java.io.OutputStreamWriter' |
| 584 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
9 |
Avoid unused imports such as 'java.io.StringWriter' |
| 585 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
20 |
Avoid unused imports such as 'com.sun.org.apache.xalan.internal.xsltc.trax.OutputSettings' |
| 586 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
26 |
Avoid excessively long variable names like _applicationContext |
| 587 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
28 |
Avoid unused private fields such as '_act'. |
| 588 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
28 |
Perhaps '_act' could be replaced by a local variable. |
| 589 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
33 |
Document empty constructor |
| 590 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
33 |
It is a good practice to call super() in a constructor |
| 591 |
regenstrief\mw\medication\PrintMedicationListCommand.java |
52 |
Avoid excessively long variable names like applicationContext |
| 592 |
regenstrief\mw\medication\RouteComposition.java |
58 |
Perhaps 'action' could be replaced by a local variable. |
| 593 |
regenstrief\mw\medication\RouteComposition.java |
60 |
Avoid excessively long variable names like applicationContext |
| 594 |
regenstrief\mw\medication\RouteComposition.java |
68 |
Private field 'clearables' could be made final; it is only initialized in the declaration or constructor. |
| 595 |
regenstrief\mw\medication\RouteComposition.java |
82 |
Avoid variables with short names like sa |
| 596 |
regenstrief\mw\medication\RouteComposition.java |
83 |
Avoid using if statements without curly braces |
| 597 |
regenstrief\mw\medication\RouteComposition.java |
84 |
A method should have only one exit point, and that should be the last statement in the method |
| 598 |
regenstrief\mw\medication\RouteComposition.java |
88 |
Use equals() to compare strings instead of '==' or '!=' |
| 599 |
regenstrief\mw\medication\RouteComposition.java |
89 |
A method should have only one exit point, and that should be the last statement in the method |
| 600 |
regenstrief\mw\medication\RouteComposition.java |
89 |
Avoid using if...else statements without curly braces |
| 601 |
regenstrief\mw\medication\RouteComposition.java |
91 |
Found 'DD'-anomaly for variable 'displayTxt' (lines '91'-'93'). |
| 602 |
regenstrief\mw\medication\RouteComposition.java |
91 |
Found 'DD'-anomaly for variable 'displayTxt' (lines '91'-'95'). |
| 603 |
regenstrief\mw\medication\RouteComposition.java |
91 |
Found 'DU'-anomaly for variable 'displayTxt' (lines '91'-'108'). |
| 604 |
regenstrief\mw\medication\RouteComposition.java |
92 |
A method should have only one exit point, and that should be the last statement in the method |
| 605 |
regenstrief\mw\medication\RouteComposition.java |
92 |
Avoid using if...else statements without curly braces |
| 606 |
regenstrief\mw\medication\RouteComposition.java |
93 |
Avoid using if...else statements without curly braces |
| 607 |
regenstrief\mw\medication\RouteComposition.java |
95 |
Avoid using if...else statements without curly braces |
| 608 |
regenstrief\mw\medication\RouteComposition.java |
97 |
String.indexOf(char) is faster than String.indexOf(String). |
| 609 |
regenstrief\mw\medication\RouteComposition.java |
99 |
A method should have only one exit point, and that should be the last statement in the method |
| 610 |
regenstrief\mw\medication\RouteComposition.java |
100 |
Avoid empty if statements |
| 611 |
regenstrief\mw\medication\RouteComposition.java |
102 |
A method should have only one exit point, and that should be the last statement in the method |
| 612 |
regenstrief\mw\medication\RouteComposition.java |
105 |
A method should have only one exit point, and that should be the last statement in the method |
| 613 |
regenstrief\mw\medication\RouteComposition.java |
105 |
Avoid using if...else statements without curly braces |
| 614 |
regenstrief\mw\medication\RouteComposition.java |
107 |
Avoid using if...else statements without curly braces |
| 615 |
regenstrief\mw\medication\RouteComposition.java |
114 |
Avoid variables with short names like sa |
| 616 |
regenstrief\mw\medication\RouteComposition.java |
123 |
The field name indicates a constant but its modifiers do not |
| 617 |
regenstrief\mw\medication\RouteComposition.java |
124 |
This final field could be made static |
| 618 |
regenstrief\mw\medication\RouteComposition.java |
130 |
Found 'DD'-anomaly for variable 'actionTxt' (lines '130'-'133'). |
| 619 |
regenstrief\mw\medication\RouteComposition.java |
130 |
Found 'DU'-anomaly for variable 'actionTxt' (lines '130'-'167'). |
| 620 |
regenstrief\mw\medication\RouteComposition.java |
133 |
Found 'DU'-anomaly for variable 'actionTxt' (lines '133'-'167'). |
| 621 |
regenstrief\mw\medication\RouteComposition.java |
138 |
Avoid unused local variables such as 'action'. |
| 622 |
regenstrief\mw\medication\RouteComposition.java |
138 |
Found 'DU'-anomaly for variable 'action' (lines '138'-'167'). |
| 623 |
regenstrief\mw\medication\RouteComposition.java |
139 |
Found 'DD'-anomaly for variable 'routeCode' (lines '139'-'141'). |
| 624 |
regenstrief\mw\medication\RouteComposition.java |
139 |
Found 'DD'-anomaly for variable 'routeCode' (lines '139'-'150'). |
| 625 |
regenstrief\mw\medication\RouteComposition.java |
140 |
Avoid if (x != y) ..; else ..; |
| 626 |
regenstrief\mw\medication\RouteComposition.java |
157 |
Avoid if (x != y) ..; else ..; |
| 627 |
regenstrief\mw\medication\RouteComposition.java |
158 |
Avoid using if...else statements without curly braces |
| 628 |
regenstrief\mw\medication\RouteComposition.java |
160 |
Avoid using if...else statements without curly braces |
| 629 |
regenstrief\mw\medication\RouteComposition.java |
165 |
Avoid using if...else statements without curly braces |
| 630 |
regenstrief\mw\medication\RouteComposition.java |
174 |
Found 'DU'-anomaly for variable 'isFresh' (lines '174'-'196'). |
| 631 |
regenstrief\mw\medication\RouteComposition.java |
179 |
Avoid using if...else statements without curly braces |
| 632 |
regenstrief\mw\medication\RouteComposition.java |
179 |
Found 'DU'-anomaly for variable 'isFresh' (lines '179'-'196'). |
| 633 |
regenstrief\mw\medication\RouteComposition.java |
181 |
Avoid appending characters as strings in StringBuffer.append. |
| 634 |
regenstrief\mw\medication\RouteComposition.java |
181 |
Avoid using if...else statements without curly braces |
| 635 |
regenstrief\mw\medication\RouteComposition.java |
186 |
Avoid using if...else statements without curly braces |
| 636 |
regenstrief\mw\medication\RouteComposition.java |
186 |
Found 'DU'-anomaly for variable 'isFresh' (lines '186'-'196'). |
| 637 |
regenstrief\mw\medication\RouteComposition.java |
188 |
Avoid appending characters as strings in StringBuffer.append. |
| 638 |
regenstrief\mw\medication\RouteComposition.java |
188 |
Avoid using if...else statements without curly braces |
| 639 |
regenstrief\mw\medication\RouteComposition.java |
203 |
This final field could be made static |
| 640 |
regenstrief\mw\medication\RouteComposition.java |
209 |
Avoid using if statements without curly braces |
| 641 |
regenstrief\mw\medication\RouteComposition.java |
223 |
The class 'ActionWord' has a Cyclomatic Complexity of 3 (Highest = 12). |
| 642 |
regenstrief\mw\medication\RouteComposition.java |
226 |
Avoid excessively long variable names like applicationContext |
| 643 |
regenstrief\mw\medication\RouteComposition.java |
230 |
Avoid variables with short names like sa |
| 644 |
regenstrief\mw\medication\RouteComposition.java |
230 |
The method 'format' has a Cyclomatic Complexity of 12. |
| 645 |
regenstrief\mw\medication\RouteComposition.java |
231 |
Avoid using if statements without curly braces |
| 646 |
regenstrief\mw\medication\RouteComposition.java |
232 |
A method should have only one exit point, and that should be the last statement in the method |
| 647 |
regenstrief\mw\medication\RouteComposition.java |
235 |
These nested if statements could be combined |
| 648 |
regenstrief\mw\medication\RouteComposition.java |
237 |
Use equals() to compare strings instead of '==' or '!=' |
| 649 |
regenstrief\mw\medication\RouteComposition.java |
238 |
A method should have only one exit point, and that should be the last statement in the method |
| 650 |
regenstrief\mw\medication\RouteComposition.java |
238 |
Avoid using if...else statements without curly braces |
| 651 |
regenstrief\mw\medication\RouteComposition.java |
240 |
Found 'DD'-anomaly for variable 'displayTxt' (lines '240'-'242'). |
| 652 |
regenstrief\mw\medication\RouteComposition.java |
240 |
Found 'DD'-anomaly for variable 'displayTxt' (lines '240'-'244'). |
| 653 |
regenstrief\mw\medication\RouteComposition.java |
242 |
Avoid using if...else statements without curly braces |
| 654 |
regenstrief\mw\medication\RouteComposition.java |
244 |
Avoid using if...else statements without curly braces |
| 655 |
regenstrief\mw\medication\RouteComposition.java |
247 |
String.indexOf(char) is faster than String.indexOf(String). |
| 656 |
regenstrief\mw\medication\RouteComposition.java |
248 |
A method should have only one exit point, and that should be the last statement in the method |
| 657 |
regenstrief\mw\medication\RouteComposition.java |
256 |
Use equals() to compare strings instead of '==' or '!=' |
| 658 |
regenstrief\mw\medication\RouteComposition.java |
257 |
A method should have only one exit point, and that should be the last statement in the method |
| 659 |
regenstrief\mw\medication\RouteComposition.java |
257 |
Avoid using if...else statements without curly braces |
| 660 |
regenstrief\mw\medication\RouteComposition.java |
259 |
Found 'DD'-anomaly for variable 'displayTxt' (lines '259'-'261'). |
| 661 |
regenstrief\mw\medication\RouteComposition.java |
259 |
Found 'DD'-anomaly for variable 'displayTxt' (lines '259'-'263'). |
| 662 |
regenstrief\mw\medication\RouteComposition.java |
261 |
Avoid using if...else statements without curly braces |
| 663 |
regenstrief\mw\medication\RouteComposition.java |
263 |
Avoid using if...else statements without curly braces |
| 664 |
regenstrief\mw\medication\RouteComposition.java |
266 |
String.indexOf(char) is faster than String.indexOf(String). |
| 665 |
regenstrief\mw\medication\RouteComposition.java |
267 |
A method should have only one exit point, and that should be the last statement in the method |
| 666 |
regenstrief\mw\medication\RouteComposition.java |
269 |
A method should have only one exit point, and that should be the last statement in the method |
| 667 |
regenstrief\mw\medication\RouteComposition.java |
272 |
A method should have only one exit point, and that should be the last statement in the method |
| 668 |
regenstrief\mw\medication\RouteComposition.java |
272 |
Avoid using if...else statements without curly braces |
| 669 |
regenstrief\mw\medication\RouteComposition.java |
294 |
Avoid variables with short names like sa |
| 670 |
regenstrief\mw\medication\RouteComposition.java |
295 |
Document empty method |
| 671 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
33 |
Avoid excessively long variable names like MEDICATION_DISPLAY_NAME |
| 672 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
35 |
Avoid excessively long variable names like _substanceAdministrationAdapter |
| 673 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
35 |
Private field '_substanceAdministrationAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 674 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
44 |
Avoid reassigning parameters such as 'expressionString' |
| 675 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
48 |
Use equals() to compare object references. |
| 676 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
50 |
A method should have only one exit point, and that should be the last statement in the method |
| 677 |
regenstrief\mw\medication\SubstanceAdministrationSubjectViewAdapter.java |
62 |
Avoid excessively long variable names like newObservationComponentRelationship |
| 678 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
24 |
Avoid unused imports such as 'java.util.ArrayList' |
| 679 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
25 |
Avoid unused imports such as 'java.util.Collection' |
| 680 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
32 |
Avoid unused imports such as 'org.hl7.hpath.EvaluationIterator' |
| 681 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
35 |
Avoid unused imports such as 'org.hl7.rim.Entity' |
| 682 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
39 |
Avoid unused imports such as 'org.hl7.rim.impl.ParticipationImpl' |
| 683 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
41 |
Avoid unused imports such as 'org.hl7.types.ST' |
| 684 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
42 |
Avoid unused imports such as 'org.hl7.types.ValueFactory' |
| 685 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
46 |
Avoid unused imports such as 'org.hl7.types.enums.RoleClass' |
| 686 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
47 |
Avoid unused imports such as 'org.hl7.types.impl.RTOnull' |
| 687 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
50 |
Avoid unused imports such as 'org.regenstrief.mw.medication.formulation.FormulationRoleViewAdapter' |
| 688 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
54 |
The class 'SubstanceAdministrationViewAdapter' has a Cyclomatic Complexity of 5 (Highest = 17). |
| 689 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
57 |
Avoid excessively long variable names like EXPR_CONSUMABLE_NAME |
| 690 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
63 |
Avoid excessively long variable names like EXPR_FORMULATION_NAME |
| 691 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
71 |
Avoid excessively long variable names like _supplyListAdapter |
| 692 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
73 |
Avoid excessively long variable names like _participationAdapter |
| 693 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
75 |
Avoid excessively long variable names like EXPR_PARTICIPATIONS_TO_CLONE |
| 694 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
75 |
This final field could be made static |
| 695 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
76 |
Avoid excessively long variable names like PARTICIPATIONS_TO_CLONE |
| 696 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
78 |
Avoid excessively long variable names like EXPR_OUTBOUND_RELATIONSHIPS_TO_CLONE |
| 697 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
78 |
This final field could be made static |
| 698 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
79 |
Avoid excessively long variable names like OUTBOUND_RELATIONSHIPS_TO_CLONE |
| 699 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
81 |
Avoid excessively long variable names like EXPR_INBOUND_RELATIONSHIPS_TO_CLONE |
| 700 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
81 |
This final field could be made static |
| 701 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
82 |
Avoid excessively long variable names like INBOUND_RELATIONSHIPS_TO_CLONE |
| 702 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
91 |
The method 'getProperty' has a Cyclomatic Complexity of 17. |
| 703 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
93 |
Found 'DU'-anomaly for variable 'original' (lines '93'-'144'). |
| 704 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
94 |
Use equals() to compare object references. |
| 705 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
96 |
Avoid using if statements without curly braces |
| 706 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
98 |
Use equals() to compare object references. |
| 707 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
102 |
A method should have only one exit point, and that should be the last statement in the method |
| 708 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
102 |
Avoid using if statements without curly braces |
| 709 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
102 |
Position literals first in String comparisons |
| 710 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
107 |
A method should have only one exit point, and that should be the last statement in the method |
| 711 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
109 |
Use equals() to compare object references. |
| 712 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
115 |
A method should have only one exit point, and that should be the last statement in the method |
| 713 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
117 |
Use equals() to compare strings instead of '==' or '!=' |
| 714 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
119 |
A method should have only one exit point, and that should be the last statement in the method |
| 715 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
119 |
Avoid unnecessary if..then..else statements when returning a boolean |
| 716 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
119 |
Avoid using if...else statements without curly braces |
| 717 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
121 |
A method should have only one exit point, and that should be the last statement in the method |
| 718 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
121 |
Avoid using if...else statements without curly braces |
| 719 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
123 |
Use equals() to compare strings instead of '==' or '!=' |
| 720 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
125 |
A method should have only one exit point, and that should be the last statement in the method |
| 721 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
125 |
Avoid unnecessary if..then..else statements when returning a boolean |
| 722 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
125 |
Avoid using if...else statements without curly braces |
| 723 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
127 |
A method should have only one exit point, and that should be the last statement in the method |
| 724 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
127 |
Avoid using if...else statements without curly braces |
| 725 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
129 |
Use equals() to compare strings instead of '==' or '!=' |
| 726 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
131 |
A method should have only one exit point, and that should be the last statement in the method |
| 727 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
131 |
Avoid unnecessary if..then..else statements when returning a boolean |
| 728 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
131 |
Avoid using if...else statements without curly braces |
| 729 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
133 |
A method should have only one exit point, and that should be the last statement in the method |
| 730 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
133 |
Avoid using if...else statements without curly braces |
| 731 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
135 |
Use equals() to compare strings instead of '==' or '!=' |
| 732 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
137 |
A method should have only one exit point, and that should be the last statement in the method |
| 733 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
137 |
Avoid unnecessary if..then..else statements when returning a boolean |
| 734 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
137 |
Avoid using if...else statements without curly braces |
| 735 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
139 |
A method should have only one exit point, and that should be the last statement in the method |
| 736 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
139 |
Avoid using if...else statements without curly braces |
| 737 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
146 |
Avoid reassigning parameters such as 'hpathExpression' |
| 738 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
149 |
Avoid unused local variables such as 'original'. |
| 739 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
149 |
Found 'DU'-anomaly for variable 'original' (lines '149'-'174'). |
| 740 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
150 |
Use equals() to compare strings instead of '==' or '!=' |
| 741 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
154 |
A method should have only one exit point, and that should be the last statement in the method |
| 742 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
156 |
Use equals() to compare strings instead of '==' or '!=' |
| 743 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
165 |
A method should have only one exit point, and that should be the last statement in the method |
| 744 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
167 |
Use equals() to compare strings instead of '==' or '!=' |
| 745 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
170 |
A method should have only one exit point, and that should be the last statement in the method |
| 746 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
173 |
Avoid using if...else statements without curly braces |
| 747 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
190 |
Avoid excessively long variable names like originalModelObject |
| 748 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
196 |
Avoid excessively long variable names like clonedParticipation |
| 749 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
204 |
Avoid excessively long variable names like newActRelationship |
| 750 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
212 |
Avoid excessively long variable names like newActRelationship |
| 751 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
226 |
Avoid unused private methods such as 'clearParticipations(T)'. |
| 752 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
228 |
Avoid using if statements without curly braces |
| 753 |
regenstrief\mw\medication\SubstanceAdministrationViewAdapter.java |
232 |
Avoid using if statements without curly braces |
| 754 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
34 |
Avoid unused imports such as 'org.hl7.rim.Observation' |
| 755 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
53 |
Avoid excessively long variable names like SUPPLY_LIST_ADAPTER |
| 756 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
53 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 757 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
53 |
The field name indicates a constant but its modifiers do not |
| 758 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
55 |
Avoid excessively long variable names like PROBLEM_NODE_ADAPTER |
| 759 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
55 |
Avoid unused private fields such as 'PROBLEM_NODE_ADAPTER'. |
| 760 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
55 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 761 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
55 |
The field name indicates a constant but its modifiers do not |
| 762 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
72 |
A method should have only one exit point, and that should be the last statement in the method |
| 763 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
74 |
A method should have only one exit point, and that should be the last statement in the method |
| 764 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
76 |
A method should have only one exit point, and that should be the last statement in the method |
| 765 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 766 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
166 |
Avoid excessively long variable names like _transitionListener |
| 767 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
215 |
Document empty method |
| 768 |
regenstrief\mw\medication\SubstanceAdministrationViewController.java |
220 |
Document empty method |
| 769 |
regenstrief\mw\medication\TimingComposition.java |
22 |
A high number of imports can indicate a high degree of coupling within an object. |
| 770 |
regenstrief\mw\medication\TimingComposition.java |
58 |
The class 'TimingComposition' has a Cyclomatic Complexity of 2 (Highest = 11). |
| 771 |
regenstrief\mw\medication\TimingComposition.java |
58 |
Too many fields |
| 772 |
regenstrief\mw\medication\TimingComposition.java |
64 |
Avoid excessively long variable names like COMPONENTS_TO_VALUES |
| 773 |
regenstrief\mw\medication\TimingComposition.java |
66 |
Avoid excessively long variable names like VALUES_TO_LITERALS |
| 774 |
regenstrief\mw\medication\TimingComposition.java |
81 |
Private field 'clearables' could be made final; it is only initialized in the declaration or constructor. |
| 775 |
regenstrief\mw\medication\TimingComposition.java |
91 |
Avoid excessively long variable names like applicationContext |
| 776 |
regenstrief\mw\medication\TimingComposition.java |
98 |
Avoid if (x != y) ..; else ..; |
| 777 |
regenstrief\mw\medication\TimingComposition.java |
99 |
Avoid if (x != y) ..; else ..; |
| 778 |
regenstrief\mw\medication\TimingComposition.java |
99 |
Avoid using if...else statements without curly braces |
| 779 |
regenstrief\mw\medication\TimingComposition.java |
102 |
Avoid using if...else statements without curly braces |
| 780 |
regenstrief\mw\medication\TimingComposition.java |
105 |
Avoid using if...else statements without curly braces |
| 781 |
regenstrief\mw\medication\TimingComposition.java |
123 |
The method 'format' has a Cyclomatic Complexity of 11. |
| 782 |
regenstrief\mw\medication\TimingComposition.java |
124 |
A method should have only one exit point, and that should be the last statement in the method |
| 783 |
regenstrief\mw\medication\TimingComposition.java |
124 |
Avoid using if statements without curly braces |
| 784 |
regenstrief\mw\medication\TimingComposition.java |
130 |
A method should have only one exit point, and that should be the last statement in the method |
| 785 |
regenstrief\mw\medication\TimingComposition.java |
130 |
Avoid using if statements without curly braces |
| 786 |
regenstrief\mw\medication\TimingComposition.java |
135 |
Avoid concatenating nonliterals in a StringBuffer constructor or append(). |
| 787 |
regenstrief\mw\medication\TimingComposition.java |
135 |
Avoid using if...else statements without curly braces |
| 788 |
regenstrief\mw\medication\TimingComposition.java |
136 |
Avoid using if statements without curly braces |
| 789 |
regenstrief\mw\medication\TimingComposition.java |
138 |
A method should have only one exit point, and that should be the last statement in the method |
| 790 |
regenstrief\mw\medication\TimingComposition.java |
141 |
A method should have only one exit point, and that should be the last statement in the method |
| 791 |
regenstrief\mw\medication\TimingComposition.java |
144 |
A method should have only one exit point, and that should be the last statement in the method |
| 792 |
regenstrief\mw\medication\TimingComposition.java |
147 |
Avoid using if...else statements without curly braces |
| 793 |
regenstrief\mw\medication\TimingComposition.java |
152 |
Found 'DU'-anomaly for variable 'period' (lines '152'-'166'). |
| 794 |
regenstrief\mw\medication\TimingComposition.java |
153 |
A method should have only one exit point, and that should be the last statement in the method |
| 795 |
regenstrief\mw\medication\TimingComposition.java |
153 |
Avoid using if statements without curly braces |
| 796 |
regenstrief\mw\medication\TimingComposition.java |
156 |
A method should have only one exit point, and that should be the last statement in the method |
| 797 |
regenstrief\mw\medication\TimingComposition.java |
156 |
Avoid assignments in operands |
| 798 |
regenstrief\mw\medication\TimingComposition.java |
156 |
Avoid if (x != y) ..; else ..; |
| 799 |
regenstrief\mw\medication\TimingComposition.java |
156 |
Avoid using if...else statements without curly braces |
| 800 |
regenstrief\mw\medication\TimingComposition.java |
156 |
Found 'DD'-anomaly for variable 'per' (lines '156'-'157'). |
| 801 |
regenstrief\mw\medication\TimingComposition.java |
157 |
Avoid assignments in operands |
| 802 |
regenstrief\mw\medication\TimingComposition.java |
157 |
Avoid if (x != y) ..; else ..; |
| 803 |
regenstrief\mw\medication\TimingComposition.java |
157 |
Avoid using if...else statements without curly braces |
| 804 |
regenstrief\mw\medication\TimingComposition.java |
157 |
Found 'DU'-anomaly for variable 'per' (lines '157'-'166'). |
| 805 |
regenstrief\mw\medication\TimingComposition.java |
159 |
Avoid using if...else statements without curly braces |
| 806 |
regenstrief\mw\medication\TimingComposition.java |
162 |
Avoid using if...else statements without curly braces |
| 807 |
regenstrief\mw\medication\TimingComposition.java |
163 |
A method should have only one exit point, and that should be the last statement in the method |
| 808 |
regenstrief\mw\medication\TimingComposition.java |
163 |
Avoid using if...else statements without curly braces |
| 809 |
regenstrief\mw\medication\TimingComposition.java |
165 |
Avoid using if...else statements without curly braces |
| 810 |
regenstrief\mw\medication\TimingComposition.java |
169 |
A method should have only one exit point, and that should be the last statement in the method |
| 811 |
regenstrief\mw\medication\TimingComposition.java |
169 |
Avoid using if...else statements without curly braces |
| 812 |
regenstrief\mw\medication\TimingComposition.java |
171 |
Avoid using if...else statements without curly braces |
| 813 |
regenstrief\mw\medication\TimingComposition.java |
175 |
Avoid using if statements without curly braces |
| 814 |
regenstrief\mw\medication\TimingComposition.java |
175 |
The String literal "Timing entry has already been entered." appears 9 times in this file; the first occurrence is on line 175 |
| 815 |
regenstrief\mw\medication\TimingComposition.java |
176 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 816 |
regenstrief\mw\medication\TimingComposition.java |
178 |
Avoid using if...else statements without curly braces |
| 817 |
regenstrief\mw\medication\TimingComposition.java |
182 |
Avoid using if...else statements without curly braces |
| 818 |
regenstrief\mw\medication\TimingComposition.java |
190 |
This final field could be made static |
| 819 |
regenstrief\mw\medication\TimingComposition.java |
196 |
Avoid excessively long variable names like LONG_FORM_QTY_GROUP |
| 820 |
regenstrief\mw\medication\TimingComposition.java |
197 |
Avoid excessively long variable names like LONG_FORM_DIMENSION_GROUP |
| 821 |
regenstrief\mw\medication\TimingComposition.java |
197 |
Found 'DU'-anomaly for variable 'LONG_FORM_DIMENSION_GROUP' (lines '197'-'214'). |
| 822 |
regenstrief\mw\medication\TimingComposition.java |
199 |
Avoid using if...else statements without curly braces |
| 823 |
regenstrief\mw\medication\TimingComposition.java |
199 |
Found 'DU'-anomaly for variable 'qtyString' (lines '199'-'214'). |
| 824 |
regenstrief\mw\medication\TimingComposition.java |
201 |
Avoid using if...else statements without curly braces |
| 825 |
regenstrief\mw\medication\TimingComposition.java |
202 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 826 |
regenstrief\mw\medication\TimingComposition.java |
205 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 827 |
regenstrief\mw\medication\TimingComposition.java |
209 |
Avoid using if...else statements without curly braces |
| 828 |
regenstrief\mw\medication\TimingComposition.java |
211 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 829 |
regenstrief\mw\medication\TimingComposition.java |
224 |
This final field could be made static |
| 830 |
regenstrief\mw\medication\TimingComposition.java |
226 |
The String literal "\\s*" appears 4 times in this file; the first occurrence is on line 226 |
| 831 |
regenstrief\mw\medication\TimingComposition.java |
229 |
Avoid excessively long variable names like STANDARD_QTY_IVL_GROUP |
| 832 |
regenstrief\mw\medication\TimingComposition.java |
229 |
Found 'DU'-anomaly for variable 'STANDARD_QTY_IVL_GROUP' (lines '229'-'235'). |
| 833 |
regenstrief\mw\medication\TimingComposition.java |
230 |
Avoid excessively long variable names like STANDARD_DIMENSION_IVL_GROUP |
| 834 |
regenstrief\mw\medication\TimingComposition.java |
230 |
Found 'DU'-anomaly for variable 'STANDARD_DIMENSION_IVL_GROUP' (lines '230'-'235'). |
| 835 |
regenstrief\mw\medication\TimingComposition.java |
231 |
Avoid using if...else statements without curly braces |
| 836 |
regenstrief\mw\medication\TimingComposition.java |
232 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 837 |
regenstrief\mw\medication\TimingComposition.java |
234 |
Avoid using if...else statements without curly braces |
| 838 |
regenstrief\mw\medication\TimingComposition.java |
245 |
This final field could be made static |
| 839 |
regenstrief\mw\medication\TimingComposition.java |
250 |
Avoid excessively long variable names like IVL_QUANTITY_GROUP |
| 840 |
regenstrief\mw\medication\TimingComposition.java |
251 |
Avoid excessively long variable names like IVL_DIMENSION_GROUP |
| 841 |
regenstrief\mw\medication\TimingComposition.java |
263 |
This final field could be made static |
| 842 |
regenstrief\mw\medication\TimingComposition.java |
272 |
Avoid using if...else statements without curly braces |
| 843 |
regenstrief\mw\medication\TimingComposition.java |
283 |
This final field could be made static |
| 844 |
regenstrief\mw\medication\TimingComposition.java |
288 |
Avoid excessively long variable names like LITERAL_FORM_QTY_GROUP |
| 845 |
regenstrief\mw\medication\TimingComposition.java |
288 |
Found 'DU'-anomaly for variable 'LITERAL_FORM_QTY_GROUP' (lines '288'-'294'). |
| 846 |
regenstrief\mw\medication\TimingComposition.java |
289 |
Avoid excessively long variable names like LITERAL_FORM_DIMENSION_GROUP |
| 847 |
regenstrief\mw\medication\TimingComposition.java |
289 |
Found 'DU'-anomaly for variable 'LITERAL_FORM_DIMENSION_GROUP' (lines '289'-'294'). |
| 848 |
regenstrief\mw\medication\TimingComposition.java |
290 |
Avoid using if...else statements without curly braces |
| 849 |
regenstrief\mw\medication\TimingComposition.java |
293 |
Avoid using if...else statements without curly braces |
| 850 |
regenstrief\mw\medication\TimingComposition.java |
304 |
This final field could be made static |
| 851 |
regenstrief\mw\medication\TimingComposition.java |
306 |
This final field could be made static |
| 852 |
regenstrief\mw\medication\TimingComposition.java |
314 |
Avoid using if...else statements without curly braces |
| 853 |
regenstrief\mw\medication\TimingComposition.java |
325 |
This final field could be made static |
| 854 |
regenstrief\mw\medication\TimingComposition.java |
327 |
This final field could be made static |
| 855 |
regenstrief\mw\medication\TimingComposition.java |
335 |
Avoid using if...else statements without curly braces |
| 856 |
regenstrief\mw\medication\TimingComposition.java |
352 |
This final field could be made static |
| 857 |
regenstrief\mw\medication\TimingComposition.java |
360 |
Avoid if (x != y) ..; else ..; |
| 858 |
regenstrief\mw\medication\TimingComposition.java |
370 |
Avoid if (x != y) ..; else ..; |
| 859 |
regenstrief\mw\medication\TimingComposition.java |
380 |
Avoid using if...else statements without curly braces |
| 860 |
regenstrief\mw\medication\TimingComposition.java |
391 |
This final field could be made static |
| 861 |
regenstrief\mw\medication\TimingComposition.java |
393 |
This final field could be made static |
| 862 |
regenstrief\mw\medication\TimingComposition.java |
397 |
Avoid if (x != y) ..; else ..; |
| 863 |
regenstrief\mw\medication\TimingComposition.java |
407 |
Avoid if (x != y) ..; else ..; |
| 864 |
regenstrief\mw\medication\TimingComposition.java |
417 |
Avoid using if...else statements without curly braces |
| 865 |
regenstrief\mw\medication\TimingComposition.java |
447 |
Found 'DU'-anomaly for variable 'dayInHours' (lines '447'-'451'). |
| 866 |
regenstrief\mw\medication\TimingComposition.java |
448 |
A method should have only one exit point, and that should be the last statement in the method |
| 867 |
regenstrief\mw\medication\TimingComposition.java |
448 |
Avoid using if...else statements without curly braces |
| 868 |
regenstrief\mw\medication\TimingComposition.java |
450 |
Avoid using if...else statements without curly braces |
| 869 |
regenstrief\mw\medication\TimingComposition.java |
457 |
Found 'DU'-anomaly for variable 'day' (lines '457'-'464'). |
| 870 |
regenstrief\mw\medication\TimingComposition.java |
460 |
A method should have only one exit point, and that should be the last statement in the method |
| 871 |
regenstrief\mw\medication\TimingComposition.java |
463 |
Avoid using if...else statements without curly braces |
| 872 |
regenstrief\mw\medication\TimingComposition.java |
472 |
A method should have only one exit point, and that should be the last statement in the method |
| 873 |
regenstrief\mw\medication\TimingComposition.java |
472 |
Avoid using if statements without curly braces |
| 874 |
regenstrief\mw\medication\TimingComposition.java |
473 |
Avoid variables with short names like ab |
| 875 |
regenstrief\mw\medication\TimingComposition.java |
474 |
A method should have only one exit point, and that should be the last statement in the method |
| 876 |
regenstrief\mw\medication\TimingComposition.java |
474 |
Avoid using if...else statements without curly braces |
| 877 |
regenstrief\mw\medication\TimingComposition.java |
476 |
Avoid using if...else statements without curly braces |
| 878 |
regenstrief\mw\medication\TimingComposition.java |
488 |
Avoid really long methods. |
| 879 |
regenstrief\mw\medication\TimingComposition.java |
488 |
The method initializeMaps() has an NCSS line count of 175 |
| 880 |
regenstrief\mw\medication\TimingComposition.java |
490 |
Found 'DD'-anomaly for variable 'set' (lines '490'-'521'). |
| 881 |
regenstrief\mw\medication\TimingComposition.java |
491 |
Found 'DD'-anomaly for variable 'period' (lines '491'-'492'). |
| 882 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
34 |
Avoid excessively long variable names like _formulationAdapter |
| 883 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
50 |
Avoid reassigning parameters such as 'mode' |
| 884 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
57 |
Use equals() to compare strings instead of '==' or '!=' |
| 885 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
57 |
Use equals() to compare strings instead of '==' or '!=' |
| 886 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
59 |
A method should have only one exit point, and that should be the last statement in the method |
| 887 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
61 |
Use equals() to compare strings instead of '==' or '!=' |
| 888 |
regenstrief\mw\medication\formulation\ConsumableParticipationViewAdapter.java |
64 |
Consider simply returning the value vs storing it in local variable 'modelObject' |
| 889 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
40 |
Avoid excessively long variable names like _consumableAdapter |
| 890 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
81 |
Avoid reassigning parameters such as 'mode' |
| 891 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
88 |
Use equals() to compare strings instead of '==' or '!=' |
| 892 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
88 |
Use equals() to compare strings instead of '==' or '!=' |
| 893 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
90 |
A method should have only one exit point, and that should be the last statement in the method |
| 894 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
92 |
Use equals() to compare strings instead of '==' or '!=' |
| 895 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
100 |
A method should have only one exit point, and that should be the last statement in the method |
| 896 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
102 |
Use equals() to compare strings instead of '==' or '!=' |
| 897 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
105 |
Consider simply returning the value vs storing it in local variable 'modelObject' |
| 898 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
125 |
Avoid excessively long variable names like participationClone |
| 899 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
142 |
Avoid reassigning parameters such as 'queryMode' |
| 900 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
146 |
Use equals() to compare strings instead of '==' or '!=' |
| 901 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
152 |
A method should have only one exit point, and that should be the last statement in the method |
| 902 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
161 |
Avoid reassigning parameters such as 'queryMode' |
| 903 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
165 |
Use equals() to compare strings instead of '==' or '!=' |
| 904 |
regenstrief\mw\medication\formulation\FormulationRoleViewAdapter.java |
169 |
Use equals() to compare strings instead of '==' or '!=' |
| 905 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
38 |
Avoid excessively long variable names like _participationAdapter |
| 906 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
61 |
Avoid reassigning parameters such as 'mode' |
| 907 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
68 |
Use equals() to compare strings instead of '==' or '!=' |
| 908 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
68 |
Use equals() to compare strings instead of '==' or '!=' |
| 909 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
70 |
A method should have only one exit point, and that should be the last statement in the method |
| 910 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
72 |
Use equals() to compare strings instead of '==' or '!=' |
| 911 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
80 |
A method should have only one exit point, and that should be the last statement in the method |
| 912 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
82 |
Use equals() to compare strings instead of '==' or '!=' |
| 913 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
85 |
Consider simply returning the value vs storing it in local variable 'modelObject' |
| 914 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
105 |
Avoid excessively long variable names like participationClone |
| 915 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
122 |
Avoid reassigning parameters such as 'queryMode' |
| 916 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
126 |
Use equals() to compare strings instead of '==' or '!=' |
| 917 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
132 |
A method should have only one exit point, and that should be the last statement in the method |
| 918 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
141 |
Avoid reassigning parameters such as 'queryMode' |
| 919 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
145 |
Use equals() to compare strings instead of '==' or '!=' |
| 920 |
regenstrief\mw\medication\formulation\IngredientRoleViewAdapter.java |
149 |
Use equals() to compare strings instead of '==' or '!=' |
| 921 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
22 |
A high number of imports can indicate a high degree of coupling within an object. |
| 922 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
80 |
The class 'MedicationComposition' has a Cyclomatic Complexity of 4 (Highest = 27). |
| 923 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
85 |
The field name indicates a constant but its modifiers do not |
| 924 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
87 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 925 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
87 |
The field name indicates a constant but its modifiers do not |
| 926 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
89 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 927 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
89 |
The field name indicates a constant but its modifiers do not |
| 928 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
91 |
Avoid excessively long variable names like applicationContext |
| 929 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
94 |
Avoid using if statements without curly braces |
| 930 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
95 |
Avoid using if statements without curly braces |
| 931 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
102 |
Private field 'clearables' could be made final; it is only initialized in the declaration or constructor. |
| 932 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
112 |
Avoid reassigning parameters such as 'text' |
| 933 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
113 |
String.indexOf(char) is faster than String.indexOf(String). |
| 934 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
114 |
Avoid using if statements without curly braces |
| 935 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
118 |
Found 'DD'-anomaly for variable 'drugCompletions' (lines '118'-'118'). |
| 936 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
118 |
Found 'DU'-anomaly for variable 'drugCompletions' (lines '118'-'132'). |
| 937 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
120 |
Avoid using if...else statements without curly braces |
| 938 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
122 |
Found 'DD'-anomaly for variable 'mergedResults' (lines '122'-'122'). |
| 939 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
122 |
Found 'DU'-anomaly for variable 'mergedResults' (lines '122'-'132'). |
| 940 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
124 |
Avoid using for statements without curly braces |
| 941 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
139 |
Avoid variables with short names like sa |
| 942 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
150 |
Avoid using if statements without curly braces |
| 943 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
158 |
Found 'DU'-anomaly for variable 'patientid' (lines '158'-'172'). |
| 944 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
179 |
Avoid using for statements without curly braces |
| 945 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
180 |
Avoid using if statements without curly braces |
| 946 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
188 |
Avoid excessively long variable names like activeMoietyPlayers |
| 947 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
188 |
Found 'DU'-anomaly for variable 'activeMoietyPlayers' (lines '188'-'201'). |
| 948 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
192 |
Avoid if (x != y) ..; else ..; |
| 949 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
193 |
Avoid using for statements without curly braces |
| 950 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
194 |
Avoid using if statements without curly braces |
| 951 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
197 |
A method should have only one exit point, and that should be the last statement in the method |
| 952 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
197 |
Avoid using if...else statements without curly braces |
| 953 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
235 |
Avoid reassigning parameters such as 'name' |
| 954 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
239 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 955 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
240 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 956 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
241 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 957 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
243 |
A method should have only one exit point, and that should be the last statement in the method |
| 958 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
251 |
Avoid reassigning parameters such as 'name' |
| 959 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
255 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 960 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
256 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 961 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
257 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 962 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
259 |
A method should have only one exit point, and that should be the last statement in the method |
| 963 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
268 |
Avoid really long methods. |
| 964 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
268 |
The method 'addToFormulation' has a Cyclomatic Complexity of 19. |
| 965 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
268 |
The method addToFormulation() has an NPath complexity of 10584 |
| 966 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
287 |
Avoid using if statements without curly braces |
| 967 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
301 |
A method should have only one exit point, and that should be the last statement in the method |
| 968 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
306 |
Found 'DD'-anomaly for variable 'drugMaterial' (lines '306'-'317'). |
| 969 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
309 |
Found 'DD'-anomaly for variable 'isCandidate' (lines '309'-'312'). |
| 970 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
311 |
Avoid using for statements without curly braces |
| 971 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
312 |
Avoid using if statements without curly braces |
| 972 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
312 |
Found 'DD'-anomaly for variable 'isCandidate' (lines '312'-'312'). |
| 973 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
314 |
Avoid unnecessary comparisons in boolean expressions |
| 974 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
314 |
Avoid using if statements without curly braces |
| 975 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
317 |
Found 'DD'-anomaly for variable 'drugMaterial' (lines '317'-'329'). |
| 976 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
320 |
Avoid using if statements without curly braces |
| 977 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
321 |
Avoid using if statements without curly braces |
| 978 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
321 |
Found 'DD'-anomaly for variable 'drugMaterial' (lines '321'-'317'). |
| 979 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
321 |
Found 'DD'-anomaly for variable 'drugMaterial' (lines '321'-'329'). |
| 980 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
324 |
Avoid using if statements without curly braces |
| 981 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
324 |
Found 'DD'-anomaly for variable 'formulation' (lines '324'-'356'). |
| 982 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
324 |
Found 'DU'-anomaly for variable 'formulation' (lines '324'-'376'). |
| 983 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
334 |
Found 'DU'-anomaly for variable 'ingredient' (lines '334'-'376'). |
| 984 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
346 |
Avoid excessively long variable names like originalingredients |
| 985 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
359 |
Avoid using if...else statements without curly braces |
| 986 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
365 |
Avoid if (x != y) ..; else ..; |
| 987 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
370 |
Avoid using if statements without curly braces |
| 988 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
373 |
Avoid using if...else statements without curly braces |
| 989 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
406 |
Avoid using for statements without curly braces |
| 990 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
407 |
Avoid using if statements without curly braces |
| 991 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
412 |
Avoid using if statements without curly braces |
| 992 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
416 |
Avoid using if statements without curly braces |
| 993 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
428 |
The field name indicates a constant but its modifiers do not |
| 994 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
429 |
This final field could be made static |
| 995 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
436 |
Avoid using if statements without curly braces |
| 996 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
452 |
This final field could be made static |
| 997 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
460 |
Avoid using if...else statements without curly braces |
| 998 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
463 |
Avoid using if...else statements without curly braces |
| 999 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
477 |
Found 'DD'-anomaly for variable 'counter' (lines '477'-'479'). |
| 1000 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
478 |
Avoid using for statements without curly braces |
| 1001 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
479 |
Avoid using if statements without curly braces |
| 1002 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
479 |
Found 'DD'-anomaly for variable 'counter' (lines '479'-'479'). |
| 1003 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
492 |
Avoid using for statements without curly braces |
| 1004 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
493 |
Avoid concatenating nonliterals in a StringBuffer constructor or append(). |
| 1005 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
493 |
Avoid using if statements without curly braces |
| 1006 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
493 |
Method call on object which may be null |
| 1007 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
496 |
Avoid using for statements without curly braces |
| 1008 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
514 |
Found 'DD'-anomaly for variable 'counter' (lines '514'-'516'). |
| 1009 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
515 |
Avoid using for statements without curly braces |
| 1010 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
516 |
Avoid using if statements without curly braces |
| 1011 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
516 |
Found 'DD'-anomaly for variable 'counter' (lines '516'-'516'). |
| 1012 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
522 |
The method 'getMedNames' has a Cyclomatic Complexity of 11. |
| 1013 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
525 |
Found 'DD'-anomaly for variable 'foundname' (lines '525'-'530'). |
| 1014 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
536 |
Avoid using if...else statements without curly braces |
| 1015 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
536 |
Position literals first in String comparisons |
| 1016 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
538 |
Avoid using if...else statements without curly braces |
| 1017 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
543 |
Avoid using if...else statements without curly braces |
| 1018 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
543 |
Position literals first in String comparisons |
| 1019 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
545 |
Avoid using if...else statements without curly braces |
| 1020 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
551 |
Avoid unnecessary comparisons in boolean expressions |
| 1021 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
551 |
Avoid using for statements without curly braces |
| 1022 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
551 |
Avoid using if statements without curly braces |
| 1023 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
559 |
Avoid variables with short names like sa |
| 1024 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
559 |
The method 'format' has a Cyclomatic Complexity of 27. |
| 1025 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
559 |
The method format() has an NPath complexity of 316800 |
| 1026 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
566 |
Found 'DU'-anomaly for variable 'finalname' (lines '566'-'623'). |
| 1027 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
568 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 1028 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
570 |
A method should have only one exit point, and that should be the last statement in the method |
| 1029 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
570 |
Avoid using if statements without curly braces |
| 1030 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
585 |
Found 'DD'-anomaly for variable 'formname' (lines '585'-'588'). |
| 1031 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
586 |
Avoid using if statements without curly braces |
| 1032 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
589 |
A method should have only one exit point, and that should be the last statement in the method |
| 1033 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
595 |
Avoid using if statements without curly braces |
| 1034 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
598 |
A method should have only one exit point, and that should be the last statement in the method |
| 1035 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
605 |
Avoid if (x != y) ..; else ..; |
| 1036 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
605 |
Avoid using for statements without curly braces |
| 1037 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
605 |
Avoid using if...else statements without curly braces |
| 1038 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
609 |
Avoid using if...else statements without curly braces |
| 1039 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
613 |
Avoid using if statements without curly braces |
| 1040 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
615 |
Avoid concatenating nonliterals in a StringBuffer constructor or append(). |
| 1041 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
615 |
Avoid using if statements without curly braces |
| 1042 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
619 |
Avoid concatenating nonliterals in a StringBuffer constructor or append(). |
| 1043 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
626 |
Avoid reassigning parameters such as 'role' |
| 1044 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
626 |
Avoid reassigning parameters such as 'role' |
| 1045 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
628 |
Avoid using if...else statements without curly braces |
| 1046 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
631 |
A method should have only one exit point, and that should be the last statement in the method |
| 1047 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
631 |
Avoid using if...else statements without curly braces |
| 1048 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
631 |
Substitute calls to size() == 0 (or size() != 0) with calls to isEmpty() |
| 1049 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
633 |
Avoid using if...else statements without curly braces |
| 1050 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
635 |
A method should have only one exit point, and that should be the last statement in the method |
| 1051 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
635 |
Avoid using if statements without curly braces |
| 1052 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
639 |
A method should have only one exit point, and that should be the last statement in the method |
| 1053 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
639 |
Avoid using if statements without curly braces |
| 1054 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
642 |
A method should have only one exit point, and that should be the last statement in the method |
| 1055 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
642 |
Avoid using if...else statements without curly braces |
| 1056 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
642 |
Position literals first in String comparisons |
| 1057 |
regenstrief\mw\medication\formulation\MedicationComposition.java |
644 |
Avoid using if...else statements without curly braces |
| 1058 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
37 |
Avoid excessively long variable names like _formulationAdapter |
| 1059 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
37 |
Private field '_formulationAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 1060 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
48 |
Switch statements should have a default label |
| 1061 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
51 |
A method should have only one exit point, and that should be the last statement in the method |
| 1062 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
53 |
A method should have only one exit point, and that should be the last statement in the method |
| 1063 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
55 |
A method should have only one exit point, and that should be the last statement in the method |
| 1064 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
57 |
A method should have only one exit point, and that should be the last statement in the method |
| 1065 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
59 |
A method should have only one exit point, and that should be the last statement in the method |
| 1066 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
61 |
A method should have only one exit point, and that should be the last statement in the method |
| 1067 |
regenstrief\mw\medication\formulation\MedicationEntityController.java |
63 |
A method should have only one exit point, and that should be the last statement in the method |
| 1068 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
50 |
The class 'MedicineMaterialViewAdapter' has a Cyclomatic Complexity of 5 (Highest = 11). |
| 1069 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
54 |
Avoid excessively long variable names like EXPR_MATERIAL_NAME |
| 1070 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
63 |
Use equals() to compare strings instead of '==' or '!=' |
| 1071 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
64 |
A method should have only one exit point, and that should be the last statement in the method |
| 1072 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
66 |
Use equals() to compare strings instead of '==' or '!=' |
| 1073 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
83 |
A method should have only one exit point, and that should be the last statement in the method |
| 1074 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
86 |
Avoid using if...else statements without curly braces |
| 1075 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
92 |
Use equals() to compare object references. |
| 1076 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
93 |
Use equals() to compare object references. |
| 1077 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
107 |
A method should have only one exit point, and that should be the last statement in the method |
| 1078 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
109 |
Use equals() to compare object references. |
| 1079 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
110 |
Use equals() to compare strings instead of '==' or '!=' |
| 1080 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
112 |
A method should have only one exit point, and that should be the last statement in the method |
| 1081 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
120 |
Avoid if (x != y) ..; else ..; |
| 1082 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
122 |
A method should have only one exit point, and that should be the last statement in the method |
| 1083 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
125 |
A method should have only one exit point, and that should be the last statement in the method |
| 1084 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
125 |
Avoid using if...else statements without curly braces |
| 1085 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
127 |
Use equals() to compare strings instead of '==' or '!=' |
| 1086 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
129 |
A method should have only one exit point, and that should be the last statement in the method |
| 1087 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
132 |
Avoid using if...else statements without curly braces |
| 1088 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
143 |
Avoid reassigning parameters such as 'mode' |
| 1089 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
144 |
Avoid using if statements without curly braces |
| 1090 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
147 |
Use equals() to compare strings instead of '==' or '!=' |
| 1091 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
147 |
Use equals() to compare strings instead of '==' or '!=' |
| 1092 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
148 |
A method should have only one exit point, and that should be the last statement in the method |
| 1093 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
150 |
Use equals() to compare strings instead of '==' or '!=' |
| 1094 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
152 |
Consider simply returning the value vs storing it in local variable 'modelObject' |
| 1095 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
154 |
Avoid using if...else statements without curly braces |
| 1096 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
161 |
Avoid using if statements without curly braces |
| 1097 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
176 |
Avoid excessively long variable names like NAME_SQL_PARAM_TYPES |
| 1098 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
179 |
Avoid reassigning parameters such as 'queryMode' |
| 1099 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
179 |
The method 'setCriteria' has a Cyclomatic Complexity of 11. |
| 1100 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
188 |
Use equals() to compare strings instead of '==' or '!=' |
| 1101 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
190 |
Avoid using if statements without curly braces |
| 1102 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
197 |
Use equals() to compare strings instead of '==' or '!=' |
| 1103 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
201 |
Avoid using if statements without curly braces |
| 1104 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
223 |
Avoid using if statements without curly braces |
| 1105 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
226 |
Avoid excessively long variable names like ingredientCriteria |
| 1106 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
234 |
Avoid excessively long variable names like ingredientMaterial |
| 1107 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
235 |
Avoid using if statements without curly braces |
| 1108 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
235 |
No need to check for null before an instanceof |
| 1109 |
regenstrief\mw\medication\formulation\MedicineMaterialViewAdapter.java |
245 |
Avoid using if...else statements without curly braces |
| 1110 |
regenstrief\mw\medication\reconciliation\ReconciliationComponentViewAdapter.java |
56 |
Avoid excessively long variable names like originalModelObject |
| 1111 |
regenstrief\mw\medication\reconciliation\ReconciliationListViewAdapter.java |
34 |
Avoid excessively long variable names like RECONCILIATION_COMPONENT_ADAPTER |
| 1112 |
regenstrief\mw\medication\reconciliation\ReconciliationListViewAdapter.java |
45 |
Avoid excessively long variable names like fulfillsRelationship |
| 1113 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
28 |
Avoid using implementation types like 'ArrayList'; use the interface instead |
| 1114 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
30 |
Use explicit scoping instead of the default package private level |
| 1115 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
40 |
Document empty constructor |
| 1116 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
40 |
It is a good practice to call super() in a constructor |
| 1117 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
72 |
Avoid using implementation types like 'ArrayList'; use the interface instead |
| 1118 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
82 |
Avoid variables with short names like va |
| 1119 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
83 |
Avoid unused local variables such as 'supply'. |
| 1120 |
regenstrief\mw\medication\renew\RenewMedicationListCommand.java |
83 |
Found 'DU'-anomaly for variable 'supply' (lines '83'-'99'). |
| 1121 |
regenstrief\mw\medication\supply\SupplyComponentViewAdapter.java |
56 |
Avoid variables with short names like ts |
| 1122 |
regenstrief\mw\medication\supply\SupplyComponentViewAdapter.java |
92 |
Avoid reassigning parameters such as 'hpathExpression' |
| 1123 |
regenstrief\mw\medication\supply\SupplyComponentViewAdapter.java |
98 |
Use equals() to compare strings instead of '==' or '!=' |
| 1124 |
regenstrief\mw\medication\supply\SupplyListViewAdapter.java |
35 |
Avoid excessively long variable names like SUPPLY_COMPONENT_ADAPTER |
| 1125 |
regenstrief\mw\medication\supply\SupplyListViewAdapter.java |
46 |
Avoid excessively long variable names like supplyRelationship |
| 1126 |
regenstrief\mw\observation\ChangeObservationStatusCommand.java |
20 |
Avoid using implementation types like 'ArrayList'; use the interface instead |
| 1127 |
regenstrief\mw\observation\ChangeObservationStatusCommand.java |
25 |
Document empty constructor |
| 1128 |
regenstrief\mw\observation\ChangeObservationStatusCommand.java |
25 |
It is a good practice to call super() in a constructor |
| 1129 |
regenstrief\mw\observation\ChangeObservationStatusCommand.java |
36 |
A switch statement does not contain a break |
| 1130 |
regenstrief\mw\observation\ChangeObservationStatusCommand.java |
38 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 1131 |
regenstrief\mw\observation\ChangeObservationStatusCommand.java |
39 |
A method should have only one exit point, and that should be the last statement in the method |
| 1132 |
regenstrief\mw\observation\EditObservationCommand.java |
59 |
Document empty constructor |
| 1133 |
regenstrief\mw\observation\EditObservationCommand.java |
59 |
It is a good practice to call super() in a constructor |
| 1134 |
regenstrief\mw\observation\EditObservationCommand.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 1135 |
regenstrief\mw\observation\ObservationListController.java |
55 |
The String literal "item" appears 6 times in this file; the first occurrence is on line 55 |
| 1136 |
regenstrief\mw\observation\ObservationListController.java |
71 |
A method should have only one exit point, and that should be the last statement in the method |
| 1137 |
regenstrief\mw\observation\ObservationListController.java |
74 |
A method should have only one exit point, and that should be the last statement in the method |
| 1138 |
regenstrief\mw\observation\ObservationListController.java |
81 |
Private field '_code' could be made final; it is only initialized in the declaration or constructor. |
| 1139 |
regenstrief\mw\observation\ObservationListController.java |
107 |
Avoid using if...else statements without curly braces |
| 1140 |
regenstrief\mw\observation\ObservationViewAdapter.java |
22 |
A high number of imports can indicate a high degree of coupling within an object. |
| 1141 |
regenstrief\mw\observation\ObservationViewAdapter.java |
80 |
The class 'ObservationViewAdapter' has a Cyclomatic Complexity of 10 (Highest = 50). |
| 1142 |
regenstrief\mw\observation\ObservationViewAdapter.java |
83 |
When instantiating a SimpleDateFormat object, specify a Locale |
| 1143 |
regenstrief\mw\observation\ObservationViewAdapter.java |
91 |
Avoid excessively long variable names like _rangeValueComposition |
| 1144 |
regenstrief\mw\observation\ObservationViewAdapter.java |
99 |
Avoid using if statements without curly braces |
| 1145 |
regenstrief\mw\observation\ObservationViewAdapter.java |
102 |
Avoid excessively long variable names like COMPONENT_OBSERVATIONS |
| 1146 |
regenstrief\mw\observation\ObservationViewAdapter.java |
103 |
Avoid variables with short names like ar |
| 1147 |
regenstrief\mw\observation\ObservationViewAdapter.java |
103 |
JUnit 4 tests that execute tests should use the @Test annotation |
| 1148 |
regenstrief\mw\observation\ObservationViewAdapter.java |
109 |
Avoid really long methods. |
| 1149 |
regenstrief\mw\observation\ObservationViewAdapter.java |
109 |
Avoid reassigning parameters such as 'propertyName' |
| 1150 |
regenstrief\mw\observation\ObservationViewAdapter.java |
109 |
The method 'getProperty' has a Cyclomatic Complexity of 50. |
| 1151 |
regenstrief\mw\observation\ObservationViewAdapter.java |
109 |
The method getProperty() has an NCSS line count of 117 |
| 1152 |
regenstrief\mw\observation\ObservationViewAdapter.java |
109 |
The method getProperty() has an NPath complexity of 525 |
| 1153 |
regenstrief\mw\observation\ObservationViewAdapter.java |
113 |
Use equals() to compare strings instead of '==' or '!=' |
| 1154 |
regenstrief\mw\observation\ObservationViewAdapter.java |
114 |
A method should have only one exit point, and that should be the last statement in the method |
| 1155 |
regenstrief\mw\observation\ObservationViewAdapter.java |
114 |
Avoid using if statements without curly braces |
| 1156 |
regenstrief\mw\observation\ObservationViewAdapter.java |
116 |
A method should have only one exit point, and that should be the last statement in the method |
| 1157 |
regenstrief\mw\observation\ObservationViewAdapter.java |
116 |
Avoid using if statements without curly braces |
| 1158 |
regenstrief\mw\observation\ObservationViewAdapter.java |
125 |
Avoid using if statements without curly braces |
| 1159 |
regenstrief\mw\observation\ObservationViewAdapter.java |
126 |
Avoid using if statements without curly braces |
| 1160 |
regenstrief\mw\observation\ObservationViewAdapter.java |
131 |
A method should have only one exit point, and that should be the last statement in the method |
| 1161 |
regenstrief\mw\observation\ObservationViewAdapter.java |
131 |
Avoid using if...else statements without curly braces |
| 1162 |
regenstrief\mw\observation\ObservationViewAdapter.java |
132 |
A method should have only one exit point, and that should be the last statement in the method |
| 1163 |
regenstrief\mw\observation\ObservationViewAdapter.java |
132 |
Avoid using if...else statements without curly braces |
| 1164 |
regenstrief\mw\observation\ObservationViewAdapter.java |
134 |
A method should have only one exit point, and that should be the last statement in the method |
| 1165 |
regenstrief\mw\observation\ObservationViewAdapter.java |
134 |
Avoid using if...else statements without curly braces |
| 1166 |
regenstrief\mw\observation\ObservationViewAdapter.java |
135 |
A method should have only one exit point, and that should be the last statement in the method |
| 1167 |
regenstrief\mw\observation\ObservationViewAdapter.java |
135 |
Avoid using if...else statements without curly braces |
| 1168 |
regenstrief\mw\observation\ObservationViewAdapter.java |
136 |
A method should have only one exit point, and that should be the last statement in the method |
| 1169 |
regenstrief\mw\observation\ObservationViewAdapter.java |
136 |
Avoid using if...else statements without curly braces |
| 1170 |
regenstrief\mw\observation\ObservationViewAdapter.java |
137 |
A method should have only one exit point, and that should be the last statement in the method |
| 1171 |
regenstrief\mw\observation\ObservationViewAdapter.java |
137 |
Avoid using if...else statements without curly braces |
| 1172 |
regenstrief\mw\observation\ObservationViewAdapter.java |
139 |
A method should have only one exit point, and that should be the last statement in the method |
| 1173 |
regenstrief\mw\observation\ObservationViewAdapter.java |
139 |
Avoid using if...else statements without curly braces |
| 1174 |
regenstrief\mw\observation\ObservationViewAdapter.java |
141 |
Use equals() to compare strings instead of '==' or '!=' |
| 1175 |
regenstrief\mw\observation\ObservationViewAdapter.java |
142 |
A method should have only one exit point, and that should be the last statement in the method |
| 1176 |
regenstrief\mw\observation\ObservationViewAdapter.java |
142 |
Avoid using if statements without curly braces |
| 1177 |
regenstrief\mw\observation\ObservationViewAdapter.java |
143 |
Found 'DD'-anomaly for variable 'unit' (lines '143'-'152'). |
| 1178 |
regenstrief\mw\observation\ObservationViewAdapter.java |
143 |
Found 'DD'-anomaly for variable 'unit' (lines '143'-'154'). |
| 1179 |
regenstrief\mw\observation\ObservationViewAdapter.java |
148 |
Avoid using if statements without curly braces |
| 1180 |
regenstrief\mw\observation\ObservationViewAdapter.java |
150 |
Avoid using if statements without curly braces |
| 1181 |
regenstrief\mw\observation\ObservationViewAdapter.java |
152 |
Avoid using if statements without curly braces |
| 1182 |
regenstrief\mw\observation\ObservationViewAdapter.java |
156 |
A method should have only one exit point, and that should be the last statement in the method |
| 1183 |
regenstrief\mw\observation\ObservationViewAdapter.java |
157 |
Use equals() to compare strings instead of '==' or '!=' |
| 1184 |
regenstrief\mw\observation\ObservationViewAdapter.java |
158 |
A method should have only one exit point, and that should be the last statement in the method |
| 1185 |
regenstrief\mw\observation\ObservationViewAdapter.java |
158 |
Avoid using if statements without curly braces |
| 1186 |
regenstrief\mw\observation\ObservationViewAdapter.java |
161 |
Avoid using if statements without curly braces |
| 1187 |
regenstrief\mw\observation\ObservationViewAdapter.java |
162 |
Avoid using for statements without curly braces |
| 1188 |
regenstrief\mw\observation\ObservationViewAdapter.java |
163 |
Avoid using if statements without curly braces |
| 1189 |
regenstrief\mw\observation\ObservationViewAdapter.java |
164 |
A method should have only one exit point, and that should be the last statement in the method |
| 1190 |
regenstrief\mw\observation\ObservationViewAdapter.java |
166 |
A method should have only one exit point, and that should be the last statement in the method |
| 1191 |
regenstrief\mw\observation\ObservationViewAdapter.java |
168 |
Use equals() to compare strings instead of '==' or '!=' |
| 1192 |
regenstrief\mw\observation\ObservationViewAdapter.java |
169 |
A method should have only one exit point, and that should be the last statement in the method |
| 1193 |
regenstrief\mw\observation\ObservationViewAdapter.java |
169 |
Avoid using if statements without curly braces |
| 1194 |
regenstrief\mw\observation\ObservationViewAdapter.java |
180 |
Avoid if (x != y) ..; else ..; |
| 1195 |
regenstrief\mw\observation\ObservationViewAdapter.java |
181 |
Avoid using if...else statements without curly braces |
| 1196 |
regenstrief\mw\observation\ObservationViewAdapter.java |
182 |
Avoid using if...else statements without curly braces |
| 1197 |
regenstrief\mw\observation\ObservationViewAdapter.java |
183 |
Avoid using if...else statements without curly braces |
| 1198 |
regenstrief\mw\observation\ObservationViewAdapter.java |
184 |
Avoid using if...else statements without curly braces |
| 1199 |
regenstrief\mw\observation\ObservationViewAdapter.java |
187 |
Avoid using if...else statements without curly braces |
| 1200 |
regenstrief\mw\observation\ObservationViewAdapter.java |
188 |
Avoid using if...else statements without curly braces |
| 1201 |
regenstrief\mw\observation\ObservationViewAdapter.java |
190 |
A method should have only one exit point, and that should be the last statement in the method |
| 1202 |
regenstrief\mw\observation\ObservationViewAdapter.java |
190 |
Avoid using if...else statements without curly braces |
| 1203 |
regenstrief\mw\observation\ObservationViewAdapter.java |
191 |
Avoid using if...else statements without curly braces |
| 1204 |
regenstrief\mw\observation\ObservationViewAdapter.java |
193 |
A method should have only one exit point, and that should be the last statement in the method |
| 1205 |
regenstrief\mw\observation\ObservationViewAdapter.java |
193 |
Avoid using if...else statements without curly braces |
| 1206 |
regenstrief\mw\observation\ObservationViewAdapter.java |
195 |
A method should have only one exit point, and that should be the last statement in the method |
| 1207 |
regenstrief\mw\observation\ObservationViewAdapter.java |
195 |
Avoid using if...else statements without curly braces |
| 1208 |
regenstrief\mw\observation\ObservationViewAdapter.java |
197 |
Avoid using if statements without curly braces |
| 1209 |
regenstrief\mw\observation\ObservationViewAdapter.java |
200 |
A method should have only one exit point, and that should be the last statement in the method |
| 1210 |
regenstrief\mw\observation\ObservationViewAdapter.java |
201 |
Use equals() to compare strings instead of '==' or '!=' |
| 1211 |
regenstrief\mw\observation\ObservationViewAdapter.java |
202 |
A method should have only one exit point, and that should be the last statement in the method |
| 1212 |
regenstrief\mw\observation\ObservationViewAdapter.java |
202 |
Avoid using if statements without curly braces |
| 1213 |
regenstrief\mw\observation\ObservationViewAdapter.java |
205 |
A method should have only one exit point, and that should be the last statement in the method |
| 1214 |
regenstrief\mw\observation\ObservationViewAdapter.java |
208 |
Avoid using if...else statements without curly braces |
| 1215 |
regenstrief\mw\observation\ObservationViewAdapter.java |
213 |
The method 'setProperty' has a Cyclomatic Complexity of 24. |
| 1216 |
regenstrief\mw\observation\ObservationViewAdapter.java |
214 |
A method should have only one exit point, and that should be the last statement in the method |
| 1217 |
regenstrief\mw\observation\ObservationViewAdapter.java |
214 |
Avoid using if statements without curly braces |
| 1218 |
regenstrief\mw\observation\ObservationViewAdapter.java |
217 |
Use equals() to compare object references. |
| 1219 |
regenstrief\mw\observation\ObservationViewAdapter.java |
234 |
Avoid using if statements without curly braces |
| 1220 |
regenstrief\mw\observation\ObservationViewAdapter.java |
236 |
A method should have only one exit point, and that should be the last statement in the method |
| 1221 |
regenstrief\mw\observation\ObservationViewAdapter.java |
239 |
A method should have only one exit point, and that should be the last statement in the method |
| 1222 |
regenstrief\mw\observation\ObservationViewAdapter.java |
241 |
Use equals() to compare strings instead of '==' or '!=' |
| 1223 |
regenstrief\mw\observation\ObservationViewAdapter.java |
243 |
Avoid using if statements without curly braces |
| 1224 |
regenstrief\mw\observation\ObservationViewAdapter.java |
250 |
A method should have only one exit point, and that should be the last statement in the method |
| 1225 |
regenstrief\mw\observation\ObservationViewAdapter.java |
252 |
Use equals() to compare object references. |
| 1226 |
regenstrief\mw\observation\ObservationViewAdapter.java |
255 |
A method should have only one exit point, and that should be the last statement in the method |
| 1227 |
regenstrief\mw\observation\ObservationViewAdapter.java |
255 |
Avoid using if statements without curly braces |
| 1228 |
regenstrief\mw\observation\ObservationViewAdapter.java |
258 |
Avoid variables with short names like ts |
| 1229 |
regenstrief\mw\observation\ObservationViewAdapter.java |
258 |
Static DateFormatter objects should be accessed in a synchronized manner |
| 1230 |
regenstrief\mw\observation\ObservationViewAdapter.java |
260 |
A method should have only one exit point, and that should be the last statement in the method |
| 1231 |
regenstrief\mw\observation\ObservationViewAdapter.java |
260 |
Avoid using if statements without curly braces |
| 1232 |
regenstrief\mw\observation\ObservationViewAdapter.java |
264 |
A method should have only one exit point, and that should be the last statement in the method |
| 1233 |
regenstrief\mw\observation\ObservationViewAdapter.java |
266 |
Use equals() to compare strings instead of '==' or '!=' |
| 1234 |
regenstrief\mw\observation\ObservationViewAdapter.java |
268 |
Found 'DU'-anomaly for variable 'newStandard' (lines '268'-'301'). |
| 1235 |
regenstrief\mw\observation\ObservationViewAdapter.java |
273 |
A method should have only one exit point, and that should be the last statement in the method |
| 1236 |
regenstrief\mw\observation\ObservationViewAdapter.java |
273 |
Avoid using if statements without curly braces |
| 1237 |
regenstrief\mw\observation\ObservationViewAdapter.java |
276 |
A method should have only one exit point, and that should be the last statement in the method |
| 1238 |
regenstrief\mw\observation\ObservationViewAdapter.java |
278 |
A method should have only one exit point, and that should be the last statement in the method |
| 1239 |
regenstrief\mw\observation\ObservationViewAdapter.java |
278 |
Avoid using if statements without curly braces |
| 1240 |
regenstrief\mw\observation\ObservationViewAdapter.java |
280 |
A method should have only one exit point, and that should be the last statement in the method |
| 1241 |
regenstrief\mw\observation\ObservationViewAdapter.java |
282 |
A method should have only one exit point, and that should be the last statement in the method |
| 1242 |
regenstrief\mw\observation\ObservationViewAdapter.java |
282 |
Avoid using if...else statements without curly braces |
| 1243 |
regenstrief\mw\observation\ObservationViewAdapter.java |
284 |
Use equals() to compare object references. |
| 1244 |
regenstrief\mw\observation\ObservationViewAdapter.java |
286 |
A method should have only one exit point, and that should be the last statement in the method |
| 1245 |
regenstrief\mw\observation\ObservationViewAdapter.java |
288 |
Use equals() to compare strings instead of '==' or '!=' |
| 1246 |
regenstrief\mw\observation\ObservationViewAdapter.java |
293 |
Avoid if (x != y) ..; else ..; |
| 1247 |
regenstrief\mw\observation\ObservationViewAdapter.java |
295 |
A method should have only one exit point, and that should be the last statement in the method |
| 1248 |
regenstrief\mw\observation\ObservationViewAdapter.java |
297 |
A method should have only one exit point, and that should be the last statement in the method |
| 1249 |
regenstrief\mw\observation\ObservationViewAdapter.java |
297 |
Avoid using if...else statements without curly braces |
| 1250 |
regenstrief\mw\observation\ObservationViewAdapter.java |
300 |
Avoid using if...else statements without curly braces |
| 1251 |
regenstrief\mw\observation\ObservationViewAdapter.java |
305 |
Avoid variables with short names like ts |
| 1252 |
regenstrief\mw\observation\ObservationViewAdapter.java |
327 |
Avoid reassigning parameters such as 'mode' |
| 1253 |
regenstrief\mw\observation\ObservationViewAdapter.java |
328 |
Avoid using if statements without curly braces |
| 1254 |
regenstrief\mw\observation\ObservationViewAdapter.java |
330 |
Use equals() to compare strings instead of '==' or '!=' |
| 1255 |
regenstrief\mw\observation\ObservationViewAdapter.java |
330 |
Use equals() to compare strings instead of '==' or '!=' |
| 1256 |
regenstrief\mw\observation\ObservationViewAdapter.java |
331 |
A method should have only one exit point, and that should be the last statement in the method |
| 1257 |
regenstrief\mw\observation\ObservationViewAdapter.java |
333 |
Use equals() to compare strings instead of '==' or '!=' |
| 1258 |
regenstrief\mw\observation\ObservationViewAdapter.java |
343 |
Avoid using if...else statements without curly braces |
| 1259 |
regenstrief\mw\observation\ObservationViewAdapter.java |
346 |
The method 'createModelObject' has a Cyclomatic Complexity of 12. |
| 1260 |
regenstrief\mw\observation\ObservationViewAdapter.java |
357 |
Avoid using for statements without curly braces |
| 1261 |
regenstrief\mw\observation\ObservationViewAdapter.java |
357 |
Avoid using if statements without curly braces |
| 1262 |
regenstrief\mw\observation\ObservationViewAdapter.java |
362 |
Avoid using if statements without curly braces |
| 1263 |
regenstrief\mw\observation\ObservationViewAdapter.java |
367 |
Avoid excessively long variable names like outboundRelationships |
| 1264 |
regenstrief\mw\observation\ObservationViewAdapter.java |
368 |
Avoid using if statements without curly braces |
| 1265 |
regenstrief\mw\observation\ObservationViewAdapter.java |
369 |
Avoid using for statements without curly braces |
| 1266 |
regenstrief\mw\observation\ObservationViewAdapter.java |
372 |
Avoid excessively long variable names like newActRelationship |
| 1267 |
regenstrief\mw\observation\ObservationViewAdapter.java |
375 |
Avoid using if statements without curly braces |
| 1268 |
regenstrief\mw\observation\ObservationViewAdapter.java |
379 |
Avoid using if statements without curly braces |
| 1269 |
regenstrief\mw\observation\ObservationViewAdapter.java |
392 |
Avoid excessively long variable names like definedComponentIterator |
| 1270 |
regenstrief\mw\observation\ObservationViewAdapter.java |
399 |
Avoid excessively long variable names like componentObservation |
| 1271 |
regenstrief\mw\observation\ObservationViewAdapter.java |
409 |
Avoid using if statements without curly braces |
| 1272 |
regenstrief\mw\observation\ObservationViewAdapter.java |
431 |
Avoid excessively long variable names like actualComponentIterator |
| 1273 |
regenstrief\mw\observation\RangeValueComposition.java |
25 |
Avoid unused imports such as 'java.util.regex.Pattern' |
| 1274 |
regenstrief\mw\observation\RangeValueComposition.java |
49 |
The class 'RangeValueComposition' has a Cyclomatic Complexity of 2 (Highest = 14). |
| 1275 |
regenstrief\mw\observation\RangeValueComposition.java |
51 |
Avoid excessively long variable names like applicationContext |
| 1276 |
regenstrief\mw\observation\RangeValueComposition.java |
61 |
Avoid excessively long variable names like GREATER_OR_LESS_THAN |
| 1277 |
regenstrief\mw\observation\RangeValueComposition.java |
93 |
The method 'setObject' has a Cyclomatic Complexity of 14. |
| 1278 |
regenstrief\mw\observation\RangeValueComposition.java |
93 |
The method setObject() has an NPath complexity of 840 |
| 1279 |
regenstrief\mw\observation\RangeValueComposition.java |
95 |
Found 'DU'-anomaly for variable 'real2' (lines '95'-'178'). |
| 1280 |
regenstrief\mw\observation\RangeValueComposition.java |
96 |
Found 'DU'-anomaly for variable 'unit1' (lines '96'-'178'). |
| 1281 |
regenstrief\mw\observation\RangeValueComposition.java |
97 |
Found 'DU'-anomaly for variable 'unit2' (lines '97'-'178'). |
| 1282 |
regenstrief\mw\observation\RangeValueComposition.java |
101 |
Avoid excessively long variable names like representativeValue |
| 1283 |
regenstrief\mw\observation\RangeValueComposition.java |
104 |
Found 'DD'-anomaly for variable 'defaultUnit' (lines '104'-'106'). |
| 1284 |
regenstrief\mw\observation\RangeValueComposition.java |
104 |
Found 'DU'-anomaly for variable 'defaultUnit' (lines '104'-'178'). |
| 1285 |
regenstrief\mw\observation\RangeValueComposition.java |
106 |
Found 'DU'-anomaly for variable 'defaultUnit' (lines '106'-'178'). |
| 1286 |
regenstrief\mw\observation\RangeValueComposition.java |
109 |
Found 'DD'-anomaly for variable 'currentUnit' (lines '109'-'129'). |
| 1287 |
regenstrief\mw\observation\RangeValueComposition.java |
109 |
Found 'DU'-anomaly for variable 'currentUnit' (lines '109'-'178'). |
| 1288 |
regenstrief\mw\observation\RangeValueComposition.java |
129 |
Found 'DU'-anomaly for variable 'currentUnit' (lines '129'-'178'). |
| 1289 |
regenstrief\mw\observation\RangeValueComposition.java |
133 |
Avoid if (x != y) ..; else ..; |
| 1290 |
regenstrief\mw\observation\RangeValueComposition.java |
138 |
Position literals first in String comparisons |
| 1291 |
regenstrief\mw\observation\RangeValueComposition.java |
149 |
Avoid if (x != y) ..; else ..; |
| 1292 |
regenstrief\mw\observation\RangeValueComposition.java |
159 |
Avoid if (x != y) ..; else ..; |
| 1293 |
regenstrief\mw\observation\RangeValueComposition.java |
164 |
Avoid if (x != y) ..; else ..; |
| 1294 |
regenstrief\mw\observation\RangeValueComposition.java |
182 |
Avoid excessively long variable names like representativeValue |
| 1295 |
regenstrief\mw\observation\RangeValueComposition.java |
189 |
Avoid excessively long variable names like representativeValue |
| 1296 |
regenstrief\mw\observation\RangeValueComposition.java |
190 |
Avoid if (x != y) ..; else ..; |
| 1297 |
regenstrief\mw\observation\RangeValueComposition.java |
191 |
A method should have only one exit point, and that should be the last statement in the method |
| 1298 |
regenstrief\mw\observation\RangeValueComposition.java |
192 |
Avoid if (x != y) ..; else ..; |
| 1299 |
regenstrief\mw\observation\RangeValueComposition.java |
193 |
A method should have only one exit point, and that should be the last statement in the method |
| 1300 |
regenstrief\mw\observation\RangeValueComposition.java |
194 |
Avoid if (x != y) ..; else ..; |
| 1301 |
regenstrief\mw\observation\RangeValueComposition.java |
195 |
A method should have only one exit point, and that should be the last statement in the method |
| 1302 |
regenstrief\mw\observation\RangeValueComposition.java |
197 |
A method should have only one exit point, and that should be the last statement in the method |
| 1303 |
regenstrief\mw\observation\RangeValueComposition.java |
199 |
A method should have only one exit point, and that should be the last statement in the method |
| 1304 |
regenstrief\mw\observation\RangeValueComposition.java |
225 |
This final field could be made static |
| 1305 |
regenstrief\mw\observation\RangeValueComposition.java |
231 |
Found 'DU'-anomaly for variable 'QTY_GROUP' (lines '231'-'246'). |
| 1306 |
regenstrief\mw\observation\RangeValueComposition.java |
232 |
Found 'DU'-anomaly for variable 'UNIT_GROUP' (lines '232'-'246'). |
| 1307 |
regenstrief\mw\observation\RangeValueComposition.java |
258 |
This final field could be made static |
| 1308 |
regenstrief\mw\observation\RangeValueComposition.java |
285 |
This final field could be made static |
| 1309 |
regenstrief\mw\observation\RangeValueComposition.java |
291 |
Avoid if (x != y) ..; else ..; |
| 1310 |
regenstrief\mw\observation\RangeValueComposition.java |
294 |
Avoid if (x != y) ..; else ..; |
| 1311 |
regenstrief\mw\observation\RangeValueComposition.java |
315 |
This final field could be made static |
| 1312 |
regenstrief\mw\observation\RangeValueComposition.java |
345 |
These nested if statements could be combined |
| 1313 |
regenstrief\mw\patient\EditPatientCommand.java |
37 |
Avoid excessively long variable names like _isNewPatientCommand |
| 1314 |
regenstrief\mw\patient\EditPatientCommand.java |
37 |
Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
| 1315 |
regenstrief\mw\patient\EditPatientCommand.java |
39 |
Private field 'VFACTORY' could be made final; it is only initialized in the declaration or constructor. |
| 1316 |
regenstrief\mw\patient\EditPatientCommand.java |
39 |
The field name indicates a constant but its modifiers do not |
| 1317 |
regenstrief\mw\patient\EditPatientCommand.java |
49 |
Document empty constructor |
| 1318 |
regenstrief\mw\patient\EditPatientCommand.java |
49 |
It is a good practice to call super() in a constructor |
| 1319 |
regenstrief\mw\patient\EditPatientCommand.java |
100 |
Use equals() to compare object references. |
| 1320 |
regenstrief\mw\patient\EditPatientCommand.java |
103 |
A method should have only one exit point, and that should be the last statement in the method |
| 1321 |
regenstrief\mw\patient\EditPatientCommand.java |
106 |
Found 'DU'-anomaly for variable 'duplicates' (lines '106'-'136'). |
| 1322 |
regenstrief\mw\patient\EditPatientCommand.java |
118 |
A method should have only one exit point, and that should be the last statement in the method |
| 1323 |
regenstrief\mw\patient\EditPatientCommand.java |
155 |
Avoid if (x != y) ..; else ..; |
| 1324 |
regenstrief\mw\patient\EditPatientCommand.java |
159 |
A method should have only one exit point, and that should be the last statement in the method |
| 1325 |
regenstrief\mw\patient\PatientCloseCommand.java |
32 |
Document empty constructor |
| 1326 |
regenstrief\mw\patient\PatientCloseCommand.java |
32 |
It is a good practice to call super() in a constructor |
| 1327 |
regenstrief\mw\patient\PatientCloseCommand.java |
44 |
Found 'DD'-anomaly for variable 'parent' (lines '44'-'46'). |
| 1328 |
regenstrief\mw\patient\PatientCloseCommand.java |
46 |
Avoid assignments in operands |
| 1329 |
regenstrief\mw\patient\PatientCloseCommand.java |
46 |
Found 'DU'-anomaly for variable 'parent' (lines '46'-'64'). |
| 1330 |
regenstrief\mw\patient\PatientComposition.java |
25 |
Avoid unused imports such as 'java.util.regex.Pattern' |
| 1331 |
regenstrief\mw\patient\PatientComposition.java |
41 |
Avoid excessively long variable names like _personViewAdapter |
| 1332 |
regenstrief\mw\patient\PatientComposition.java |
41 |
Possible unsafe assignment to a non-final static field in a constructor. |
| 1333 |
regenstrief\mw\patient\PatientComposition.java |
43 |
Avoid excessively long variable names like _patientViewAdapter |
| 1334 |
regenstrief\mw\patient\PatientComposition.java |
43 |
Private field '_patientViewAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 1335 |
regenstrief\mw\patient\PatientComposition.java |
45 |
Avoid excessively long variable names like applicationContext |
| 1336 |
regenstrief\mw\patient\PatientComposition.java |
56 |
Private field 'clearables' could be made final; it is only initialized in the declaration or constructor. |
| 1337 |
regenstrief\mw\patient\PatientComposition.java |
92 |
This final field could be made static |
| 1338 |
regenstrief\mw\patient\PatientComposition.java |
94 |
This final field could be made static |
| 1339 |
regenstrief\mw\patient\PatientComposition.java |
121 |
This final field could be made static |
| 1340 |
regenstrief\mw\patient\PatientComposition.java |
123 |
This final field could be made static |
| 1341 |
regenstrief\mw\patient\PatientComposition.java |
136 |
Found 'DD'-anomaly for variable 'name' (lines '136'-'138'). |
| 1342 |
regenstrief\mw\patient\PatientComposition.java |
153 |
This final field could be made static |
| 1343 |
regenstrief\mw\patient\PatientComposition.java |
155 |
This final field could be made static |
| 1344 |
regenstrief\mw\patient\PatientComposition.java |
168 |
Found 'DD'-anomaly for variable 'name' (lines '168'-'170'). |
| 1345 |
regenstrief\mw\patient\PatientComposition.java |
185 |
This final field could be made static |
| 1346 |
regenstrief\mw\patient\PatientComposition.java |
187 |
This final field could be made static |
| 1347 |
regenstrief\mw\patient\PatientListViewController.java |
48 |
A method should have only one exit point, and that should be the last statement in the method |
| 1348 |
regenstrief\mw\patient\PatientListViewController.java |
100 |
Document empty method |
| 1349 |
regenstrief\mw\patient\PatientListViewController.java |
105 |
Document empty method |
| 1350 |
regenstrief\mw\patient\PatientViewAdapter.java |
45 |
Avoid excessively long variable names like _personViewAdapter |
| 1351 |
regenstrief\mw\patient\PatientViewAdapter.java |
45 |
Private field '_personViewAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 1352 |
regenstrief\mw\patient\PatientViewAdapter.java |
47 |
Avoid excessively long variable names like applicationContext |
| 1353 |
regenstrief\mw\patient\PatientViewAdapter.java |
54 |
Avoid reassigning parameters such as 'hpathExpression' |
| 1354 |
regenstrief\mw\patient\PatientViewAdapter.java |
58 |
A method should have only one exit point, and that should be the last statement in the method |
| 1355 |
regenstrief\mw\patient\PatientViewAdapter.java |
62 |
Use equals() to compare object references. |
| 1356 |
regenstrief\mw\patient\PatientViewAdapter.java |
65 |
Avoid using if...else statements without curly braces |
| 1357 |
regenstrief\mw\patient\PatientViewAdapter.java |
68 |
Avoid variables with short names like c |
| 1358 |
regenstrief\mw\patient\PatientViewAdapter.java |
74 |
A method should have only one exit point, and that should be the last statement in the method |
| 1359 |
regenstrief\mw\patient\PatientViewAdapter.java |
104 |
Position literals first in String comparisons |
| 1360 |
regenstrief\mw\patient\PersonViewAdapter.java |
50 |
The class 'PersonViewAdapter' has a Cyclomatic Complexity of 5 (Highest = 10). |
| 1361 |
regenstrief\mw\patient\PersonViewAdapter.java |
55 |
Avoid excessively long variable names like EXPR_GENDER_CODE_NAME |
| 1362 |
regenstrief\mw\patient\PersonViewAdapter.java |
78 |
The method 'setCriteria' has a Cyclomatic Complexity of 10. |
| 1363 |
regenstrief\mw\patient\PersonViewAdapter.java |
86 |
Avoid using if statements without curly braces |
| 1364 |
regenstrief\mw\patient\PersonViewAdapter.java |
86 |
Position literals first in String comparisons |
| 1365 |
regenstrief\mw\patient\PersonViewAdapter.java |
92 |
Avoid using if statements without curly braces |
| 1366 |
regenstrief\mw\patient\PersonViewAdapter.java |
92 |
Position literals first in String comparisons |
| 1367 |
regenstrief\mw\patient\PersonViewAdapter.java |
99 |
Avoid using if statements without curly braces |
| 1368 |
regenstrief\mw\patient\PersonViewAdapter.java |
103 |
Avoid using if statements without curly braces |
| 1369 |
regenstrief\mw\patient\PersonViewAdapter.java |
111 |
Avoid unused private methods such as 'setAddressItem(T,String,AddressPartType)'. |
| 1370 |
regenstrief\mw\patient\PersonViewAdapter.java |
113 |
Found 'DD'-anomaly for variable 'shouldUpdate' (lines '113'-'136'). |
| 1371 |
regenstrief\mw\patient\PersonViewAdapter.java |
114 |
Found 'DD'-anomaly for variable 'addTo' (lines '114'-'129'). |
| 1372 |
regenstrief\mw\patient\PersonViewAdapter.java |
114 |
Found 'DU'-anomaly for variable 'addTo' (lines '114'-'146'). |
| 1373 |
regenstrief\mw\patient\PersonViewAdapter.java |
115 |
Found 'DU'-anomaly for variable 'adCollection' (lines '115'-'146'). |
| 1374 |
regenstrief\mw\patient\PersonViewAdapter.java |
123 |
Avoid variables with short names like ad |
| 1375 |
regenstrief\mw\patient\PersonViewAdapter.java |
128 |
Found 'DD'-anomaly for variable 'shouldUpdate' (lines '128'-'136'). |
| 1376 |
regenstrief\mw\patient\PersonViewAdapter.java |
129 |
Found 'DD'-anomaly for variable 'addTo' (lines '129'-'129'). |
| 1377 |
regenstrief\mw\patient\PersonViewAdapter.java |
136 |
Found 'DD'-anomaly for variable 'shouldUpdate' (lines '136'-'136'). |
| 1378 |
regenstrief\mw\patient\PersonViewAdapter.java |
137 |
Found 'DD'-anomaly for variable 'addTo' (lines '137'-'129'). |
| 1379 |
regenstrief\mw\patient\PersonViewAdapter.java |
137 |
Found 'DU'-anomaly for variable 'addTo' (lines '137'-'146'). |
| 1380 |
regenstrief\mw\problem\ChangeProblemStatusCommand.java |
20 |
Avoid using implementation types like 'ArrayList'; use the interface instead |
| 1381 |
regenstrief\mw\problem\ChangeProblemStatusCommand.java |
25 |
Document empty constructor |
| 1382 |
regenstrief\mw\problem\ChangeProblemStatusCommand.java |
25 |
It is a good practice to call super() in a constructor |
| 1383 |
regenstrief\mw\problem\ChangeProblemStatusCommand.java |
36 |
A switch statement does not contain a break |
| 1384 |
regenstrief\mw\problem\ChangeProblemStatusCommand.java |
38 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 1385 |
regenstrief\mw\problem\ChangeProblemStatusCommand.java |
39 |
A method should have only one exit point, and that should be the last statement in the method |
| 1386 |
regenstrief\mw\problem\EditProblemCommand.java |
38 |
The field name indicates a constant but its modifiers do not |
| 1387 |
regenstrief\mw\problem\EditProblemCommand.java |
44 |
Document empty constructor |
| 1388 |
regenstrief\mw\problem\EditProblemCommand.java |
44 |
It is a good practice to call super() in a constructor |
| 1389 |
regenstrief\mw\problem\EditProblemCommand.java |
57 |
Avoid if (x != y) ..; else ..; |
| 1390 |
regenstrief\mw\problem\EditProblemCommand.java |
84 |
A method should have only one exit point, and that should be the last statement in the method |
| 1391 |
regenstrief\mw\problem\EditProblemCommand.java |
119 |
Avoid if (x != y) ..; else ..; |
| 1392 |
regenstrief\mw\problem\EditProblemCommand.java |
132 |
Overriding method merely calls super |
| 1393 |
regenstrief\mw\problem\ProblemListController.java |
30 |
Avoid unused imports such as 'net.sf.xwsf.server.events.SelectionEvent' |
| 1394 |
regenstrief\mw\problem\ProblemListController.java |
32 |
Avoid unused imports such as 'old.net.sf.xwsf.gui.Execution' |
| 1395 |
regenstrief\mw\problem\ProblemListController.java |
48 |
Avoid reassigning parameters such as 'view' |
| 1396 |
regenstrief\mw\problem\ProblemListController.java |
49 |
Avoid using final local variables, turn them into fields |
| 1397 |
regenstrief\mw\problem\ProblemListController.java |
51 |
The String literal "item" appears 6 times in this file; the first occurrence is on line 51 |
| 1398 |
regenstrief\mw\problem\ProblemListController.java |
57 |
The String literal "list" appears 6 times in this file; the first occurrence is on line 57 |
| 1399 |
regenstrief\mw\problem\ProblemListController.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 1400 |
regenstrief\mw\problem\ProblemListController.java |
80 |
A method should have only one exit point, and that should be the last statement in the method |
| 1401 |
regenstrief\mw\problem\ProblemListController.java |
87 |
Private field '_code' could be made final; it is only initialized in the declaration or constructor. |
| 1402 |
regenstrief\mw\problem\ProblemListController.java |
105 |
Avoid using if...else statements without curly braces |
| 1403 |
regenstrief\mw\problem\ProblemListController.java |
122 |
Avoid unused local variables such as 'foo'. |
| 1404 |
regenstrief\mw\problem\ProblemListController.java |
122 |
Found 'DU'-anomaly for variable 'foo' (lines '122'-'134'). |
| 1405 |
regenstrief\mw\problem\ProblemListController.java |
145 |
Document empty method |
| 1406 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
34 |
Avoid excessively long variable names like PROBLEM_DISPLAY_NAME |
| 1407 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
36 |
Avoid excessively long variable names like _problemNodeAdapter |
| 1408 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
36 |
Private field '_problemNodeAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 1409 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
45 |
Avoid reassigning parameters such as 'expressionString' |
| 1410 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
49 |
Use equals() to compare object references. |
| 1411 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
51 |
A method should have only one exit point, and that should be the last statement in the method |
| 1412 |
regenstrief\mw\problem\ProblemNodeSubjectViewAdapter.java |
63 |
Avoid excessively long variable names like newObservationComponentRelationship |
| 1413 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
45 |
The class 'ProblemNodeViewAdapter' has a Cyclomatic Complexity of 4 (Highest = 10). |
| 1414 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
46 |
Avoid excessively long variable names like NODE_DISPLAY_STRING |
| 1415 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
49 |
Avoid excessively long variable names like _observationAdapter |
| 1416 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
56 |
Avoid variables with short names like a |
| 1417 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
59 |
Use equals() to compare object references. |
| 1418 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
62 |
A method should have only one exit point, and that should be the last statement in the method |
| 1419 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
62 |
Avoid using if...else statements without curly braces |
| 1420 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
64 |
A method should have only one exit point, and that should be the last statement in the method |
| 1421 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
64 |
Avoid using if...else statements without curly braces |
| 1422 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
66 |
Avoid using if...else statements without curly braces |
| 1423 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
72 |
Avoid using final local variables, turn them into fields |
| 1424 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
94 |
Avoid reassigning parameters such as 'mode' |
| 1425 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
95 |
Avoid using if statements without curly braces |
| 1426 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
98 |
Use equals() to compare strings instead of '==' or '!=' |
| 1427 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
98 |
Use equals() to compare strings instead of '==' or '!=' |
| 1428 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
99 |
A method should have only one exit point, and that should be the last statement in the method |
| 1429 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
99 |
Avoid using if...else statements without curly braces |
| 1430 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
100 |
Use equals() to compare strings instead of '==' or '!=' |
| 1431 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
109 |
A method should have only one exit point, and that should be the last statement in the method |
| 1432 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
110 |
Use equals() to compare strings instead of '==' or '!=' |
| 1433 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
117 |
Avoid using if...else statements without curly braces |
| 1434 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
120 |
The method 'createModelObject' has a Cyclomatic Complexity of 10. |
| 1435 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
129 |
Avoid using if statements without curly braces |
| 1436 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
130 |
Avoid using for statements without curly braces |
| 1437 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
135 |
Avoid using if statements without curly braces |
| 1438 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
139 |
Found 'DU'-anomaly for variable 'command' (lines '139'-'159'). |
| 1439 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
140 |
Avoid excessively long variable names like outboundRelationships |
| 1440 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
141 |
Avoid using if statements without curly braces |
| 1441 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
144 |
Avoid excessively long variable names like newActRelationship |
| 1442 |
regenstrief\mw\problem\ProblemNodeViewAdapter.java |
147 |
Avoid using if statements without curly braces |
| 1443 |
regenstrief\mw\user\EditUserCommand.java |
3 |
Avoid unused imports such as 'java.util.List' |
| 1444 |
regenstrief\mw\user\EditUserCommand.java |
5 |
Avoid unused imports such as 'net.sf.xwsf.server.SharedVariable' |
| 1445 |
regenstrief\mw\user\EditUserCommand.java |
8 |
Avoid unused imports such as 'org.hl7.rim.Patient' |
| 1446 |
regenstrief\mw\user\EditUserCommand.java |
15 |
Avoid unused private fields such as 'VFACTORY'. |
| 1447 |
regenstrief\mw\user\EditUserCommand.java |
15 |
Private field 'VFACTORY' could be made final; it is only initialized in the declaration or constructor. |
| 1448 |
regenstrief\mw\user\EditUserCommand.java |
15 |
The field name indicates a constant but its modifiers do not |
| 1449 |
regenstrief\mw\user\EditUserCommand.java |
16 |
Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
| 1450 |
regenstrief\mw\user\EditUserCommand.java |
21 |
Document empty constructor |
| 1451 |
regenstrief\mw\user\EditUserCommand.java |
21 |
It is a good practice to call super() in a constructor |
| 1452 |
regenstrief\mw\user\EditUserCommand.java |
60 |
Use equals() to compare object references. |
| 1453 |
regenstrief\mw\user\EditUserCommand.java |
63 |
A method should have only one exit point, and that should be the last statement in the method |
| 1454 |
regenstrief\mw\user\UserAuthenticateCommand.java |
45 |
Avoid excessively long variable names like _userParticipationViewAdapter |
| 1455 |
regenstrief\mw\user\UserAuthenticateCommand.java |
56 |
Avoid excessively long variable names like _authenticationStatusVariable |
| 1456 |
regenstrief\mw\user\UserAuthenticateCommand.java |
56 |
Private field '_authenticationStatusVariable' could be made final; it is only initialized in the declaration or constructor. |
| 1457 |
regenstrief\mw\user\UserAuthenticateCommand.java |
59 |
Avoid excessively long variable names like _authenticatedSession |
| 1458 |
regenstrief\mw\user\UserAuthenticateCommand.java |
67 |
Document empty constructor |
| 1459 |
regenstrief\mw\user\UserAuthenticateCommand.java |
67 |
It is a good practice to call super() in a constructor |
| 1460 |
regenstrief\mw\user\UserAuthenticateCommand.java |
90 |
Avoid excessively long variable names like authorParticipation |
| 1461 |
regenstrief\mw\user\UserAuthenticateCommand.java |
94 |
A method should have only one exit point, and that should be the last statement in the method |
| 1462 |
regenstrief\mw\user\UserAuthenticateCommand.java |
101 |
A method should have only one exit point, and that should be the last statement in the method |
| 1463 |
regenstrief\mw\user\UserAuthenticateCommand.java |
118 |
A method should have only one exit point, and that should be the last statement in the method |
| 1464 |
regenstrief\mw\user\UserAuthenticateCommand.java |
140 |
Avoid excessively long variable names like authenticationMessage |
| 1465 |
regenstrief\mw\user\UserLogOffCommand.java |
34 |
Document empty constructor |
| 1466 |
regenstrief\mw\user\UserLogOffCommand.java |
34 |
It is a good practice to call super() in a constructor |
| 1467 |
regenstrief\mw\user\UserLogOffCommand.java |
46 |
Found 'DD'-anomaly for variable 'parent' (lines '46'-'49'). |
| 1468 |
regenstrief\mw\user\UserLogOffCommand.java |
48 |
Found 'DD'-anomaly for variable 'session' (lines '48'-'53'). |
| 1469 |
regenstrief\mw\user\UserLogOffCommand.java |
48 |
Found 'DU'-anomaly for variable 'session' (lines '48'-'70'). |
| 1470 |
regenstrief\mw\user\UserLogOffCommand.java |
49 |
Avoid assignments in operands |
| 1471 |
regenstrief\mw\user\UserLogOffCommand.java |
49 |
Found 'DU'-anomaly for variable 'parent' (lines '49'-'70'). |
| 1472 |
regenstrief\mw\user\UserLogOffCommand.java |
74 |
Private field '_msg' could be made final; it is only initialized in the declaration or constructor. |
| 1473 |
regenstrief\mw\user\UserParticipationViewAdapter.java |
33 |
Private field '_userViewAdapter' could be made final; it is only initialized in the declaration or constructor. |
| 1474 |
regenstrief\mw\user\UserViewAdapter.java |
68 |
Avoid reassigning parameters such as 'hpathExpression' |
| 1475 |
regenstrief\mw\user\UserViewAdapter.java |
73 |
A method should have only one exit point, and that should be the last statement in the method |
| 1476 |
regenstrief\mw\user\UserViewAdapter.java |
77 |
Use equals() to compare object references. |
| 1477 |
regenstrief\mw\user\UserViewAdapter.java |
81 |
A method should have only one exit point, and that should be the last statement in the method |
| 1478 |
regenstrief\mw\user\UserViewAdapter.java |
94 |
Avoid unused private fields such as 'EMPTY_LIST'. |
| 1479 |
regenstrief\mw\user\UserViewAdapter.java |
94 |
The field name indicates a constant but its modifiers do not |
| 1480 |
regenstrief\mw\user\UserViewAdapter.java |
96 |
Avoid reassigning parameters such as 'queryMode' |
| 1481 |
regenstrief\mw\user\UserViewAdapter.java |
99 |
Use equals() to compare strings instead of '==' or '!=' |
| 1482 |
regenstrief\mw\user\UserViewAdapter.java |
103 |
Avoid variables with short names like id |
| 1483 |
regenstrief\mw\user\UserViewAdapter.java |
119 |
A method should have only one exit point, and that should be the last statement in the method |
| 1484 |
regenstrief\mw\util\BAGasAStringConversion.java |
51 |
A class which only has private constructors should be final |
| 1485 |
regenstrief\mw\util\BAGasAStringConversion.java |
53 |
It is somewhat confusing to have a field name with the same name as a method |
| 1486 |
regenstrief\mw\util\BAGasAStringConversion.java |
53 |
It is somewhat confusing to have a field name with the same name as a method |
| 1487 |
regenstrief\mw\util\BAGasAStringConversion.java |
55 |
Private field 'evaluator' could be made final; it is only initialized in the declaration or constructor. |
| 1488 |
regenstrief\mw\util\BAGasAStringConversion.java |
64 |
Avoid excessively long variable names like previousExpression |
| 1489 |
regenstrief\mw\util\BAGasAStringConversion.java |
70 |
Avoid excessively long variable names like previousExpression |
| 1490 |
regenstrief\mw\util\BAGasAStringConversion.java |
88 |
Use equals() to compare strings instead of '==' or '!=' |
| 1491 |
regenstrief\mw\util\BAGasAStringConversion.java |
90 |
A method should have only one exit point, and that should be the last statement in the method |
| 1492 |
regenstrief\mw\util\BAGasAStringConversion.java |
92 |
Use equals() to compare strings instead of '==' or '!=' |
| 1493 |
regenstrief\mw\util\BAGasAStringConversion.java |
94 |
A method should have only one exit point, and that should be the last statement in the method |
| 1494 |
regenstrief\mw\util\BAGasAStringConversion.java |
96 |
Use equals() to compare strings instead of '==' or '!=' |
| 1495 |
regenstrief\mw\util\BAGasAStringConversion.java |
98 |
A method should have only one exit point, and that should be the last statement in the method |
| 1496 |
regenstrief\mw\util\BAGasAStringConversion.java |
100 |
Use equals() to compare strings instead of '==' or '!=' |
| 1497 |
regenstrief\mw\util\BAGasAStringConversion.java |
102 |
A method should have only one exit point, and that should be the last statement in the method |
| 1498 |
regenstrief\mw\util\BAGasAStringConversion.java |
104 |
Position literals first in String comparisons |
| 1499 |
regenstrief\mw\util\BAGasAStringConversion.java |
106 |
A method should have only one exit point, and that should be the last statement in the method |
| 1500 |
regenstrief\mw\util\BAGasAStringConversion.java |
108 |
Use equals() to compare strings instead of '==' or '!=' |
| 1501 |
regenstrief\mw\util\BAGasAStringConversion.java |
110 |
A method should have only one exit point, and that should be the last statement in the method |
| 1502 |
regenstrief\mw\util\BAGasAStringConversion.java |
112 |
Use equals() to compare strings instead of '==' or '!=' |
| 1503 |
regenstrief\mw\util\BAGasAStringConversion.java |
114 |
A method should have only one exit point, and that should be the last statement in the method |
| 1504 |
regenstrief\mw\util\BAGasAStringConversion.java |
126 |
Avoid unused local variables such as 'theBag'. |
| 1505 |
regenstrief\mw\util\BAGasAStringConversion.java |
126 |
Found 'DU'-anomaly for variable 'theBag' (lines '126'-'159'). |
| 1506 |
regenstrief\mw\util\BAGasAStringConversion.java |
127 |
Use equals() to compare strings instead of '==' or '!=' |
| 1507 |
regenstrief\mw\util\BAGasAStringConversion.java |
129 |
A method should have only one exit point, and that should be the last statement in the method |
| 1508 |
regenstrief\mw\util\BAGasAStringConversion.java |
131 |
Use equals() to compare strings instead of '==' or '!=' |
| 1509 |
regenstrief\mw\util\BAGasAStringConversion.java |
133 |
A method should have only one exit point, and that should be the last statement in the method |
| 1510 |
regenstrief\mw\util\BAGasAStringConversion.java |
135 |
Use equals() to compare strings instead of '==' or '!=' |
| 1511 |
regenstrief\mw\util\BAGasAStringConversion.java |
137 |
A method should have only one exit point, and that should be the last statement in the method |
| 1512 |
regenstrief\mw\util\BAGasAStringConversion.java |
139 |
Use equals() to compare strings instead of '==' or '!=' |
| 1513 |
regenstrief\mw\util\BAGasAStringConversion.java |
141 |
A method should have only one exit point, and that should be the last statement in the method |
| 1514 |
regenstrief\mw\util\BAGasAStringConversion.java |
143 |
Use equals() to compare strings instead of '==' or '!=' |
| 1515 |
regenstrief\mw\util\BAGasAStringConversion.java |
145 |
A method should have only one exit point, and that should be the last statement in the method |
| 1516 |
regenstrief\mw\util\BAGasAStringConversion.java |
147 |
Use equals() to compare strings instead of '==' or '!=' |
| 1517 |
regenstrief\mw\util\BAGasAStringConversion.java |
149 |
A method should have only one exit point, and that should be the last statement in the method |
| 1518 |
regenstrief\mw\util\BAGasAStringConversion.java |
151 |
Use equals() to compare strings instead of '==' or '!=' |
| 1519 |
regenstrief\mw\util\BAGasAStringConversion.java |
153 |
A method should have only one exit point, and that should be the last statement in the method |
| 1520 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
50 |
A class which only has private constructors should be final |
| 1521 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
52 |
It is somewhat confusing to have a field name with the same name as a method |
| 1522 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
52 |
It is somewhat confusing to have a field name with the same name as a method |
| 1523 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
61 |
Avoid excessively long variable names like previousExpression |
| 1524 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
67 |
Avoid excessively long variable names like previousExpression |
| 1525 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
86 |
Found 'DU'-anomaly for variable 'criterionCode' (lines '86'-'96'). |
| 1526 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
90 |
A method should have only one exit point, and that should be the last statement in the method |
| 1527 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
103 |
A method should have only one exit point, and that should be the last statement in the method |
| 1528 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
110 |
Found 'DU'-anomaly for variable 'criterionCode' (lines '110'-'122'). |
| 1529 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
112 |
Avoid unused local variables such as 'queryName'. |
| 1530 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
112 |
Found 'DU'-anomaly for variable 'queryName' (lines '112'-'122'). |
| 1531 |
regenstrief\mw\util\CVasBooleanConditionConversion.java |
116 |
A method should have only one exit point, and that should be the last statement in the method |
| 1532 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
49 |
A class which only has private constructors should be final |
| 1533 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
49 |
The class 'CVasDisplayStringConversion' has a Cyclomatic Complexity of 4 (Highest = 12). |
| 1534 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
51 |
It is somewhat confusing to have a field name with the same name as a method |
| 1535 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
51 |
It is somewhat confusing to have a field name with the same name as a method |
| 1536 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
58 |
Avoid excessively long variable names like previousExpression |
| 1537 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
65 |
Avoid excessively long variable names like previousExpression |
| 1538 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 1539 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
84 |
A method should have only one exit point, and that should be the last statement in the method |
| 1540 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
86 |
A method should have only one exit point, and that should be the last statement in the method |
| 1541 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
94 |
Avoid reassigning parameters such as 'displayName' |
| 1542 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
94 |
The method 'convertBackward' has a Cyclomatic Complexity of 12. |
| 1543 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
94 |
The method convertBackward() has an NPath complexity of 288 |
| 1544 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
96 |
A method should have only one exit point, and that should be the last statement in the method |
| 1545 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
104 |
Found 'DD'-anomaly for variable 'observationCode' (lines '104'-'110'). |
| 1546 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
105 |
Avoid excessively long variable names like observationCodeSystem |
| 1547 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
105 |
Found 'DD'-anomaly for variable 'observationCodeSystem' (lines '105'-'111'). |
| 1548 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
108 |
Avoid variables with short names like cd |
| 1549 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
115 |
Found 'DD'-anomaly for variable 'code' (lines '115'-'120'). |
| 1550 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
115 |
Found 'DD'-anomaly for variable 'code' (lines '115'-'129'). |
| 1551 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
115 |
Found 'DD'-anomaly for variable 'code' (lines '115'-'138'). |
| 1552 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
125 |
Position literals first in String comparisons |
| 1553 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
128 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 1554 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
133 |
When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
| 1555 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
149 |
A method should have only one exit point, and that should be the last statement in the method |
| 1556 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
149 |
Avoid using if...else statements without curly braces |
| 1557 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
150 |
Position literals first in String comparisons |
| 1558 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
151 |
Avoid using if...else statements without curly braces |
| 1559 |
regenstrief\mw\util\CVasDisplayStringConversion.java |
153 |
Avoid using if...else statements without curly braces |
| 1560 |
regenstrief\mw\util\CVasImageConversion.java |
47 |
A class which only has private constructors should be final |
| 1561 |
regenstrief\mw\util\CVasImageConversion.java |
50 |
It is somewhat confusing to have a field name with the same name as a method |
| 1562 |
regenstrief\mw\util\CVasImageConversion.java |
50 |
It is somewhat confusing to have a field name with the same name as a method |
| 1563 |
regenstrief\mw\util\CVasImageConversion.java |
59 |
Avoid excessively long variable names like previousExpression |
| 1564 |
regenstrief\mw\util\CVasImageConversion.java |
65 |
Avoid excessively long variable names like previousExpression |
| 1565 |
regenstrief\mw\util\CVasImageConversion.java |
83 |
Found 'DD'-anomaly for variable 'param0' (lines '83'-'89'). |
| 1566 |
regenstrief\mw\util\CVasImageConversion.java |
103 |
A method should have only one exit point, and that should be the last statement in the method |
| 1567 |
regenstrief\mw\util\CommonAdapterUtils.java |
20 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 1568 |
regenstrief\mw\util\CommonAdapterUtils.java |
43 |
Avoid excessively long variable names like ESCAPED_CHARACTERS |
| 1569 |
regenstrief\mw\util\CommonAdapterUtils.java |
58 |
Avoid excessively long variable names like patientParticipation |
| 1570 |
regenstrief\mw\util\CommonAdapterUtils.java |
67 |
Avoid if (x != y) ..; else ..; |
| 1571 |
regenstrief\mw\util\CommonAdapterUtils.java |
68 |
A method should have only one exit point, and that should be the last statement in the method |
| 1572 |
regenstrief\mw\util\CommonAdapterUtils.java |
81 |
Avoid if (x != y) ..; else ..; |
| 1573 |
regenstrief\mw\util\CommonAdapterUtils.java |
106 |
Avoid unnecessary if..then..else statements when returning a boolean |
| 1574 |
regenstrief\mw\util\CommonAdapterUtils.java |
111 |
A method should have only one exit point, and that should be the last statement in the method |
| 1575 |
regenstrief\mw\util\CommonAdapterUtils.java |
117 |
Use explicit scoping instead of the default package private level |
| 1576 |
regenstrief\mw\util\DatabaseProperties.java |
20 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 1577 |
regenstrief\mw\util\DatabaseProperties.java |
31 |
It is somewhat confusing to have a field name with the same name as a method |
| 1578 |
regenstrief\mw\util\DatabaseProperties.java |
33 |
It is somewhat confusing to have a field name with the same name as a method |
| 1579 |
regenstrief\mw\util\DatabaseProperties.java |
35 |
It is somewhat confusing to have a field name with the same name as a method |
| 1580 |
regenstrief\mw\util\DatabaseProperties.java |
37 |
It is somewhat confusing to have a field name with the same name as a method |
| 1581 |
regenstrief\mw\util\DatabaseProperties.java |
39 |
It is somewhat confusing to have a field name with the same name as a method |
| 1582 |
regenstrief\mw\util\DatabaseProperties.java |
45 |
It is somewhat confusing to have a field name with the same name as a method |
| 1583 |
regenstrief\mw\util\DatabaseProperties.java |
45 |
The field name indicates a constant but its modifiers do not |
| 1584 |
regenstrief\mw\util\DatabaseURIResolver.java |
56 |
Avoid variables with short names like c |
| 1585 |
regenstrief\mw\util\DatabaseURIResolver.java |
67 |
System.exit() should not be used in J2EE/JEE apps |
| 1586 |
regenstrief\mw\util\DatabaseURIResolver.java |
76 |
System.exit() should not be used in J2EE/JEE apps |
| 1587 |
regenstrief\mw\util\DatabaseURIResolver.java |
80 |
Avoid reassigning parameters such as 'href' |
| 1588 |
regenstrief\mw\util\DatabaseURIResolver.java |
84 |
Found 'DD'-anomaly for variable 'result' (lines '84'-'93'). |
| 1589 |
regenstrief\mw\util\DatabaseURIResolver.java |
84 |
Found 'DD'-anomaly for variable 'result' (lines '84'-'99'). |
| 1590 |
regenstrief\mw\util\DatabaseURIResolver.java |
88 |
Found 'DD'-anomaly for variable 'url' (lines '88'-'92'). |
| 1591 |
regenstrief\mw\util\DatabaseURIResolver.java |
88 |
Found 'DD'-anomaly for variable 'url' (lines '88'-'98'). |
| 1592 |
regenstrief\mw\util\DatabaseURIResolver.java |
93 |
Found 'DU'-anomaly for variable 'result' (lines '93'-'144'). |
| 1593 |
regenstrief\mw\util\DatabaseURIResolver.java |
99 |
Found 'DU'-anomaly for variable 'result' (lines '99'-'144'). |
| 1594 |
regenstrief\mw\util\DatabaseURIResolver.java |
106 |
Avoid if (x != y) ..; else ..; |
| 1595 |
regenstrief\mw\util\DatabaseURIResolver.java |
109 |
A method should have only one exit point, and that should be the last statement in the method |
| 1596 |
regenstrief\mw\util\DatabaseURIResolver.java |
119 |
A method should have only one exit point, and that should be the last statement in the method |
| 1597 |
regenstrief\mw\util\DatabaseURIResolver.java |
124 |
Found 'DD'-anomaly for variable 'result' (lines '124'-'132'). |
| 1598 |
regenstrief\mw\util\DatabaseURIResolver.java |
130 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 1599 |
regenstrief\mw\util\DatabaseURIResolver.java |
130 |
Avoid variables with short names like rs |
| 1600 |
regenstrief\mw\util\DatabaseURIResolver.java |
130 |
Ensure that resources like this ResultSet object are closed after use |
| 1601 |
regenstrief\mw\util\DatabaseURIResolver.java |
140 |
Avoid variables with short names like sr |
| 1602 |
regenstrief\mw\util\DatabaseURIResolver.java |
146 |
Avoid reassigning parameters such as 'uri' |
| 1603 |
regenstrief\mw\util\DatabaseURIResolver.java |
156 |
A method should have only one exit point, and that should be the last statement in the method |
| 1604 |
regenstrief\mw\util\DatabaseURIResolver.java |
175 |
A method should have only one exit point, and that should be the last statement in the method |
| 1605 |
regenstrief\mw\util\InitializeDatabase.java |
20 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 1606 |
regenstrief\mw\util\Printer.java |
41 |
Avoid duplicate imports such as 'net.sf.xwsf.server.ApplicationContext' |
| 1607 |
regenstrief\mw\util\Printer.java |
42 |
Avoid duplicate imports such as 'net.sf.xwsf.server.ViewFactory' |
| 1608 |
regenstrief\mw\util\Printer.java |
51 |
Avoid excessively long variable names like PRINT_TRANSFORM_URI |
| 1609 |
regenstrief\mw\util\Printer.java |
53 |
Avoid excessively long variable names like _transformerFactory |
| 1610 |
regenstrief\mw\util\Printer.java |
114 |
Avoid excessively long variable names like applicationContext |
| 1611 |
regenstrief\mw\util\Printer.java |
122 |
Avoid excessively long variable names like destinationConnection |
| 1612 |
regenstrief\mw\util\Printer.java |
126 |
Found 'DU'-anomaly for variable 'rightNow' (lines '126'-'198'). |
| 1613 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
13 |
A class which only has private constructors should be final |
| 1614 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
15 |
It is somewhat confusing to have a field name with the same name as a method |
| 1615 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
15 |
It is somewhat confusing to have a field name with the same name as a method |
| 1616 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
26 |
Avoid excessively long variable names like previousExpression |
| 1617 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
32 |
Avoid excessively long variable names like previousExpression |
| 1618 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
41 |
Position literals first in String comparisons |
| 1619 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
43 |
A method should have only one exit point, and that should be the last statement in the method |
| 1620 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
47 |
Avoid variables with short names like st |
| 1621 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
48 |
Found 'DD'-anomaly for variable 'numeratorString' (lines '48'-'52'). |
| 1622 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
49 |
Found 'DD'-anomaly for variable 'denominatorString' (lines '49'-'56'). |
| 1623 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
67 |
Avoid if (x != y) ..; else ..; |
| 1624 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
74 |
A method should have only one exit point, and that should be the last statement in the method |
| 1625 |
regenstrief\mw\util\RTOasDisplayStringConversion.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 1626 |
regenstrief\mw\util\SQLutils.java |
63 |
Avoid variables with short names like c |
| 1627 |
regenstrief\mw\util\SQLutils.java |
128 |
System.exit() should not be used in J2EE/JEE apps |
| 1628 |
regenstrief\mw\util\SQLutils.java |
138 |
System.exit() should not be used in J2EE/JEE apps |
| 1629 |
regenstrief\mw\util\SQLutils.java |
145 |
Avoid variables with short names like id |
| 1630 |
regenstrief\mw\util\SQLutils.java |
145 |
Method names should not contain underscores |
| 1631 |
regenstrief\mw\util\SQLutils.java |
147 |
Found 'DD'-anomaly for variable 'content' (lines '147'-'155'). |
| 1632 |
regenstrief\mw\util\SQLutils.java |
152 |
Avoid variables with short names like x |
| 1633 |
regenstrief\mw\util\SQLutils.java |
153 |
Avoid variables with short names like b |
| 1634 |
regenstrief\mw\util\SQLutils.java |
164 |
Avoid reassigning parameters such as 'content' |
| 1635 |
regenstrief\mw\util\SQLutils.java |
164 |
Avoid reassigning parameters such as 'id' |
| 1636 |
regenstrief\mw\util\SQLutils.java |
164 |
Avoid variables with short names like id |
| 1637 |
regenstrief\mw\util\SQLutils.java |
164 |
Method names should not contain underscores |
| 1638 |
regenstrief\mw\util\SQLutils.java |
178 |
The String literal " where id=\'" appears 4 times in this file; the first occurrence is on line 178 |
| 1639 |
regenstrief\mw\util\SQLutils.java |
178 |
The String literal "\';" appears 5 times in this file; the first occurrence is on line 178 |
| 1640 |
regenstrief\mw\util\SQLutils.java |
180 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 1641 |
regenstrief\mw\util\SQLutils.java |
180 |
Avoid variables with short names like rs |
| 1642 |
regenstrief\mw\util\SQLutils.java |
180 |
Ensure that resources like this ResultSet object are closed after use |
| 1643 |
regenstrief\mw\util\SQLutils.java |
207 |
Method names should not contain underscores |
| 1644 |
regenstrief\mw\util\SQLutils.java |
236 |
Avoid variables with short names like id |
| 1645 |
regenstrief\mw\util\SQLutils.java |
236 |
Method names should not contain underscores |
| 1646 |
regenstrief\mw\util\SQLutils.java |
246 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 1647 |
regenstrief\mw\util\SQLutils.java |
246 |
Avoid variables with short names like rs |
| 1648 |
regenstrief\mw\util\SQLutils.java |
246 |
Ensure that resources like this ResultSet object are closed after use |
| 1649 |
regenstrief\mw\util\SQLutils.java |
249 |
Avoid variables with short names like p |
| 1650 |
regenstrief\mw\util\SQLutils.java |
269 |
Method names should not contain underscores |
| 1651 |
regenstrief\mw\util\SQLutils.java |
281 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 1652 |
regenstrief\mw\util\SQLutils.java |
281 |
Ensure that resources like this ResultSet object are closed after use |
| 1653 |
regenstrief\mw\util\SQLutils.java |
291 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 1654 |
regenstrief\mw\util\SQLutils.java |
291 |
Ensure that resources like this ResultSet object are closed after use |
| 1655 |
regenstrief\mw\util\SQLutils.java |
293 |
Avoid excessively long variable names like xsltgui_xsl_rs_string |
| 1656 |
regenstrief\mw\util\SQLutils.java |
336 |
String.indexOf(char) is faster than String.indexOf(String). |
| 1657 |
regenstrief\mw\util\StatusGuru.java |
37 |
The class 'StatusGuru' has a Cyclomatic Complexity of 14 (Highest = 27). |
| 1658 |
regenstrief\mw\util\StatusGuru.java |
40 |
Avoid really long methods. |
| 1659 |
regenstrief\mw\util\StatusGuru.java |
40 |
The method 'getSetMode' has a Cyclomatic Complexity of 27. |
| 1660 |
regenstrief\mw\util\StatusGuru.java |
47 |
Avoid if (x != y) ..; else ..; |
| 1661 |
regenstrief\mw\util\StatusGuru.java |
53 |
A method should have only one exit point, and that should be the last statement in the method |
| 1662 |
regenstrief\mw\util\StatusGuru.java |
57 |
A method should have only one exit point, and that should be the last statement in the method |
| 1663 |
regenstrief\mw\util\StatusGuru.java |
61 |
A method should have only one exit point, and that should be the last statement in the method |
| 1664 |
regenstrief\mw\util\StatusGuru.java |
65 |
A method should have only one exit point, and that should be the last statement in the method |
| 1665 |
regenstrief\mw\util\StatusGuru.java |
72 |
A method should have only one exit point, and that should be the last statement in the method |
| 1666 |
regenstrief\mw\util\StatusGuru.java |
76 |
A method should have only one exit point, and that should be the last statement in the method |
| 1667 |
regenstrief\mw\util\StatusGuru.java |
80 |
A method should have only one exit point, and that should be the last statement in the method |
| 1668 |
regenstrief\mw\util\StatusGuru.java |
84 |
A method should have only one exit point, and that should be the last statement in the method |
| 1669 |
regenstrief\mw\util\StatusGuru.java |
91 |
A method should have only one exit point, and that should be the last statement in the method |
| 1670 |
regenstrief\mw\util\StatusGuru.java |
95 |
A method should have only one exit point, and that should be the last statement in the method |
| 1671 |
regenstrief\mw\util\StatusGuru.java |
99 |
A method should have only one exit point, and that should be the last statement in the method |
| 1672 |
regenstrief\mw\util\StatusGuru.java |
103 |
A method should have only one exit point, and that should be the last statement in the method |
| 1673 |
regenstrief\mw\util\StatusGuru.java |
108 |
A method should have only one exit point, and that should be the last statement in the method |
| 1674 |
regenstrief\mw\util\StatusGuru.java |
113 |
A method should have only one exit point, and that should be the last statement in the method |
| 1675 |
regenstrief\mw\util\StatusGuru.java |
122 |
A method should have only one exit point, and that should be the last statement in the method |
| 1676 |
regenstrief\mw\util\StatusGuru.java |
136 |
A method should have only one exit point, and that should be the last statement in the method |
| 1677 |
regenstrief\mw\util\StatusGuru.java |
140 |
A method should have only one exit point, and that should be the last statement in the method |
| 1678 |
regenstrief\mw\util\StatusGuru.java |
145 |
A method should have only one exit point, and that should be the last statement in the method |
| 1679 |
regenstrief\mw\util\StatusGuru.java |
154 |
A method should have only one exit point, and that should be the last statement in the method |
| 1680 |
regenstrief\mw\util\StatusGuru.java |
158 |
A method should have only one exit point, and that should be the last statement in the method |
| 1681 |
regenstrief\mw\util\WordIndexer.java |
65 |
The class 'WordIndexer' has a Cyclomatic Complexity of 4 (Highest = 12). |
| 1682 |
regenstrief\mw\util\WordIndexer.java |
77 |
Private field '_template' could be made final; it is only initialized in the declaration or constructor. |
| 1683 |
regenstrief\mw\util\WordIndexer.java |
78 |
Private field '_parameterFormats' could be made final; it is only initialized in the declaration or constructor. |
| 1684 |
regenstrief\mw\util\WordIndexer.java |
80 |
Private field '_stringBuilder' could be made final; it is only initialized in the declaration or constructor. |
| 1685 |
regenstrief\mw\util\WordIndexer.java |
81 |
Private field '_formatter' could be made final; it is only initialized in the declaration or constructor. |
| 1686 |
regenstrief\mw\util\WordIndexer.java |
95 |
Avoid variables with short names like i |
| 1687 |
regenstrief\mw\util\WordIndexer.java |
115 |
Use explicit scoping instead of the default package private level |
| 1688 |
regenstrief\mw\util\WordIndexer.java |
205 |
The String literal "INSERT INTO CONCEPT_NAME(code,codeSystem,displayName,className,attributeName)\n" appears 5 times in this file; the first occurrence is on line 205 |
| 1689 |
regenstrief\mw\util\WordIndexer.java |
311 |
The method 'indexValueSets' has a Cyclomatic Complexity of 12. |
| 1690 |
regenstrief\mw\util\WordIndexer.java |
314 |
Avoid variables with short names like rs |
| 1691 |
regenstrief\mw\util\WordIndexer.java |
314 |
Ensure that resources like this ResultSet object are closed after use |
| 1692 |
regenstrief\mw\util\WordIndexer.java |
320 |
Found 'DU'-anomaly for variable 'insertStmt' (lines '320'-'365'). |
| 1693 |
regenstrief\mw\util\WordIndexer.java |
324 |
Found 'DD'-anomaly for variable 'observationCode' (lines '324'-'324'). |
| 1694 |
regenstrief\mw\util\WordIndexer.java |
324 |
Found 'DU'-anomaly for variable 'observationCode' (lines '324'-'365'). |
| 1695 |
regenstrief\mw\util\WordIndexer.java |
325 |
Avoid excessively long variable names like observationCodeSystem |
| 1696 |
regenstrief\mw\util\WordIndexer.java |
325 |
Found 'DD'-anomaly for variable 'observationCodeSystem' (lines '325'-'325'). |
| 1697 |
regenstrief\mw\util\WordIndexer.java |
325 |
Found 'DU'-anomaly for variable 'observationCodeSystem' (lines '325'-'365'). |
| 1698 |
regenstrief\mw\util\WordIndexer.java |
327 |
Avoid variables with short names like is |
| 1699 |
regenstrief\mw\util\WordIndexer.java |
327 |
Found 'DD'-anomaly for variable 'is' (lines '327'-'327'). |
| 1700 |
regenstrief\mw\util\WordIndexer.java |
327 |
Found 'DU'-anomaly for variable 'is' (lines '327'-'365'). |
| 1701 |
regenstrief\mw\util\WordIndexer.java |
333 |
Avoid using if statements without curly braces |
| 1702 |
regenstrief\mw\util\WordIndexer.java |
336 |
Deeply nested if..then statements are hard to read |
| 1703 |
regenstrief\mw\util\WordIndexer.java |
336 |
No need to check for null before an instanceof |
| 1704 |
regenstrief\mw\util\WordIndexer.java |
379 |
Avoid variables with short names like rs |
| 1705 |
regenstrief\mw\util\WordIndexer.java |
393 |
Avoid excessively long variable names like insertRelationshipStmt |
| 1706 |
regenstrief\mw\util\WordIndexer.java |
398 |
Found 'DD'-anomaly for variable 'code' (lines '398'-'398'). |
| 1707 |
regenstrief\mw\util\WordIndexer.java |
398 |
Found 'DU'-anomaly for variable 'code' (lines '398'-'463'). |
| 1708 |
regenstrief\mw\util\WordIndexer.java |
401 |
Found 'DD'-anomaly for variable 'className' (lines '401'-'401'). |
| 1709 |
regenstrief\mw\util\WordIndexer.java |
401 |
Found 'DU'-anomaly for variable 'className' (lines '401'-'463'). |
| 1710 |
regenstrief\mw\util\WordIndexer.java |
402 |
Found 'DD'-anomaly for variable 'attributeName' (lines '402'-'402'). |
| 1711 |
regenstrief\mw\util\WordIndexer.java |
402 |
Found 'DU'-anomaly for variable 'attributeName' (lines '402'-'463'). |
| 1712 |
regenstrief\mw\util\WordIndexer.java |
404 |
Avoid using if statements without curly braces |
| 1713 |
regenstrief\mw\util\WordIndexer.java |
404 |
Position literals first in String comparisons |
| 1714 |
regenstrief\mw\util\WordIndexer.java |
407 |
Avoid excessively long variable names like uninvertedDisplayName |
| 1715 |
regenstrief\mw\util\WordIndexer.java |
428 |
Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. |
| 1716 |
regenstrief\mw\util\WordIndexer.java |
431 |
Avoid excessively long variable names like leadTermCodeSystem |
| 1717 |
regenstrief\mw\util\WordIndexer.java |
433 |
Found 'DD'-anomaly for variable 'leadTermCode' (lines '433'-'433'). |
| 1718 |
regenstrief\mw\util\WordIndexer.java |
433 |
Found 'DU'-anomaly for variable 'leadTermCode' (lines '433'-'463'). |
| 1719 |
regenstrief\mw\util\WordIndexer.java |
434 |
Found 'DD'-anomaly for variable 'leadTermCodeSystem' (lines '434'-'434'). |
| 1720 |
regenstrief\mw\util\WordIndexer.java |
434 |
Found 'DU'-anomaly for variable 'leadTermCodeSystem' (lines '434'-'463'). |
| 1721 |
regenstrief\mw\util\WordIndexer.java |
436 |
Deeply nested if..then statements are hard to read |
| 1722 |
regenstrief\mw\util\WordIndexer.java |
487 |
Avoid reassigning parameters such as 'string' |
| 1723 |
regenstrief\mw\util\WordIndexer.java |
491 |
A method should have only one exit point, and that should be the last statement in the method |
| 1724 |
regenstrief\mw\util\WordIndexer.java |
491 |
Avoid using if...else statements without curly braces |
| 1725 |
regenstrief\mw\util\WordIndexer.java |
495 |
A method should have only one exit point, and that should be the last statement in the method |
| 1726 |
regenstrief\mw\util\WordIndexer.java |
495 |
Avoid using if...else statements without curly braces |
| 1727 |
regenstrief\mw\util\WordIndexer.java |
497 |
Avoid using if...else statements without curly braces |
| 1728 |
regenstrief\mw\util\WordIndexer.java |
503 |
Avoid using final local variables, turn them into fields |
| 1729 |
regenstrief\mw\util\WordIndexer.java |
537 |
Avoid variables with short names like n |
| 1730 |
regenstrief\mw\util\WordIndexer.java |
540 |
Avoid assignments in operands |
| 1731 |
regenstrief\mw\util\WordIndexer.java |
540 |
Avoid using while statements without curly braces |
| 1732 |
regenstrief\mw\util\WordIndexer.java |
558 |
The String literal "INSERT INTO CODE_IMAGE_MAP(code, codeSystem, priorityNumber, param0, imagePath)\n" appears 8 times in this file; the first occurrence is on line 558 |
| 1733 |
regenstrief\mw\util\WordIndexer.java |
588 |
Avoid variables with short names like rs |
| 1734 |
regenstrief\mw\util\WordIndexer.java |
599 |
Found 'DD'-anomaly for variable 'code' (lines '599'-'599'). |
| 1735 |
regenstrief\mw\util\WordIndexer.java |
599 |
Found 'DU'-anomaly for variable 'code' (lines '599'-'636'). |
| 1736 |
regenstrief\mw\util\WordIndexer.java |
600 |
Found 'DD'-anomaly for variable 'codeSystem' (lines '600'-'600'). |
| 1737 |
regenstrief\mw\util\WordIndexer.java |
600 |
Found 'DU'-anomaly for variable 'codeSystem' (lines '600'-'636'). |
| 1738 |
regenstrief\mw\util\WordIndexer.java |
602 |
Found 'DD'-anomaly for variable 'className' (lines '602'-'602'). |
| 1739 |
regenstrief\mw\util\WordIndexer.java |
602 |
Found 'DU'-anomaly for variable 'className' (lines '602'-'636'). |
| 1740 |
regenstrief\mw\util\WordIndexer.java |
603 |
Found 'DD'-anomaly for variable 'attributeName' (lines '603'-'603'). |
| 1741 |
regenstrief\mw\util\WordIndexer.java |
603 |
Found 'DU'-anomaly for variable 'attributeName' (lines '603'-'636'). |
| 1742 |
regenstrief\mw\util\WordIndexer.java |
604 |
Found 'DD'-anomaly for variable 'observationCode' (lines '604'-'604'). |
| 1743 |
regenstrief\mw\util\WordIndexer.java |
604 |
Found 'DU'-anomaly for variable 'observationCode' (lines '604'-'636'). |
| 1744 |
regenstrief\mw\util\WordIndexer.java |
605 |
Avoid excessively long variable names like observationCodeSystem |
| 1745 |
regenstrief\mw\util\WordIndexer.java |
605 |
Found 'DD'-anomaly for variable 'observationCodeSystem' (lines '605'-'605'). |
| 1746 |
regenstrief\mw\util\WordIndexer.java |
605 |
Found 'DU'-anomaly for variable 'observationCodeSystem' (lines '605'-'636'). |
| 1747 |
regenstrief\mw\util\WordIndexer.java |
609 |
Avoid variables with short names like st |
| 1748 |
regenstrief\mw\util\WordIndexer.java |
613 |
Found 'DD'-anomaly for variable 'index' (lines '613'-'613'). |
| 1749 |
regenstrief\mw\util\WordIndexer.java |
613 |
Found 'DU'-anomaly for variable 'index' (lines '613'-'636'). |
| 1750 |
regenstrief\mw\util\WordIndexer.java |
628 |
Found 'DD'-anomaly for variable 'index' (lines '628'-'613'). |
| 1751 |
regenstrief\mw\util\WordIndexer.java |
628 |
Found 'DU'-anomaly for variable 'index' (lines '628'-'636'). |
| 1752 |
regenstrief\mw\util\WordIndexer.java |
642 |
Avoid excessively long variable names like queryPartitionSQLFormat |
| 1753 |
regenstrief\mw\util\WordIndexer.java |
650 |
Avoid variables with short names like rs |
| 1754 |
regenstrief\mw\util\WordIndexer.java |
659 |
Found 'DD'-anomaly for variable 'entityInternalId' (lines '659'-'659'). |
| 1755 |
regenstrief\mw\util\WordIndexer.java |
659 |
Found 'DU'-anomaly for variable 'entityInternalId' (lines '659'-'691'). |
| 1756 |
regenstrief\mw\util\WordIndexer.java |
660 |
Found 'DD'-anomaly for variable 'component' (lines '660'-'660'). |
| 1757 |
regenstrief\mw\util\WordIndexer.java |
660 |
Found 'DU'-anomaly for variable 'component' (lines '660'-'691'). |
| 1758 |
regenstrief\mw\util\WordIndexer.java |
662 |
Avoid excessively long variable names like internalDiscriminator |
| 1759 |
regenstrief\mw\util\WordIndexer.java |
662 |
Found 'DD'-anomaly for variable 'internalDiscriminator' (lines '662'-'662'). |
| 1760 |
regenstrief\mw\util\WordIndexer.java |
662 |
Found 'DU'-anomaly for variable 'internalDiscriminator' (lines '662'-'691'). |
| 1761 |
regenstrief\mw\util\WordIndexer.java |
667 |
Avoid variables with short names like st |
| 1762 |
regenstrief\mw\util\WordIndexer.java |
671 |
Found 'DD'-anomaly for variable 'index' (lines '671'-'671'). |
| 1763 |
regenstrief\mw\util\WordIndexer.java |
671 |
Found 'DU'-anomaly for variable 'index' (lines '671'-'691'). |
| 1764 |
regenstrief\mw\util\WordIndexer.java |
683 |
Found 'DD'-anomaly for variable 'index' (lines '683'-'671'). |
| 1765 |
regenstrief\mw\util\WordIndexer.java |
683 |
Found 'DU'-anomaly for variable 'index' (lines '683'-'691'). |
| 1766 |
regenstrief\mw\util\WordIndexer.java |
695 |
Avoid variables with short names like st |
| 1767 |
regenstrief\mw\util\WordIndexer.java |
697 |
Avoid using if statements without curly braces |
| 1768 |
regenstrief\mw\util\WordIndexer.java |
715 |
Avoid variables with short names like t |
| 1769 |
regenstrief\mw\util\WordIndexer.java |
718 |
Avoid using while statements without curly braces |
| 1770 |
regenstrief\mw\util\WordIndexer.java |
719 |
Avoid appending characters as strings in StringBuffer.append. |
| 1771 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
29 |
The class 'ApplicationActViewAdapter' has a Cyclomatic Complexity of 7 (Highest = 12). |
| 1772 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
42 |
The method 'getProperty' has a Cyclomatic Complexity of 12. |
| 1773 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
46 |
A method should have only one exit point, and that should be the last statement in the method |
| 1774 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
50 |
Use equals() to compare object references. |
| 1775 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
52 |
A method should have only one exit point, and that should be the last statement in the method |
| 1776 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
53 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 1777 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
55 |
Use equals() to compare strings instead of '==' or '!=' |
| 1778 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
57 |
A method should have only one exit point, and that should be the last statement in the method |
| 1779 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
59 |
Use equals() to compare object references. |
| 1780 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
63 |
A method should have only one exit point, and that should be the last statement in the method |
| 1781 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
70 |
A method should have only one exit point, and that should be the last statement in the method |
| 1782 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
74 |
Avoid variables with short names like sb |
| 1783 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
85 |
Avoid appending characters as strings in StringBuffer.append. |
| 1784 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
90 |
Avoid appending characters as strings in StringBuffer.append. |
| 1785 |
regenstrief\mw\viewadapters\ApplicationActViewAdapter.java |
102 |
A method should have only one exit point, and that should be the last statement in the method |
| 1786 |
regenstrief\mw\viewadapters\ComponentRelationshipViewAdapter.java |
40 |
Avoid excessively long variable names like newComponentRelationship |
| 1787 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
55 |
Avoid excessively long variable names like _applicationContext |
| 1788 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
55 |
Private field '_applicationContext' could be made final; it is only initialized in the declaration or constructor. |
| 1789 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
57 |
Avoid excessively long variable names like QUERY_RESULTS_LIMIT |
| 1790 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
59 |
Avoid excessively long variable names like applicationContext |
| 1791 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
59 |
It is a good practice to call super() in a constructor |
| 1792 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
68 |
Avoid unnecessary if..then..else statements when returning a boolean |
| 1793 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
70 |
A method should have only one exit point, and that should be the last statement in the method |
| 1794 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
118 |
Document empty method |
| 1795 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
123 |
Avoid excessively long variable names like newSubjectRelationship |
| 1796 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
140 |
Found 'DU'-anomaly for variable 'garbage' (lines '140'-'176'). |
| 1797 |
regenstrief\mw\viewadapters\GenericViewAdapter.java |
151 |
A method should have only one exit point, and that should be the last statement in the method |
| 1798 |
regenstrief\mw\viewadapters\HPathEvaluator.java |
36 |
When instantiating a SimpleDateFormat object, specify a Locale |
| 1799 |
regenstrief\mw\viewadapters\HPathEvaluator.java |
62 |
A method should have only one exit point, and that should be the last statement in the method |
| 1800 |
regenstrief\mw\viewadapters\HPathEvaluator.java |
68 |
A method should have only one exit point, and that should be the last statement in the method |
| 1801 |
regenstrief\mw\viewadapters\HPathEvaluator.java |
72 |
A method should have only one exit point, and that should be the last statement in the method |
| 1802 |
regenstrief\mw\viewadapters\HPathEvaluator.java |
94 |
Use equals() to compare object references. |
| 1803 |
regenstrief\mw\viewadapters\NotificationChannelViewAdapter.java |
36 |
Avoid excessively long variable names like applicationContext |
| 1804 |
regenstrief\mw\viewadapters\NotificationChannelViewAdapter.java |
42 |
Avoid excessively long variable names like notificationChannel |
| 1805 |
regenstrief\mw\viewadapters\NotificationChannelViewAdapter.java |
45 |
Use equals() to compare strings instead of '==' or '!=' |
| 1806 |
regenstrief\mw\viewadapters\NotificationChannelViewAdapter.java |
47 |
A method should have only one exit point, and that should be the last statement in the method |
| 1807 |
regenstrief\mw\viewadapters\ObjectBuffer.java |
38 |
Avoid modifiers which are implied by the context |
| 1808 |
regenstrief\mw\viewadapters\ObjectBuffer.java |
47 |
Avoid modifiers which are implied by the context |
| 1809 |
regenstrief\mw\viewadapters\ObjectBuffer.java |
49 |
Avoid modifiers which are implied by the context |
| 1810 |
regenstrief\mw\viewadapters\ObjectBuffer.java |
51 |
Avoid modifiers which are implied by the context |
| 1811 |
regenstrief\mw\viewadapters\PrintViewAdapter.java |
37 |
Avoid excessively long variable names like applicationContext |
| 1812 |
regenstrief\mw\viewadapters\PrintViewAdapter.java |
57 |
Avoid if (x != y) ..; else ..; |
| 1813 |
regenstrief\mw\viewadapters\PrintViewAdapter.java |
59 |
A method should have only one exit point, and that should be the last statement in the method |
| 1814 |
regenstrief\mw\viewadapters\PrintViewAdapter.java |
70 |
A method should have only one exit point, and that should be the last statement in the method |
| 1815 |
regenstrief\mw\viewadapters\PrintViewAdapter.java |
75 |
A method should have only one exit point, and that should be the last statement in the method |
| 1816 |
regenstrief\protocol\db\DatabaseURLConnection.java |
43 |
Private field '_url' could be made final; it is only initialized in the declaration or constructor. |
| 1817 |
regenstrief\protocol\db\DatabaseURLConnection.java |
65 |
String.indexOf(char) is faster than String.indexOf(String). |
| 1818 |
regenstrief\protocol\db\DatabaseURLConnection.java |
93 |
Avoid variables with short names like c |
| 1819 |
regenstrief\protocol\db\DatabaseURLConnection.java |
93 |
Ensure that resources like this Connection object are closed after use |
| 1820 |
regenstrief\protocol\db\DatabaseURLConnection.java |
96 |
Avoid variables with short names like ps |
| 1821 |
regenstrief\protocol\db\DatabaseURLConnection.java |
97 |
Avoid variables with short names like rs |
| 1822 |
regenstrief\protocol\db\DatabaseURLConnection.java |
97 |
Ensure that resources like this ResultSet object are closed after use |
| 1823 |
regenstrief\protocol\db\Handler.java |
27 |
Avoid unnecessary constructors - the compiler will generate these for you |
| 1824 |
regenstrief\protocol\db\Handler.java |
27 |
Document empty constructor |
| 1825 |
regenstrief\protocol\db\Handler.java |
27 |
It is a good practice to call super() in a constructor |
| 1826 |
regenstrief\protocol\tcp\Handler.java |
42 |
A method should have only one exit point, and that should be the last statement in the method |
| 1827 |
regenstrief\util\PersistenceGhost.java |
22 |
Document empty constructor |
| 1828 |
regenstrief\util\PersistenceGhost.java |
22 |
It is a good practice to call super() in a constructor |
| 1829 |
regenstrief\util\PersistenceGhost.java |
31 |
Avoid unused constructor parameters such as 'password'. |
| 1830 |
regenstrief\util\PersistenceGhost.java |
31 |
Avoid unused constructor parameters such as 'username'. |
| 1831 |
regenstrief\util\PersistenceGhost.java |
31 |
It is a good practice to call super() in a constructor |
| 1832 |
regenstrief\util\PersistenceGhost.java |
52 |
Avoid variables with short names like id |
| 1833 |
regenstrief\util\PersistenceGhost.java |
59 |
Document empty method |
| 1834 |
regenstrief\util\PersistenceGhost.java |
100 |
Document empty method |
| 1835 |
regenstrief\util\PersistenceGhost.java |
105 |
Document empty method |
| 1836 |
regenstrief\util\PersistenceGhost.java |
110 |
Document empty method |
| 1837 |
regenstrief\util\PersistenceGhost.java |
115 |
Document empty method |
| 1838 |
regenstrief\util\PersistenceGhost.java |
120 |
Document empty method |
| 1839 |
regenstrief\util\PersistenceGhost.java |
137 |
Document empty method |
| 1840 |
regenstrief\util\PersistenceGhost.java |
148 |
Document empty method |
| 1841 |
regenstrief\util\TRAXPipe.java |
25 |
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
| 1842 |
regenstrief\util\TRAXPipe.java |
63 |
The class 'TRAXPipe' has a Cyclomatic Complexity of 39 (Highest = 38). |
| 1843 |
regenstrief\util\TRAXPipe.java |
64 |
Use explicit scoping instead of the default package private level |
| 1844 |
regenstrief\util\TRAXPipe.java |
66 |
Avoid really long methods. |
| 1845 |
regenstrief\util\TRAXPipe.java |
66 |
The method 'main' has a Cyclomatic Complexity of 38. |
| 1846 |
regenstrief\util\TRAXPipe.java |
66 |
The method main() has an NCSS line count of 137 |
| 1847 |
regenstrief\util\TRAXPipe.java |
66 |
The method main() has an NPath complexity of 2073600 |
| 1848 |
regenstrief\util\TRAXPipe.java |
69 |
Found 'DD'-anomaly for variable 'reader' (lines '69'-'85'). |
| 1849 |
regenstrief\util\TRAXPipe.java |
70 |
Avoid unused local variables such as 'xslreader'. |
| 1850 |
regenstrief\util\TRAXPipe.java |
70 |
Found 'DU'-anomaly for variable 'xslreader' (lines '70'-'245'). |
| 1851 |
regenstrief\util\TRAXPipe.java |
72 |
Found 'DD'-anomaly for variable 'sourceParser' (lines '72'-'84'). |
| 1852 |
regenstrief\util\TRAXPipe.java |
106 |
Avoid using if statements without curly braces |
| 1853 |
regenstrief\util\TRAXPipe.java |
115 |
Avoid using if statements without curly braces |
| 1854 |
regenstrief\util\TRAXPipe.java |
118 |
Avoid using if statements without curly braces |
| 1855 |
regenstrief\util\TRAXPipe.java |
121 |
Avoid using if statements without curly braces |
| 1856 |
regenstrief\util\TRAXPipe.java |
125 |
Found 'DD'-anomaly for variable 'argiLastTransform' (lines '125'-'128'). |
| 1857 |
regenstrief\util\TRAXPipe.java |
127 |
Avoid using if statements without curly braces |
| 1858 |
regenstrief\util\TRAXPipe.java |
128 |
Found 'DD'-anomaly for variable 'argiLastTransform' (lines '128'-'128'). |
| 1859 |
regenstrief\util\TRAXPipe.java |
139 |
Found 'DU'-anomaly for variable 'reader' (lines '139'-'245'). |
| 1860 |
regenstrief\util\TRAXPipe.java |
149 |
Found 'DU'-anomaly for variable 'reader' (lines '149'-'245'). |
| 1861 |
regenstrief\util\TRAXPipe.java |
160 |
Avoid using if...else statements without curly braces |
| 1862 |
regenstrief\util\TRAXPipe.java |
166 |
Found 'DD'-anomaly for variable 'transformer' (lines '166'-'170'). |
| 1863 |
regenstrief\util\TRAXPipe.java |
166 |
Found 'DD'-anomaly for variable 'transformer' (lines '166'-'177'). |
| 1864 |
regenstrief\util\TRAXPipe.java |
166 |
Found 'DU'-anomaly for variable 'transformer' (lines '166'-'245'). |
| 1865 |
regenstrief\util\TRAXPipe.java |
182 |
Found 'DD'-anomaly for variable 'arg' (lines '182'-'182'). |
| 1866 |
regenstrief\util\TRAXPipe.java |
182 |
Found 'DU'-anomaly for variable 'arg' (lines '182'-'245'). |
| 1867 |
regenstrief\util\TRAXPipe.java |
184 |
Avoid using if statements without curly braces |
| 1868 |
regenstrief\util\TRAXPipe.java |
190 |
Found 'DD'-anomaly for variable 'input' (lines '190'-'197'). |
| 1869 |
regenstrief\util\TRAXPipe.java |
190 |
Found 'DD'-anomaly for variable 'input' (lines '190'-'204'). |
| 1870 |
regenstrief\util\TRAXPipe.java |
190 |
Found 'DD'-anomaly for variable 'input' (lines '190'-'208'). |
| 1871 |
regenstrief\util\TRAXPipe.java |
191 |
Found 'DD'-anomaly for variable 'result' (lines '191'-'213'). |
| 1872 |
regenstrief\util\TRAXPipe.java |
191 |
Found 'DD'-anomaly for variable 'result' (lines '191'-'219'). |
| 1873 |
regenstrief\util\TRAXPipe.java |
193 |
Found 'DD'-anomaly for variable 'inputDirectory' (lines '193'-'202'). |
| 1874 |
regenstrief\util\TRAXPipe.java |
193 |
Found 'DU'-anomaly for variable 'inputDirectory' (lines '193'-'245'). |
| 1875 |
regenstrief\util\TRAXPipe.java |
194 |
Found 'DD'-anomaly for variable 'outputDirectory' (lines '194'-'217'). |
| 1876 |
regenstrief\util\TRAXPipe.java |
194 |
Found 'DU'-anomaly for variable 'outputDirectory' (lines '194'-'245'). |
| 1877 |
regenstrief\util\TRAXPipe.java |
202 |
Found 'DU'-anomaly for variable 'inputDirectory' (lines '202'-'245'). |
| 1878 |
regenstrief\util\TRAXPipe.java |
217 |
Found 'DU'-anomaly for variable 'outputDirectory' (lines '217'-'245'). |
| 1879 |
regenstrief\util\TRAXPipe.java |
223 |
Avoid if (x != y) ..; else ..; |
| 1880 |
regenstrief\util\TRAXPipe.java |
224 |
Avoid using if...else statements without curly braces |
| 1881 |
regenstrief\util\TRAXPipe.java |
225 |
Avoid if (x != y) ..; else ..; |
| 1882 |
regenstrief\util\TRAXPipe.java |
244 |
Avoid using if...else statements without curly braces |
| 1883 |
regenstrief\util\URLCat.java |
44 |
Use explicit scoping instead of the default package private level |
| 1884 |
regenstrief\util\URLCat.java |
64 |
Found 'DU'-anomaly for variable 'destination' (lines '64'-'70'). |
| 1885 |
regenstrief\util\URLCat.java |
91 |
Found 'DD'-anomaly for variable 'filenameFilter' (lines '91'-'97'). |
| 1886 |
regenstrief\util\URLCat.java |
91 |
Found 'DU'-anomaly for variable 'filenameFilter' (lines '91'-'114'). |
| 1887 |
regenstrief\util\URLCat.java |
97 |
Found 'DU'-anomaly for variable 'filenameFilter' (lines '97'-'114'). |
| 1888 |
regenstrief\util\URLCat.java |
116 |
Avoid unused private fields such as '_error'. |
| 1889 |
regenstrief\util\URLCat.java |
119 |
Avoid excessively long variable names like destinationConnection |
| 1890 |
regenstrief\util\URLCat.java |
124 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 1891 |
regenstrief\util\URLCat.java |
125 |
A method should have only one exit point, and that should be the last statement in the method |
| 1892 |
regenstrief\util\URLCat.java |
135 |
Avoid unused method parameters such as 'sourceObject'. |
| 1893 |
regenstrief\util\URLCat.java |
136 |
Avoid excessively long variable names like destinationConnection |
| 1894 |
regenstrief\util\URLCat.java |
182 |
Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |
| 1895 |
regenstrief\util\XMLInspector.java |
33 |
Avoid unnecessary constructors - the compiler will generate these for you |
| 1896 |
regenstrief\util\XMLInspector.java |
33 |
Document empty constructor |
| 1897 |
regenstrief\util\XMLInspector.java |
33 |
It is a good practice to call super() in a constructor |
| 1898 |
regenstrief\util\XMLJoiner.java |
51 |
Avoid unnecessary constructors - the compiler will generate these for you |
| 1899 |
regenstrief\util\XMLJoiner.java |
51 |
Document empty constructor |
| 1900 |
regenstrief\util\XMLJoiner.java |
51 |
It is a good practice to call super() in a constructor |
| 1901 |
regenstrief\util\XMLJoiner.java |
67 |
A method should have only one exit point, and that should be the last statement in the method |
| 1902 |
regenstrief\util\XMLJoiner.java |
71 |
Avoid variables with short names like p |
| 1903 |
regenstrief\util\XMLJoiner.java |
78 |
A method should have only one exit point, and that should be the last statement in the method |
| 1904 |
regenstrief\util\XMLJoiner.java |
101 |
Avoid if (x != y) ..; else ..; |
| 1905 |
regenstrief\util\XMLMultiTransformer.java |
61 |
Use explicit scoping instead of the default package private level |
| 1906 |
regenstrief\util\XMLMultiTransformer.java |
63 |
Avoid excessively long variable names like _saxTransformerFactory |
| 1907 |
regenstrief\util\XMLMultiTransformer.java |
63 |
Use explicit scoping instead of the default package private level |
| 1908 |
regenstrief\util\XMLMultiTransformer.java |
65 |
Use explicit scoping instead of the default package private level |
| 1909 |
regenstrief\util\XMLMultiTransformer.java |
68 |
Avoid excessively long variable names like saxTransformerFactory |
| 1910 |
regenstrief\util\XMLMultiTransformer.java |
68 |
It is a good practice to call super() in a constructor |
| 1911 |
regenstrief\util\XMLMultiTransformer.java |
75 |
Use explicit scoping instead of the default package private level |
| 1912 |
regenstrief\util\XMLMultiTransformer.java |
86 |
Avoid excessively long variable names like _transformerParameters |
| 1913 |
regenstrief\util\XMLMultiTransformer.java |
86 |
Use explicit scoping instead of the default package private level |
| 1914 |
regenstrief\util\XMLMultiTransformer.java |
88 |
Avoid excessively long variable names like _transformerOutputProperties |
| 1915 |
regenstrief\util\XMLMultiTransformer.java |
88 |
Use explicit scoping instead of the default package private level |
| 1916 |
regenstrief\util\XMLMultiTransformer.java |
110 |
Avoid excessively long variable names like transformerHandler |
| 1917 |
regenstrief\util\XMLMultiTransformer.java |
110 |
Use explicit scoping instead of the default package private level |
| 1918 |
regenstrief\util\XMLMultiTransformer.java |
117 |
Avoid if (x != y) ..; else ..; |
| 1919 |
regenstrief\util\XMLMultiTransformer.java |
138 |
Found 'DU'-anomaly for variable 'transformer' (lines '138'-'179'). |
| 1920 |
regenstrief\util\XMLMultiTransformer.java |
165 |
Avoid empty catch blocks |
| 1921 |
regenstrief\util\XMLMultiTransformer.java |
186 |
Assigning an Object to null is a code smell. Consider refactoring. |
| 1922 |
regenstrief\util\XMLSplitter.java |
40 |
Avoid unnecessary constructors - the compiler will generate these for you |
| 1923 |
regenstrief\util\XMLSplitter.java |
40 |
Document empty constructor |
| 1924 |
regenstrief\util\XMLSplitter.java |
40 |
It is a good practice to call super() in a constructor |
| 1925 |
regenstrief\util\XMLSplitter.java |
44 |
Variables that are not final should not contain underscores (except for underscores in standard prefix/suffix). |