"PHP" / Говнокод #9271 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
/*
 @param array $menu_array Array of pages
 @param string $current Current page
 @return string $menu Menu.
*/
function CreateMenu($menu_array, $current)
{
    foreach($menu_array as $key => $value)
    {
        if($key == $current) $active = " class=\"active\"";
        
        $menu .= "<li$active><a href=\"$key\">$value</a></li>\n";
        unset($active);
    }
    
    return $menu;
}

Запостил: varg242 varg242, (Updated )

Комментарии (6) RSS

    • ava varg242 # Ссылка на оригинал 0 ⇈

      $active выдаёт notice, т.к. 13 стр. - unset();

      $menu не инициализировано - опять notice.

      Нет обработки ошибок.
      Нет возможности получения в массиве.
      Ответить

      • ava nobody # Ссылка на оригинал 0 ⇈

        тоесть -
        function CreateMenu($menu_array, $current)
        {
        $menu = '';
        if(is_array($menu_array)) {
        foreach($menu_array as $key => $value)
        {
        if($key == $current)
        $active = " class=\"active\"";
        else
        $active = "";

        $menu .= "<li$active><a href=\"$key\">$value</a></li>\n";
        }
        }

        return $menu;
        }
        Уже не гавно ? Мне кажется это не критично все ... по мимо того что не проверяется массив ли входные данные, а то иначе foreach будет ругаться ...
        Ответить

    • ava Vasiliy # Ссылка на оригинал 0 ⇈

      Классика говна. Нет возможности управлять отображением. Я вот например хочу класс CSS добавить и что лезть в код. А если я в двух и более местах меню использую городить огороды с контейнерами ? В целом говно. Но если используется для одного хомяка и не претендует на гордое название "модуль построения меню CMS" то не страшно.
      Ответить

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓