如何用布尔清除器拨打电话？布尔陷阱

如@ benjamin-gruenbaum在评论中所指出的，这称为布尔陷阱：

说我有一个这样的功能

UpdateRow(var item, bool externalCall);

在我的控制器中，该值externalCall始终为TRUE。调用此函数的最佳方法是什么？我通常写

UpdateRow(item, true);

但是我问自己，我是否应该声明一个布尔值，以表明“真”值代表什么？您可以通过查看函数的声明来知道这一点，但是如果您看到类似以下内容，显然会更快，更清晰

bool externalCall = true;
UpdateRow(item, externalCall);

PD：不确定这个问题是否真的适合这里，如果不合适，我在哪里可以得到更多有关此的信息？

PD2：我没有标记任何语言，因为我认为这是一个非常普遍的问题。无论如何，我使用c＃，并且可接受的答案适用于c＃

并非总是有完美的解决方案，但是您有许多选择可供选择：

• 如果您的语言可用，请使用命名参数。这非常有效，并且没有任何特定的缺点。在某些语言中，任何参数都可以作为命名参数传递，例如updateRow(item, externalCall: true)（C＃）或update_row(item, external_call=True)（Python）。

  您建议使用单独的变量是模拟命名参数的一种方法，但没有相关的安全性（不能保证您为该参数使用了正确的变量名）。

• 为您的公共接口使用不同的功能，并使用更好的名称。这是通过将参数值放在名称中来模拟命名参数的另一种方法。

  这是很容易理解的，但是会给编写这些功能的人带来很多样板。当存在多个布尔参数时，它也不能很好地应对组合爆炸。一个重要的缺点是客户端无法动态设置此值，但必须使用if / else来调用正确的函数。

• 使用一个枚举。布尔值的问题在于它们分别称为“ true”和“ false”。因此，请改用名称更好的类型（例如enum CallType { INTERNAL, EXTERNAL }）。另外一个好处是，这增加了程序的类型安全性（如果您的语言将枚举实现为不同的类型）。枚举的缺点是它们会向您的公开可见API添加类型。对于纯内部函数，这无关紧要，枚举没有明显的缺点。

  在没有枚举的语言中，有时会使用短字符串代替。这行得通，甚至可能比原始布尔值更好，但很容易产生错字。然后，该函数应立即断言该参数与一组可能的值匹配。

这些解决方案都不会对性能产生过大的影响。命名参数和枚举可以在编译时完全解析（对于已编译的语言）。使用字符串可能涉及字符串比较，但是对于小的字符串文字和大多数类型的应用程序而言，其成本可以忽略不计。

正确的解决方案是按照您的建议进行操作，但是将其打包成一个迷你外观：

void updateRowExternally() {
  bool externalCall = true;
  UpdateRow(item, externalCall);
}

可读性胜过微观优化。您可以负担得起额外的函数调用，这肯定比您可以负担得起甚至必须一次查找布尔标志的语义的开发人员的负担更好。

说我有一个像UpdateRow（var item，bool externalCall）;

为什么会有这样的功能？

在什么情况下，您可以将externalCall参数设置为不同的值来调用它？

如果一个来自外部客户端应用程序，而另一个来自同一程序（即不同的代码模块），那么我认为您应该有两种不同的方法，每种方法一种，甚至可能在不同的情况下定义。接口。

但是，如果您基于从非程序源（例如，读取的配置文件或数据库）获取的某些数据进行调用，则布尔传递方法会更有意义。

虽然我同意使用语言功能来增强可读性和值安全性是理想的，但您也可以选择一种实用的方法：呼叫时注释。喜欢：

UpdateRow(item, true /* row is an external call */);

要么：

UpdateRow(item, true); // true means call is external

或（正确地，按照Frax的建议）：

UpdateRow(item, /* externalCall */true);

1. 您可以“命名”您的布尔值。以下是OO语言的示例（可以在提供的类中表达UpdateRow()），但是概念本身可以以任何语言应用：

   class Table
   {
   public:
       static const bool WithExternalCall = true;
       static const bool WithoutExternalCall = false;
   

   并在呼叫站点中：

   UpdateRow(item, Table::WithExternalCall);
   

2. 我相信项目1更好，但是在使用函数时，它不会强制用户使用新变量。如果类型安全性对您很重要，则可以创建一个enum类型并使其UpdateRow()接受而不是bool：

   UpdateRow(var item, ExternalCallAvailability ec);

3. 您可以通过某种方式修改函数名称，使其bool更好地反映参数的含义。不太确定，但也许：

   UpdateRowWithExternalCall(var item, bool externalCall)

我在这里还没有阅读的另一个选项是：使用现代IDE。

例如的IntelliJ IDEA打印的方法，如果你传递一个文字如你调用变量的变量名true或null或username + “@company.com。这是用小字体完成的，因此它不会在屏幕上占用太多空间，并且看起来与实际代码有很大不同。

我仍然不是说在任何地方都放入布尔值是一个好主意。说您读代码的频率比编写代码的频率高的论点通常非常有力，但是在这种情况下，它确实很大程度上取决于您（和您的同事！）用来支持工作的技术。与IDE相比，IDE的问题要少得多。

我的测试设置部分的修改示例：

2天，没有人提到多态吗？

target.UpdateRow(item);

当我想用项目更新行时，我不想考虑数据库名称，微服务的URL，用于通信的协议或是否使用外部调用。需要被用来实现它。不要再向我推这些细节了。只需拿走我的物品，就可以在某处更新行了。

这样做，它便成为施工问题的一部分。这可以通过多种方式解决。这是一个：

Target xyzTargetFactory(TargetBuilder targetBuilder) {
    return targetBuilder
        .connectionString("some connection string")
        .table("table_name")
        .external()
        .build()
    ; 
}

如果您正在查看并想到“但是我的语言已经命名了参数，那么我就不需要了”。很好，很好，去使用它们。只是把这些废话放在甚至不知道在说什么的主叫客户端之外。

应该注意的是，这不仅仅是语义问题。这也是一个设计问题。传递布尔值，您将不得不使用分支来处理它。不是很面向对象。有两个或多个布尔值可以传递吗？希望您的语言有多次派遣吗？查看将嵌套的集合可以做什么。正确的数据结构可以使您的生活更加轻松。

如果可以修改updateRow功能，也许可以将其重构为两个功能。鉴于该函数带有布尔参数，我怀疑它看起来像这样：

function updateRow(var item, bool externalCall) {
  Database.update(item);

  if (externalCall) {
    Service.call();
  }
}

这可能有点代码味道。根据externalCall变量的设置，函数可能具有截然不同的行为，在这种情况下，它具有两种不同的职责。将其重构为只有一个职责的两个功能可以提高可读性：

function updateRowAndService(var item) {
  updateRow(item);
  Service.call();
}

function updateRow(var item) {
  Database.update(item);
}

现在，在任何调用这些函数的地方，您都可以一目了然地判断是否正在调用外部服务。

当然，情况并非总是如此。这是情境和品味的问题。重构将布尔参数转换为两个函数的函数通常值得考虑，但并非总是最佳选择。

如果UpdateRow在受控代码库中，则应考虑策略模式：

public delegate void Request(string query);    
public void UpdateRow(Item item, Request request);

其中，Request代表某种DAO（平凡情况下的回调）。

真实情况：

UpdateRow(item, query =>  queryDatabase(query) ); // perform remote call

错误的情况：

UpdateRow(item, query => readLocalCache(query) ); // skip remote call

通常，端点实现将在更高的抽象级别上进行配置，我将在此处使用它。作为有用的免费副作用，这使我可以选择实现另一种访问远程数据的方式，而无需更改代码：

UpdateRow(item, query => {
  var data = readLocalCache(query);
  if (data == null) {
    data = queryDatabase(query);
  }
  return data;
} );

通常，这种控制反转可确保数据存储和模型之间的耦合程度较低。