1

我有一个连接到 CodeClimate 的应用程序，它向我显示了一个Method upstream_transactions has a Cognitive Complexity of 6 (exceeds 5 allowed)带有以下代码的错误：

def upstream_transactions(bank_account:, external_account:, external_transactions:)
  external_transactions.each do |transaction|
    next if transaction.currency != 'USD'

    transfer = BankTransfer.unscoped.find_or_create_by(

添加 line 后出现此错误next if transaction.currency != 'USD'。

def upstream_transactions(bank_account:, external_account:, external_transactions:)
  external_transactions.each do |transaction|
    next if transaction.currency != 'USD'

    transfer = BankTransfer.unscoped.find_or_create_by(
      customer_id: customer.id, transaction_identifier: transaction.transaction_id
    )
    next if transfer.deleted_at?

    transfer.update(
      date: transaction.booked_at,
      # (...) some other params
    )
  end
  Success(bank_account: bank_account, external_account: external_account)
end

如何避免这样的错误？

4

2 回答 2

1

您可以通过制作一些可以调用的辅助方法来降低复杂性。也许

def obtain_transfer(transaction)
  BankTransfer.unscoped.find_or_create_by(
      customer_id: customer.id, transaction_identifier: transaction.transaction_id)
end

中的“不”更少

   next unless transaction.currency == 'USD'

虽然你甚至可以

    external_transactions.select { |t| t.currency == 'USD' }.each do |transaction|

应该调用该方法upstream_USD_transactions吗？潜在调用者以某种方式知道它将丢弃所有非美元转账是否合理？

于 2020-10-21T14:31:00.903 回答
0

这不是错误，认知复杂性是：

衡量一个代码单元直观理解的难易程度

因此，当您添加额外的流控制语句next if transaction.currency != 'USD'时，它会增加代码测试的复杂性。添加诸如if, unless,之类的语句&&以及任何添加额外逻辑流控制的语句（例如 switch 语句）都会增加复杂性。

您可以采取两种方法来尽量减少这种复杂性：

  1. 修改您添加的语句
  2. 修改现有方法

如果在一个已经很复杂的方法中添加了一条小语句，那么请尝试查看该现有方法中可以简化的内容。就像@nullTerminator 在他们的回答中所建议的那样，将逻辑划分为不同的辅助方法可以降低复杂性。较小的方法将更容易测试，并且每个方法都应该具有较小的认知复杂性。

于 2020-12-02T18:23:15.793 回答