Question

我正在创建一个动态菜单，根据设置的'模式'（通过ajax传递）显示项目。关闭它会创建菜单，禁用和隐藏与该模式无关的图标。

问题是，在我的实现中有很多if条件。谁能告诉我一个更清洁的方式来做我想要实现的目标？

我的代码是：

public function gridMenu()
  {
    $mode = Validate::sanitize($_POST['mode']);

    $modes = array(
        'products' => array('edit', 'delete', 'archive')
    );

    $output = '<div id="hexContainer">';
    for($i = 1; $i < 7; $i++) {
      $img = '';
      $output .= '<div class="hex hex' . $i;
      if($i == 1)
      {
        if(in_array('edit', $modes[$mode]))
        {
          $output .= ' hex-selectable';
          $img = '<img data-option="Edit" src="' . ROOT . 'images/edit.png">';
        } else {
          $output .= ' hex-disabled';
        }
      }
      if($i == 2)
      {
        if(in_array('zzz', $modes[$mode]))
        {
          $output .= ' hex-selectable';
        } else {
          $output .= ' hex-disabled';
        }
      }
      if($i == 3)
      {
        if(in_array('delete', $modes[$mode]))
        {
          $output .= ' hex-selectable';
          $img = '<img data-option="Delete" src="' . ROOT . 'images/delete.png">';
        } else {
          $output .= ' hex-disabled';
        }
      }
      if($i == 4)
      {
        if(in_array('xxx', $modes[$mode]))
        {
          $output .= ' hex-selectable';
        } else {
          $output .= ' hex-disabled';
        }
      }
      if($i == 5)
      {
        if(in_array('archive', $modes[$mode]))
        {
          $output .= ' hex-selectable';
          $img = '<img data-option="Archive" src="' . ROOT . 'images/archive.png">';
        } else {
          $output .= ' hex-disabled';
        }
      }
      if($i == 6)
      {
        if(in_array('zzz', $modes[$mode]))
        {
          $output .= ' hex-selectable';
        } else {
          $output .= ' hex-disabled';
        }
      }

      $output .= '">';
      $output .= $img;
      $output .= '</div>';
    }
    $output .= '<div class="hex hex-mid"></div>';
    $output .= '</div>';
    echo $output;
  }

Answer 1

使用switch功能：

switch ($i){
   case 1:
       //code for $i==1
       break;
   case 2:
       // code for $i==2
       break;

   //...

}

Answer 2

尚未经过测试，但您可以尝试一下这方面的内容。

基本上从你的代码我假设你试图只在奇数轮上插入一个图像。因此，我创建了$_modes和$_mode变量，这些变量将保存每轮的值。

使用这些值，我们将使用您在代码中设置的规则自动创建HTML类，如果它是奇数，则在每一轮中创建HTML类，然后在数组中我们也添加图像。

public function gridMenu() {
    $mode = Validate::sanitize($_POST['mode']);

    $modes = array(
        'products' => array('edit', 'delete', 'archive')
    );

    $_modes = array(
        false,
        array('Edit', 'edit'),
        array('ZZZ', 'zzz'),
        array('Delete', 'delete'),
        array('XXX', 'xxx'),
        array('Archive', 'archive'),
        array('ZZZ', 'zzz'),
    );

    $output = '<div id="hexContainer">';

    for($i = 1; $i < 7; $i++) {
        $_mode = $_modes[$i];

        $img = '';
        $classnames = array('hex', 'hex' . $i);

        if ( ($i % 2) === 0 && in_array($_mode[1], $modes[$mode])) {
            $img = '<img data-option="' . $_mode[0] . '" src="' . ROOT . 'images/' . $_mode[1] . '.png">';
        }

        $classnames[] = (in_array($_mode[1], $modes[$mode]) ? 'hex-selectable' : 'hex-disabled');

        $output .= '<div class="' . implode(' ', $classnames) . '">';
        $output .= $img;
        $output .= '</div>';
    }

    $output .= '<div class="hex hex-mid"></div>';
    $output .= '</div>';

    echo $output;
}

更有效的方式来构建动态菜单

2 个答案: