0

参考:RuboCop | Style/FormatString - Qiita

参考リンクにある通り、RuboCopで%を使うのをやめてformatを使うようにメッセージを出力する方法があります。
同じように 配列のhoge[1, 2]を使うのをやめてhoge.slice(1,2)を使うようにさせることはできますか?

values_atと混合しやすいため、sliceを使えという警告を出すことによってvalues_atのつもりで使ってしまっていたのに気づくやすくなると考えています。

2 件の回答 2

2

先に回答している方がおっしゃるとおり、そのようなCopは存在しません(ちなみに、CopというのはRuboCopにおけるルールの単位です)。

また、偽陽性の警告が非常に多くなると思われるため、そのようなCopを作成したとしても、RuboCop自身に含むことはないとおもいます。

例えば、以下のように String クラスにも [] メソッドは定義されています。
https://docs.ruby-lang.org/ja/latest/class/String.html#I_--5B--5D

x = "hoge"
x[1, 2] # => "og"

そのため、単純にコードを見ただけでは hoge[1, 2] は Array に対するメソッド呼び出しなのか String に対するものなのかが区別できないため、偽陽性の原因となってしまいます。

そのため、RuboCop ではなく別のツールを使ってみてはいかがでしょうか?
例えば querly というツールでは、このようなルールを手軽に自分で定義することが可能です。
https://github.com/soutaro/querly

# querly.yaml
rules:
  - id: array.slice
    pattern: "[](_, _)"
    message: 'array[x, y] でなく、array.slice(x, y) を使用して下さい'
    before: 'array[x, y]'
    after:  'array.slice(x, y)'

上記の内容をquerly.yamlとして保存した上で、querly check .とコマンドを実行することで期待した検査を実行することが可能です。

このツールは Rubygems で配布されているため、gem install querlyとしてインストールすることが可能です。

0

もしそういうCopがあれば、リポジトリをsliceで検索すれば、指摘メッセージの一部として存在するだろう、という予測の元に調べてみたら、ありませんでした。

というわけで、今のところなさそうなので、Issueで提案するか自分でCopを書くか、でしょうか。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。