Говнокод #7326 — PHP — Говнокод.ру

PHP / Говнокод #7326

+164
1. 1
2. 2
3. 3
```
<a href="?page=news" <?php if (isset($_GET['page'])) if ($_GET['page']=='news') echo 'class="active"' ?> >Новости</a>
<a href="?page=photo" <?php if (isset($_GET['page'])) if ($_GET['page']=='photo') echo 'class="active"' ?> >Фото</a>
<a href="/" <?php if (isset($_GET['page'])); else echo ' class="active"'?> >Главная</a>
```
Подсветка той ссылки, на которой сейчас находишься (присвоение класса active).
Проверка if (isset($_GET['page'])) сделана для того, чтобы PHP не ругался на то, что page не передан (такое происходит при переходе на главную).

Запостил: opex_jr, 23 Июля 2011

Tweet
Комментарии (27) RSS
- guest 23.07.2011 21:17 # −19
  
  показать все, что скрытоПомни, guest, за тобой могут пиписюнчик!
  Ответить
  - KirAmp 23.07.2011 23:25 # +9
    
    Ты всегда приходишь первым и всегда постишь про свои маленькие проблемы.. Неужели у тебя нет совсем никакой личной жизни?
    Ответить
  - nethak 24.07.2011 14:28 # +3
    
    Я понимаю ты поймал эту шутку, и думаешь это смешно, отнюдь, выставляешь себя критином умственно отсталым.
    Если ты думаешь что ты кому-то действуешь на нервы (малолетнии убогие критины любят это делать), то скажу тебе что всем безразличен твой бред, просто какой смысл, ты тупо теряешь своё время.
    Ты мудак короче.
    Ответить
    - eth0 24.07.2011 21:40 # +2
      
      Рыцари грамматика, который освобождает, давно бы вас КРОВЬКИШКИРАСПИДОРАСИЛОпустили под нож, комрад.
      Ответить
- Yurik 23.07.2011 21:21 # 0
  
  <?php if($page): ?>class="active"<?php endif; ?>
  Ответить
  - opex_jr 23.07.2011 21:49 # −1
    
    А как обойти ошибку "Undefined variable: page"? На главной же page не передаётся.
    Ответить
    - guest 23.07.2011 21:55 # −14
      
      показать все, что скрытоОчевидно, require_once("пиписюнчик")
      Ответить
    - opex_jr 23.07.2011 21:56 # 0
      
      <?php if (isset($_GET['page'])) $page=$_GET['page']; else $page=""?>
      А Ваш код не сработал: просто присваивает класс всем ссылкам подряд.
      Ответить
      
      Yurik 23.07.2011 23:03 # −1
      
      Не в шаблоне:
      
      $photo = (isset($_GET['page']) and $_GET['page'] == 'photo') ? true : false;
      
      Потом в шаблоне:
      
      <?php if($photo): ?>class="active"<?php endif; ?>
      Ответить
      
      Ring 23.07.2011 23:33 # 0
      
      На мой взгляд не есть здорово, много левых переменных. Тогда уж лучше что-то типа:
      
      Не в шаблоне
      
      public function checkActive($page){ echo (isset($_GET["page"]) && $_GET["page"] == $page) ? " class='active' " : ""; }
      
      в шаблоне
      
      <a href="?page=news"<?php $this->checkActive("news"); ?>>Новости</a>
      Ответить
      
      Yurik 24.07.2011 00:12 # +2
      
      Ну это каждому свое, мне, например, совсем не нравится печатать HTML отдельными функциями. Плюс, можно вместо переменных использвать один массив.
      Ответить
      
      Yurik 24.07.2011 00:19 # −1
      
      Хотя, возможно, вы правы и так лучше, я пока кроме гостевух ничего не писал и в больших проектах не учавствовал :)
      Ответить
      
      Ring 24.07.2011 00:47 # 0
      
      Ну, опыт дело наживное :)
      А по-хорошему, то, что я написал, тоже не самая лучшая реализация.
      Ответить
      
      bugmenot 24.07.2011 08:13 # −3
      
      пиххипишники в раскаянии?
      Ответить
      
      wvxvw 24.07.2011 12:30 # +2
      
      Просто замечательная ветка... и echo вместо return, и true : false как результат тернарного оператора, и название функции, которая по сути предикат, и при этом называется checkFact вместо isFact, и повторяющиеся строковые константы...
      Ответить
      
      nethak 24.07.2011 14:31 # −2
      
      И тут явился ГУРУ.
      Ответить
      
      Lure Of Chaos 24.07.2011 15:06 # −1
      
      > которая по сути предикат
      вначале прочиталось "пердикат" :) простите
      Ответить
      
      bugmenot 24.07.2011 15:12 # 0
      
      есть такое словечко - brainfart
      Ответить
      
      wvxvw 24.07.2011 21:13 # +4
      
      Предикат, это часть предложения, которая описывает подлежащее, например, homo homini lupus est - lupus est - предикат. Когда-то, когда единственным языком программирования был натуральный язык, а единственным православным языком программирования был древне-итальянский, функции-предикаты строили по принципу: однин аргумент, он же подлежащее, и, либо наречие, либо отглагольное существительное, либо глагол в страдательном залоге. Назначение такой функции, описав в названии ее сущность, дать ответ с минимальным количеством вариаций. Такой подход так же нужен в классической логике для построения утверждений типа силлогизмов, когда на основании двух предикатов доказывается третий.
      В современных языках предикаты, это, как правило, функции принимающие один аргумент - читай: подлежащее, и выносящие суждение о проверяемом свойстве - ложно оно или истинно. Типичный пример, в ECMAScript - isNaN(n). Или, если мы попробуем создать предикат на основе первого примера: estLupus(homo).
      :)
      Ответить
      
      Lure Of Chaos 24.07.2011 21:21 # +3
      
      я знал, что такое предикат, но все равно спасибо = )
      Ответить
      
      guest 24.07.2011 21:45 # −2
      
      КО? Спасибо.
      Ответить
      
      wvxvw 24.07.2011 23:39 # 0
      
      Главноe - правильно выбрать стиль повествования.
      Ответить
      
      qfox 29.07.2011 00:25 # −2
      
      $page = @$_GET['page'];
      foreach ($menuItems = array(
      'news' => 'Новости',
      'hunews' => 'хуевости',
      ) as $url => $name):
      ?><a href="<?= $url ?>" <?= $page == $url ? 'class="active"' : '' ?>><?= $name ?></a><?
      foreachend;
      
      а вообще, конечно, это в целом говно.
      Ответить
      
      qfox 29.07.2011 00:27 # −2
      
      $photo = (isset($_GET['page']) and $_GET['page'] == 'photo') ? true : false;
      надо переписать как:
      $photo = (isset($_GET['page']) === true and false !== ($_GET['page'] == 'photo')) ? true : false;
      if($photo) $photo = true; else $photo = false;
      if(!$photo) $photo = false;
      и еще пару строк на всякий случай.
      Ответить
    - opex_jr 29.07.2011 16:19 # 0
      
      Понял, ошибки подавляются собакой.
      <a href="?page=news" <?php if (@$_GET['page']=='news') echo 'class="active"' ?> >Новости</a>
      Ответить
- sayidandrtfm 24.07.2011 20:43 # +1
  
  <a href="?page=news" <?php echo ('class='.((isset($_GET['page']) && $_GET['page']=='news')?'act':'pass').'ive');?> >Новости</a>
  Ответить
- AlexanderC 03.08.2011 16:48 # 0
  
  <?=@$_GET['page']=='news'?'active':''?> или с isset(), ток длиннее будет
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

PHP / Говнокод #7326

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

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