Nie podoba mnie się poniższy kod, jak zrobić to ładniej?
A może jest jakaś guava do podobnych rzeczy?
private Boolean checkingMethod(List<Book> books) {
Boolean check = null;
for (Book book: books) {
if (book.getValue() != null) {
check = true;
} else {
check = false;
}
}
return check;
Z góry thx za pomoc.
W ogole to zle to zrobilem... poniewaz po kazdym if statement nadpisuje tego checka, wiec do d**y cos takiego :D
a właściwie co ta metoda robi skoro iterujesz się po całej liście a i tak zapamiętujesz wynik z ostatniego elementu :)?
Dokładnie, sam to zauważyłem i się wyśmiałem. Trochę się zapętliłem bo to większy wycinek całości i mało czytelne się to robi...
No to coś takiego...
private Boolean checkingMethod(List<Book> books) {
Boolean check = null;
for (Book book: books) {
if (book.getValue() == null) {
check = false;
break;
} else {
check = true;
}
}
return check;
Ale pewnie można to ładniej napisać heh.
check = book.getValue() != null;
if(!check)
break;
boolean result = someObjects.stream().anyMatch(obj -> some_condition_met);
aka
return books.stream().anyMatch(book -> book.getValue() != null);
Nawet lepiej
private Boolean checkingMethod(List<Book> books) {
for (Book book: books) {
if (book.getValue() == null) {
return false;
}
}
return true;
}
czyli co do zasady metoda sprawdza czy wszystkie wartosci kolekcji nie maja wartosci null i jesci takowa sie pojawi to zwraca false dobrze zrozumialem ? czemu zwraczasz obiekt boolean potrzebujesz go gdzies ?
Dodaj do Book metodę hasValue lub zwróć Optional<Value>, a później użyj sugestii @spartanPAGE
Rossik napisał(a):
czyli co do zasady metoda sprawdza czy wszystkie wartosci kolekcji nie maja wartosci null i jesci takowa sie pojawi to zwraca false dobrze zrozumialem ? czemu zwraczasz obiekt boolean potrzebujesz go gdzies ?
Dokładnie.
Tak, wartość jaką uzyskam będę używał gdzieś indziej.
I mam do dyspozycji java 7 i guave.
Krwawy Szczur napisał(a):
Nawet lepiej
private Boolean checkingMethod(List<Book> books) {
for (Book book: books) {
if (book.getValue() == null) {
return false;
}
}
return true;
}
w sumie nie wygląda tak źle :p
Guava ma też np.
isPresent() - Returns true if this holder contains a (non-null) instance.
albo cuda jak
Optional.fromNullable(var).orNull(