Question about the repository and specification design about eshoponweb

[wyshmily]

And I don't think such specifications are needed by the applicationcore.

Want some guide, thanks!

[ardalis]

Specification is meant to encapsulate complex queries. If you just want to know if a record exists, you can do a GetById() and see that it returns null - no Spec needed.

[wyshmily]

but in my case, the query is not by id, but other field like studentnumber

[ardalis]

In that case if you're already using specs in your design, it would be easy to create a spec for StudentByStudentNumber(int studentNumber) that would include the required lambda expression to fetch student(s) with that number.

Alternately unless you expect it to be a regular occurrence for student creation to result in potential duplicates, you could just attempt the create and catch the exception when the record already exists. If it truly is an exceptional condition, this would be an appropriate design as well.

[wyshmily]

In fact it would not cause an exception, because I use soft delete, with a IsDeleted field in each table.
Such checks can only be performed manually since there is no unique index.
So the answer is creating specs to do the job?

[ardalis]

If you're already set up to use Specifications (that is, your Repository can accept an ISpecification<T> or similar) then yes, a spec would do the job easily. If you're not using the Specification pattern or don't want to use it for this case, the simplest approach would be to create a method on your interface (example: IStudentRepository has method GetStudentByStudentNumber(int number) ). The reason I don't prefer this approach is it results in bigger repositories and more repository interfaces and classes. I'd rather have the specification classes, which better follow SRP, than many inconsistent IWhateverRepository implementations, given the choice. Hope that helps.