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

    +65

    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
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    $quGroup = mysql_query($x = "
            SELECT
                a.latitude, a.longitude, a.catid, a.id, a.title, a.arttype, a.userid, b.firstname, b.lastname, b.usertype,
                a.latitude2, a.longitude2, a.latitude3, a.longitude3, a.latitude4, a.longitude4, a.latitude5, a.longitude5,
                a.latitude6, a.longitude6, a.latitude7, a.longitude7, a.latitude8, a.longitude8, a.latitude9, a.longitude9,
                a.latitude10, a.longitude10, a.latitude11, a.longitude11
            from materials a, users b
            where a.status='1' and a.userid=b.id and add_date > $day)
            $sql_cond");
        while ($flGroup = mysql_fetch_row($quGroup)) {
            $j++;
            $a1 = $a2 = array ();
            $l1 = $flGroup[0];
            $l2 = $flGroup[1];
            if ($l1 && $l2) {
                $a1[] = $l1;
                $a2[] = $l2;
            }
    
            $latitude2 = $flGroup[10];
            $longitude2 = $flGroup[11];
            if ($latitude2 && $longitude2) {
                $a1[] = $latitude2;
                $a2[] = $longitude2;
            }
    
            $latitude3 = $flGroup[12];
            $longitude3 = $flGroup[13];
            if ($latitude3 && $longitude3) {
                $a1[] = $latitude3;
                $a2[] = $longitude3;
            }
    
            $latitude4 = $flGroup[14];
            $longitude4 = $flGroup[15];
            if ($latitude4 && $longitude4) {
                $a1[] = $latitude4;
                $a2[] = $longitude4;
            }
    
            $latitude5 = $flGroup[16];
            $longitude5 = $flGroup[17];
            if ($latitude5 && $longitude5) {
                $a1[] = $latitude5;
                $a2[] = $longitude5;
            }
    
            $latitude6 = $flGroup[18];
            $longitude6 = $flGroup[19];
            if ($latitude6 && $longitude6) {
                $a1[] = $latitude6;
                $a2[] = $longitude6;
            }
    
            $latitude7 = $flGroup[20];
            $longitude7 = $flGroup[21];
            if ($latitude7 && $longitude7) {
                $a1[] = $latitude7;
                $a2[] = $longitude7;
            }
    
            $latitude8 = $flGroup[22];
            $longitude8 = $flGroup[23];
            if ($latitude8 && $longitude8) {
                $a1[] = $latitude8;
                $a2[] = $longitude8;
            }
    
            $latitude9 = $flGroup[24];
            $longitude9 = $flGroup[25];
            if ($latitude9 && $longitude9) {
                $a1[] = $latitude9;
                $a2[] = $longitude9;
            }
    
            $latitude10 = $flGroup[26];
            $longitude10 = $flGroup[27];
            if ($latitude10 && $longitude10) {
                $a1[] = $latitude10;
                $a2[] = $longitude10;
            }
    
            $latitude11 = $flGroup[28];
            $longitude11 = $flGroup[29];
            if ($latitude11 && $longitude11) {
                $a1[] = $latitude11;
                $a2[] = $longitude11;
            }
    
            if (!$l1 && !$l2 && !$latitude2 && !$longitude2 && !$latitude3 && !$longitude3 && !$latitude4 && !$longitude4 && !$latitude5 && !$longitude5 && !$latitude6 && !$longitude6 && !$latitude7 && !$longitude7 && !$latitude8 && !$longitude8 && !$latitude9 && !$longitude9 && !$latitude10 && !$longitude10 && !$latitude11 && !$longitude11)
                continue;
    
    ...
    }

    как изящно

    shmaltorhbooks, 28 Июня 2012

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

    +65

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    if ($_GET['where'] != '') {
    	if ($_GET['where'] != '') {
    		die ("<meta http-equiv='refresh' content='0; url=".$page."?where=".$_GET['where']."'>");
    	} else {
    		die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
    	}
    } else {
    	die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
    }

    Интересно, в каком случае выполнится второй die()?

    domaster, 27 Июня 2012

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

    +71

    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
    # Разбор time()
    function GetTime( $time ) {
    	
    	# Получение массива из элементов даты
    	$time = getdate( $time );
    	
    	# Все элементы с ведущими нулями
    	$d = ( $time['mday'] > 9 ) ? $time['mday'] : '0' . $time['mday'];
    	$w = ( $time['mon'] > 9 ) ? $time['mon'] : '0' . $time['mon'];
    	$y = $time['year'];
    	$h = ( $time['hours'] > 9 ) ? $time['hours'] : '0' . $time['hours'];
    	$m = ( $time['minutes'] > 9 ) ? $time['minutes'] : '0' . $time['minutes'];
    	$s = ( $time['seconds'] > 9 ) ? $time['seconds'] : '0' . $time['seconds'];
    	
    	# Сборка и возврат результата
    	return $d . '/' . $w . '/' . $y . '&nbsp;' . $h . ':' . $m . ':' . $s;
    }

    Про date() явно не в курсе...

    nethak, 27 Июня 2012

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

    +62

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    function find( &$db, $id) {
    		$statement = Contact::findStatement() . " WHERE id = $id";
    		$dbResult = ActiveRecord::abstractFind( $db, $statement);
    		foreach ($dbResult as $dbRow) {
    			return Contact::load( $db, $dbRow);
    		}
    		return null;
    	}

    shmaltorhbooks, 26 Июня 2012

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

    +141

    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
    <?php
    if(!$this->model_forum->community_once(array('id'=>$community))) {
       	 echo file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/no_community.tpl');
    } 
    
    else if(isset($_GET['topic']) && is_numeric($_GET['topic']) && $topic = $this->model_forum->topic_once(array('id'=>(int)$_GET['topic']))) {
       	 $tpl =  file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/add_response_topic.tpl');
    	 $tpl =  str_replace('{topic_id}', (int)$_GET['topic'] ,$tpl);
    	 echo $tpl;
    }
    
    else {
        $tpl = file_get_contents($_SERVER['DOCUMENT_ROOT'].'/style/templates/mobile/add_topic.tpl');
        $tpl = str_replace('{community}',$community, $tpl);
    	$tpl = str_replace('{topic_date_closing}', '23', $tpl);
    	echo $tpl;
    	
    } ?>

    оцените на предмет гавнокода

    nepster, 26 Июня 2012

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

    +67

    1. 1
    $this->bgcolor = $bgcolor ? ($bgcolor <= 0xFFFFFF) ? ($bgcolor > 0xFFFFFF) ? 0xFFFFFF : $bgcolor : 0x000000 : 0xFFFFFF;

    FFFFFFFFFFFFFFFFFFFFFFF

    Razban_Guestov, 26 Июня 2012

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

    +72

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    public function isAdmin() {
            $admins = Model::factory('User')->where('access_level_id', 1)->find_many();
            $ids = array();
            foreach($admins as $admin) {
                $ids[]= $admin->id;
            }
            return in_array($this->id, $ids) ? true : false;
        }

    Метод проверки на админе из модели пользователей. Не, ну а чо.

    mr.The, 26 Июня 2012

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

    +61

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    if (is_page('orders_item')) {
    	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
    } elseif (is_page('orders_garant')) {
    	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_garant.php?pid=".$_GET['pid']."'>");
    } else {
    	die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
    }
    exit();

    Хоть опросник пиши:
    - чем первый die() отличается от последнего?
    - почему их 3?
    - кто будет экранировать $_GET?
    - зачем в конце exit() подпирает die()?

    domaster, 25 Июня 2012

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

    +57

    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
    class GetParameter
    	{
    		var $dbhost = '';
    		var $dbname = '';
    		var $dblogin = '';
    		var $dbpassword = '';
    
    		var $deepmenu1itemstable = '`deepmenu1items`';
    		var $deepmenu2itemstable = '`deepmenu2items`';
    		var $topmenu2itemstable = '`topmenu2items`';
    		var $topmenu3itemstable = '`topmenu3items`';
    		var $topmenu4itemstable = '`topmenu4items`';
    		var $news1table = '`news1`';
    		var $news2table = '`news2`';
    		var $news3table = '`news3`';
    		var $news4table = '`news4`';
    		var $sites2table = '`sites2`';
    		var $chattable = '`chat`';
    		var $votestable = '`votes`';
    		
    
    		var $varstable = '`vars`';
    		var $sitestable = '`sites`';
    		var $orderstable = '`orders`';
    		var $fotoreporttable = '`fotoreport`';
    		var $fotortable = '`fotor`';
    		var $articlestable = '`articles`';
    		var $menuitemstable = '`menuitems`';
    		var $topmenuitemstable = '`topmenuitems`';
    		var $streetstable = '`streets`';
    		var $regionstable = '`regions`';
    		var $zastrojstable = '`zastrojs`';
    		var $naznachofisstable = '`naznachofiss`';
    		var $managerstable = '`managers`';
    		var $opensellstable = '`opensells`';
    		var $objectstable = '`objects`';
    		var $kodstable = '`kods`';
    		var $namestable = '`names`';
    		var $adminstable = '`admins`';
    		var $peoplestable = '`peoples`';
    
    		var $userstable = '`users`';
    }

    Сайт написан за 1500$
    молчу про остальные классы и функции.

    Samedi, 24 Июня 2012

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

    +47

    1. 1
    2. 2
    3. 3
    4. 4
    return PartnersProjectDaily::getInstance()->getSummary(
              array('date' => Utils::dateFormat($_POST['date']), 
                       'options' =>new StatSearchOptions($_POST['options']), 
                        'group' => $_POST['group']));

    Инкапсулировать параметризация полностью, руки не дошли?

    sl4mmer, 24 Июня 2012

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