我在游戏中有一个实体,它有一个更新方法,它需要定位最近的僵尸,目前僵尸列表只是我访问的一个全局对象,但这似乎是错误的,我可以将列表传递给更新方法,但我'不确定这是否是最好的方法?
这是我的更新方法的简化版本:
this.update = function () {
var targetedZombie = null;
//TODO: should not be using the zombies object - tight coupling should be removed
var alivezombies = [];
for (var zombie in zombies) {
if (zombies[zombie].Alive) {
alivezombies.push(zombies[zombie]);
}
}
targetedZombie = this.GetClosestEntity(alivezombies);
if (targetedZombie) {
Fire(this, targetedZombie);
}
});