我是一名初级开发人员(目前正在练习 Java),我对我的代码的正确性有一个疑问,下面是一个示例: 我正在用 Java 编写一个简单的 MMO 游戏表示,我有 2 个类(角色和法术)。 角色有属性(mName、mHealth 和 mEnergy),法术类有属性(mSpellName、mSpellCost、mSpellDamage)。而Spell类也有一个叫execute的方法,这里是一段代码
public void spellExecute(Character caster, Character target) {
caster.mEnergy -= this.spellCost;
target.mHealth -= this.spellDamage
}
这种结构意味着 Character 字段是公共(public)的并且可以直接访问,但是在一些示例中我看到所有字段都必须是私有(private)的并且只能通过 get/set 方法访问。我的问题是:一般来说,哪种方式更正确?这对我很重要,因为我想写出好的代码:)
最佳答案
通常,您会使用 get/set 方法,因为它们允许类通过这些方法控制访问
任何其他使用您的类的类,应该只能按照您描述的方式访问和更改您的字段。
例如让我们看一个简单的(过于简化的)燃油泵
class Pump
{
constant int PRICE_PER_LITRE = 100; //in pence
private int litresDispensed;
private bool nozzleUp;
private int litresAvailable;
private int price; //in pence
construct()
{
litresDispensed = 0;
nozzleUp = false;
}
startFuelling()
{
nozzleUp = true;
}
stopFuelling()
{
nozzleUp = false;
}
takeFuel(int litresTaken)
{
if(nozzleUp = true)
{
litresAvailable -= litresTaken;
price += litresTaken * PRICE_PER_LITRE;
}
else
{
error("You must lift the nozzle before taking fuel!");
}
}
getPrice()
{
if(nozzleUp = true)
{
error("You can't pay until you've finished fuelling! Please return the nozzle");
}
else
{
return price;
}
}
}
我们的最终 get 方法对于确保在用户尝试付款之前完成其余交易非常重要。
如果我们允许直接访问价格,他们可以在完成加油之前完成!这会让他们偷走我们所有的燃料。
如图所示,get 方法可以保护该字段免受外部影响。它仍然可以被操纵,但只能以我们希望允许它被操纵的方式。另请注意,此字段根本没有设置方法:我们不希望用户能够设置自己的价格,只能设置我们指定的价格!
如果您编写的 get/set 方法只返回和设置字段,没有任何验证或检查,那么您可以简单地将字段公开(或者,您需要决定是否应该直接访问该字段): 也就是说,最好尽可能使用 get/set 方法,因为它允许您在将来添加验证而不会破坏代码。
关于java - 面向对象编程私有(private)类字段+获取/设置或公共(public)类字段?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/28546195/