1

这是一个低级设计问题。我正在尝试掌握面向对象的编程。

我正在编写一个 Java 类，其方法是设置值对象的数据成员的值。这些值是由我的班级根据一些业务逻辑计算的。最后，我的类应该返回如此填充的值对象。这样做最面向对象的方式是什么？

一种可能的方法，我似乎要走的路是，在我的 Java 类中为我需要设置的每个数据成员（VO）创建 compute() 方法。这些计算方法中的每一个都返回我使用值对象的相应 setter 方法设置的所需值。但是，我认为这种方法比面向对象更程序化，因为我只是按顺序按程序调用这些计算方法并填充我的值对象。

那么我可以使用/查看哪些设计（......设计模式，也许？）或最佳实践，以便此代码成为面向对象的。

代表代码（我现在拥有的）：

class Adapter {   // this is my Java class
    MyValueObject applyBusinessLogic(Input object)
    {
        MyValueObject vo = new MyValueObject();
        vo.setA(computeA());
        vo.setB(computeB());
        vo.setC(computeC());
        return vo;
    }

    String computeA() { ...some logic... return a;}
    String computeB() { ...some logic... return b;}
    String computeC() { ...some logic... return c;}
}

class MyValueObject {
      String a;
      String b;
      String c;

      public void setA(String a)
      {
           this.a = a;
      }

      public void setB(String b)
      {
           this.b = b;
      }

      public void setC(String c)
      {
           this.c = c;
      }
}
4

3 回答 3

1

恕我直言，这实际上取决于职责和计算方法的作用。您应该牢记 SRP (http://en.wikipedia.org/wiki/Single_responsibility_principle) 来进行 OO 设计。还有高内聚和松耦合原则。

如果 computeA()、B 和 C 不相关，则应拆分您的 Adapter 类。如果它们是相关的，那么您的 Adapter 类应该以不同的方式命名以表达它的实际作用。例如：税务计算器。

然后你可以创建一个新的类来接收一个 TaxCalculator 和一个 MyValueObject 来应用。或者只是将应用程序留在计算器中（尽管一个类需要两个职责，这通常很糟糕）。在后一种情况下，也许您可​​以调用类 TaxCalculationsSetter 或类似的东西，并将计算方法设为私有。等等，你会再次满足 SRP。;-)

永远记住，“什么是责任”是一个有点主观的东西。

我希望这已经澄清了一些事情。

于 2012-12-21T12:02:39.813 回答
1

正如izaera 已经提到的，这在很大程度上取决于您没有提供的细节或问题。但是，至少有一件事在我看来是不对的：

您的值对象的构造似乎需要一些逻辑。现在的问题是，它应该去哪里？您已经在适配器中完成了它。但是，这可能不对，因为VO的构造属于VO而不是Adapter。因此，我肯定会将该逻辑移至 VO 本身——作为其构造的一部分。

因此，在我看来，将该逻辑移至 VO 会改进设计。

...
MyValueObject vo = new MyValueObject(); 
vo.setA(required input);
vo.setB(required input);
vo.setC(required input);

class MyValueObject {
      String a;
      String b;
      String c;

      public void setA(required input)
      {  
          ... some logic
          this.a = result of logic
      }

      public void setB(required input)
      {  
          ... some logic
          this.b = result of logic
      }

      public void setC(Srequired input)
      {
             ... some logic
          this.c = result of logic
      }
}

显然，如果使用构造函数立即完成构造，整个事情会变得更好，除非你必须使用设置器，如果说，你正在使用一些 ORM 等。

于 2012-12-21T16:37:16.997 回答
0

对我来说看起来不错。您不应该总是寻找要使用的编程模式。最重要的是 KISS（保持愚蠢和简单）

我会考虑是否可以使 MyValueObject 不可变。

class Adapter {   // this is my Java class
    MyValueObject applyBusinessLogic(Input object){
        return new MyValueObject(computeA(),computeB(),computeC());
    }

    String computeA() { ...some logic... return a;}
    String computeB() { ...some logic... return b;}
    String computeC() { ...some logic... return c;}
}

class MyValueObject {
    final String a;
    final String b;
    final String c;

    public MyValueObject(String a, String b, String c){
        this.a=a;
        this.b=b;
        this.c=c;
    }

}
于 2012-12-21T12:03:58.700 回答