Hi there,
I'm in the Skinning StyleNode code and I see that the 'get' methods return Iterators
from the good ol' days.
It seems to me that it is better if they just return Lists so the code that iterates over
the values is cleaner using 5.0's for(String foo : yyy) construct.
Does anyone see why I wouldn't want these to return List instead of Iterator?

Here's a code snippet. Thanks, Jeanne
--

 public Iterator<IncludePropertyNode> getIncludedProperties()
 {
   if(_includedProperties == null)
   {
     List<IncludePropertyNode> list = Collections.emptyList();
     return list.iterator();
   }
   else
     return (Arrays.asList(_includedProperties)).iterator();
 }

 /**
  * Gets the properties specified by this node's parent that should be
  * ignored. This method will return an empty iterator if
  * [EMAIL PROTECTED] #isInhibitingAll()} returns <code>true</code>
  *
  * @return an iterator over the properties that should be ignored, an
  *         empty iterator if all properties should be.
  */
 public Iterator<String> getInhibitedProperties()
 {
   if(_inhibitedProperties == null)
   {
     List<String> list = Collections.emptyList();
     return list.iterator();
   }
   else
   {
     return _inhibitedProperties.iterator();
   }
 }

Reply via email to