Хорошая практика: Сохранение локальной переменной видимости VS-зависимой инъекции

Question

Хорошая практика диктует, что переменные должны быть объявлены как локальные, насколько это возможно. Это, однако, мешает хорошей практике инъекции зависимостей для модульного тестирования.

Пример:

Class A { 

    public A() {} 

    public void start() { 
     Map<Integer, VirtualMachine> vms = VirtualMachine.getAllVMs(); 

     // more code here 
    } 
} 

Теперь я не могу модульное тестирование методы start(). (getAllVMs() оказывается статическим методом в этом случае, но он мог бы так же легко быть объектом new. note: Я не хочу использовать powermock).

Так что я могу использовать google guice, чтобы ввести его, но если я это сделаю, мне нужно дать класс vms, даже если я использую его только локально в этом методе.

Есть ли способ обойти это или я вынужден сделать это поле класса?

Благодаря

Answer 1

Если значение этой переменной требуется только в одном вызове метода, то передать его в качестве параметра метода. Если он применяется во многих независимых вызовах метода, обеспечивая общий контекст, тогда логическая область «больше», чем метод, и она должна быть полем.