пытаюсь сделать нечто такое

       $items = Cart::getContent();
           foreach ($items as $item) {
               $condition = new \Darryldecode\Cart\CartCondition([
                   'name' => 'COUPON',
                   'type' => 'coupon',
                   'attributes' => [
                       'promocode' => $this->code
                   ],
                   'value' =>  $item->attributes[0] != 0  ? 0 : -1 * $this->procent . '%' ,
               ]);
               Cart::addItemCondition($item->id, $condition);
           }

Все работает, но по факту не создается Cart::getConditions()

поправьте меня пожалуйста, если видите какие то неточности https://maxyc.ru/programming/replace-conditional-with-polymorphism/

получается был иф элс, а стало свитч кейс

не совсем. я вынес  на разные классы реализацию. а выбор нужного класса вынес в отдельный фабричный метод

вот такой прикол красивее и короче будет через мапинг по карте в конфиге

А если там карты в картах и карты в картах...

поправьте меня пожалуйста, если видите какие то неточности https://maxyc.ru/programming/replace-conditional-with-polymorphism/

спробую накидати варіанти
1 - якщо вже if поміняв на switch то мені більше подобається варіант з ключ значення  (скрін з прикладом який був під руками) https://github.com/PHPOffice/PhpSpreadsheet/blob/master/src/PhpSpreadsheet/IOFactory.php

Это и есть маппинг, про который речь шла выше)

о, а я і не побачив, уже зовсім оченятка злипаються
в такому випадку я підтримую твій варіант)