Mercurial > dive4elements > river
comparison artifacts/src/main/java/org/dive4elements/river/artifacts/states/minfo/BedQualityState.java @ 7254:fe32a7f9655e
Reduce usage of careless 'for (int i = 0; i < x.size(); i++)' pattern.
author | Sascha L. Teichmann <teichmann@intevation.de> |
---|---|
date | Sat, 05 Oct 2013 17:52:16 +0200 |
parents | ea6225010871 |
children | 98ca61cb58fb |
comparison
equal
deleted
inserted
replaced
7253:90a3bcd8060a | 7254:fe32a7f9655e |
---|---|
126 String stateId, | 126 String stateId, |
127 String hash) { | 127 String hash) { |
128 List<String> diameters = access.getBedDiameter(); | 128 List<String> diameters = access.getBedDiameter(); |
129 List<String> loadDiameters = access.getBedloadDiameter(); | 129 List<String> loadDiameters = access.getBedloadDiameter(); |
130 List<DateRange> ranges = access.getDateRanges(); | 130 List<DateRange> ranges = access.getDateRanges(); |
131 for (int i = 0; i < ranges.size(); i++) { | 131 for (int i = 0, R = ranges.size(); i < R; i++) { |
132 DateRange range = ranges.get(i); | 132 DateRange range = ranges.get(i); |
133 for (String diameter: diameters) { | 133 for (String diameter: diameters) { |
134 int ndxTop = generateIndex(diameter); | 134 int ndxTop = generateIndex(diameter); |
135 int ndxSub = generateIndex(diameter); | 135 int ndxSub = ndxTop; // TODO: Is this correct? |
136 ndxTop += 1; | 136 ndxTop += 1; |
137 ndxTop = ndxTop << 3; | 137 ndxTop = ndxTop << 3; |
138 ndxSub = ndxSub << 3; | 138 ndxSub = ndxSub << 3; |
139 ndxTop += i; | 139 ndxTop += i; |
140 ndxSub += i; | 140 ndxSub += i; |