iluwatar#184 Fluent interface pattern, added cached initialization to anonymous iterator for lazy fluentiterable, small documentation changes

Author: hannomalie

fluentinterface/src/main/java/com/iluwatar/fluentinterface/fluentiterable/lazy/DecoratingIterator.java

@@ -4,7 +4,7 @@
 
 /**
  * This class is used to realize LazyFluentIterables. It decorates
- * a given iterator.
+ * a given iterator. Does not support consecutive hasNext() calls.
  * @param <TYPE>
  */
 public abstract class DecoratingIterator<TYPE> implements Iterator<TYPE> {

fluentinterface/src/main/java/com/iluwatar/fluentinterface/fluentiterable/lazy/LazyFluentIterable.java

@@ -126,19 +126,14 @@ public FluentIterable<TYPE> last(int count) {return new LazyFluentIterable<TYPE>
         @Override
         public Iterator<TYPE> iterator() {
             return new DecoratingIterator<TYPE>(iterable.iterator()) {
-                int currentIndex = 0;
+                public int stopIndex;
+                public int totalElementsCount;
+                private List<TYPE> list;
+                private int currentIndex = 0;
 
                 @Override
                 public TYPE computeNext() {
-                    List<TYPE> list = new ArrayList<>();
-
-                    Iterator<TYPE> newIterator = iterable.iterator();
-                    while(newIterator.hasNext()) {
-                        list.add(newIterator.next());
-                    }
-
-                    int totalElementsCount = list.size();
-                    int stopIndex = totalElementsCount - count;
+                    initialize();
 
                     TYPE candidate = null;
                     while(currentIndex < stopIndex && fromIterator.hasNext()) {
@@ -150,6 +145,19 @@ public TYPE computeNext() {
                     }
                     return candidate;
                 }
+
+                private void initialize() {
+                    if(list == null) {
+                       list = new ArrayList<>();
+                        Iterator<TYPE> newIterator = iterable.iterator();
+                        while(newIterator.hasNext()) {
+                            list.add(newIterator.next());
+                        }
+
+                        totalElementsCount = list.size();
+                        stopIndex = totalElementsCount - count;
+                    }
+                }
             };
         }
     };

fluentinterface/src/main/java/com/iluwatar/fluentinterface/fluentiterable/simple/SimpleFluentIterable.java

@@ -10,6 +10,7 @@
 /**
  * This is a simple implementation of the FluentIterable interface. It evaluates
  * all chained operations eagerly.
+ * This implementation would be costly to be utilized in real applications.
  * @param <TYPE> the type of the objects the iteration is about
  */
 public class SimpleFluentIterable<TYPE> implements FluentIterable<TYPE> {