問題タブ [reek]

Rails プロジェクトで reek を実行すると、次のような警告が表示されます。

[36]:ArborReloaded::UserStoryService#destroy_stories には約 8 つのステートメントがあります (TooManyStatements)

メソッドは次のとおりです。

この方法をどのように最小化できますか?

私は現在、テストツールの非常に簡単なスタートを書いています。アイデアは、私の「テスト可能な」モジュールを含むクラスを持つことができるということです。例えば：

Testable モジュールには次のものがあります。

Reek がこのファイルをチェックアウトすると、以下が返されます。

でもそこがインクルードメソッドのポイントではないでしょうか。

Reekチェックのこの側面をオフにできることはわかっていますが、ここでのアドバイスに従うにはどうすればよいでしょうか? この場合、クラスは私がすぐに知っているものではありません。他の人が作成したクラスには、私のモジュールが含まれます。

繰り返しますが、チェックをオフにできることはわかっていますが、他のコンテキストでチェックが必要なようです。それで、おそらく「ミックスイン」アプローチを間違っているのではないかと思い始めました。それが Reek が指摘していることです。

クラスのこの単純なメソッドはstatus、安全なナビゲーション演算子を使用してメソッドを実行するだけです。

しかし、reek は次の警告を報告します。

Nil Check を回避するために、このようなメソッドを適切に記述するにはどうすればよいですか?

思考ボットからのこの投稿も確認しましたが、安全なナビゲーション オペレーターには「多すぎる」ように思えます。

ルビー 2.3.1