Каков лучший способ, если условие с несколькими ИЛИ в Java? Java8 предпочтительнее - PullRequest
Новая
       8

Каков лучший способ, если условие с несколькими ИЛИ в Java? Java8 предпочтительнее

0 голосов
/ 08 сентября 2018

Как мне сделать это лучше? Я предпочитаю синтаксис java8.

Эти логические условия могут расти. 

boolean imageType = filetype.startsWith("image");

boolean videoType = filetype.startsWith("video");

boolean archiveType = filetype.contains("archive");

boolean jarType = filetype.contains("java-archive");

boolean audioType = filetype.startsWith("audio");

boolean zipType = filetype.contains("zip");

boolean xTarType = filetype.contains("x-tar");

boolean rarType = filetype.contains("rar");

if(!(imageType || videoType || archiveType || jarType || audioType || zipType || xTarType)) {
         //doSomething         
}

Ответы [ 3 ]

0 голосов
/ 08 сентября 2018

1) Перегруппируйте свои условия в Предикат с. Возьмите случай с Enum: 

public enum PredicateEnum {

    IMAGE   (filetype -> filetype.startsWith("image")),
    VIDEO   (filetype -> filetype.startsWith("video")),
    ARCHIVE (filetype -> filetype.contains("archive")),
    JAR     (filetype -> filetype.contains("java-archive")),
    AUDIO   (filetype -> filetype.startsWith("audio")),
    ZIP     (filetype -> filetype.contains("zip")),
    X_TAR   (filetype -> filetype.contains("x-tar")),
    RAR     (filetype -> filetype.contains("rar"));

    private Predicate<String> predicate;

    PredicateEnum(Predicate<String> predicate) {
        this.predicate = predicate;
    }

    public Predicate<String> getPredicate() {
        return predicate;
    }
}

2) Используйте Stream # Reduce и Predicate # или , чтобы создать один предикат, который является результатом всех ваших предикатов, связанных логическими операторами ИЛИ: 

Predicate<String> predicateOr = Stream.of(PredicateEnum.values())
        .map(PredicateEnum::getPredicate)
        .reduce(Predicate::or)
        .orElse(s -> false);

System.out.println("image.png: "      + predicateOr.test("image.png"));
System.out.println("my-archive.txt: " + predicateOr.test("my-archive.txt"));
System.out.println("foo : "           + predicateOr.test("foo"));

3) Используйте результат Predicate # test в вашем if утверждении. Например, приведенный выше код выводит на печать: 

image.png: true
my-archive.txt: true
foo : false
0 голосов
/ 08 сентября 2018

Вот несколько способов сделать это более масштабируемым.

  1. Используйте регулярное выражение: 

    Pattern p = Pattern.compile("^video|^audio|^image|zip|rar|java-archive|x-tar");
if (!p.matcher(filetype).find()) {
    // do stuff
}

  2. Использовать массивы или списки. Например: 

    String[] prefixes = new String[]{"video", "audio", "images"};
String[] contains = new String[]{"zip", "rar", "x-tar", "jar-archive"};
boolean match = false;
for (String p : prefixes) {
    if (filetype.startsWith(p)) {
        match = true;
    }
}
...
if (!match) {
    // do stuff
}

Очевидно, подход регулярных выражений является более кратким, но подход с использованием массивов, вероятно, более эффективен (если это имеет значение!). Это зависит от того, как движок регулярных выражений справляется с множеством альтернатив.

Оба подхода будут масштабироваться; например, обновив регулярное выражение или добавив строки в массивы.

В обоих случаях вы можете легко загрузить соответствующие критерии из файла свойств или аналогичного файла ... и избежать внесения изменений в код.

Я не уверен, что лямбды и потоки Java 8 хорошо подходят для этой проблемы.

0 голосов
/ 08 сентября 2018

Можно также использовать более объектно-ориентированный подход, чтобы дать вам немного больше информации о типе файла. Я могу представить, что это будет полезно позже в вашей программе.

Вы можете сделать что-то вроде объявления всех ваших типов файлов в Enum: 

public enum FileType {
    IMAGE("a"),
    VIDEO("b"),
    ARCHIVE("c"),
    JAR("d"),
    AUDIO("e"),
    ZIP("f"),
    XTAR("g");

    private String str;

    FileType(String str) {
        this.str = str;
    }

    public String getStr() {
        return str;
    }

    public static FileType getFileTypeForStr(String str) {
        for (FileType fileType : FileType.values()) {
            if (fileType.getStr().equalsIgnoreCase(str)) {
                return fileType;
            }
        }

        return null;
    }
}

Затем в своей функции вы можете заменить все свои Booleans проверкой, чтобы увидеть, является ли введенный вами String1 тип файла для включения: 

FileType fileType = FileType.getFileTypeForStr(String1); //And String2, String3, String4...
if (fileType != null) {
    System.out.printf("File type found of type %s", fileType.name());
} else {
    System.out.printf("No file type found for input %s", String1);
}

Поскольку у вас есть 7 различных строк для проверки, вы можете добавить простую проверку, чтобы проверить, все ли переменные String1 совпадают: 

boolean isNotFileType = Stream
    .of(String1, String2, String3, String4, String5, String6, String7)
    .map(FileType::getFileTypeForStr)
    .anyMatch(Objects::isNull);
...