Можно ли улучшить этот код, чтобы избежать повторения?

Question

Можно ли улучшить этот код, чтобы избежать повторения?

Код ниже является оператором switch, написанным на php. Линия $historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date); повторяется в каждом случае. Это нарушает принцип СУХОЙ (не повторяйся). Есть ли способ улучшить код, чтобы придерживаться DRY?

 switch ($term)
{
case "1":
$term = 'XXX_1_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
break;
case "2":
$term = 'XXX_2_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
break;
case "5":
$term = 'XXX_5_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
break;
default:
print ("Invalid  parameter.");
}

1

cakephp dry php

Решение

Другие решения

Вы можете сделать что-то вроде этого:

 switch ($term)
{
case "1":
case "2":
case "5":
$term = 'XXX_'.$term.'_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
break;
default:
print ("Invalid  parameter.");
}

2

Источник

Accepted Answer

очевидное улучшение:

switch ($term) {
case "1":
case "2":
case "5":
$term = 'XXX_'.$term'_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
break;
default:
print ("Invalid parameter.");
}

или же

if ( ($term == '1') || ($term == '2') || ($term == '5') ) {
$term = 'XXX_'.$term'_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
} else {
print ("Invalid parameter.");
}

или же

if (in_array($term, array('1', '2', '5'))) {
$term = 'XXX_'.$term'_year';
$historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date);
} else {
print ("Invalid parameter.");
}

4