构造函数参数中的对象：创建新实例，还是分配指针？

Question

如果一个类包含一个对象作为实例变量，并且其中一个构造函数包含一个与参数类型相同的对象，那么最佳做法是将参数指定给实例变量，还是从参数创建一个新对象，然后将新对象指定给实例变量。

下面是我正在做的一个练习的一个例子：

public class MyCircle {

  private MyPoint center; //contains variables int x and int y
  private int radius;

  //Non-controversial Constructor:
  public MyCircle(int x, int y, int radius ) {
    //creates new center using a valid MyPoint constructor
    this.center = new MyPoint(x, y);
    this.radius = radius;
  }

  //OPTION A
  public MyCircle( MyPoint center, int radius ) {
    this.center = center;
    this.radius = radius;
  }

  //OPTION B
  public MyCircle( MyPoint center, int radius ){
    this.center = new MyPoint( center.getX(), center.getY() );
    this.radius = radius;
  }
}

起初，我输入了选项A，但我认为如果这样做，可能会产生错误行为。中心引用了可能被无意中间接修改的现有对象。我想，另一种思考方式是，这为创建共享一个中心的多个对象创建了一条途径，移动一个中心会有意移动共享该中心的所有圆。

Answer 1

这两个选项都可以，但是正如你所说的，一个对象可能会随着时间的推移而改变，当你想同时修改多个对象中的中心时，选项A是可以的，例如在圆圈列表中，但是如果你想拥有唯一和独立的中心点，选项B是正确的。所以你为什么不同时拥有两个构造函数，并根据应用程序中的许多情况使用一个或另一个，使用任何你认为更好的东西，保留两个，这是我的建议。

希望对你有帮助。

Answer 2

我认为这取决于你的程序，如果你希望圆圈有一个对MyPoint对象的引用，那么你必须传递它，否则，为什么不传递实际上x和y值本身。

例如，选项B可以写成：

public MyCircle(int x, int y, int radius) {
   // rest 
}

Answer 3

因为Java没有指针（至少对开发人员来说不是），所以这个选项将被放弃，而且不是一个好办法。。现在：

public MyCircle( MyPoint center, int radius ) {
    this.center = center;
    this.radius = radius;   }

在我看来更好，你可以只分配中心，不需要制作MyPoint类的风险副本...我说有风险是因为如果你想这样做，你至少应该检查该参数的非空条件......

如果选项A中的center为null，你是对的，这是可能发生的，那么你可以通过抛出非法参数来处理它，或者只是将该对象分配给默认值。...

但正如我之前所说，这是我的观点。。