Rubocopを使ってRubyプロジェクトのスタイル管理などを行っているのですが、そのルールの中にNaming/BinaryOperatorParameterName
というものがあります。
これは、こちらのスタイルガイドにもあるように、「([]
や<<
を除く)2項演算子を実装するときには、仮引数名をother
にする」というものです。
もちろん、普通の数値演算などであればother
で特に問題ないのですが、時刻や(Rubyにはないですが)ポインタに対する加算のように、「別の種類の値しか足し算できない」ようなものについては、other
を使うことで逆にわかりにくくもなるように思えます。
ここで質問なのですが、
- 「このような状況でも
other
を使うべき」なのでしょうか、それとも「別種の値であれば名前を変えるべき」なのでしょうか。 - 名前を変える場合に、Rubocopのエラー対策としては「
Naming/BinaryOperatorParameterName
自体を止める」「演算子実装のところで#rubocop
コメントを入れる」など、どのような方法が適切でしょうか。
なお、「プロジェクト内に確立したコーディング規約はない」という条件で考えていただければ幸いです。
回答1件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2017/11/15 05:46