是否可以改进此代码以避免重复？

Question

下面的代码是用php写的switch语句。 $historical_term = $this->MonthlyCurves->getHistorical($term, $start_date, $end_date); 行在每种情况下都会重复。这违反了 DRY（不要重复自己）原则。有什么方法可以改进代码以遵守 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.");
            }

Answer 1

你可以这样做：

 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.");
            }

Answer 2

明显改善：

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.");
}

是否可以改进此代码以避免重复？

Can this code be improved to avoid repetition?

php

cakephp

dry