Использование Java-поставщика / функции для передачи метода в статический метод - PullRequest
0 голосов
/ 12 мая 2018

У меня есть пара классов:

public class TextContent {
    private String externalId;
}

public class ImageContent {
    private String externalImageId;
}

public static void validateImageInput(List<ImageContent> imageAssets, String requestId) {
    if(CollectionUtils.isEmpty(imageAssets)) {
        throw some Error;
    }

    Set<String> uniqueIds = imageAssets.stream().map(ImageContent::externalImageId).collect(Collectors.toSet());
    if(uniqueIds.size() != imageAssets().size()) {
        throw some Error;
    }

    //Do some processing
}

public static void validateTextInput(List<TextContent> textAssets, String requestId) {
    if(CollectionUtils.isEmpty(textAssets)) {
        throw some Error;
    }

    Set<String> uniqueIds = textAssets.stream().map(ImageContent::externalId).collect(Collectors.toSet());
    if(uniqueIds.size() != textAssets().size()) {
        throw some Error;
    }

    //Do some processing
}

Как видите, часть проверки одинакова для обоих этих классов. И я хотел попробовать сделать это распространенным методом. Для этого:

public static void validateInput(List<?> assets, String requestId, Supplier<String> mapper) {
    if(CollectionUtils.isEmpty(assets)) {
        throw some error;
    }

    Set<String> uniqueIds = assets.stream().map(x -> mapper.get()).collect(Collectors.toSet());
    if(uniqueIds.size() != assets().size()) {
        throw some Error;
    }
}

и затем вызовите его с помощью:

public static void validateAllInputs(List<ImageContent> imageAssets, List<TextContent> textAssets, String requestId) {
    validateInput(imageAssets, requestId, ImageContent::externalImageId);
    validateInput(textAssets, requestId, TextContent::externalId);
    doSomeProcessingWithText(textAssets, requestId);
    doSomeProcessingWithImage(imageAssets, requestId);
}

Но я получаю сообщение об ошибке Non static method cannot be referenced from static context.

Edit: Другой вариант, который я пробовал, использовал Function, то есть я передаю <TextContent, String> mapper, а в своем потоке я использую .map(x -> mapper.apply(x). Однако, когда я пытаюсь передать его в функцию validateInputs(textAsset, requestId, TextContent::externalId), я получаю ту же ошибку Non static method cannot be referenced from static context.

Ответы [ 2 ]

0 голосов
/ 13 мая 2018

Сделано это с помощью дженериков:

public static <T> void validateInput(List<T> assets, String requestId, Function<T, String> mapper) {
    if(CollectionUtils.isEmpty(assets)) {
        throw some error;
    }

    Set<String> uniqueIds = assets.stream().map(mapper).collect(Collectors.toSet());
    if(uniqueIds.size() != assets().size()) {
        throw some Error;
    }
}

А затем позвоните, используя:

validateInput(imageAssets, requestId, ImageContent::externalImageId);
0 голосов
/ 12 мая 2018

Я бы предложил, чтобы оба типа реализовывали один и тот же интерфейс:

public interface Content {
    public String getExternalId();

    public static String externalId(Content content) {
        return content == null ? null : content.getExternalId();
    }
}

Затем измените подпись validateInput на:

public static void validateInput(List<? extends Content> assets, String requestId) {
    if(CollectionUtils.isEmpty(assets)) {
        throw some error;
    }

    Set<String> uniqueIds = assets.stream().map(Content::externalId).collect(Collectors.toSet());
    if(uniqueIds.size() != assets.size()) {
        throw some error;
    }

    //Do some processing
}

И если вы все еще хотите извлечь общий метод здесь, вы можете написать так:

public static <E> void commonValidate(List<E> assets, String requestId, Function<E, String> mapper) {
    Set<String> uniqueIds = assets.stream().map(mapper).collect(Collectors.toSet());
}
...