Składnia Java z MongoDB

Question

Przechodzę przez wprowadzenie do MongoDB dla java. Istnieje przykładowy kod do pobierania wszystkich dokumentów z kolekcji. Kod działa, ale uważam, że jest trochę ... niepewny z braku lepszego słowa. Zastanawiam się, czy jest jakiś konkretny powód, który sprawia, że ​​jest to konieczne. Podany przykład jest:

FindIterable<Document> iterable = db.getCollection("restaurants").find(); 
iterable.forEach(new Block<Document>() { 
    @Override 
    public void apply(final Document document) { 
     System.out.println(document); 
    } 
}); 

Czy istnieje jakiś powód instancja Block musi być tworzone w każdej iteracji forEach w powyższym przykładzie? Dlaczego nie coś trochę bardziej proste jak:

FindIterable<Document> iterable = db.getCollection("restaurants").find(); 
for (Document document : iterable) { 
    System.out.println(document); 
} 

Answer 1

Choć z pewnością można użyć formularza, który zasugerował:

for (Document document : col.find()) { 
    // do something 
} 

wprowadza problem, gdy ciało pętli for zgłasza wyjątek: jeśli tak się stanie, kursor nie zostanie zamknięty. Prawidłowe idiom, aby ustrzec się przed że jest użycie MongoCursor (która implementuje zamykane) wyraźnie:

try (MongoCursor<Document> cursor = col.find().iterator()) { 
    while (cursor.hasNext()) { 
     System.out.println(cursor.next()); 
    } 
} 

Metoda forEach jest tylko trochę cukru składniowej aby uniknąć konieczności kodu aplikacji martwić konieczności zamykania kursor ręcznie w ten sposób.

Jeśli nie chcesz, aby utworzyć nowy blok dla każdej iteracji można byłaby kod wyciągnąć anonimowy wewnętrzną utworzenie klasy, np:

Block<Document> block = new Block<Document>() { 
    @Override 
    public void apply(final Document document) { 
     System.out.println(document); 
    } 
}; 
col.find().forEach(block); 

Oczywiście to jeszcze clunkier, więc jeśli są w stanie używać Javy 8, można wymienić całość z lambda:

col.find().forEach((Block<Document>) document -> { 
    System.out.println(document); 
}); 

lub w tym przypadku prościej:

col.find().forEach((Block<Document>) System.out::println); 

Dzięki lambda metafactory nie zostaną utworzone niepotrzebne obiekty.

Answer 2

zadałem sobie to samo pytanie i znalazłem dość łatwe następujący kod do obsługi tej sytuacji:

List<Document> restaurants = db.getCollection("restaurants").find().into(new ArrayList<Document>()); 

for (Document restaurant : restaurants) { 
    System.out.println(restaurant); 
}