if else 类构造函数中的条件...这是好的做法吗？

Question

我编写了一个构造函数并传递一个布尔标志来决定将哪个值设置为类变量。代码如下

public PDFParagraph(PDFPhrase phrase,boolean isRtl) {
            super(phrase);
            if(isRtl)
                    this.setAlignment(Element.ALIGN_RIGHT);
            else
                    this.setAlignment(Element.ALIGN_LEFT);
    }

现在我很困惑，不确定是否应该在构造函数中添加 if...else 条件。设置类变量值的风格是否良好？

谢谢， 哈努曼特。

Answer 1

构造函数中的条件本身没有问题。但是，在这种情况下，我倾向于像这样编写构造函数：

public PDFParagraph(PDFPhrase phrase, boolean isRtl) {
    super(phrase);
    setAlignment(isRtl ? Element.ALIGN_RIGHT : Element.ALIGN_LEFT);
}

Answer 2

使用 if / else 来执行此操作不存在样式问题。但是：

• 你可以写得更简单：

  setAlignment(isRtl ? Element.ALIGN_RIGHT : Element.ALIGN_LEFT);
  

• 很多人（包括我自己）认为你应该总是在“then”和“else”语句两边加上大括号。

---

相关的一点是：如果您发现自己编写了一个如下所示的构造函数：

public Thing(boolean cond, ...) {
    super(...);
    if (cond) {
        // Lots of statements
    } else {
        // Lots of different statements
    }
    ...
}

这可能表明您需要重构构造函数。 （或者可能不是……这取决于细节。）

Answer 3

如果您只是将初始对齐传递给 set，那么对于构造函数的用户来说可能会更清楚，特别是如果它是 Enum 并且这些是唯一的可能性。但是，如果您尝试限制初始对齐并且它类似于 String，那么您所做的似乎没问题。不过，为了清楚起见，您可能仍然需要考虑使用Enum。它比传递给构造函数的 true 或 false 更容易阅读。

Answer 4

当然，您可以在构造函数中添加 if/else 语句。

通常，最佳实践是使方法尽可能原子（简短且切中要害）且定义清晰。这也适用于构造函数。您希望构造函数根据您传递的参数来设置您的对象。

如果您需要 if/else，那么您可以添加一个。
如果需要的话，您甚至可以疯狂地放入 for 循环！ ;)

Answer 5

我建议您使用枚举和 switch 语句而不是布尔值。添加另一个对齐方式时会发生什么？