| Issue 52: | prefer the enhanced for loop | |
| Back to list |
In utils.DataNodeArrayItemizedOverlay.updateItem()
Use the more efficient extended for-loop (foreach) instead of the explicite,index-driven for-loop.
replace
<code:java>
for (int i = 0; i < overlayItems.size(); ++i)
if (overlayItems.get(i).second.intValue() == id) {
OverlayItem oi = Helper.getOverlayItem(pos,
overlayItems.get(i).first.getMarker());
overlayItems.get(i).first = oi; // update the OverlayItem
</code>
with
<code:java>
for ( Pair<OverlayItem,Integer> overlayItem : overlayItems) {
if (overlayItem.second.intValue() == id) {
OverlayItem oi = Helper.getOverlayItem(pos,
overlayItem.first.getMarker());
overlayItem.first = oi; // update the OverlayItem
</code>
Mar 21, 2011
Can you proove that the extended for-loop is really more efficient?
Mar 22, 2011
(No comment was entered for this change.)
Status:
Started
Mar 22, 2011
(No comment was entered for this change.)
Status:
Fixed
|
Owner: benpi...@googlemail.com