Sha256: 675659a4b1615873e138a1cfe6d0432fc9d8b9d511d3e1afbdedadd6824a638d
Contents?: true
Size: 1.85 KB
Versions: 2
Compression:
Stored size: 1.85 KB
Contents
# frozen_string_literal: true module RuboCop module Cop module Performance # This cop is used to identify usages of `count` on an # `Array` and `Hash` and change them to `size`. # # @example # # bad # [1, 2, 3].count # (1..3).to_a.count # Array[*1..3].count # Array(1..3).count # # # bad # {a: 1, b: 2, c: 3}.count # [[:foo, :bar], [1, 2]].to_h.count # Hash[*('a'..'z')].count # Hash(key: :value).count # # # good # [1, 2, 3].size # (1..3).to_a.size # Array[*1..3].size # Array(1..3).size # # # good # {a: 1, b: 2, c: 3}.size # [[:foo, :bar], [1, 2]].to_h.size # Hash[*('a'..'z')].size # Hash(key: :value).size # # # good # [1, 2, 3].count { |e| e > 2 } # TODO: Add advanced detection of variables that could # have been assigned to an array or a hash. class Size < Base extend AutoCorrector MSG = 'Use `size` instead of `count`.' def_node_matcher :array?, <<~PATTERN { [!nil? array_type?] (send _ :to_a) (send (const nil? :Array) :[] _) (send nil? :Array _) } PATTERN def_node_matcher :hash?, <<~PATTERN { [!nil? hash_type?] (send _ :to_h) (send (const nil? :Hash) :[] _) (send nil? :Hash _) } PATTERN def_node_matcher :count?, <<~PATTERN (send {#array? #hash?} :count) PATTERN def on_send(node) return if node.parent&.block_type? || !count?(node) add_offense(node.loc.selector) do |corrector| corrector.replace(node.loc.selector, 'size') end end end end end end
Version data entries
2 entries across 2 versions & 1 rubygems
Version | Path |
---|---|
rubocop-performance-1.8.1 | lib/rubocop/cop/performance/size.rb |
rubocop-performance-1.8.0 | lib/rubocop/cop/performance/size.rb |