我用动物代替了我的真实代码,所以这可能看起来有点愚蠢,但这是一个真正的问题。
我的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();
}
}
谢谢。
最佳答案
问自己一个问题:每种动物都有自己的 HashMap 有意义吗?听起来答案是否定的,因为这是“全局”的“共享”数据。
将其设为静态
变量是合理的。我建议还考虑其他选择。例如,听起来 HashMap 存储了有关动物生活的世界的信息。因此,这个变量可能属于另一个类,World
或 Farm
,而不是 Animal
类。
关于用于数据存储的 Java 静态变量与本地变量,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/48345498/