Question

我根据API的输入显示星级评分（注意：只显示，根本没有收到评级）。下面的代码完全按照我的需要运行，但它是一个非常大的逻辑，用于感觉它可以大规模简化。

我在这里读了一些其他的票，表明使用数组可能比交换机案例更有效。但是因为逻辑取决于每个情况在一个数字范围内是真实的（例如超过2.5但小于3）我不确定数组在这种情况下是否会起作用。

底线是：这个代码可以以某种方式大规模简化吗？

$stars = 3.5;

switch ($stars) {
  case ($stars > 1 && $stars <= 1.5):
    $star2 = 'dashicons-star-half';
    $star3 = 'dashicons-star-empty';
    $star4 = 'dashicons-star-empty';
    $star5 = 'dashicons-star-empty';
    break;
  case($stars > 1.5 && $stars <= 2):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-empty';
    $star4 = 'dashicons-star-empty';
    $star5 = 'dashicons-star-empty';
    break;
  case($stars > 2 && $stars <= 2.5):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-half';
    $star4 = 'dashicons-star-empty';
    $star5 = 'dashicons-star-empty';
    break;
  case($stars > 2.5 && $stars <= 3):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-filled';
    $star4 = 'dashicons-star-empty';
    $star5 = 'dashicons-star-empty';
    break;
  case($stars > 3 && $stars <= 3.5):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-filled';
    $star4 = 'dashicons-star-half';
    $star5 = 'dashicons-star-empty';
    break;
  case($stars > 3.5 && $stars <= 4):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-filled';
    $star4 = 'dashicons-star-filled';
    $star5 = 'dashicons-star-empty';
    break;
  case($stars > 4 && $stars <= 4.5):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-filled';
    $star4 = 'dashicons-star-filled';
    $star5 = 'dashicons-star-half';
    break;
  case($stars > 4.5):
    $star2 = 'dashicons-star-filled';
    $star3 = 'dashicons-star-filled';
    $star4 = 'dashicons-star-filled';
    $star5 = 'dashicons-star-filled';
    break;
  default:
  $star2 = 'dashicons-star-empty';
  $star3 = 'dashicons-star-empty';
  $star4 = 'dashicons-star-empty';
  $star5 = 'dashicons-star-empty';
}

?>
 <div class="wporg-ratings" title="<?php echo $stars; ?> out of 5 stars" style="color:#e6b800;">
   <span class="dashicons dashicons-star-filled"></span>
   <span class="dashicons <?php echo $star2; ?>"></span>
   <span class="dashicons <?php echo $star3; ?>"></span>
   <span class="dashicons <?php echo $star4; ?>"></span>
   <span class="dashicons <?php echo $star5; ?>"></span>
 </div>

Answer 1

我认为不需要阵列或开关。你可以使用for循环，并根据你的循环变量检查$stars值，看看应该使用哪个图标。

<div class="wporg-ratings" title="<?php echo $stars; ?> out of 5 stars" style="color:#e6b800;">
<?php
// for loop: one iteration for each of five possible stars
// (for loops are generally best for repeating code a specific number of times)
for ($i=0; $i < 5; $i++) {
    if ($stars <= $i ) {
        // empty stars are displayed when the iterator (i) is >= the star value
        echo  '<span class="dashicons dashicons-star-empty"></span>';
    } elseif ($stars <= $i + 0.5) {
        // half stars are displayed for star values between i and i+0.5 
        echo  '<span class="dashicons dashicons-star-half"></span>';
    } else {
        // whole stars are displayed when the star value is > i+0.5
        echo  '<span class="dashicons dashicons-star-filled"></span>';
    }
}
?>
</div>

为了帮助理解其工作原理，请通过循环获取理论值。我们可以使用您问题3.5中的那个。

第一次迭代：3.5＆gt; 0,3.5＆gt; 0.5，所以你得到了其他值（填充星）
第二次迭代：3.5＆gt; 1,3.5＆gt; 1.5，所以你得到了其他值（填充星）
第三次迭代：3.5＆gt; 2,3.5> 2.5，所以你得到了其他值（填充星）
第四次迭代：3.5＆gt; 3,3.5 = 3.5，所以你得到了elseif值（半星）
第五次迭代：3.5＆lt; 4，所以你得到if值（空星）

Answer 2

这是快速版：

$stars = 3.5;
$d = array(
    ($stars >= 1.0) ? 'dashicons-star-filled' : (($stars >= 0.5) ? 'dashicons-star-half' : 'dashicons-star-empty'),
    ($stars >= 2.0) ? 'dashicons-star-filled' : (($stars >= 1.5) ? 'dashicons-star-half' : 'dashicons-star-empty'),
    ($stars >= 3.0) ? 'dashicons-star-filled' : (($stars >= 2.5) ? 'dashicons-star-half' : 'dashicons-star-empty'),
    ($stars >= 4.0) ? 'dashicons-star-filled' : (($stars >= 3.5) ? 'dashicons-star-half' : 'dashicons-star-empty'),
    ($stars >= 5.0) ? 'dashicons-star-filled' : (($stars >= 4.5) ? 'dashicons-star-half' : 'dashicons-star-empty')
);
echo '<div class="wporg-ratings" title="' . $stars . ' out of 5 stars" style="color:#e6b800;">';
foreach ($d as $value) echo '<span class="dashicons ' . $value . '"></span>';
echo '</div>';

它与原始逻辑不同（当第一颗星总是被填充时），但你可以将它用作方向

下一步可能是将重复的代码移动到函数

Answer 3

另一种可能性是定义一个哈希数组。让我们说数组中的每个元素都包含＆＃34;最小值和三个字符串，＆＃34;并且数组按升序排序。您的if / then逻辑可以被该数组中的循环替换，并尽快突破。

但是： ＆＃34; Clarity 胜过所有其他问题。＆＃34;如果代码可以运行并且可以合理维护，谁会关心它是否会发臭？

您的现有代码可以简化，以消除任何价值的可能性＆＃34;通过裂缝＆＃34;通过注意if条件的前半部分可以省略：如果你达到第二种情况，那么值必须大于1.5，依此类推。

对于这种情况，阵列是否比交换机情况更有效？

3 个答案: