DDDのリポジトリについての質問

Question

前提・実現したいこと

DDDで、リポジトリクラスに定義するメソッドについて質問があります。
例えば User集約というのがあるとします。
以下のように User集約は一意に識別するNoを持っているとします。

新しくUserを登録する場合は、既存のUserのNoに1加算した数値を、これから登録するUserのNoとしたいとします。
そのときに、「既存のUserのNo」を取得するために、以下のようなサービスとリポジトリの実装を検討しました。

該当のソースコード

class UserRegisterService
{
    private IUserRepository $userRepository;

    public function register(string $name): void
    {
        $latestUser = $this->userRepository->findLatestUser();
        $this->userRepository->save(
            new User(
                new No($latestUser->No()->value() + 1),
                $name
            )
        );
    }
}

// IUserRepository のInterface定義は省略してます。
class UserRepository implements IUserRepository
{
    public function findLatestUser(): User
    {
        // 以下のクエリに相当するものをORMか何かで取得して、User集約に変換して返す。
        $query = 'SELECT * FROM Users ORDER BY No Desc Limit 1';
        return;
    }
    public function save(): void
    {
        // 保存処理
    }
}

疑問

リポジトリにこういったメソッドって追加し続けてよいのですか？？
上の例では最新のUserを取得している例だけですが、
最古のUserのNoが欲しいとか、ちょうど中間のUserのNoが欲しいとかも
リポジトリのメソッドに追加して行ったら肥大化しまくるんですが、よいのか気になりました。

リポジトリの立ち位置がよくわからないので教えてください。

Answer 1

そもそも

$latestUser = $this->userRepository->findLatestUser();

が複数のサーバで同時に実行されたら同じ番号のUserが保存されます。IDが重複するとINSERTに失敗します。サーバが一台なら問題ないですが、複数サーバで運用することが普通なので、通常はこういう採番はしません。

普通はAUTO_INCREMENTを使います。が、他にもいろいろな方法があります。

私が書いた、以下の記事を参考にしてみてください。

https://zenn.dev/j5ik2o/articles/a085ab3e3d0f197f6559

Answer 2

必要なメソッドを追加すること自体は個別の用途に合わせていけば良いと思うのですが、
そもそもこのロジックには穴があります。

基本的にDBへのアクセスは並列に行われますので、２つの処理「処理1」と「処理2」が同時に同じことを行う場合に次のようなケースが起こりえます。

• 処理1 : 最新のNoを取得 (No = 3)
• 処理2 : 最新のNoを取得 (No = 3)
• 処理1 : ユーザーAを登録 (No = 4)
• 処理2 : ユーザーBを登録 (No = 4) => ユーザーBが INSERT で書き込めない

解決方法は色々あります。調べてみてください。