Упростить код с одинаковым действием для нескольких условий

Есть один супер декоратор у меня

PHP:

1. public function getDefense($country_code)
2. {
3.     // Германия - бонус наземной техники
4.     if ($country_code == 'de' and $this->type == 'ground') {
5.         $this->defense *= 1.2;
6.     }
7.  
8.     // Россия - бонус морской техники
9.     if ($country_code == 'ru' and $this->type == 'marine') {
10.         $this->defense *= 1.2;
11.     }
12.  
13.     // США - бонус воздушной техники
14.     if ($country_code == 'us' and $this->type == 'air') {
15.         $this->defense *= 1.2;
16.     }
17.  
18.     // Если нет изменений, то Китай, Казахстан, Украина, Беларусь: значение по дефолту
19.     return $this->defense;
20. }

Что не нравится: мозолят глаза однотипные вроде как условия с одинаковым действием.

Есть идея в скобках через OR их все запилить в одно условие, но может у кого-то опыта побольше и есть какие-нибудь покруче наработки, поделитесь.

 

@Drunkenmunky только теперь в массиве повторы - 1.2

PHP:

1. public function getDefense($country_code)
2. {
3.     $array = ['de' => ['ground' => 1.2], 'ru' => ['marine' => 1.2], 'us' => ['air' => 1.2]];
4.  
5.     if (isset($array[$country_code][$this->type])) {
6.         $this->defense *= $array[$country_code][$this->type];
7.     }
8.  
9.     return $this->defense;
10. }

хотя нормально вроде, вдруг там бонус меняться будет или разный для каждой страны, оставлю пожалуй так

 

это где так у меня?

у меня 7 стран и три типа техники, 20% бонус только трём странам на определённую техну, если условие(я): {страна-техника} не совпали, то дефолт вернёт без бонуса ЖЕ

я УЖЕ подумал что на случай изменения размера(ов) бонуса то подойдёт и с таким массивом

 

ты обрезал код

там в конце метода

PHP:

1. // Если нет изменений, то Китай, Казахстан, Украина, Беларусь: значение по дефолту
2. return $this->defense;

--- Добавлено ---
@Drunkenmunky понял?

дефолт 100

если например Россия, то 120, если Беларусь, то 100 так и вернётся

условие изменяет возвращаемое значение свойства при совпадении

 

ага, мне эта мысль как раз тоже мелькнула, когда я массив под твой пример рисовал, тоже надо попробовать, спс...
--- Добавлено ---
@Drunkenmunky твой пример хорош тем, что можно отдельно под каждую страну бонус назначить

 

а я наоборот глухо забил по 1.2, а теперь думаю ё-маё, так можно же будет менять

чёт тупанул, ок, вроде порешали

 

@MouseZver благодарю, буду пробовать

подтолкнули в этой теме меня смотреть в сторону массивов, нашёл я такую вот функцию: возыращает ключ совпадающего значения, я не фанат работы с массивами, правильно ли я применил функцию?

PHP:

1. public function getDefense($country_code)
2. {
3.     if ($country_code == array_search($this->type, ['de' => 'ground', 'ru' => 'marine', 'us' => 'air'])) {
4.         $this->defense *= 1.2;
5.     }
6.  
7.     return $this->defense;
8. }

--- Добавлено ---
@MouseZver мне варик с in_array приколол больше всего, я таких фокусов ещё не видел

 

@MouseZver я на 7.4 сижу, забрал варик с in_array, всё гениальное - просто, блин это же надо: тупо конкатенировать и сравнить, я чёт не допёр, нормально так