問題タブ [ruby-style-guide]

複雑な割り当てを設定するこのメソッドを調整しようとしており、この関数が警官を通過させるための他のオプションを探しています。

誰かが私を正しい方向に向けるための考えを持っていますか?

現在、内部の 2 つの .map 呼び出しを分割しようとしています。

失敗した警官

問題のコード

ヘルパー メソッド

更新されたエラー

仕様: 引数の数が間違っています (指定された 2 に対して 1 が期待されます)。Failure/Error: SELECT = ->(opt) { opt['type'] == 'product_options' }

parse_items の割り当て分岐条件のサイズが大きすぎます。[17/15]

更新されたコード

多くのエラーが発生する可能性のあるアプリケーションを開発しています。そのため、私 (およびこのアプリケーションを使用する管理者) は、すべての関連情報をログに記録することに大きな関心を持っています。しかし、ルビー スタイル ガイドに苦労しています。rubocop のデフォルトが大好きで、通常のコードはほとんどの場合に適合します。しかし、ロギングを使用すると、通常のアプリケーションよりも多くのコード行が必要になります。

例については、この構造を参照してください

スタイルガイドに準拠した方法で sich エラーをログに記録するためのトリックまたは一般的なパターンはありますか? または、この場合スタイルガイドを無視する必要がありますか?

私のテストファイルで Ruboco に問題がありました。最初に、これは私のコードです：

これが Ruboco のエラー出力です。

そこで、スタイルガイドで JSON の正しい位置合わせを検索しました。インデントと改行のあらゆる組み合わせを実際に試しましたが、Rubocop は毎回同じエラーをスローします。Andy ちなみに、JSON 全体を 1 行にまとめることも解決策ではありません。Rubocoが満足するように、正しい配置がどのように見えるかを誰か説明できますか?

Rubocop /コミュニティ主導の Ruby スタイル ガイドがメソッド定義で括弧を推奨するのはなぜですか?

状況に応じて、括弧の有無にかかわらずメソッド呼び出しが許可されます。ただし、私の第一印象は、メソッド呼び出しに括弧がないことは、メソッド定義に括弧がないことよりも潜在的にあいまいであるということです。コードをより簡単にするためなど、その背後に理由があったのでしょうか、それとも「歴史的な理由」や「最も普及したスタイルだったから」という理由でそうなったのでしょうか?

説明:

どちらのスタイルが読みやすいかについて意見を求めているわけではありません。

lintは、あいまいでバグの原因とLint/AmbiguousOperatorなるという考えに基づいています ( Link )。これは(リンク)の場合と同じではないかと思いました。do_something *some_arrayStyle/MethodDefParentheses

それらの警官の実際の名前を見つけるために戻った後、私の最善の推測では、「技術的な」理由はなく、1つは適切な「糸くず」であり、もう1つは「スタイル」の問題です.