Исходный код
function ShowUserRowStatus($key, $value){
// Check value
settype($value, 'integer');
switch($value){
case'1':{
$content='~s_enable~';
break;
}
default:{
$content='~s_disable~';
}
}
//exit
return $content;
}
Что не так в этом коде
- Функция называется
ShowUserRowStatus
, хотя она ничего не показывает, а лишь возвращает специфическую строку. - В алгоритме всего две альтернативы, однако применён оператор
switch
, что явно излишне. - Присутствует переменная
$content
, без которой совершенно спокойно можно обойтись. - Параметр
$key
не используется. - Название параметра
$value
слишком общее, не отражает его смысла.
Как отрефакторить исходный код
function GetUserRowStatus($status) {
settype($status, 'integer');
return $status == '1' ? '~s_enable~' : '~s_disable~';
}
Вполне вероятно, что без приведения статуса к целому числу тоже можно обойтись.