DDD: il servizio contiene due repository

Non è raro che un servizio abbia due repository, ma spesso è un suggerimento di un design scadente. Vale la pena guardare, per vedere se è possibile migliorare il tuo design ma, se guardi e non riesci a migliorarlo, non dovresti preoccuparti troppo.

Nel tuo caso, penso che tu abbia bisogno di guardare il concetto di Aggregates in DDD.

Aggregates are groups of things that belong together. An Aggregate Root is the thing that holds them all together.

Se Passenger e Passport non appartengono insieme, non so cosa faccia. La tua radice aggregata in questo caso dovrebbe essere ovviamente Passenger.

Sono totalmente d'accordo con @pdr che non devi incolpare te stesso a causa di un repository aggiuntivo nel tuo servizio.

A proposito dei metodi del passaporto, come addPassport , penso che dovrebbero rimanere in PassengerService e nel repository di passaporto. Qualcosa come:

public class PassengerService : ServiceBase<Passenger>, IPassengerService {

    private readonly IPassportRepository _passport_repository;
    private readonly IPassengerRepository _pass_repository

    public PassengerService(IPassportRepository passport_repository,     IPassengerRepository passenger_repository) {
        ...

    }


   public void removePassport(...) {
       _passport_repository.remove(...);

   }


}