書籍を購入しEC−CUBEの勉強は始めたところです。

リファクタリングガイドラインに

action 関数に, ビジネスロジックを直接記述したり, SQL を実行する処理を記述しないこと.

というのがありました。

購入した書籍ではaction関数でSQLを実行しているので？？？
となったのですが、これは、SQLを実行する処理を行う関数を作成し、action関数からはそれを呼び出して使うべきと考えていいのでしょうか？

よろしくお願いいたします。

はい。hidekota2 さんが仰られている方向は、「原則」としては正しい方向だと思います。

個別のサイトのカスタマイズなどで、ルールを知った上でルールを破るといった例外が、ときに最適解だったりする事もあるとは思いますが。

(最適と判断すれば、テンプレートから SQL を実行することも個人的にはやっています。)

----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

引用：