逻辑编程帮助 - II

Question

A

if (  ( empty($infoA) || empty($infoB) ) && ( !empty($inputA) && !empty($inputB) )  ) 
{
  //add
}

B

if (  ( !empty($infoA) || !empty($infoB) ) && ( empty($inputA) && empty($inputB) )  )
{
  //remove
}

C

if (  ( !empty($infoA) || !empty($infoB) ) && ( ($inputA != $infoA) || ($inputB != $infoB) )  ) 
{
  //add
}

---

因此，为了不重复添加，我们可以：

if (A || C) 
{
  //add
}
elseif(B) 
{
 //remove
}

在您的选项上有更好的逻辑吗？

语境： 我相信这应该做什么无关紧要。我的意思是，这是一个逻辑问题。 ：s 不知道在这里写什么...:(

这是一个表单：一些输入将来自数据库，其他输入来自输入字段。 我们在这里做一些比较。

A上下文： 如果来自数据库的值为空，并且输入字段A和B不为空，则添加到数据库。

B 上下文： 如果来自数据库的值不为空，并且输入字段 A 和 B 为空，请从数据库中删除。

C 上下文： 如果来自数据库的值不为空，并且输入字段A!等于 infoA或输入字段B不等于数据库值infoB，那么， 添加到数据库。

请指教。 MEM

Answer 1

几个问题：

• C真的是Add，还是update？
• 您真的需要将 inputA 和 inputB 的处理与 infoA 和 infoB 联系起来吗？

一般来说，我会通过以下方式重构它：将

• 添加/删除决策拆分为单独的 process_input 函数，该函数仅处理一对输入/信息对象。
• 将添加/删除活动拆分为它们自己的函数，这些函数将处理数据的基本空处理。

逻辑的唯一简化是调用 add() 或 remove() 的决定仅基于输入数据是否为空。这就是真正需要的一切。 add() 和remove() 函数执行自己的数据验证，以确保操作对于提供的数据有效。

此重构演示了两种重要的编程技术：

• DRY（不要重复自己）原则...我们不应该多次检查对象上的empty()。
• SRP（单一责任原则）——每个操作应该只做一件事。
  • 输入处理程序负责将输入数据与现有信息数据配对，然后调用 process_input。
  • process_input函数只负责决定是否需要执行add()或remove()操作。
  • add() 和remove() 函数各自负责自己不同的操作，可以独立于其余代码进行测试

（请原谅我的php...它有点生疏）

function remove($data)
{
    if (empty($data))
        return;

    // Do remove operation
}

function add($data)
{
    if (empty($data))
        return;

    // Do add operation
}

function process_input($input, $info)
{
    if (empty($input))
        remove($info);
    else
        add($input);
}

process_input($inputA, $infoA);
process_input($inputB, $infoB);

Answer 2

我认为展开条件并使用 else 或 else-if 语句对于代码的可读性和理解有很大帮助。虽然您提出的答案减少了代码行数，但它将很难理解和调试，特别是如果您将其传递给其他人。

这是我的看法，

// check if to update in DB or not

$value = db_adapter::get_var($sql);
if (empty($value))
{
  if (!empty($field_A) && !empty($field_B))
    add($field_A, $field_B);  

} else 
// you can roll the following block up into a function
{


  if (empty($field_A) && empty($field_B))
    remove($field_A, $field_B);
   else {
      if ($field_A != $value)
       update($field_A);  
     else if ($field_B != $value)
       update($field_B);
   }

}

其他每个都可以汇总到一个函数中；我没有这样做，因为我想不出任何逻辑来调用 else 块中的代码块（处理可能删除更新？确定更新或删除？）