在避免RuboCop的ABC大小警告的同时，还有一种更好的分配Ruby哈希的方法吗？

问题描述

我有一个方法可以构建笔记本电脑的属性，但前提是该属性存在于该方法的行中：

def build_laptop_attributes desk_id,row,laptop
  attributes = {}
  attributes[:desk_number] = room_id if laptop && desk_id
  attributes[:status] = row[:state].downcase if row[:state]
  attributes[:ip_address] = row[:ip_address] if row[:ip_address]
  attributes[:model] = row[:model] if row[:model]
  attributes
end

当前，RuboCop表示Metric/AbcSize太高了，我想知道是否存在一种明显且干净的方式来分配这些属性？

解决方法

样式指南提供“最佳做法”；评估并在需要时进行调整

首先，RuboCop是建议。仅仅因为RuboCop抱怨某事并不意味着从某种绝对意义上说是错误的。这只是意味着您应该（在执行操作时）花费更多的头骨汗水，以查看您执行的操作是否有意义。

第二，您还没有提供一个独立的可执行示例。这使得SO读者无法可靠地对其进行重构，因为如果没有原始帖子中未提供的示例输入和预期输出，当前无法对其进行测试。您还需要这些东西来评估和重构自己的代码。

最后，ABC Metric查看分配，分支和条件。您有五个分配，四个条件和看起来像一个方法调用的对象。那很多吗如果您尚未调优Rubocop，答案为“ RuboCop如此认为”。您是否同意取决于您和您的团队。

如果您想尝试喂Rubocop，可以做一些可能有助于降低指标的事情：

1. 重构任务的数量和复杂性。一些可能的示例包括：

   • 用安全的导航器（&.）替换后缀if语句，以防止在nil上调用方法。

   • 将您的某些分支逻辑和条件提取到“做正确的事”的方法上，从而有可能通过四个方法调用将当前方法缩减为单个分配。例如：

     attributes = { desk_number: location,status: laptop_status,... }
     

   • 用解构性作业替换所有多个作业（尽管Rubocop也经常抱怨这些作业）。

2. 首先检查是否具有正确的数据结构。也许您真的只想要一个OpenStruct或其他一些数据对象。

您当前的代码似乎可读，那么榨汁真的值得吗？如果您确定在这种情况下RuboCop被误导了，并且您的代码可以正常工作并且在内部代码审核中通过了汇总，那么您可以仅在该部分的.rubocop.yml或disable that particular metric中调整指标的敏感度源代码。

,

在阅读@Todd A. Jacobs的答案后，您可能想要（或不希望）写这样的东西：

def build_laptop_attributes desk_id,row,laptop
  desk_number = room_id if laptop && desk_id

  {
    desk_number: desk_number,status: row[:state]&.downcase,ip_address: = row[:ip_address],model: row[:model]
  }.compact
end

这种减少的优点是减少了对[]=的呼叫次数，并且在单个if中分解了许多compact。

我认为它更易读，因为它更简洁，并且重点完全放在键和值之间的对应上。

,

减少条件数量的替代版本（假设您要检查nil /初始化值）：

def build_laptop_attributes desk_id,laptop
  attributes = {}
  attributes[:desk_number] = room_id if laptop && desk_id
  attributes[:status] = row[:state]&.downcase
  attributes[:ip_address] = row[:ip_address]
  attributes[:model] = row[:model]

  attributes.compact
end

还有另外的.compact作为取消分配检查的费用。