[PHP] コードの可読性を高めるにはクラス化するべきか、関数化すべきか

Question

#分からないこと

• クラス化と関数化の使い分けが分からない
• そもそもコードの可読性を高めるために意識することが知りたいです。

#可読性を高める目的

• 自分以外の方もストレスなく開発できること
• デバックしやすい状況を作りたい

一応、今書いているコードのメイン処理文を書きます。

  global $wpdb;
  $ids = getExampleIds($wpdb);
  foreach ($ids as $id) {
    $exampleId = $id->meta_value;
    $json = getJsonData($exampleId);
    $exampleData = getExampleData($json, $exampleId);

    insertDataToExamplesTable($exampleData, $wpdb);

    insertDataToTermsTable($exampleData, $wpdb);

    insertDataToTermTaxonomyTable($exampleData, $wpdb);

    insertDataToTermRelationships($id, $exampleData, $wpdb);
  }

関数化することで、多少可読性が高まりましたが、これで良かったのか不明です。
どんなことでも良いので、ご助言を頂戴できると幸いです。
宜しくお願い致します。

Answer 1

関数化とは？

機能（振る舞い）の共通化です。
非常に単純な概念ですので関数化すべきか？では、あまり悩まないはずです。

クラス化とは？

主な目的は、機能（振る舞い）とデータのカプセル化です。

カプセル化とは、

• 機能（振る舞い）とデータの紐付け
• 複雑な処理のブラックボックス化（外部に対しての抽象化）
• 状態やフラグなどの隠蔽

を指します。
あとは継承により再利用できるか否かです。

主なメリットは、複雑さを外部から隠蔽できることと、データと機能がひとつにまとまる事です。

使い分けのポイント

クラス化するメリットがあるかを考えればよいでしょう。
コードを拝見すると、プレフィックス（前置詞）と共通の引数（データ）が多いのでクラス化することで可読性が上がると思います。
逆にクラス化していて可読性が下がるというのは、クラス設計に問題があるように思えます。

ただし、注意してほしいのはクラス設計は大切なんですが、早すぎる最適化はしないことです。
まだ小さいプログラムで可読性に問題ないのであれば、現状どちらでもかまいません。
このプログラムが小さいプログラムで済む可能性があるので、ここで設計に悩むのは時間の無駄です。

大切な事は、クラス化するメリットと目的を十分に理解する事です。

Answer 2

関数化については機能ごとにプログラムをきりわけるものですから
可読性はあまり重要ではないかも。どちらかと言うと汎用性・再利用性を
気にしたほうがいいですね。
当然関数の中身を書くときはデバグの必要もあるので可読性を気にしてください
関数を利用して構成することでER図を書いた時の流れがわかりやすくなると思います。

クラスについては機能群＝アプリケーションとして動かすためのものです。
最終的には必要性に迫られてクラス化を模索することになると思いますが
たとえばこのサイトでも簡単なプログラムを例示するときにわざわざクラスを書かないように
全体感を見ながら書いていくことになるでしょう