@mone_piposted at 2023-02-07

[rubocop] Wrap hash in { and }.

Q&A

Closed

Ruby Rails RuboCop

解決したいこと

rubocopでWrap hash in { and }.の指摘を修正したい

発生している問題・エラー

以下のincludesの部分でrubocop指摘されました。

Advertisement.includes(program: [:admin, campaign: { donor: :credit }])

テーブル構成

Advertisement
┗program
　┣campaign
　┃　┗donor
　┃　 ┗credit
　┗admin

自分で試したこと

[:admin, campaign: { donor: :credit }]の部分が指摘されているのかなと思うのですが、対処方法が見つからなかったので、質問させていただきました。

参考にしたサイト
・Rails の ActiveRecord で孫モデルやひ孫モデルに join や include する方法（深いネストのincludes）
・Rails ActiveRecord/SQL 小技集
・Railsで複数テーブルのjoins/includesとwhere検索

2Answer

@uasi posted at 2023-02-07

指摘に Style/HashAsLastArrayItem と RuboCop ルール名が出ていると思います。修正方法を知るにはこのルール名で検索してください。 https://docs.rubocop.org/rubocop/cops_style.html#stylehashaslastarrayitem のように公式ドキュメントの修正方法が見つかります。

指摘の内容は、配列の最後にあるハッシュだけは {} で囲まずに書ける言語仕様があるが、書くことを推奨するというものです。 [:admin, { campaign: { donor: :credit } }] のように修正すれば解消します。

1Like

@mone_pi posted at 2023-02-07

ありがとうございます！！
助かりました！！

0Like

Are you sure you want to delete the question?