Null pointer with iterator.next

Question

Ниже приведен фрагмент кода в java.

Collection contextPages = (Collection) getContextPages(); 
Iterator contextPageIter = contextPages.iterator(); 
while (contextPageIter.hasNext()) { 
    Page contextPage = (Page) contextPageIter.next(); 
    String result = contextPage.getResult();  // <-- Null pointer exception here 
    // Other stuff which this API does. 
} 

Этот код был в эксплуатации некоторое время. Но по какой-то причине мы нанесли нулевой указатель на String result = contextPage.getResult();.

Так выглядит, хотя мы сделали чек на hasNext(), метод next() возвращает null.

Одна из возможностей заключается в том, что сама коллекция имеет null, но возможно ли, что этот код может привести к нулевому указателю в многопоточной среде?

Answer 1

Некоторые коллекции позволяют добавлять нуль, поэтому iter.next будет возвращать его, и когда вы его используете -> NPE

Чтобы продемонстрировать

@Test 
public void testNull() { 
    Collection<Object> c = new ArrayList<>(); 
    c.add(null); 
    Iterator<Object> iterator = c.iterator(); 
    Assert.assertTrue(iterator.hasNext()); 
    Object o = iterator.next(); 
    Assert.assertNull(o); 
} 

Так убедитесь, что вы не добавляете null ссылки на коллекции или чек на null перед использованием результата Iterator#next()

Page contextPage = (Page) contextPageIter.next(); 
String result = null; 
if(contextPage != null) { 
     result = contextPage.getResult(); 
} 

Там не должно быть никакой разницы между одно- и многопоточными приложениями, просто убедитесь, что вы не добавляете null.

Answer 2

Лучше проверить нуль перед использованием результата Iterator.next()

Попробуйте это -

Page contextPage = (Page) contextPageIter.next(); 
String result = null; 
if(contextPage != null) { 
     result = contextPage.getResult(); 
}