Re: How to not use casting to invoke the methods of a List of objects

From:

Robert Klemme <shortcutter@googlemail.com>

Newsgroups:

comp.lang.java.programmer

Date:

Fri, 20 Jul 2012 18:03:08 +0200

Message-ID:

<a6tdqdF3eiU1@mid.individual.net>

On 20.07.2012 17:45, Joerg Meier wrote:

On Fri, 20 Jul 2012 06:58:49 -0700 (PDT), clusardi2k@aol.com wrote:

The below start of replacement code fails because it skips through the list. There wasn't a good reason for me to finish the code! The loop can't even iterate the required number of loops.

         Iterator <Route> it = objList.iterator();

         int size = objList.size();

         for (int i = 0;i < size; i++)
         {
            it.next();
         }

That is not really how you use iterators. Proper way:

Iterator <Route> it = objList.iterator();

while (it.hasNext()) {
it.next();
}

In 2012 I'd rather say the proper way for a simple iteration (i.e.
without removing elements or such) is

for (final Route r : objList) {
   // camel case and accessor added:
   System.out.println(r.getLocationId());
}

Assuming objList is assignment compatible to Iterable<Route>. We
haven't seen the declaration in the original posting though so we can
only speculate.

I do have to agree that the original question was quite a bit dark.

Cheers

    robert

--
remember.guy do |as, often| as.you_can - without end
http://blog.rubybestpractices.com/