【构造器中给域赋值】是调用赋值方法好，还是直接赋值好？

Question

通常情况下，构造器中给域赋值貌似都是直接进行赋值的，例如书上这段代码：class Shape
{
  private double width;
  private double height;
  
  double getWidth() {return width;}
  double getHeight() {return height;}
  void setWidth(double w) {width = w;}
  void setHeight(double h) {height = h;}
  
  Shape(double w, double h) //直接赋值的构造器
  {
    width = w;
    height = h;
  }
}联系到java的封装概念，为了避免直接赋值带来的麻烦，是不是在构造器中调用方法去赋值更好呢？
修改后的代码如下：class Shape
{
  private double width;
  private double height;
  
  double getWidth() {return width;}
  double getHeight() {return height;}
  void setWidth(double w) {width = w;}
  void setHeight(double h) {height = h;}
  
  Shape(double w, double h) //调用方法赋值的构造器
  {
    setWidth(w);
    setHeight(h);
  }
}

Answer 1

定义成private已经封装了,何必多此一举?

Answer 2

恩 这个看个人习惯吧 当然一般是喜欢把赋值放在构造方法里我一直想问你的那个签名什么意思啊？是蒙语吗？还有你的ID实在是太YD了，竟然叫Big Bro？！

Answer 3

那个是泰语还有，楼上的大大不会是把bro看成bra了吧-_-|||

Answer 4

比较喜欢 this.** = **郁闷eclipse老是把我写好的改成set**(**）

Answer 5

两个big撞一起了,当bug遇上bra……

Answer 6

封装应该是指对外吧，如果对内还封装，那是不是有点过了。不过对内的，我认为应该还要加个限制，只能读，不要该，如果是改的话还是要用method来统一入口吧个人意见

Answer 7

ZangXT，火龙果，老紫竹等大侠可以来个最后的定论么？

Answer 8

this.* = * 这种写法是有问题的
但是大多程序员认为这种写法带来的实惠更大一些

Answer 9

这个根据情况而定，从数据安全的角度上来讲，尽量不提供set方法，而是采用构造函数直接初始化的方法，否则数据什么时候被意外修改了自己都不知道,建议如下：
class Shape
{
  private double width;
  private double height;
  
  double getWidth() {return width;}
  double getHeight() {return height;}
  
  Shape(double w, double h)   {
    this.width = w;
    this.height = h;
  }
}

Answer 10

在自己的类中使用成员变量是直接用变量，还是用这个变量的访问器去调用，这存在一定的分歧。可以参考 Martin Fowler 的《重构——改善既有代码的设计》第 8.1 节中相关部分。

Answer 11

这本来就是个争论很大的问题 贴一段出自某Spring教程的话Type2设值注入的优势
1． 对于习惯了传统JavaBean开发的程序员而言，通过setter方法设定依赖关系显得更加直观，更加自然。
2． 如果依赖关系（或继承关系）较为复杂，那么Type3模式的构造函数也会相当庞大（我们需要在构造函数中设定所有依赖关系），此时Type2模式往往更为简洁。
3． 对于某些第三方类库而言，可能要求我们的组件必须提供一个默认的构造函数（如Struts中的Action），此时Type3类型的依赖注入机制就体现出其局限性，难以完成我们期望的功能。
Type3构造子注入的优势：
1． “在构造期即创建一个完整、合法的对象”，对于这条Java设计原则，Type3无疑是最好的响应者。
2． 避免了繁琐的setter方法的编写，所有依赖关系均在构造函数中设定，依赖关系集中呈现，更加易读。
3． 由于没有setter方法，依赖关系在构造时由容器一次性设定，因此组件在被创建之后即处于相对“不变”的稳定状态，无需担心上层代码在调用过程中执行setter方法对组件依赖关系产生破坏，特别是对于Singleton模式的组件而言，这可能对整个系统产生重大的影响。
4． 同样，由于关联关系仅在构造函数中表达，只有组件创建者需要关心组件内部的依赖关系。对调用者而言，组件中的依赖关系处于黑盒之中。对上层屏蔽不必要的信息，也为系统的层次清晰性提供了保证。
5． 通过构造子注入，意味着我们可以在构造函数中决定依赖关系的注入顺序，对于一个大量依赖外部服务的组件而言，依赖关系的获得顺序可能非常重要，比如某个依赖关系注入的先决条件是组件的UserDao及相关资源已经被设定。

Answer 12

昨天晚上关于这个问题的新体会，从软件工程的角度上来考虑，还是用在构造器中调用set方法来给域赋值比较好，这样一来，给域赋值的行为就集中在一个地方，不需要重复，而且以后更易于维护，需要改变内部赋值过程的时候，只需要修改set方法就可以了