删除 if 语句的嵌套

Question

我有一段代码，我不知道如何重构它。它的可读性不是很好，我想让它可读。这是一个问题，

数据库中有两列，它们可以是 NULL、0 或每列都有一个值。在网页上，这两列中的每一列都有一个复选框 - 启用和文本框 - 值。

x = checkbox1
z = textbox1
y = checkbox2
w = textbox2

逻辑是，如果两个复选框均未选中，则两个值都应为 0。如果其中一个被选中而另一个未选中，则其他值应为 NULL。对于所选的文本框，如果文本框为空，则其值应为 NULL，否则应为文本框中的值

if{x}
{
   if(z)
   {
      a = NULL;
   }
   else
   {
      a = z;
   }
   if(y)
   {
      if(w)
      {
          b=w;
      }
      else
      {
          b = NULL;
      }
   }
   else
   {
      b = null
   }
}
else
{
   if(y)
   {
      a = NULL;
      if(w)
      {
          b=w;
      }
      else
      {
          b = NULL;
      }
   }
   else
   {
      a = 0;
      b = 0;
   }
}

相信我，这是一个有效的场景。让我知道这是否有意义或者我应该提供更多信息

Answer 1

使用一些逻辑与和非，我们得到更具可读性的东西。
我们可以通过默认为 NULL 来节省一点（因此不需要将另一个设置为 NULL）。我们还可以通过将检查文本框是否已设置或使用 null 的代码放入一个小函数中来节省。

在伪代码中：

a = NULL
b = NULL
if (not checkbox1) and (not checkbox2):
  a = 0
  b = 0
if (checkbox1):
  a = valueornull(textbox1)
if (checkbox2):
  b = valueornull(textbox2)


function valueornull(textbox):
  if textbox value:
    return value
  else:
    return null

Answer 2

我认为使用比这里的单个字母更具描述性的名称会有所帮助，但假设这是 C 代码，那么使用内联 if 语句看起来会更整洁：

if(x)
{
   a = z ? NULL : z;
   b = (y && w) ? w : NULL;
}
else
{
   a = y ? NULL : 0;
   b = (y && w) ? w : 0;
}