Início / coding / Perguntas / 78749107
Accepted
PatPanda
Asked: 2024-07-15 17:30:56 +0800 CST

Problema de refatoração if-else-if string.contains("X") para alternar entre maiúsculas e minúsculas (ou melhor)

  • 772
  • Inclua detalhes sobre seu objetivo:

Gostaria de refatorar if-else-if string.contains("X")instruções em java para mudar de caso ou mapa.

  • Mostre algum código:

Eu tenho um trecho de código, muito simples, que se parece com:

     public String question(String sentence) {
        if (sentence.contains("eat")) {
            return getFoodFromSentence(sentence);
        } else if (sentence.contains("drink")) {
            return getBeverageFromSentence(sentence);
        } else if (sentence.contains("drive")) {
            return getVehicleFromSentence(sentence);
        } else if (sentence.contains("travel")) {
            return getDestinationFromSentence(sentence);
    [...
        many many other else if
    ...]
        } else {
            return "sentence with unknown verb";
        }
    }

    private String getDestinationFromSentence(String sentence) {
        return ""; //some complex code to extract some info from the sentence
    }

    private String getVehicleFromSentence(String sentence) {
        return ""; //some complex code to extract some info from the sentence
    }
[... all the other actual handling methods ]

Desejo refatorar isso usando um switch case:

  • Descreva o que tentei:

Usando caixa de comutação:

   public String question(String sentence) {
        switch (???) { //issue here, I am not able to come up with the expression
            case (sentence.contains("eat")): // issue here, I am not able to have the cases representing the if
                return getFoodFromSentence(sentence);
            case (sentence.contains("drink")):
                return getBeverageFromSentence(sentence);
            case (sentence.contains("drive")):
                return getVehicleFromSentence(sentence);
            case (sentence.contains("travel")):
                return getDestinationFromSentence(sentence);
        [ ... the other case conditions ... ]
            default:
                return "sentence with unknown verb";
        }
    }

Também tentei usar o padrão Map, algo como:

 public String question(String sentence) {
        Map<Predicate<String>, Function<String, String>> map =
                Map.of(s -> s.contains("eat"), s -> getFoodFromSentence(s),
                        s -> s.contains("drink"), s -> getBeverageFromSentence(s),
                        s -> s.contains("drive"), s -> getVehicleFromSentence(s),
                        s -> s.contains("travel"), s -> getDestinationFromSentence(s));
        return map.get(???).apply(sentence); // issue here, how to get the correct key of the map?
    }
  • Pergunta:

Posso perguntar como resolver os problemas de uso do switch case?

Ou talvez como resolver o Mapa?

Existe outra técnica para refatorar isso if-else-if string.contains("X")?

java

3 respostas

  1. Para a abordagem baseada em mapa, você poderia tentar isto:

    public String question(String sentence) {
    Map<String, Function<String, String>> map = Map.of(
            "eat", this::getFoodFromSentence,
            "drink", this::getBeverageFromSentence,
            "drive", this::getVehicleFromSentence,
            "travel", this::getDestinationFromSentence
    );
    for (Map.Entry<String, Function<String, String>> entry : map.entrySet()) {
        if (sentence.contains(entry.getKey())) {
            return entry.getValue().apply(sentence);
        }
    }
    return null;
}
    • 2
  2. Best Answer

    casedeclarações podem fazer uso da whenpalavra-chave:

    public String question(String sentence) {
    switch (sentence) {
        case String s when sentence.contains("eat"):
            return getFoodFromSentence(sentence);
        case String s when sentence.contains("drink"):
            return getBeverageFromSentence(sentence);
        case String s when sentence.contains("drive"):
            return getVehicleFromSentence(sentence);
        case String s when sentence.contains("travel"):
            return getDestinationFromSentence(sentence);
        default:
            return "sentence with unknown verb";
    }
}
    • 2

  3. Uma solução semelhante com a abordagem de mapa de tobias_k apenas com um LinkedHashMap para garantir a ordem e usando streams e opcionais para possivelmente melhorar um pouco a legibilidade:

    import java.util.LinkedHashMap;
import java.util.Map;
import java.util.function.UnaryOperator;

public class Example {

    public static void main(String[] args) {

        Example ex = new Example();

        String sentence1 = "What do you eat?";
        System.out.println(ex.question(sentence1));

        String sentence2 = "What do you drink?";
        System.out.println(ex.question(sentence2));

        String sentence3 = "What do you drive?";
        System.out.println(ex.question(sentence3));

        String sentence4 = "Where do you travel to?";
        System.out.println(ex.question(sentence4));

        String sentence5 = "What did you do yesterday?";
        System.out.println(ex.question(sentence5));
    }

    public String question(String sentence) {

        Map<String, UnaryOperator<String>> map = new LinkedHashMap<>();
        map.put("eat",    this::getFoodFromSentence);
        map.put("drink",  this::getBeverageFromSentence);
        map.put("drive",  this::getVehicleFromSentence);
        map.put("travel", this::getDestinationFromSentence);

        return map.keySet().stream()
                           .filter(sentence::contains)
                           .findFirst()
                           .map(e -> map.get(e).apply(sentence))
                           .orElse("Sentence with unknown verb");
    }

    private String getFoodFromSentence(String sentence) {
        return "Haggis";
    }

    private String getBeverageFromSentence(String sentence) {
        return "IRN-BRU";
    }

    private String getDestinationFromSentence(String sentence) {
        return "Ring of Brodgar";
    }

    private String getVehicleFromSentence(String sentence) {
        return "Galloway";
    }
}
    • 1

relate perguntas