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

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

    +159

    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
    18. 18
    function _getConditionWithCategoryConj( $condition, $categoryID ) //fetch products from current category
    {
      $category_condition = "";
      $q = db_query("select productID from ".
                CATEGORIY_PRODUCT_TABLE." where categoryID=".(int)$categoryID);
      $icounter = 0;
      while( $product = db_fetch_row( $q ) )
      {
        if ( $icounter == 0 )
          $category_condition .= " productID IN ('";
        if ( $icounter > 0 )
          $category_condition .= ", ";
        $category_condition .= (int)$product[0];
        $icounter++;
      }
      if ( $icounter>0 ) {
        $category_condition .= "')";
      }

    shopcms.
    нет слов больше.

    zealotous, 01 Марта 2011

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

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $difference = strtotime($VipAcc['DateTime']) - time();
    
    $d = floor($difference/60/60/24);
    $h = floor(($difference - $d*60*60*24)/60/60);
    $m = floor(($difference - $d*60*60*24 - $h*60*60)/60);

    так и не выучил функций по работе со временем

    ($VipAcc['DateTime'] - значение взято з базы)

    Sergunik, 28 Февраля 2011

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

    +159

    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
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    <?
    			if(($socle == 1302 || $socle == 2723 || $socle == 2723 || $socle == 211) && ($power == 1816 || $power = 1296 || $power = 2741 || $power = 2741 || $power = 216  || $power = 1854))//G9 40W
    				{echo '<br/><a href="/catalog/?id=1&section=2163&dep=12">Купить лампочку: цоколь G9, мощность 40W</a>';}
    			else if(($socle == 1324 || $socle == 2964 || $socle == 212) && ($power == 1826 || $power == 2942 || $power == 1325 || $power == 1861 || $power == 217  || $power == 1734))//GU10 50W
    				{echo '<br/><a href="/catalog/?id=2&section=2163&dep=12">Купить лампочку: цоколь GU10, мощность 50W</a>';}
    			else if(($socle == 2855 || $socle == 1294 || $socle == 2733 || $socle == 1883 || $socle == 1721) && ($power == 1816 || $power = 1296 || $power = 2741 || $power = 2741 || $power = 216  || $power = 1854))//E14 40W
    				{
    				echo '<br/><a href="/catalog/?id=3&section=2163&dep=12">Купить лампочку: цоколь E14, мощность 40W</a><br/>';
    				echo '<br/><a href="/catalog/?id=4&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E14, мощность 7W</a>';
    				}
    			else if(($socle == 2748 || $socle == 1305 || $socle == 3051 || $socle == 203 || $socle == 1724) && ($power == 1816 || $power = 1296 || $power = 2741 || $power = 2741 || $power = 216  || $power = 1854))//E27 40W
    				{
    				echo '<br/><a href="/catalog/?id=5&section=2162&dep=12">Купить лампочку: цоколь E27, мощность 40W</a><br/>';
    				echo '<br/><a href="/catalog/?id=6&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E27, мощность 11W</a>';
    				}
    			else if(($socle == 2748 || $socle == 1305 || $socle == 3051 || $socle == 203 || $socle == 1724) && ($power == 1919 || $power = 1817 || $power = 5163 || $power = 1306 || $power = 2727 || $power = 205 || $power = 3052 || $power = 1850))//E27 60W
    				{
    				echo '<br/><a href="/catalog/?id=7&section=2162&dep=12">Купить лампочку: цоколь E27, мощность 60W</a><br/>';
    				echo '<br/><a href="/catalog/?id=8&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E27, мощность 11W</a>';
    				}
    			else if(($socle == 2748 || $socle == 1305 || $socle == 3051 || $socle == 203 || $socle == 1724) && ($power == 1919 || $power = 1817 || $power = 5163 || $power = 1306 || $power = 2727 || $power = 205 || $power = 3052 || $power = 1850))//E27 60W
    				{
    				echo '<br/><a href="/catalog/?id=9&section=2162&dep=12">Купить лампочку: цоколь E27, мощность 60W</a><br/>';
    				echo '<br/><a href="/catalog/?id=10&section=4212&dep=12">Купить энергосберегающую лампочку: цоколь E27, мощность 11W</a>';
    				}		
    		?>

    Говнокодеры не спят, они какают и какают :))))
    Учитесь, как надо давать покупателю возможность выбрать подходящую лампочку)

    greevex, 25 Февраля 2011

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

    +159

    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
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    //старт сессии
    function Sessions_Start () {
            global $SecureKey, $SecureCookieName, $NumIPRanks, $AuthSalt;
     
        $LifeTime = 0; 
        $Path = '/';
        $Domain = '';
        $Secure = FALSE;
        $HttpOnly = TRUE;
        session_set_cookie_params($LifeTime, $Path, $Domain, $Secure, $HttpOnly);
            
            session_start ();
            
            if (!isset ($_SESSION['UserData'])) {
                    $_SESSION['UserData'] = array ();
                    $_SESSION['UserData']['UserType'] = 'guest';    
            }
            else{
                    $Type = $_SESSION['UserData']['UserType'];
                    $SessionValidator = 0;
                    
                    if ($Type == 'member') {
                            if (!isset ($_SESSION['UserData']['UserID']) or !isset ($_SESSION['UserData']['UserName']) or!isset ($_SESSION['UserData']['GroupID']) or!isset ($_SESSION['UserData'][$SecureKey]) or!isset ($_SESSION['UserData'][$SecureCookieName])) {
                                    $SessionValidator++;
                            }
                            
                            if (!isset ($_COOKIE[$SecureCookieName])) {
                                    $SessionValidator++;
                            }                       
                            
                            //секретная строка
                            $IPArray = explode ('.', $_SERVER['REMOTE_ADDR']);
                            $IPRanks = array ();
                            for ($i=0; $i < $NumIPRanks; $i++) {
                                    $IPRanks[] = $IPArray[$i];
                            }
                            $IPFinalString = sha1 (implode ('.', $IPRanks) . $AuthSalt);
                            $CurUserAgent = sha1 (substr ($_SERVER['HTTP_USER_AGENT'], 0, 50) . $AuthSalt);
                            $SecureString = sha1 ($IPFinalString . $CurUserAgent . $AuthSalt);
                            
                            if ($_SESSION['UserData'][$SecureKey] <> $SecureString) {
                                    $SessionValidator++;
                            }
                            
                            if ($_SESSION['UserData'][$SecureCookieName] <> $_COOKIE[$SecureCookieName]) {
                                    $SessionValidator++;
                            }
                            
                            if ($SessionValidator > 0) {
                                    session_regenerate_id ();
                                    $_SESSION['UserData'] = array ();
                                    $_SESSION['UserData']['UserType'] = 'guest';
                            }
                            
                    }
            }       
            return (session_id ());
    }

    qbasic, 23 Февраля 2011

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

    +159

    1. 1
    $this->last_error = $success;

    кстати о названиях переменных :))

    youROCK, 22 Февраля 2011

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

    +159

    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
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    class forms {
        //put your code here
        
        function __construct($name,$id,$method,$action) {
            echo  "<form id='$id' name='id' method='$method' action='$action'>";
        }
        
        static private function attr($param)    {
            if ($param) {
                foreach ($param as $nam_attr => $val_attr) {
                    $attribs.=$nam_attr."='".$val_attr."'";
                }
                return $attribs; 
            }
        }
        static private function label($id,$text,$attr)    {
            $key = array_search('regue', $attr, true);
            if ($key = 1) $text.="<font color='red'>*</font>";
            return "<label for='$id'>$text</label>";
        }
     
        static public function inputs($text,$type,$name,$id,$value=false,$class=false, $attr=false)    {
           
            return self::label($id,$text,$attr)."<input type='$type' name='$name' id='$id' value='$value' class='$class' ".self::attr($attr)."/>";
        }    
        
        static public function inputs_chek($text,$type,$name,$id,$value=false,$class=false, $attr=false)    {
            return  self::label($id,$text,$attr)."<input type='$type' name='$name' id='$id' class='$class' ".self::attr($attr)."/>$value";
        }   
        
        static public function area($text,$name,$id,$value=false,$class=false, $attr=false)    {
            return  self::label($id,$text,$attr)."<TEXTAREA NAME='$name' id='$id' WRAP='virtual' COLS='40' ROWS='3' ".self::attr($attr).">$value</TEXTAREA>";
        } 
        
        static public function select($text,$name,$id,$value=false,$class=false, $attr=false)    {
            $select="<select name='$name' id='$id' ".self::attr($attr).">";
            foreach($value as $value => $val_text)  {
                $select.="<option value='$value'>$val_text</option>";
            }
           $select.="</select>";
           return  self::label($id,$text,$attr).$select;
        } 
        
        function __destruct() {
            echo "<input type='submit'/><input type='reset' value='Очистить'></form>";
        }   
    }

    qbasic, 21 Февраля 2011

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

    +159

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    $query = "SELECT id, idTovar FROM $table";
     $res = mysql_query_assoc($query) or die(mysql_error());
     foreach ($res as $id => $idTovar) 
      {
       $idTovar_array = explode(",", $idTovar);
       if($key = array_search($delete, $idTovar_array))
         unset($idTovar_array[$key]);
       $idTovar1 = implode(",", $idTovar_array);
       if ($idTovar==$idTovar1);
       else
       mysql_query("UPDATE $table SET idTovar = '$idTovar' WHERE id = '$id'");
      }

    qbasic, 19 Февраля 2011

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

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    $this->setLayout('empty');
            $this->setTemplate('step3');
    
            if ($this->getBrokerId() == 19 || $this->getBrokerId() == 29)
            {
                $this->setLayout('empty');
                $this->setTemplate('step3');
                $this->brokerId = $this->getBrokerId();
            }

    Нашел в чужом проекте. При этом $this->getBrokerId() не просто возвращает переменную, а производит сравнение строк (то есть вызывать этот метод 3 раза - явный перебор).

    blaster999, 15 Февраля 2011

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

    +159

    1. 1
    2. 2
    3. 3
    $ar=$arFields['GROUP_ID'];
    $arFields['GROUP_ID']=array();
    foreach($ar as $a) $arFields['GROUP_ID'][]=$a;

    Смысл данной конструкции не ясен, если так нужно обнулить ключи для массива то array_values() в помощь.

    tkf, 15 Февраля 2011

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

    +159

    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
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    <?php
    ...
    // Проверяем нужны ли стрелки назад
    if ($page != 1) $pervpage = '<a href=catalog.php?page=1>Первая</a> |
    <a href=catalog.php?page='. ($page - 1) .'>Предыдущая</a> | ';
    // Проверяем нужны ли стрелки вперед
    if ($page != $total) $nextpage = ' | <a href=catalog.php?page='. ($page + 1) .
    '>Следующая</a> | <a href=catalog.php?page=' .$total. '>Последняя</a>';
    // Находим две ближайшие станицы с обоих краев, если они есть
    if($page - 5 > 0) $page5left = ' <a href=catalog.php?page='. ($page - 5) .'>'.
    ($page - 5) .'</a> | ';
    if($page - 4 > 0) $page4left = ' <a href=catalog.php?page='. ($page - 4) .'>'.
    ($page - 4) .'</a> | ';
    if($page - 3 > 0) $page3left = ' <a href=catalog.php?page='. ($page - 3) .'>'.
    ($page - 3) .'</a> | ';
    if($page - 2 > 0) $page2left = ' <a href=catalog.php?page='. ($page - 2) .'>'.
    ($page - 2) .'</a> | ';
    if($page - 1 > 0) $page1left = '<a href=catalog.php?page='. ($page - 1) .'>'.
    ($page - 1) .'</a> | ';
    if($page + 5 <= $total) $page5right = ' | <a href=catalog.php?page='. ($page + 5)
    .'>'. ($page + 5) .'</a>';
    if($page + 4 <= $total) $page4right = ' | <a href=catalog.php?page='. ($page + 4)
    .'>'. ($page + 4) .'</a>';
    if($page + 3 <= $total) $page3right = ' | <a href=catalog.php?page='. ($page + 3)
    .'>'. ($page + 3) .'</a>';
    if($page + 2 <= $total) $page2right = ' | <a href=catalog.php?page='. ($page + 2)
    .'>'. ($page + 2) .'</a>';
    if($page + 1 <= $total) $page1right = ' | <a href=catalog.php?page='. ($page + 1)
    .'>'. ($page + 1) .'</a>';
    ...
    ?>

    магазин на диване...
    http://www.php.ru/forum/viewtopic.php?t=30808#263199

    Devzirom, 14 Февраля 2011

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