如何以改进的方式完成此代码 [关闭]

Question

关闭。 这个问题是基于意见的。 它目前不接受答案。

---

想改善这个问题吗？ 更新问题，以便通过编辑这篇文章用事实和引文来回答问题。

6年前关闭。

改进这个问题

我想用正确的代码替换这里的 switch 语句。 我需要以改进的方式执行此操作的建议。

switch(true) {
    case ($value === 'test1'):
    $testArray['class'] = 'incomplete';
    return $this->icon(
        'test_0.png',
        $testArray
    );
    case ($value === 'test2'):
    $testArray['class'] = 'progress';
    return $this->icon(
        'test_1.png',
        $testArray
    );
    case ($value === 'test3'):
    $testArray['class'] = 'complete';
    return $this->icon(
        'test_3.png',
        $testArray
    );
}

Answer 1

尝试使用'incomplete' 、 'progress'或'complete'设置$value 。 将 imgs 重命名为incomplete.png .png、 progress.png和complete.png

然后这样做：

$testArray['class'] = $value; 
return $this->icon(
    $value.'.png',
    $testArray
);

这一切都取决于您设置$value的方式。 但是，如果您设法以正确的方式进行设置，它会更短且更易于使用。

Answer 2

改进的 switch 语句是，

    switch($value) {
    case 'test1':
        $testArray['class'] = 'incomplete';
        $image = 'test_0.png';
        break;
    case 'test2':
        $testArray['class'] = 'progress';
        $image = 'test_1.png';
        break;
    case 'test3':
        $testArray['class'] = 'complete';
        $image = 'test_3.png';
        break;
    default :
        break;
}
return $this->icon($image,$testArray);

或使用 if 语句，

if ($value == 'test1') {
    $testArray['class'] = 'incomplete';
    $image = 'test_0.png';
} elseif ($value == 'test2') {
    $testArray['class'] = 'progress';
    $image = 'test_1.png';
} else {
    $testArray['class'] = 'complete';
    $image = 'test_3.png';
}

return $this->icon($image,$testArray);