1. Лучший говнокод

    В номинации:
    За время:
  2. PHP / Говнокод #13549

    +149

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    <?php if ($tags) { ?>
      <div class="tags"><b><?php echo $text_tags; ?></b>
    	<?php for ($i = 0; $i < count($tags); $i++) { ?>
    	<?php if ($i < (count($tags) - 1)) { ?>
    	<a href="<?php echo $tags[$i]['href']; ?>"><?php echo $tags[$i]['tag']; ?></a>,
    	<?php } else { ?>
    	<a href="<?php echo $tags[$i]['href']; ?>"><?php echo $tags[$i]['tag']; ?></a>
    	<?php } ?>
    	<?php } ?>
      </div>
      <?php } ?>

    opencart 1.5.6

    rombik, 04 Августа 2013

    Комментарии (11)
  3. PHP / Говнокод #13537

    +149

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    // Проверка наличия интернета по доступности яндекса и рамблера
      public function is_inet($wait=false) {
        if ($wait) {
    
          while(!($this->ping('yandex.ru')&&$this->ping('rambler.ru'))) {
            echo date('Y/m/d H:i:s >> ').'Internet disconnect, wait...'."<br/>\n";
            }
          return true;
          }
    
          elseif ($this->ping('yandex.ru')&&$this->ping('rambler.ru')) return true;
          else return false;
        }

    madfriend, 02 Августа 2013

    Комментарии (2)
  4. JavaScript / Говнокод #13529

    +149

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    где-то в коде:
                  /* коментарий №1 (см внизу)*/
    
    в самом конце файла:
    //----------------------------------------------- Старые куски кода -----------------------------------------------------------------------------
    // №1
    /*   
    тут простыня кода на 2 экрана
    */

    управление версиями, чо

    clauclauclau, 01 Августа 2013

    Комментарии (4)
  5. PHP / Говнокод #13448

    +149

    1. 1
    if('null'==$this->user_data['e_mail']) $this->user_data['e_mail']=null;

    clgs, 18 Июля 2013

    Комментарии (2)
  6. PHP / Говнокод #13438

    +149

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    if (!class_exists($module, false))
    {
    // Get content from php file
    $filepath = _PS_MODULE_DIR_.$module.'/'.$module.'.php';
    $file = trim(file_get_contents(_PS_MODULE_DIR_.$module.'/'.$module.'.php'));
    if (substr($file, 0, 5) == '<?php')
    	$file = substr($file, 5);
    if (substr($file, -2) == '?>')
    	$file = substr($file, 0, -2);
    
    // If (false) is a trick to not load the class with "eval".
    // This way require_once will works correctly
    if (eval('if (false){	'.$file.' }') !== false)
    	require_once( _PS_MODULE_DIR_.$module.'/'.$module.'.php' );
    else
    	$errors[] = sprintf(Tools::displayError('%1$s (parse error in %2$s)'), $module, substr($filepath, strlen(_PS_ROOT_DIR_)));
    }

    Это оригинал кода из знаменитого Prestashop ...
    Попросили написать модуль. Долго мучался, почему валидный код не работает. Потом нашел.
    Смысл логики в строках 13-17 : мы не хотим принимать файлы модулей с eval функцией.
    Поэтому мы сначала проверяем, есть ли eval в файле.
    Как проверяем?
    А давайте его просто выполним!
    Если выполнится - тогда - require_once
    В общем я О....ЕЛ! Других слов просто нет.
    Мало того, так еще нужно следить, чтобы ваш файл модуля был валидным для eval функции.
    Иначе - Parse error: syntax error ...

    corner578, 17 Июля 2013

    Комментарии (6)
  7. PHP / Говнокод #13264

    +149

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    <?php
    while($menu = mysql_fetch_assoc($q))
    {
    	?>
    	<option value="<?=$menu['id']?>"<?=($id === $menu['id']) ? ' selected="selected"' : ''?>><?=$menu['title']?></option>
    	<?php
    }
    ?>

    Адовейший пиздец наговнокодил сегодня. Ну не люблю я HTML-теги в PHP писать, о шаблонизаторах никогда не слышал

    3Jlou_OguHo4ka, 30 Июня 2013

    Комментарии (32)
  8. PHP / Говнокод #13194

    +149

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    public static function getUserByOwnerId($ownerId)
        {
            $user = DB::query(Database::SELECT,
                '
                SELECT
                 users.username, users.email, users.refer
                FROM
                 owners
                WHERE
                 owners.user_id='.Model_Index::escape($ownerId)
                )
            ->execute();
    
            return $user;
        }

    во как

    coll3ctor, 19 Июня 2013

    Комментарии (17)
  9. PHP / Говнокод #13116

    +149

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    $startYear = date('Y', $programs[$i][1]);
                    $startMounth = date('m', $programs[$i][1]);
                    $startDay = date('d', $programs[$i][1]);
                    $startHour = date('H', $programs[$i][1]);
                    $startMinute = date('i', $programs[$i][1]);
                    $startSecond = date('s', $programs[$i][1]);
                    $startDate = mktime($startHour + $timeoffset, $startMinute, $startSecond, $startMounth, $startDay, $startYear);

    Поправка на часовой пояс

    stsaranchin, 06 Июня 2013

    Комментарии (0)
  10. JavaScript / Говнокод #13068

    +149

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    <a onclick="document.getElementById('callback').style.display='block'">Заказать обратный звонок</a>
    
    =====================================================================================
    
    <img src="img/close_callback.png" onclick="document.getElementById('callback').style.display='none'">
    
    =====================================================================================
    
    <div class='main_menu'>
        <ul>
            <li style='width:185px;margin:0 5.5px 0 0' class='ie_menu'><a href='' class='active' style='width:155px' onmouseover='document.getElementById("drop_top_1").style.display="block"' onmouseout='document.getElementById("drop_top_1").style.display="none"'>каталог строительных <br/>материалов</a>
                <div class='drop_top' id='drop_top_1' onmouseover='this.style.display="block"'  onmouseout='this.style.display="none"'>

    Мамонт-стайл

    dsxack, 29 Мая 2013

    Комментарии (5)
  11. PHP / Говнокод #13046

    +149

    1. 1
    $years = range(date('Y', $now), date('Y', strtotime('+ 1 year', $now)));

    Пятница в моей голове... А надо было всеголтшь действующий и следующий

    nonamez, 24 Мая 2013

    Комментарии (1)