Этот вопрос мотивирован тем, что я в последнее время слишком часто вижу, структурой if..else if..else. Несмотря на то, что он прост и имеет свое применение, что-то в нем постоянно повторяет мне, что его можно заменить чем-то более тонким, элегантным и просто в целом более актуальным.
Чтобы быть как можно более конкретным, вот что я имею в виду:
if (i == 1) {
doOne();
} else if (i == 2) {
doTwo();
} else if (i == 3) {
doThree();
} else {
doNone();
}
Я могу придумать два простых способа переписать это, либо троичным (это просто еще один способ написания той же структуры):
(i == 1) ? doOne() :
(i == 2) ? doTwo() :
(i == 3) ? doThree() : doNone();
или используя Map (на Java и я тоже думаю на C #), Dictionary или любую другую структуру K / V, например:
public interface IFunctor() {
void call();
}
public class OneFunctor implemets IFunctor() {
void call() {
ref.doOne();
}
}
/* etc. */
Map<Integer, IFunctor> methods = new HashMap<Integer, IFunctor>();
methods.put(1, new OneFunctor());
methods.put(2, new TwoFunctor());
methods.put(3, new ThreeFunctor());
/* .. */
(methods.get(i) != null) ? methods.get(i).call() : doNone();
На самом деле метод Map, описанный выше, это то, чем я занимался в прошлый раз, но теперь я не могу перестать думать, что для этой конкретной проблемы в целом должны быть лучшие альтернативы.
Итак, какие другие - и, скорее всего, лучшие - способы заменить if..else if .. еще там, и какой из них ваш любимый?
Ваши мысли под этой строкой!
Хорошо, вот ваши мысли:
Во-первых, самым популярным ответом был оператор switch, например:
switch (i) {
case 1: doOne(); break;
case 2: doTwo(); break;
case 3: doThree(); break;
default: doNone(); break;
}
Это работает только для значений, которые могут использоваться в коммутаторах, что, по крайней мере, в Java, является довольно ограничивающим фактором. Приемлемо для простых случаев, хотя, естественно.
Другой и, возможно, более причудливый способ, которым вы, похоже, предлагаете, - это сделать это с помощью полиморфизма. Лекция на Youtube, связанная с CMS, - это отличные часы, посмотрите ее здесь: «Чистые разговоры по коду - наследование, полиморфизм и тестирование» Насколько я понял, это будет примерно так:
public interface Doer {
void do();
}
public class OneDoer implements Doer {
public void do() {
doOne();
}
}
/* etc. */
/* some method of dependency injection like Factory: */
public class DoerFactory() {
public static Doer getDoer(int i) {
switch (i) {
case 1: return new OneDoer();
case 2: return new TwoDoer();
case 3: return new ThreeDoer();
default: return new NoneDoer();
}
}
}
/* in actual code */
Doer operation = DoerFactory.getDoer(i);
operation.do();
Два интересных момента из Google Talk:
- Использовать нулевые объекты вместо возврата нулей (и, пожалуйста, выбрасывайте только исключения времени выполнения)
- Попробуйте написать небольшой проект без if: s.
Кроме того, на мой взгляд, стоит упомянуть один пост - CDR, который предоставил нам свои извращенные привычки, и хотя его не рекомендуется использовать, просто очень интересно посмотреть.
Спасибо всем за ответы (пока), думаю, я мог бы кое-что узнал сегодня!