How to use "continue" in groovy's each loop

Question

I am new to groovy (worked on java), trying to write some test cases using Spock framework. I need the following Java snippet converted into groovy snippet using "each loop"

Java Snippet:

List<String> myList = Arrays.asList("Hello", "World!", "How", "Are", "You");
for( String myObj : myList){
    if(myObj==null) {
        continue;   // need to convert this part in groovy using each loop
    }
    System.out.println("My Object is "+ myObj);
}

Groovy Snippet:

def myObj = ["Hello", "World!", "How", "Are", "You"]
myList.each{ myObj->
    if(myObj==null){
        //here I need to continue
    }
    println("My Object is " + myObj)
}

Possible duplicate of [Best pattern for simulating "continue" in Groovy closure](http://stackoverflow.com/questions/205660/best-pattern-for-simulating-continue-in-groovy-closure) — kriegaex, Jan 04 '17 at 14:52

Vampire · Accepted Answer · 2017-01-10T10:11:43.317

71

Either use return, as the closure basically is a method that is called with each element as parameter like

def myObj = ["Hello", "World!", "How", "Are", "You"]
myList.each{ myObj->
    if(myObj==null){
        return
    }
    println("My Object is " + myObj)
}

Or switch your pattern to

def myObj = ["Hello", "World!", "How", "Are", "You"]
myList.each{ myObj->
    if(myObj!=null){
        println("My Object is " + myObj)
    }
}

Or use a findAll before to filter out null objects

def myList = ["Hello", "World!", "How", "Are", null, "You"]
myList.findAll { it != null }.each{ myObj->
    println("My Object is " + myObj)
}

edited Jan 10 '17 at 10:11

answered Jan 04 '17 at 15:01

Vampire

32,892
3
65
94

OP wants to filter entries and therefore `findAll {...}` is the top answer here +1 – Alex Sep 04 '20 at 13:33
1

Last version with findAll looks the niciest. But isn't that more time consuming? Aren't you iterating over the whole collection twice then? – Yaerius Sep 17 '20 at 15:26

injecteer · Answer 2 · 2017-01-04T14:59:58.927

25

you can either use a standard for loop with continue:

for( String myObj in myList ){
  if( something ) continue
  doTheRest()
}

or use return in each's closure:

myList.each{ myObj->
  if( something ) return
  doTheRest()
}

edited Jan 04 '17 at 14:59

answered Jan 04 '17 at 14:08

injecteer

17,769
3
41
79

score 1 · Answer 3 · answered Jan 04 '17 at 14:09

1

You could also only enter your if statement if the object isn't null.

def myObj = ["Hello", "World!", "How", "Are", "You"]
myList.each{ 
    myObj->
    if(myObj!=null){
        println("My Object is " + myObj)
    }
}

answered Jan 04 '17 at 14:09

Andrew_CS

2,462
1
15
36

1

code is cleaner with the "continue" approach, when there is a lot of code to be executed when the condition is not satisfied. – Pablo Pazos Jun 09 '17 at 18:00

How to use "continue" in groovy's each loop

Java Snippet:

3 Answers3