In meiner aktuellen Anwendung muss ich eine API-GET-Route „/inactive-users“ erstellen, die archivierte Benutzer zurückgeben soll (“isArchived”），并且同时处于非活动状态超过一个月（"lastVisitedDate" 字段应早于 new Date() - 1 个月).

Was soll ich in diesem Fall tun, wenn ich eine mehrschichtige Architektur (Controller/Dienst/Repository) und ein schlechtes Domänenmodell verwende?

Ich sehe, dass es zwei mögliche Ansätze gibt.

1 – Erstellen Sie eine generische Repository-Methode, um den Benutzer abzurufen und ihm die von uns benötigten Benutzerfelder zu übergeben.

@Injectable()
export class UserRepository {
  constructor(private readonly prisma: PrismaService) { }

  findAll = async ( where: { user: Partial<User>; dateTreshold: Date } ): Promise<User[]> => {
    const users = await this.prisma.user.findMany(
      { where: {...user}, lastVisitedDate: { lt: dateTreshold }; // lt - less than
    );
        
    return users.map(user => new User(user));
  }
}

@Injectable()
export class UserService {
  constructor(private readonly userRepository: UserRepository) {}
  
  getInactiveUsers = async () => {
    return this.userRepository.findAll(
      { where: {user: {isArchived: true}, dateTreshold: "// calculatedDate //"}
    ) 
  }
}

2 – Erstellen Sie die Repository-Methode, um inaktive Benutzer genau abzurufen. Die Methode weiß, welche Felder sie anfordern soll.

@Injectable()
export class UserRepository {
  constructor(private readonly prisma: PrismaService) { }

  getInactiveUsers = async (): Promise<User[]> => {
    const users = await this.prisma.user.findMany(
      { where: {isArchived: false, lastVisitedDate: { lt: "// calculatedDate //" }}; // lt - less than
    );
        
    return users.map(user => new User(user));
  }
}

@Injectable()
export class UserService {
  constructor(private readonly userRepository: UserRepository) {}
  
  getInactiveUsers = async () => {
    return this.userRepository.getInactiveUsers() 
  }
}

Welcher Weg ist besser? Das erste sieht für mich gut aus, da das Repository in diesem Fall nichts über das Domänenverständnis des „inaktiven“ Benutzers weiß. Gleichzeitig kann es jedoch recht schwierig sein, einen solchen reaktiven Ansatz zu entwickeln.

Der zweite Weg ist einfacher zu erstellen, verfügt aber gleichzeitig über ein gewisses Verständnis der „Geschäftslogik“ und weiß, dass es sich dabei um inaktive Benutzer handelt "isArchived" 等于 false. Außerdem weiß diese Repository-Methode, wie viele Tage wir verwenden müssen.

Welche Option sollte in dieser Situation gewählt werden? Oder gibt es vielleicht andere Möglichkeiten, dieses Ding zu bauen?