1. PHP / Говнокод #557

    +145.5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if ($ajax_teamID == $value['team2ID']){
                  $visitors_team = $Bet->CBetType[$Bet->betTypeIndex]->CTeam->getOne($value['team2ID'])->title;
                  $home_team     = $Bet->CBetType[$Bet->betTypeIndex]->CTeam->getOne($value['team1ID'])->title;
            } else {      
                  $home_team     = $Bet->CBetType[$Bet->betTypeIndex]->CTeam->getOne($value['team1ID'])->title;              
                  $visitors_team = $Bet->CBetType[$Bet->betTypeIndex]->CTeam->getOne($value['team2ID'])->title;
            }

    Не делайте так.

    guest, 16 Февраля 2009

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

    +141.5

    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
    $db = S4Y::getInstance()->getDb();
            $sql = 'INSERT INTO publish_groups (domain, name)
                VALUES (:domain, :name)';
            $db->query(
                $sql, array('domain' => S4Y::$domain, 'name' => $_POST['subgroup'])
            );
            if ($this->_groupId) {
                $last_insert_id = $db->lastInsertId();
                $sql = 'UPDATE publish_groups
                    SET parent_id = :parent
                    WHERE id = :id';
                $db->query(
                    $sql,
                    array('parent' => $this->_groupId, 'id' => $last_insert_id)
                );

    Кусок кода создает группу публикаций и устанавливает ей парента.
    В два приема, чтоб не напрягать мускуль более чем 2 параметрами...

    guest, 16 Февраля 2009

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

    +132.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (!empty(($_POST['text'])) {
    	$text = $_POST['text'];
    } else {
    	unset($text);
    }

    guest, 16 Февраля 2009

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

    +150.9

    1. 1
    if (FALSE !== strpos($this->lin{$i},$this->o)) {

    я б до такого сравнения даже в кошмарном сне бы не додумался.

    guest, 15 Февраля 2009

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

    +182

    1. 1
    ini_set('register_globals','on');// хуй с ними.

    guest, 14 Февраля 2009

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

    +145

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    <?
    
    $mytext = iconv('windows-1251', 'utf-8', $mytext);
    echo $mytext;
    
    //There is a little problem with iconv in such using
    //This code isn't work correctly. Solution is:
    
    $mytext = iconv('windows-1251', 'utf-8', $mytext);
    echo $mytext;
    
    ?>

    Я всегда говорил что бездумная перепечатка чужих мануалов это зло...

    guest, 13 Февраля 2009

    Комментарии (1)
  7. Pascal / Говнокод #549

    +152

    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
    if currIndex=2 then
          begin
          // разкрываем выделенные узлы
            TreeView_U.Items[r].Expanded:=true;
            if TreeView_U.Items[r].Parent <> nil then
            begin
              TreeView_U.Items[r].Parent.Expanded:=true;
              if TreeView_U.Items[r].Parent.Parent <> nil then
              begin
                TreeView_U.Items[r].Parent.Parent.Expanded:=true;
                if TreeView_U.Items[r].Parent.Parent.Parent <> nil then
                TreeView_U.Items[r].Parent.Parent.Parent.Expanded:=true;
              end;
            end;

    Наткнулся тут в прокете. Поначалу просто кусок закоментировал не разбираясь, потому что просто не нужен. Суть - попытка раскрыть полностью ветвь дерева, если в ней есть хоть один выбранный (с помеченным чекбоксом) элемент, о чём говорит currIndex=2.
    Весь офис валяется под столами :)))

    guest, 13 Февраля 2009

    Комментарии (5)
  8. Pascal / Говнокод #548

    +148

    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
    if currIndex=2 then
     begin
      // разкрываем выделенные узлы
      TreeView_U.Items[r].Expanded:=true;
      if TreeView_U.Items[r].Parent <> nil then
       begin
        TreeView_U.Items[r].Parent.Expanded:=true;
        if TreeView_U.Items[r].Parent.Parent <> nil then
         begin
          TreeView_U.Items[r].Parent.Parent.Expanded:=true;
          if TreeView_U.Items[r].Parent.Parent.Parent <> nil then
           TreeView_U.Items[r].Parent.Parent.Parent.Expanded:=true;
         end;
       end;

    Это попытка раскрыть полностью ветвь дерева, если в ней есть хоть один выбранный (с помеченным чекбоксом) элемент, о чём говорит currIndex=2
    Вообще-то оно даже работает, ибо вложенность дерева маленькая. Но увеличиться запросто может...

    guest, 13 Февраля 2009

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

    +149

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if((isset($pro['stats'][$c_l])) and ($time == $pro['stats'][$c_l])) {
         $this->lg['u'] = 'h';
    } else {
         $this->lg['u'] = 'u';
         $pro['stats'][$c_l] = $time;
    }
                                                                    }

    самое главно чтобы u всегда было равно u если оно не будет h

    guest, 13 Февраля 2009

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

    +213.9

    1. 1
    2. 2
    if ($result=='admin') {
     $result='admin'; }

    выбрал, но не поверил

    guest, 13 Февраля 2009

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