リポジトリパターンのクラス設計について

Question

###前提・実現したいこと
MySQLであるWEBサービスを作っています。
今までは処理も永続化も一つのクラスに書いていたのですが、リポジトリパターンで分けるといいみたいなので分割しました。
しかし、クラス設計で悩んでいます。
以下のような場合、どのようにクラス設計を行うのが正しいのでしょうか？

ここでは、重複禁止のメールアドレスカラム(mail)があるとします。

###分割前のソースコード

/**
 * メールアドレスを変更します
 * @param mysqli $sql メールアドレス
 * @param string $mail メールアドレス
 *
 * @throws Exception 既に登録されている
 */
public function setMail($sql,$mail){
	//引数チェック省略

	//DBのhogeテーブルのmailカラムの更新
	$stmt = $sql->prepare("UPDATE hoge SET mail=?");
	$stmt->bind_param("s", $mail);
	$stmt->execute();
	
	//もし1062エラー(重複データの格納)なら例外を投げる
	if ($sql->errno === 1062) {
		throw new Exception("このメールアドレスは登録済みです。");
	}
	
	//重複でなければセット
	$this->mail=$mail;
}

###困っていること
このクラスをリポジトリパターンで分割する場合、重複チェック(1062エラーチェック)は、どう行うのが正しいのか？
今考えているのは、setMall()関数では重複チェックを行わずに、引数チェックと、フィールドへの代入のみを行い、Repositoryクラスの永続化関数で重複チェックを行う事です。

Comments

[Chironian]

回答依頼を頂きましたが、PHP, SQL, リボジトリパターンどれも知らないので回答できません。ごめんなさい。

[kgtkr]

こちらこそいきなり依頼してすみません

Answer 1

あくまでも私の考えになりますが、重複チェックはサービス側でやるべきです。(実際に取ってくるのはデータアクセス層ですが)
また、重複チェックとUPDATEを同時にすると２つの処理をやることになってしまうので、基本的には分けるべきでしょう。(もしやる場合はDataAccess層からExceptionを投げる必要があるでしょう)

HogeService

class HogeService
{
    protected $hogeRepository;

    public function __construct(HogeRepositoryInterface $hogeRepository)
    {
        $this->hogeRepository = $hogeRepository;
    }

    public function existsEmail($email)
    {
        $this->hogeRepository->exitsEmail($email);
    }

    public function updateEmail($email)
    省略
}

HogeRepository

class HogeRepository implements HogeRepositoryInterface
{
    protected $hoge;

    public function __construct(Hoge $hoge)
    {
        $this->hoge = $hoge;
    }

    public function existsEmail($email)
    {
        // もしbool値を返さないようなメソッドを使う場合はここで処理して返す
        $this->hoge->existsEmail($email);
    }

    // 省略
}

HogeRepositoryInterface
省略します。

Hoge(データアクセス)

class Hoge
{
    // ここではあえてEmailに限定させましたが、カラムも別途していできるようにしても良いと思います
    public function existsEmail($email)
    {
        $stmt = $sql->prepare("SELECT count(*) FROM hoge WHERE mail = :mail");
        $stmt->bind_param("mail", $mail);
        $stmt->execute();

        return !!$stmt->fetchColumn();
    }

    // 省略
}

メインコード

// クラスで管理している場合はメンバ変数に入れておきましょう。
// より良いのはDIコンテナなどを利用してRepositoryはDIするのが良いでしょう。
$hogeService = new HogeService(new HogeRepository); 

if ($hogeService->exitsEmail($email)) {
    // メールアドレス存在してるのでエラーを返す
}

// ここはtry-catchで囲っておいても良いかもしれません
$hogeService->updateEmail($email);

って今気づきましたが１年越しの回答でした