避免在儲存庫層中包含領域邏輯
在我目前的應用程式中，我需要建立一個api GET 路由“/inactive-users”，它應該會傳回已存檔的使用者（“isArchived”），並且同時處於非活動狀態超過一個月（"lastVisitedDate" 欄位應早於new Date() - 1 個月）。
採用分層架構（控制器/服務/儲存庫），領域模型貧乏，這種情況下我該怎麼辦？
我看到有兩種可能的方法。
1 - 建立通用儲存庫方法來取得使用者並將我們需要的使用者欄位傳遞給它。
@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 - 準確地建立儲存庫方法來檢索不活動的用戶，該方法將知道它應該要求哪些欄位。
@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()
}
}
哪種方式比較好？ 第一個對我來說看起來不錯，因為在這種情況下，儲存庫對「非活動」用戶的網域理解一無所知。但同時 - 建立這樣一個響應式方法可能相當困難。
第二種方式更容易構建，但同時 - 它具有一些“業務”邏輯理解，並且知道不活動用戶是那些 "isArchived" 等於 false 的用戶。而且，這個儲存庫方法知道我們需要使用的天數。
在這種情況下應該選擇哪個選項？或者也許還有其他方法來建造這個東西？
