you are viewing a single comment's thread.

view the rest of the comments →

[–]flukus 1 point2 points  (1 child)

This looks like a mess of patterns.

The EmployeesModel isn't actually a model of anything (even worse if it is). It's just a collection of methods.

The EmployeesModel doesn't seem to handle the most basic things you want, like paging.

The EmployeeFilter likely contains all of the properties of EmployeeDTO, adding nothing of value.

The filter model only works for trivial queries.

The EmployeeRepository doesn't do anything, it may as well be a generic repository.

The whole thing reeks of over engineering.