оптимизация — PHP Switch Условный оператор лучше
Каков наилучший способ оптимизировать этот код? Я знаю, что операторы if и else будут быстрее, но я хочу что-то, что будет чистым и лаконичным. Есть идеи?
switch ($data['months']) {
case ($data['months'] >= 400):
$data['months'] = 400;
break;
case ($data['months'] >= 360):
$data['months'] = 360;
break;
case ($data['months'] >= 60):
$data['months'] = 60;
break;
case ($data['months'] >= 48):
$data['months'] = 48;
break;
case ($data['months'] >= 36):
$data['months'] = 36;
break;
case ($data['months'] >= 24):
$data['months'] = 24;
break;
case ($data['months'] >= 12):
$data['months'] = 12;
break;
case ($data['months'] >= 9):
$data['months'] = 9;
break;
case ($data['months'] >= 6):
$data['months'] = 6;
break;
case ($data['months'] >= 3):
$data['months'] = 3;
break;
case ($data['months'] >= 1):
$data['months'] = 1;
break;
default:
$data['months'] = 12;
}
Решение
Я думаю, вы могли бы использовать для каждого с массивом точек останова.
$breakpoints = array(400, 360, 60, 48, 36, 24, 12, 9, 6, 3, 1);
// Loop through each breakpoints
foreach($breakpoints as $breakpoint){
// Proceed to next breakpoint
if($data['months'] < $breakpoint)
continue;
// No need to loop through the rest of the array
$data['months'] = $breakpoint;
break;
}
// Handle the cases where $data['month'] is less than 1
if($data['months'] < 1)
$data['months'] = 12;
Обновить
Альтернативой будет проверка, если $data['months'] меньше 1 до цикла:
if($data['months'] < 1)
$data['months'] = 12;
else{
$breakpoints = array(400, 360, 60, 48, 36, 24, 12, 9, 6, 3, 1);
// Loop through each breakpoints
foreach($breakpoints as $breakpoint){
// Proceed to next breakpoint
if($data['months'] < $breakpoint)
continue;
// No need to loop through the rest of the array
$data['months'] = $breakpoint;
break;
}
}
Другие решения
Нечто подобное должно сработать.
$breakpoints = [1, 3, 6, 9, 12, 24, 36, 48, 60, 360, 400];
foreach($breakpoints as $breakpoint) {
if($data['months'] >= $breakpoint) {
$value = $breakpoint;
}
}
if(!isset($value)) { $value = 12; }
Вот что я закончил в конце.
$breakpoints = [400, 360, 240, 120, 60, 48, 36, 24, 12, 9, 6, 3, 1, null];
foreach($breakpoints as $breakpoint){
if($data['months'] >= $breakpoint) {
$data['months'] = $breakpoint ? $breakpoint : 12;
break;
}
}