multi-stan php

Question

napisałem prosty skrypt z wielu warunków:

$item =12; 
    if($item < 5) 
    { 
    //display icon buyer 
    } 
    elseif ($item < 10) 
    { 
    //display icon buyer 
    } 
    elseif ($item < 15) 
    { 
    //display icon good buyer 
    } 
    elseif ($item < 20) 
    { 
    //display icon top buyer 
    } 
    // etc.... 

To długa wielo stan i wiem, jest bardzo zła.

Jak mogę zoptymalizować kod?

Uwaga.switch nie jest możliwe, ponieważ używam operatorów < i > itd

Answer 1

lepiej niż bukietem warunkowych jest patrzeć na wzór, który pozwala po prostu obliczyć.

W twoim przypadku wygląda to tak, że kroki są 5 - tak odpowiednie dla podziału.

---

lub zmapować go:

$item = 12; 

$icons = [ 
    10 => 'buyer', 
    15 => 'good buyer', 
    20 => 'top buyer', 
]; 

foreach ($icons as $value => $icon) { 
    if ($item < $value) { 
     return $icon; 
    } 
} 
return null; 

co umożliwia bardziej elastyczne kroki (jak masz to w twoich danych w pytaniu).

Answer 2

dać to odchodzenie (demo):

<?php 
function icon($item) { 
    $icons = array(
     'buyer', 
     'buyer', 
     'good buyer', 
     'top buyer' 
    ); 
    $index = max(0, min(3, $item/5)); 
    return $icons[$index]; 
} 

$item = 12; 
echo icon($item) . "\n"; 

$item = 1; 
echo icon($item) . "\n"; 

$item = 100; 
echo icon($item) . "\n"; 

---

Wystarczy dodać do tablicy $icons więcej opcji/ikony.