Java：使用本地与实例变量？

Question

我的应用中有 3 个classes：

1. Runner / Main（调用服务类）

2。服务类（执行业务逻辑）

3.存储库类（由服务调用以进行数据库查询）

我不确定在服务类中实现变量的最佳方式。以下 2 种方法中哪种方法最好？为什么？

例如我应该有实例变量：

public class DogService{

    List<Dogs> dogList= new ArrayList<Dog>(); //instance var

    public DogService(){}

    public List<dogs> getAllDogs(){

    dogList=dogRepository.getAll();

    return dogList;

        }

    }

或方法中的局部变量：

public class DogService{

       public DogService(){}

        public List<dogs> getAllDogs(){

       List<Dogs> dogList= new ArrayList<Dog>(); //local var to method

        dogList=dogRepository.getAll();

        return dogList;

            }

        }

服务类使用示例：

public class Runner {

    List<Dogs> listOfAllDogs = new ArrayList<Dog>();

    DogService dogService = new DogService();

    public static void main(String[] args) {

    listOfAllDogs = dogService.getAllDogs();

}

Answer 1

在第一种情况下，您使用您的实例创建一个新的 ArrayList，并在离开该方法后保留对狗列表的引用。你在浪费内存。

此外，它是您的类中没有使用的字段，因此它会使您的代码杂乱无章，您可以删除这些无用的行。

它也可能是错误的来源。该变量已声明，并且具有暗示其用途的名称。稍后，其他开发人员可能会尝试将其用于其他用途，并且根据之前是否调用该方法，它会工作还是崩溃。

在第二种情况下，变量没有用，因为您可以立即返回 getter 的结果。但是编译器会为你处理好这些，所以你不必担心。

Answer 2

这完全是个人观点，但您误解了服务层的典型用途，即：

public class DogService{
  Repository repository;
  public DogService(Repository repo){
    this.repository = repo;
  }

  public List<dogs> getAllDogs(){
    return this.repository.getAll();
  }
}

service 有责任知道在哪里寻找狗。它不会参与尝试记住特定的狗或查找它们：它delegates 对底层存储库负责。

回答您的问题，这意味着 方法和实例都不应该记住狗的列表。如果另一个方法，例如getAllDogNames 需要做一些时髦的事情，它可能需要一个实例变量：

public List<String> getAllDogNames(String prefix){
  List<Dog> dogs = this.getAllDogs();
  List<String> names = new ArrayList<String>();
  for (dog : dogs) {
    names.add(prefix + dog.getName()); //Or whatever
  }
}

但这应该推迟到适当的包装器。

Answer 3

如果dogList 不会更改，那么将其作为字段将允许您潜在地缓存它。对于dogs 可能不是一个好主意，它可能有小狗或死去，但如果它是一个静态列表或其他东西，它会有一些用途。

例如

 if (dogList == null) {
    dogList= new ArrayList<Dog>(); 
    dogList=dogRepository.getAll();
}

return dogList;