VPF::Убрать дубляж кода - Форум программистов

ДобренькийПапаша

Дата 6.7.2011, 14:48 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1278
Регистрация: 14.1.2006
Где: г.Москва

Репутация: нет
Всего: 7

Ситуация такая. Есть два цикла, тело которых почти идентично, исключая то, что оба они опираются на два разных ObservableCollection (заход в какой-либо из них происходит по условию):

Код


if(.......)
{
for (int i = 0, j=0; i < MoneyCoinsNominals.Count; i++)
                { 
                }
}
else
{
for (int i = 0; i < MoneyNotesNominals.Count; i++)
               {
               }
}

В теле цикла происходит обращение к полям класса, который содержится в коллекции. Есть способ как-то перенести всё в один метод?
Ну, передам как Type, но толку-то? В общем не понимаю как убрать ветвление.

Код


private Grid FormBlocks(PaymentDevices device, Type collection, string typeOfMoney)
        {
            
        }

--------------------

Меня зовут Себастьян Парейра, торговец чёрным деревом.

jonie

Дата 6.7.2011, 15:07 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 22
Всего: 118

Код


ObservableCollection some = FistCollection;
if(...) some = SecondCollection;
//use some here...

не?

--------------------

Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...

ДобренькийПапаша

Дата 6.7.2011, 15:42 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1278
Регистрация: 14.1.2006
Где: г.Москва

Репутация: нет
Всего: 7

Цитата(jonie @ 6.7.2011, 15:07

)

не?

Не думаю

Код


switch(paymentDevice.Type)
            {
                case DbInfrastructure.DeviceType.CoinDispenser:
                    {
                        typeOfMoney = "монет";
                        return FormBlocks(paymentDevice, MoneyCoinsNominals.GetType(), typeOfMoney);
                    }
                case DbInfrastructure.DeviceType.CashDispenser:
                    {
                        typeOfMoney = "купюр";
                        return FormBlocks(paymentDevice, MoneyNotesNominals.GetType(), typeOfMoney);
                    }
            }

Код


private Grid FormBlocks(PaymentDevices device, Type collection, string typeOfMoney)
        {
            
        }

Если буду передавать коллекцию, а в методе аргумент будет типа ObservableCollection, то приведение не будет выполнено - нужно специфицировать тип обобщения, т.е. <T>. А если специфицировать, то либо два метода разных и снова дубляж кода, либо в одном методе проверка на тип и снова дубляж кода из-за ветвления.

Это сообщение отредактировал(а) ДобренькийПапаша - 6.7.2011, 15:44

--------------------

Меня зовут Себастьян Парейра, торговец чёрным деревом.

neutrino

Дата 15.7.2011, 19:17 (ссылка)

(нет голосов)

Загрузка ...

Gothic soul

Профиль
Группа: Модератор
Сообщений: 3041
Регистрация: 25.3.2002
Где: Верхняя Галилея, Кармиэль

Репутация: 3
Всего: 62

запость весь код...

--------------------

The truth comes from within ...

Покойся с миром, Vit

Fitc

Дата 15.7.2011, 22:43 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 129
Регистрация: 30.1.2010

Репутация: 3
Всего: 4

Можно использовать что то вроде этого:

Код


Action<MyClass> invoke=delegate(MyClass param)
{
//сюда можно впихнуть внешние данные (переменные)
};
Array.Foreach(myCollectionOfMyClassItems1,invoke);
Array.Foreach(myCollectionOfMyClassItems2,invoke);

Это сообщение отредактировал(а) Fitc - 15.7.2011, 22:44

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Общие вопросы по .NET и C# \| Следующая тема »