使用带有静态变量的随机 getter

Using a randomized getter with static variables

所以我正在使用一些随机发生器为我的口袋妖怪生成统计数据。我的口袋妖怪构造函数是:

public Pokemon(String name, int health, int attack, int speed, boolean status) {

    this.name = name;
    Pokemon.health = health;
    Pokemon.attack = attack;
    Pokemon.speed = speed;
    Pokemon.status = status;
}

 private static int randomHealth(int min, int max) {
    int range = (max - min) + 1;
    return (int)(Math.random() * range) + min;
}
private static int randomAttack(int min, int max) {
    int range = (max - min) + 1;
    return (int)(Math.random() * range) + min;
}
private static int randomSpeed(int min, int max) {
    int range = (max - min) + 1;
    return (int)(Math.random() * range) + min;
}

我的随机数发生器:

private Random rand = new Random();
private List<Pokemon> pokemon;


WildPokemon(){
    pokemon = new ArrayList();

    Pokemon rattata = new Pokemon("Rattata",randomHealth(15,20),randomAttack(2,5),randomSpeed(3,6), true);
    pokemonZone1.add(rattata);
    Pokemon pidgey = new Pokemon("Pidgey",randomHealth(10,17),randomAttack(3,4),randomSpeed(2,5), true);
    pokemonZone1.add(pidgey);

getter:

Pokemon getRandomPokemon() {
    int n = rand.nextInt(pokemon.size());
    return pokemon.get(n);
}

但是,我面临的问题是,当使用我的战斗方法(为简洁起见省略)从列表中选择随机口袋妖怪时,为随机口袋妖怪生成的统计数据总是从最近构建的口袋妖怪中提取(在这种情况下是 Pidgey)。

例如,我刚刚提取了一个随机口袋妖怪的统计数据并得到了 Rattata 的以下数据:

健康:14

攻击:3

速度:2

如果我的隐含逻辑是正确的,小拉达不可能有这么低的生命值或速度。

我觉得好像问题出在"static"变量的使用上,因为我还没有完全理解它们。可能是在 运行 我的 getRandomPokemon() 方法时,它会为这些值提取最后已知的最小值和最大值。在这种情况下是 Pidgey 的。

battle(starter, wildPokemon.getRandomPokemon());

我为混乱的例子道歉,但我发现这是我最好的学习方式。谢谢!


更新:

感谢大家的帮助。我设法通过将我的变量从静态变量更改为非静态变量来解决这个问题,并且通常只是清理代码。 public class 精灵宝可梦 {

public static int starterHealth;
String name;
public int health;
public int attack;
public int speed;
public boolean status;

我还制作了 Battle 自己的 class 以保留自己的方法并清理代码。 总的来说,我了解到如果我将 Pokemon class 中的变量设为静态,它会为所有 Pokemon 提供相同的统计数据。理解静态变量花了很多功夫,但我最终明白了。

谢谢大家!

你说得对。这是你的静态变量。

静态变量只存在一次句号(不是每个神奇宝贝一次)。因此,您所有的神奇宝贝都采用最后一个神奇宝贝的值,在这种情况下是 Pidgey。

很简单,您的变量不应该是静态的。