レコードの状態チェックなどの処理はｓｑｌ、アプリケーション側どちらで書いたらいいか

Question

お世話になります。

例えば、報告書テーブルがあったとしてその報告書には課長承認と部長承認２つの承認状態を持っているとして、
要件上指定した期限内に承認をとらないといけないとします。

ここでレコードのチェック状態を書くとき、それぞれのチェック状態をほしいのでチェック処理を作って、さらにその両方ともチェックされているかの処理も書くと
チェックロジックが分散してしまう気がしてしまいます。

public function is_katyou_check($id) {
  $sql = "SELECT * FROM `report` WHERE  `id` = ? && `katyou_check` = 1 && `period_date` > NOW()";
  $stmt = $this->instance->prepare($sql);
  $stmt->execute([$id]);
}
public function is_butyou_check($id) {
  $sql = "SELECT * FROM `report` WHERE  `id` = ? && `btyou_check` = 1 && `period_date` > NOW()";
  $stmt = $this->instance->prepare($sql);
  $stmt->execute([$id]);
}
public function is_katyou_butyou_check($id) {
  $sql = "SELECT * FROM `report` WHERE  `id` = ? && `katyou_check` = 1 && `btyou_check` = 1 && `period_date` > NOW()";
  $stmt = $this->instance->prepare($sql);
  $stmt->execute([$id]);
}

上記では期限内であるかのチェックもいれていますが、期限内かどうか関係なく取得したい場合もあると思うので、それぞれの条件を書いていくとかなりめんどくさい気がしてしまいます。これをアプリケーション側で書くとチェックロジックが分散せずにかけるとおもっています。

public function fetch($id) {
  $sql = "SELECT * FROM `report` WHERE  `id` = ? ";
  $stmt = $this->instance->prepare($sql);
  $stmt->execute([$id]);
}
public function is_katyou_check($report) {
  if($report->katyou_check) {
     return true;
   }
   return false;
}
public function is_butyou_check($report) {
  if($report->butyou_check) {
     return true;
   }
   return false;
}
public function is_not_expired($report) {
  if($report->period_date > time()) {
     return true;
   }
   return false;
}
public function is_katyou_butyou_check($report) {
  if($this->is_katyou_check($report) && $this->is_butyou_check($report) && $this->is_not_expired($report)) {
     return true;
   }
   return false;
}

public function check_report($id) {
  $report = $this->fetch($id);

  if($this->is_katyou_butyou_check($report) && $this->is_not_expired($report)) {
    return true;
  }

  return false;
}

上記の例だとsql側で書いたものと条件が処理がちょっと変わってるかもしれませんが、こんなイメージです。
どっちの方がいいのか悩んでいます。

皆様どのような設計で作っておられるのか、理由も含めてご意見いただければと思います。
よろしくお願いいたします。

Comments

[m.ts10806]

設計の話ではなく実装の話になってる気がしますし、
そんごくざっくり大枠しか出さないで詳細なアドバイスを得ようというのは都合が良すぎます。
具体的な回答を得たいなら具体的な内容にしてください。

Answer 1

毎回SELECTを実行するのもパフォーマンスが悪いのでSQLをちょっといじっては？

SELECT
    *
  , CASE WHEN `katyou_check` = 1 THEN '課長チェック済' END AS `kacho_checked`
  , CASE WHEN `btyou_check` = 1 THEN '部長チェック済' END AS `bucho_checked`
FROM `report`
WHERE `id` = ? AND `period_date` > NOW()";

ローマ字表はなるべくヘボン式
を使われては？

Comments

[annderber]

コメントありがとうございます。

例えば期限関係無く承認チェック済みのレコードがほしいなど色々な条件がある場合、Orlofskyさんはそれぞれのｓｑｌ文を書くイメージでしょうか

[Orlofsky]

どんな条件かによりテーブル設計やインデックス設計が変わります。
質問にCREATE TABLE, CREATE INDEX と実行を想定しているSQLを提示できた方が適切なコメントが付き易いです。
あと、英数字はシングルバイトに統一できた方が無用なトラブルが少ないです。

Answer 2

自分だったら、Reportモデルに状態判定用のメソッドを定義しますね。

class Report
{
    public function isApploved()
    {
        return $this->isApplovedByGeneralManager() &&
            $this->isApplovedByManager();
    }

    /**
     * 部長承認
     */
    public function isApplovedByGeneralManager()
    {
        return $this->apploved_by_general_manager &&
            $this->apploved_at_general_manager <= $this->period_date;
    }

    /**
     * 課長承認
     */
    public function isApplovedByManager()
    {
        return $this->apploved_by_manager &&
            $this->apploved_at_manager <= $this->period_date;
    }
}

Comments

[退会済みユーザー]

apploved_by_general_managerやapploved_by_managerがboolean前提で回答しているけど、「未処理」と「否決」の区別がつかないんだよなぁ…。

[annderber]

コメントありがとうございます。

例は簡素にして書いていますが、実際もっと複雑な条件があると思います。
むしろ複雑な方が各メソッドごとに小さく状態チェック処理を作ってそれを組み合わせて作るような設計がいいかとおもっています。あまりやり過ぎるスパゲッティ化しますが。

[退会済みユーザー]

あなたの質問の趣旨は、SQLかロジックかと書いてあり、メソッドの粒度については質問していません。
ですので、ちょっとコメントの意図が不明です。