Говнокод

function render_menu_item($tag,$value,$css=null){
$length=strlen($tag);
   
    //Validate the tags
      if($tag[0]=='<' && $tag[$length-1]=='>'){
        $closingtag=str_replace('<','</',$tag);
        if($css)
            $tag=str_replace('>',' class="'.$css.'">',$tag);        
      }
      else{
        if($css){       
        $classtag='<'.$tag.' class="'.$css.'">';
        $tag='<'.$tag.'>';
        $closingtag=str_replace('<','</',$tag);
        $tag=$classtag;
        }
        else{
        $tag='<'.$tag.'>';
        $closingtag=str_replace('<','</',$tag);
        }
      }  
  return $tag.$value.$closingtag;
}

Вот так пишут доблестные Drupal-разработчики из Иордании. Сохранено оригинальное форматирование.

Запостил:

UncleAli, 13.05.2010 (Updated 24.03.2018)

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

mrbig66 16 лет ago # 0

Я надеюсь результат хотя бы кэшируется?
И зачем три раза определять $closingtag?
ГК.
Ответить
cheef 16 лет ago # 0

регулярные выражения - это сатанинские штучки!!
Ответить
- mrbig66 16 лет ago # 0 ⇈
  Не лучше ли заготовить нарезки в виде шаблона:
```
<a href="%1$s" class="%2$s">%3$s</a>

<ul class="%1$s">%2$s</ul>

  <li class="%1$s">%2$s</li>
```
  Ответить
  - Lure Of Chaos 16 лет ago # 0 ⇈
    
    может быть и лучше. Но это же ГК
    Ответить
WarLooK 16 лет ago # 0

Где вы видели закрывающий тег "</", обычно он такой "/>"?

Какой-то вообще неработоспособный код.
Ответить
- guest 16 лет ago # 0 ⇈
  
  <a>bla</a>
  Ответить
  - WarLooK 16 лет ago # 0 ⇈
    
    $closingtag=str_replace('<','</',$tag);
    никак не подходит под <a>bla</a>
    Ответить
    - UncleAli 16 лет ago # 0 ⇈
      
      Кэп кагбэ намекает, что из открывающего тэга <a> получают соответствующий закрывающий тэг </a>
      Ответить
      - WarLooK 16 лет ago # 0 ⇈
        
        А, точно. это я гоню.
        Ответить