[GROOVY-2122] .each {} should return its collection, not void

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0
• Fix Version/s: 1.1-beta-3
• Component/s: groovy-jdk
• Assignee: Jochen Theodorou
• Reporter: Jason Smith
• Created: 13/Sep/07 13:50
• Updated: 17/Sep/07 14:49
• Resolved: 17/Sep/07 14:49

Description

The .each{} gdk method should return its own collection, unaltered. That will make it possible to chain .each{} methods, as demonstrated in the code below. It may be desirable to look at other gdk methods as well.

THE CODE AS IT IS NOW (in Groovy 1.0):

def numbers = ['one', 'two', 'three', 4, 5];
def maxSize = 0;

numbers
.collect

{ maxSize = Math.max(it.toString().size(), maxSize); return it; }

.each
{
println "${it.toString().padRight(maxSize)} : ${it.getClass().name}";
};

OUTPUT:
one : java.lang.String
two : java.lang.String
three : java.lang.String
4 : java.lang.Integer
5 : java.lang.Integer

THE CODE WHERE each{} RETURNS ITS COLLECTION:

def numbers = ['one', 'two', 'three', 4, 5];
def maxSize = 0;

numbers
.each

{ maxSize = Math.max(it.toString().size(), maxSize); }

.each
{
println "${it.toString().padRight(maxSize)} : ${it.getClass().name}";
};