Как лучше в функцию передавать много параметров?

@Zombie13 · Регистрация: 16.10.2014

Author24 — интернет-сервис помощи студентам

Есть функции что-то типа:

C#

public void ChangeFilter(string region = "", string week = "", 
            bool InclEPay = false, string EPday = "", string car = "", string verifPointName = "",
            bool getReport = true, int delay = 0, bool include = false, bool matchWithPrlInv = true,
            bool showAllTSOP = false, string appByAgency = "All", string appFac = "All", string inclCr = null)
{
......
}

Как-то не красиво выглядит, как можно это переделать? может использовать dictionary или hashtable в качестве параметров функции или создавать отдельный тип данных и передавать его?

@REV3RS · 18.12.2014, 12:09

Можно структуру использовать

@maxillion · 18.12.2014, 12:11

Для красоты и наглядности можно просто подравнять

C#

public void ChangeFilter(
            string region          = "",
            string week            = "",
            bool   InclEPay        = false,
            string EPday           = "",
            string car             = "",
            string verifPointName  = "",
            bool   getReport       = true, 
            int    delay           = 0,
            bool   include         = false,
            bool   matchWithPrlInv = true,
            bool   showAllTSOP     = false,
            string appByAgency     = "All",
            string appFac          = "All",
            string inclCr          = null)
{
 
}

Или передавать параметры при помощи класса

C#

public void foo()
{
    ChangeFilter(new MyClass() { region = "123", InclEPay = true, matchWithPrlInv = false});
}
public class MyClass
{
    public string region          = "";
    public string week            = "";
    public bool   InclEPay        = false;
    public string EPday           = "";
    public string car             = "";
    public string verifPointName  = "";
    public bool   getReport       = true;
    public int    delay           = 0;
    public bool   include         = false;
    public bool   matchWithPrlInv = true;
    public bool   showAllTSOP     = false;
    public string appByAgency     = "All";
    public string appFac          = "All";
    public string inclCr          = null;       
}
public void ChangeFilter(MyClass mc)
{
}

@tezaurismosis · 18.12.2014, 12:29

Имхо, структура - лучший вариант. По себе знаю, когда пишешь больше 5 параметров, начинаешь путаться.

@maxillion · 18.12.2014, 12:37

Сообщение от tezaurismosis

По себе знаю, когда пишешь больше 5 параметров, начинаешь путаться

Чтобы не путаться можно вызвать так

C#

1	ChangeFilter(region: "123", EPday: "456", include: true);

@tezaurismosis · 18.12.2014, 12:53

maxillion, можно, но многословно. Благо немногие методы в .NET вызываются более чем с пятью аргументами.

@REV3RS 0 / 0 / 1 Регистрация: 06.02.2014 Сообщений: 6
	18.12.2014, 12:09	2
	Можно структуру использовать 0

@tezaurismosis Администратор 9602 / 4744 / 761 Регистрация: 17.04.2012 Сообщений: 9,592 Записей в блоге: 14
	18.12.2014, 12:29	4
	Имхо, структура - лучший вариант. По себе знаю, когда пишешь больше 5 параметров, начинаешь путаться. 0

@tezaurismosis Администратор 9602 / 4744 / 761 Регистрация: 17.04.2012 Сообщений: 9,592 Записей в блоге: 14
	18.12.2014, 12:53	6
	maxillion, можно, но многословно. Благо немногие методы в .NET вызываются более чем с пятью аргументами. 0