Наиболее сжатый способ выразить этот Java условный, не проверяя значение дважды

Question

У меня есть переменная, x.

Я хочу назвать метод m()только в том случае, еслиx является одним из двух возможных значений.

При вызове m() я хочу передать ему аргумент, значение которого зависит от значения x.

Есть ли способ сделать это в Java без проверки значения x более чем один раз, и вызов/запись m() только в одном месте (т.е. не в нескольких ветвях if заявления)?

Одно из решений я развлекая:

switch (x) { 
    case 1: 
    y = "foo"; 
    break; 
    case 2: 
    y = "bar"; 
    break; 
    default: 
    y = null; 
    break; 
} 

if (y != null) m(y); 

Но я не могу помочь, но чувствую, что это технически проверки x дважды, просто скрывая этот факт, добавив «прокси» для повторной проверки.

(Чтобы уточнить, почему ограничения являются такими, какими они являются: при чтении кода у меня есть логика понимания времени, которая много разветвляется, когда существует высокая степень дублирования между ветвями - она ​​становится игрой «пятно разницы» «вместо того, чтобы просто видеть, что происходит. Я предпочитаю агрессивно реорганизовывать такое дублирование, что является привычкой, которая хорошо мне помогает в Ruby, JS и других языках, я надеюсь, что смогу научиться делать то же самое для Java и сделать код более легким для меня и других, чтобы понять с первого взгляда.)

Answer 1

Использования «Готы» или эквивалента:

void do_m_if_appropriate() { 

    // x and y are assumed to be eg. member variables 

    switch (x) { 

    case 1: 
     y = "foo"; 
     break; 

    case 2: 
     y = "bar"; 
     break; 

    default: 
     return; // this is the "goto equivalent" part 
    } 

    m(y); 
} 

Выше довольно элегантные , В случае необходимости также можно изменить его, чтобы вернуть true или false в зависимости от того, называется ли он m() или просто y или null.

---

Вы также можете делать трюки с конструкциями циклов, хотя некоторые могут сказать, что это злоупотребление конструкции петли, и вы должны прокомментировать его соответствующим образом:

do { // note: not a real loop, used to skip call to m() 
    switch (x) { 

    case 1: 
     y = "foo"; 
     break; 

    case 2: 
     y = "bar"; 
     break; 

    default: 
     continue; // "goto equivalent" part 
    } 

    m(y); 
} while(false); 

Answer 2

Почему не

switch (x) { 
    case 1: 
    y = "foo"; 
    m(y); 
    break; 
    case 2: 
    y = "bar"; 
    m(y); 
    break; 
} 

Answer 3

Я не уверен, что вы хотите сделать, но вы можете возможно использовать карту, чтобы получить «у» параметра из «х»

Map<Integer, String> map = new HashMap<>(); 
map.put(1, "foo"); 
map.put(2, "bar"); 

if (map.containsKey(x)) { 
    m(map.get(x)); 
} 

Answer 4

Вот решение с OPTIONALS (мой Java синтаксис может быть немного неправильным). Обратите внимание, что для вас код выглядит так, но реализация мудрая, это похоже на приведенный вами пример (т. Е. Проверяет, является ли y исключительным значением).

switch (x) { 
    case 1: 
    y = Optional<String>.of("foo"); 
    break; 
    case 2: 
    y = Optional<String>.of("bar"); 
    break; 
    default: 
    y = Optional<String>.empty(); 
    break; 
} 
y.map((m's class)::m); 
result = y.orElse(<value result should take if x was invalid>); 

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