Alfatyhin / loftshool

0 stars 0 forks source link

Улучшение логики #12

Closed nejtr0n closed 3 years ago

nejtr0n commented 3 years ago

Верно подметил в комете, что должна быть возможность удалить доп услуги. При текущей реализации это сделать нельзя.

https://github.com/Alfatyhin/loftshool/blob/53d9df5f301d6faa89aa7eb2d274f66507cf0e32/tasks/week_2/src/TarifAbstract.php#L39-L47

Но если ты создашь свойство классе TarifAbstract - массив сервисов. И будешь добавлять в него сервисы в методе addServices, тогда у тебя будет список сервисов добавленных к тарифу. А применять логику доп сервисов будешь в методе count().

Alfatyhin commented 3 years ago

Верно подметил в комете, что должна быть возможность удалить доп услуги. При текущей реализации это сделать нельзя.

https://github.com/Alfatyhin/loftshool/blob/53d9df5f301d6faa89aa7eb2d274f66507cf0e32/tasks/week_2/src/TarifAbstract.php#L39-L47

Но если ты создашь свойство классе TarifAbstract - массив сервисов. И будешь добавлять в него сервисы в методе addServices, тогда у тебя будет список сервисов добавленных к тарифу. А применять логику доп сервисов будешь в методе count().

сделал правда метод count считает тариф а доп услуги обрабатывает getCount так как один из тарифов переопределяет метод count и чтоб не делать в нем обработку доп услуг сделал так