此Java代码中是否有任何错误？

Question

class Creature {  
    private int yearOfBirth=10; 

    public void setYearOfBirth(int year) { 
     yearOfBirth = year; 
    } 

    void setYearOfBirth(Creature other) { 
     yearOfBirth = other.yearOfBirth; // is this correct it compiles fine 
    } 

    int getYearOfBirth() { 
     return yearOfBirth; 
    } 

    public static void main(String args[]) 
    { 
     Creature c = new Creature(); 
     c.setYearOfBirth(89); 

     Creature d = new Creature(); 
     c.setYearOfBirth(d); 

     System.out.println(c.yearOfBirth); 
    } 
} 

此代码中是否有任何错误？

“other.yearOfBirth”错了吗？我的老师说这是错误的，但它对我来说工作正常。

Answer 1

您必须要求您的教员解释为什么他们认为这是错误的（它可能是一种风格问题，甚至是一种误解），所以您可以从中学习。

这个人最终会影响你的成绩。这是与他们进行积极互动的绝好机会。你的老师对你个人的教学越是牵涉，你掌握主题的机会就越好。

如果另一方面，当你被告知某些事情是错误的，你私下离开并询问一般的互联网社区，有一种风险是，你会被告知你正确，你最终会对你的老师有一种虚假的优越感，这将会适得其反。

Answer 2

这是错误的，因为您访问的是另一个对象的私有成员（您声明为private int yearOfBirth），尽管类的类型相同。您应该使用您定义的公共吸气剂来代替：yearOfBirth = other.getYearOfBirth()

Answer 3

yearofBirth是一个私有int。因此拨打other.yearOfBirth可能会失败...

Answer 4

正如您所写，它会工作，因为你发现。尽管如此，我怀疑有一个根本性的误解。

我的精神力量告诉我，你的导师预期代码更像是以下几点：

class Creature {  
    private int yearOfBirth=10; 

    public void setYearOfBirth(int year) { 
     yearOfBirth = year; 
    } 

    public void setYearOfBirth(Creature other) { 
     yearOfBirth = other.yearOfBirth; 
    } 

    public int getYearOfBirth() { 
     return yearOfBirth; 
    } 
} 

class Program { 
    public static void main(String args[]) { 
     Creature c = new Creature(); 
     c.setYearOfBirth(89); 

     Creature d = new Creature(); 
     c.setYearOfBirth(d); 

     System.out.println(c.yearOfBirth); // This will not compile 
    } 
} 

的误解是，你只创建一个分类 - 你的主应用程序类。这实际上使yearOfBirth成为您可以从主要方法访问的混合全局值。在更典型的设计中，Creature是一个完全独立于您的主要方法的类。在这种情况下，您只能通过其public界面访问Creature。您将无法直接访问其专用字段。

---

（请注意，以任何学究那里：是的，我知道我简化。）

Answer 5

没有，是一点问题都没有用了。

看，这取决于观众的意见。但是对于给定的情况，这个代码可能是完美的。

对于其他一些情况，这可能不正确。所以这取决于如何使用。

• 直接从另一个实例访问私有成员，正确的是（并不总是可取的，虽然，例如当你继承），这就是为什么它是排在首位private。你在说“嗨，这是我的，我知道如何使用它”

• 对其他两种方法使用default访问修饰符，表示你的意图是它们不应该被包之外的其他类使用。

我可能会补充的唯一的东西就是让课程最终完成。

final class Creature 

如果你想让它可继承你可能要检讨的yearOfBirth属性的get/set，但他们的方式很是完美的我。

NOW最重要的事情这里，是你了解您的代码的每一部分做，以及它如何影响其行为。

你应该没有代码只是幸运（抱歉，我不知道这是什么正确的表达），但你应该知道你每次打字时你在做什么，你打算如何使用。

Answer 6

我看到两个“问题，”虽然我毫不犹豫地打电话给他们的错误：

1. 你明确设置生物ç的年龄为89，然后重写，年龄与未初始化的默认（！）的生物d。如果这是你打算做的事情，那么很好，但至少你会浪费几个周期来设置一个你打算在后面抛出的值。

2. 您可能违反了JavaBeans命名约定。

为了解释后点：很多Java库和框架（尤其是JSP）的依赖JavaBeans的治疗对象的组件。我没有深入探讨实际使用的机制，但从我读过的内容来看，它依赖于JavaBeans类的反省来推断属性和这些属性的类型。通过重载你的setter setYearOfBirth（）来接受一个int和一个Creature，你可以抛弃Introspection。 （请参阅here，以便对JavaBeans进行很好的介绍。）

这不是什么大问题 - 完全有可能您不会将此类用作JavaBean，并且如果这样做可能会重构它，所以它很有用。但是，你的老师可能会喜欢的东西干净了一点，如下所示：

class Creature {  
    private int yearOfBirth=10; 

    public void setYearOfBirth(int year) { 
     yearOfBirth = year; 
    } 

    int getYearOfBirth() { 
     return yearOfBirth; 
    } 

    public static void main(String args[]) 
    { 
     Creature c = new Creature(); 
     c.setYearOfBirth(89); 

     Creature d = new Creature(); 
     c.setYearOfBirth(d.getYearOfBirth()); 

     System.out.println(c.getYearOfBirth()); 
    } 
} 

现在所有的访问yearOfBirth通过公共getter方法来，这有助于封装，并防止代码破坏，如果你的主方法转移到另一个类。 （正如Greg D正确指出的那样）。

另外，这还有一个额外的好处，那就是您的代码的目的是明确，当您开始为其他人维护和修改代码时，这变得越来越重要。

Answer 7

我没有发现任何错误。

该代码有效，因为实例或类可以访问同一类的其他实例的私有成员。这是设计。