如何解决在避免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,可以做一些可能有助于降低指标的事情:
- 重构任务的数量和复杂性。一些可能的示例包括:
-
用安全的导航器(
&.
)替换后缀if语句,以防止在nil
上调用方法。 -
将您的某些分支逻辑和条件提取到“做正确的事”的方法上,从而有可能通过四个方法调用将当前方法缩减为单个分配。例如:
attributes = { desk_number: location,status: laptop_status,... }
-
用解构性作业替换所有多个作业(尽管Rubocop也经常抱怨这些作业)。
-
- 首先检查是否具有正确的数据结构。也许您真的只想要一个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
作为取消分配检查的费用。
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。