文章/答案/技术大牛

发布

社区首页 >问答首页 >Rails认知复杂度为6(允许超过5)

问Rails认知复杂度为6(允许超过5)
EN

Stack Overflow用户

提问于 2020-10-21 21:49:35

回答 2查看 561关注 0票数 0

我有一个连接到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(

这个错误是在我添加了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

如何避免这样的错误？

ruby-on-rails

ruby

code-climate

回答 2

Stack Overflow用户

发布于 2020-10-21 22:31:01

您可以通过创建一些可以调用的助手方法来降低复杂性。也许吧

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吗？潜在呼叫者不知何故知道它将丢弃所有非美元转账，这是合理的吗？

票数 0

Stack Overflow用户

发布于 2020-12-03 02:23:16

这不是一个错误，cognitive complexity是一个：

度量直观地理解代码单元的难易程度

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

您可以采取两种方法来尝试将这种复杂性降至最低：

修改您添加的现有方法statement

Modify

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

票数 0

页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持

原文链接：

https://stackoverflow.com/questions/64465013

复制

相似问题

问Rails认知复杂度为6(允许超过5)
EN

回答 2

Stack Overflow用户

Stack Overflow用户

社区

活动

圈层

关于

腾讯云开发者

热门产品

热门推荐

更多推荐

问Rails认知复杂度为6(允许超过5)EN

回答 2

Stack Overflow用户

Stack Overflow用户

社区

活动

圈层

关于

腾讯云开发者

热门产品

热门推荐

更多推荐

问Rails认知复杂度为6(允许超过5)
EN