.net - 缩短这个 IF 语句

Question

我不喜欢宽代码，尤其是当它迫使我滚动时。所以写了这个:

If _item.SubItems(pd.perioddate).Text = "N/A" Or _item.SubItems(pd.perioddate).Text = String.Empty Then
            dtpDeadlineforAP.Checked = False
End If

有没有什么好的方法可以让它变薄，让它更优雅？

Answer 1

将 _item.SubItems(pd.periodate).Text 提取到局部变量中，例如

String text = _item.SubItems(pd.perioddate).Text

If text = "N/A" Or text = String.Empty Then
            dtpDeadlineforAP.Checked = False
End If

或者，您可能希望将整个支票提取到一个单独的方法中:

If isNotFilled(_item.SubItems(pd.perioddate)) Then
            dtpDeadlineforAP.Checked = False
End If

这将使代码更具可读性，并允许您重用检查逻辑。