You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@juddi.apache.org by "Alex O'Ree (JIRA)" <ju...@ws.apache.org> on 2017/02/03 02:48:52 UTC
[jira] [Resolved] (JUDDI-967) Suspicious incrementers in for loops
[ https://issues.apache.org/jira/browse/JUDDI-967?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Alex O'Ree resolved JUDDI-967.
------------------------------
Resolution: Fixed
> Suspicious incrementers in for loops
> ------------------------------------
>
> Key: JUDDI-967
> URL: https://issues.apache.org/jira/browse/JUDDI-967
> Project: jUDDI
> Issue Type: Bug
> Reporter: JC
> Assignee: Alex O'Ree
> Fix For: 3.3.4
>
>
> In recent github snapshot, I've found suspicious use of incrementers in for loops
> juddi-core/src/main/java/org/apache/juddi/validation/vsv/Uddiuddiorgcategorizationderivedfrom.java
> {code:java}
> 154 for (int i = 0; i < items.size(); i++) {
> 155 List<String> validValues = getValidValues();
> 156 if (validValues != null) {
> 157 //ok we have some work to do
> 158 boolean valid = false;
> 159 for (int k = 0; k < validValues.size(); k++) {
> 160 if (validValues.get(i).equals(items.get(i).getKeyValue())) {
> 161 valid = true;
> 162 }
> 163 }
> 164 if (!valid) {
> 165 err += xpath + "keyedReference(" + i + ") ";
> 166 }
> 167 }
> 168 }
> {code}
> In Line 160, validValues.get( i ) should be validValues.get(j)?
> juddi-core/src/main/java/org/apache/juddi/validation/vsv/Uddiuddiorgidentifierisreplacedby.java
> {code:java}
> 303 for (int i = 0; i < items.size(); i++) {
> 304 List<String> validValues = getValidValues();
> 305 if (validValues != null) {
> 306 //ok we have some work to do
> 307 boolean valid = false;
> 308 for (int k = 0; k < validValues.size(); k++) {
> 309 if (validValues.get(i).equals(items.get(i).getKeyValue())) {
> 310 valid = true;
> 311 }
> 312 }
> 313 if (!valid) {
> 314 err += xpath + "keyedReference(" + i + ") ";
> 315 }
> 316 }
> 317 }
> {code}
> In Line 309, validValues.get( i ) should be validValues.get(j)?
> When inner loops iterate, equals always compare the same objects.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)