Шаблон фасада и массив объектов - код выглядит нестандартно? - PullRequest
Новая
       12

Шаблон фасада и массив объектов - код выглядит нестандартно?

0 голосов
/ 09 апреля 2020

Здесь начинающий программист, с ограниченными знаниями в области кодирования. У меня есть код, который (что я думаю, основываясь на том, что я читал) соответствует шаблону Facade, и я пытаюсь заставить его работать для массива.

Не то, чтобы с ним была проблема - - код работает правильно и делает то, что я собираюсь сделать. Меня беспокоит только то, что я думаю, что код выглядит странно, но я не могу объяснить, почему? Я хотел бы, чтобы кто-то, пожалуйста, указал, почему это нормально или неправильно, и что я могу сделать, чтобы улучшить это?

Обратите внимание, что в классе Order есть много других функций, которые вызываются одинаково, поэтому может потребоваться общая очистка кода - я просто не знаю, как начать. Сначала я думал перенести затронутые вызовы (например, CancelOrder) в текущий класс, так что мне не нужно было бы заниматься странной акробатикой кода, я думаю, что это также сделало бы недействительным образец? 

    public void DoSomething(Order[] orders)
    {
        //Original code had only the following to call CancelOrder(), which cancels ALL orders
        //order.CancelOrder();

        //However, since I am now receiving an array, I think my revision looks weird?
        orders[0].CancelOrder();

        foreach(Order order in orders)
        {
            ...
        }
    }
...