Блог

У меня есть переменная, 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 и сделать код более понятным для меня и других с первого взгляда.)

Я не уверен в том, что вы хотите сделать, но, возможно, вы можете использовать Map для получения параметра ‘y’ из ‘x’

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

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

Используйте «goto» или эквивалент:

 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);
}
  

Выше довольно элегантно. При необходимости также тривиально изменить его на return true or false в зависимости от того, вызвано ли оно m() , или просто y or 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);
  

Вот решение с опциями (мой синтаксис 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> );
  

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

Почему бы и нет

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