重构简单的功能

Der*_*erp 0 php refactoring

我需要以人类可读的格式显示当前的记录状态.在我的数据库中,我有一个这个字段的int,所以记录可能有状态1,2,3,4等

我写了一些代码来向用户显示当前状态:

<?php 
// code to retrieve current status from DB
// ...


if ($status == '1') {
    echo "Current status: Active";
}
if ($status == '2') {
    echo "Current status: Pending";
}
    if ($status == '3') {
    echo "Current status: Inactive";
}

// etc..

?>
Run Code Online (Sandbox Code Playgroud)

这段代码看起来非常难看,但我无法弄清楚如何重构它以使其更有效并且没有无限的一系列条件.

Mor*_*kel 5

制作一个数组:

$statvalue=array(1=>'Active',2=>'Pending',3=>'Inactive');
echo("Current status: {$statvalue[$status]}");
Run Code Online (Sandbox Code Playgroud)

  • 您应该编辑您的答案以将整数作为索引:array(0 =>'Active',...) (3认同)