用于数据存储的Java静态变量与本地变量

Question

我用动物代替了我真正的代码，所以这可能看起来有点傻，但这是一个真正的问题。

我的java程序得到一个不同动物的列表，一次遍历一个动物，对每个动物做以下事情:动物被创建，动物做一些随机运动，每个运动被计数，然后程序收集运动数据，做一些分析，并继续到列表中的下一个动物。

我有一个抽象的动物类，由不同的动物对象扩展，狗，猫等等。每个都有自己的动作。在基础动物类中，有一个散列表计算动物的随机运动。

updateMap接收移动字符串键作为输入，然后检查给定的移动是否作为键在HashMap中，如果不是，它将移动添加为键，值为1。如果移动已经是HashMap中的键，它将增加数字值。

当前的设置是HashMap在Animal类中，所以每个动物都有自己的HashMap。我的问题是，用静态updateMap方法将HashMap作为静态变量移动到一个单独的实用程序类中，会违反Java最佳实践吗？当我需要运动数据时，我必须添加一个getter方法来调用，并添加一个方法来清除将在Animal构造函数中调用的map。动物一次迭代一个，并且只迭代一次，所以更新不会有冲突。

我想创建静态实用程序类的原因是我希望它能提高性能。动物列表有数百万个项目，我想如果每个项目都不需要用这个HashMap作为类变量来创建，我可以加快一点速度（即使它几乎不引人注目）。但是我不知道对静态方法的调用是否会减慢它。如果你知道一个更好的（性能方面的）设计来跟踪运动，我将不胜感激。

伪代码:

当前：

动物.java

 public abstract class Animal {
      private HashMap counter;

      public Animal(String input){
           counter = new HashMap();
           .
           .
      }

      protected void updateMap(String key){
           if(counter.contains(key)){
                counter.update(key, counter.get(key)+1);
           } else {
                counter.add(key, 1);
           }
      }

      protected void analyze(){
           some code using counter;
      }
 }

狗.java

 public class Dog extends Animal {

      public Dog(String input){
           super(input);
      }

      private void jump(){
           .
           .
           updateMap("jump");
      }

      private void run(){
           .
           .
           updateMap("run");
      }

      private void dance(){
           .
           .
           updateMap("dance");
      }

改变：

动物.java

 public abstract class Animal {

      public Animal(String input){
           AnimalUtil.clearCounter();
           .
           .
      }

      protected void analyze(){
           HashMap count = AnimalUtil.getCounter();
           some code using counter;
      }
 }

狗.java

 public class Dog extends Animal {

      public Dog(String input){
           super(input);
      }

      private void jump(){
           .
           .
           AnimalUtil.updateMap("jump");
      }

      private void run(){
           .
           .
           AnimalUtil.updateMap("run");
      }

      private void dance(){
           .
           .
           AnimalUtil.updateMap("dance");
      }
 }

动物玩具.java

 public class AnimalUtil {
      private static HashMap counter;

      public static void updateMap(String key){
           if(counter.contains(key)){
                counter.update(key, counter.get(key)+1);
           } else {
                counter.add(key, 1);
           }
      }

      public static HashMap getCounter(){
           return counter;
      }

      public static void clearCounter(){
           counter = new HashMap();
      }
 }

谢谢。

Answer 1

问题是，大多数时候，当你创建一个实用程序类时，是因为你不知道里面的代码的责任。根据《清洁代码》，所有语言的最佳实践是，你编写的每一个代码都有自己的领域和自己的责任。因此，将您要全局使用的HashMap放在一个负责计数和观察不同动物运动的类中。这就是我的建议。

Answer 2

有一个问题要问你自己：每个动物都有自己的HashMap有意义吗？听起来答案是否定的，因为这是“共享的”数据，是“全球性的”。

将其设置为静态变量是合理的。我建议也考虑其他选择。例如，听起来哈希地图存储有关动物所居住世界的信息。因此，也许这个变量属于另一个类，世界或农场，而不是动物类。