История изменений

Исправление php-coder, 29.12.14 13:58 (текущая версия) :

Я бы завел отдельный метод (или даже два) и не использовал никаких Optional/анонимных классов:

this.winnerAccountId = getWinnerIdOrNull(auction);

private Long getWinnerIdOrNull(Auction auction) {
    if (auction == null) {
        return null;
    }

    return getWinnerIdOrNull(auction.getWinner());
}

private Long getWinnerIdOrNull(Winner winner) {
    if (winner == null) {
        return null;
    }

    return winner.getId();
}

Исходная версия php-coder, 29.12.14 13:54:

Я бы завел отдельный метод (или даже два) и не использовал никаких Optional/анонимных классов:

this.winnerAccountId = getWinnerIdOrNull(auction);

private Winner getWinnerIdOrNull(Auction auction) {
    if (auction == null) {
        return null;
    }

    return getWinnerIdOrNull(auction.getWinner());
}

private Long getWinnerIdOrNull(Winner winner) {
    if (winner == null) {
        return null;
    }

    return winner.getId();
}