Question

I have several methods in my DAL, with quite a few parameters:

public Collection<Proforma> SearchAllProforma(DateTime? startDate = null, DateTime? endDate = null,
        int? institutionID = null, int? inspectionID = null, bool? isFollowup = null, bool? excludeDeleted = null,
        bool? nutritionalOnly = null, int? parentInspectionID = null)

Should I condense these down to take an object parameter? Or leave them the way they are, using optional parameters? Or both?

Edit – I should really have said, that each one of these parameters maps to a parameter for a stored procedure.

Answer 1

I would suggest you to Create a class for all those parameter as properties.

And then send the class as parameter.

Class SerachAllProformaParameter
{
//All Properties.
}

SerachAllProformaParameter parameter= new SerachAllProformaParameter();
parameter.PropertyName="value";

public Collection<RoIVProforma> SearchAllProforma(parameter);