java - 面向对象编程私有(private)类字段+获取/设置或公共(public)类字段？

Question

我是一名初级开发人员(目前正在练习 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 方法访问。我的问题是:一般来说，哪种方式更正确？这对我很重要，因为我想写出好的代码:)

Answer 1

通常，您会使用 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 方法，因为它允许您在将来添加验证而不会破坏代码。