Java стиль: Объявление переменной в коммутаторе

Question

Следующий код не компилируется, потому что eater определяется дважды:

switch (vegetable) { 
    case TOMATO: 
     Eater eater = new Eater(Tomato.class, many parameters); 
     eater.eat(more parameters); 
     return true; 

    case POTATO: 
     Eater eater = new Eater(Potato.class, many parameters); 
     eater.eat(more parameters); 
     return true; 

    case CARROT: 
     doSomethingElse(); 
     return true; 
} 

Должен ли я:

• Используйте отдельные переменные `` tomatoEater` и potatoEater`, что делает код менее ремонтопригодным?
• Определить `eater` перед` switch`, делая его доступным больше, чем нужно?
• Определить `eater` только в первый раз, что приводит к потенциальной путанице?
• Добавить брекеты, делая код более подробным?
• Любая идея?

Answer 1

Я лично либо использовать фигурные скобки, или просто полностью отказаться от локальной переменной:

new Eater(Potato.class, many parameters) 
    .eat(more parameters); 

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

Answer 2

Почему бы не так:

switch (vegetable) 
{ 
    case TOMATO: 
     new Eater(Tomato.class, many parameters).eat(more parameters); 
     return true; 

    case POTATO: 
     new Eater(Potato.class, many parameters).eat(more parameters); 
     return true; 

    case CARROT: 
     doSomethingElse(); 
     return true; 
} 

Если вы не имеете любое использование ссылки Eater где-либо еще позже, я бы сделал это.

Answer 3

Как использовать отдельные переменные сделать код менее ремонтопригодным? (первая маркерная точка). Если бы я сказал, что это будет делать наоборот, как имя переменной лучше объясняет, что это такое. Я бы пошел с этим, если держать его в этом объеме важно для вас.

Answer 4

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

Eater eater = vegetable.getEater(many parameters); 
if (eater != null) eater.eat(more parameters); 
else doSomethingElse(); 
return true; 

Answer 5

Может быть использование переключателя не является такой хорошей идеей вообще.

---

In what better example can represent a switch statement in Java ?