Opinia na temat metody

0

Cześć,
zastanawia mnie Wasza opinia na temat tej metody:

	public List<File> getAllContryPhotos(Country country) {
		return dao.findMultiple(new StatementBuilder(Caps.class)
				.add(_Caps._country()
						.equal(_Caps.COUNTRY, country.toString()))
				.buildSelect())
				.stream()
				.map(e -> (Caps) e)
				.map(e -> new File(e.getPath()))
				.collect(Collectors.toList());
	}

Czy uważacie, że tego typu sposób pisania jest czytelny, czy też lepszym byłoby rozbicie tego wszystkiego i przypisanie do zmiennych, które z kolei przekazywalibyśmy do kolejnych metod?

0
    public List<File> getAllContryPhotos(final Country country) {
        return dao.findMultiple(new StatementBuilder(Caps.class)
                .add(this::checkIfEqual)
                .buildSelect())
                .stream()
                .map(this::toCap)
                .map(this::toFile)
                .collect(toList());
    }

tak bym to widzial.

Samo wydzielanie do metod jest spoko, czasem i redundancja może być czytelniejsza, ale w tym przypadku wolałbym skorzystać z method reference

1 użytkowników online, w tym zalogowanych: 0, gości: 1