Spring Batch : custom ItemReader

Question

I have a Spring Batch project with a simple custom reader and writer. When i run the code i end up with an endeless loop printing the first item "item 1". What am i doing wrong?

Here is my code:

Reader.java

public class Reader implements ItemReader<SimpleItem> {


public SimpleItem read() throws Exception, UnexpectedInputException, ParseException {
    if (getIterator().hasNext()) {
        return getIterator().next();
    }
    return null;
}

public Iterator<SimpleItem> getIterator() {
    List<SimpleItem> list = new ArrayList();
    list.add(new SimpleItem("item 1"));
    list.add(new SimpleItem("item 2"));
    return list.iterator();
}

}

Writer.java

public class Writer implements ItemWriter<SimpleItem> {
@Override
public void write(List<? extends SimpleItem> list) throws Exception {
    for(SimpleItem item : list) {
        System.out.println(item.getName()); // this prints item 1 endelessly
    }
}
}

Michael Minella · Accepted Answer · 2015-06-17 14:09:49Z

2

Every call to ItemReader#read begins with a call to Reader#getIterator which means you get a new list with every time you call to read. You really only want to create the list once.

We already have a List based ItemReader implementation. You can take a look at the code for it on Github here: https://github.com/spring-projects/spring-batch/blob/master/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/ListItemReader.java

answered Jun 17, 2015 at 14:09

Michael Minella

21.6k4 gold badges61 silver badges69 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Spring Batch : custom ItemReader

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related