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

Question

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

我的java程序获取不同动物的列表，并一次迭代一个动物，对每个动物执行以下操作:创建动物，动物做一些随机运动，对每个运动进行计数，然后程序收集运动数据，进行一些分析，然后继续分析列表中的下一个动物。

我有一个抽象的 Animal 类，它由不同的动物对象(狗、猫等)扩展，每个对象都有自己的 Action 。在该 Animal 基类中，有一个 HashMap 可以计算 Animal 的随机运动。

每个移动方法都以对位于 Animal 基类中的 updateMap 的调用结束。 updateMap接收movement String key作为输入，然后检查给定的movement是否在HashMap中作为key，如果没有，则添加该movement作为key，值为1。如果该movement已经是HashMap中的key它将增加数值。

当前的设置是 HashMap 位于 Animal 类中，因此每个 Animal 都有自己的。我的问题是，将 HashMap 作为具有静态 updateMap 方法的静态变量移动到单独的实用程序类是否违反 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");
      }
 }

AnimalUtil.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 有意义吗？听起来答案是否定的，因为这是“全局”的“共享”数据。

将其设为静态变量是合理的。我建议还考虑其他选择。例如，听起来 HashMap 存储了有关动物生活的世界的信息。因此，这个变量可能属于另一个类，World 或 Farm，而不是 Animal 类。