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

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

    +156

    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
    public static function getCurrentRawDate($yesterday = false) {
    
            if ($yesterday) {
                $date = strtotime("-1 days");
            } else {
                $date = time();
            }
    
            $raw = explode("-", date("Y-m-d", $date));
    
            list($year, $month, $day) = $raw;
    
            return array(
                "year" => $year,
                "month" => $month,
                "day" => $day
            );
        }

    Самое очевидное поведение функции

    deivas, 15 Октября 2014

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

    +156

    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
    class ChargifyNotFoundException extends ChargifyException {
    	var $errors;
    	var $http_code;
    	
    	public function ChargifyNotFoundException($http_code, $error) {
    		$this->http_code = $http_code;		
    
    		$message = '';
    		$this->errors = array();
    		foreach ($error as $key=>$value) {
    			if ($key == 'error') {
    				$this->errors[] = $value;
    				$message .= $value . ' ';
    			}
    		}
    
    		parent::__construct($message, intval($http_code));
    	}	
    }

    Индусам платят за количество строк

    rowdyroad, 14 Октября 2014

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

    +156

    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
    class ChargifyNotFoundException extends ChargifyException {
    	var $errors;
    	var $http_code;
    	
    	public function ChargifyNotFoundException($http_code, $error) {
    		$this->http_code = $http_code;		
    
    		$message = '';
    		$this->errors = array();
    		foreach ($error as $key=>$value) {
    			if ($key == 'error') {
    				$this->errors[] = $value;
    				$message .= $value . ' ';
    			}
    		}
    
    		parent::__construct($message, intval($http_code));
    	}	
    }

    Индусам платят за количество строк

    rowdyroad, 14 Октября 2014

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

    +156

    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
    $masterContacts = $masterClient->getContacts();
        for ($i=0; $i < count($masterContacts); $i++) {
          for ($j=$i+1; $j < count($masterContacts); $j++) { 
            if(!(strcmp($masterContacts[$i]->getFirstName(), $masterContacts[$j]->getFirstName()) && strcmp($masterContacts[$i]->getLastName(), $masterContacts[$j]->getLastName()))){
              echo $masterContacts[$i]->getFirstName() . ", " . $masterContacts[$i]->getLastName() . "==" . $masterContacts[$i]->getFirstName() . ", " . $masterContacts[$i]->getLastName() . "\n";
              foreach ($masterContacts[$i]->getFields() as $contact_i) {
                $contact_i->setContact($masterContacts[$j]);
                // $masterContacts[$i]->removeField($contact_i);
                // $masterContacts[$j]->addField($contact_i);
              }
              $em->remove($masterContacts[$i]);
              // $masterClient->removeContact($masterContacts[$i]);
              break;
            }
          }
        }
    
        foreach ($masterContacts as $masterContact) {
          for ($i=0; $i < count($masterContact->getFields()); $i++) { 
            $field_i = $masterContact->getFields()[$i];
            foreach ($masterContacts as $masterContact) {
              for ($j=$i+1; $j < count($masterContact->getFields()); $j++) { 
                $field_j = $masterContact->getFields()[$j];
                // if(!strcmp($masterContact->getFields()[$i]->getInn(), $masterContact->getFields()[$j]->getInn())){
                if(!strcmp($field_i->getXValue(), $field_j->getXValue())) {
                  // echo "compare :" . $field_i->getXValue() . " == " . $field_j->getXValue();
                  $em->remove($field_i);
                  break;
                }
              }
            }
          }
        }
    
        // echo $masterClient->getId();
    
        $mOrganizations = $masterClient->getOrganization();
        for ($i=0; $i < count($mOrganizations); $i++) { 
          for ($j=$i+1; $j < count($mOrganizations); $j++) { 
            if(!strcmp($mOrganizations[$i]->getInn(), $mOrganizations[$j]->getInn())){
              echo "compare :" . $mOrganizations[$i]->getInn() . " == " . $mOrganizations[$j]->getInn();
              $em->remove($mOrganizations[$i]);
              break;
            }
          }
        }

    mogafk, 06 Октября 2014

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

    +156

    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
    for($i = 1; $i < $conf->nb_images + 1; $i++)
    {
    	$ext_name = chr(ord('a')+$i-1);
    	?>
    	<label for="ad_picture<?php echo $i;?>"><?php echo ADSMANAGER_FORM_AD_PICTURE." ".$i; ?></label>
    	<input id="ad_picture<?php echo $i;?>" type="file" name="ad_picture<?php echo $i;?>" />
    	<?php
    	if ($isUpdateMode) {
    		$pic = $mosConfig_absolute_path."/images/$option/projects/".$ad_id.$ext_name."_t.jpg";
    		if ( file_exists( $pic)) {
    			echo "<img src='".$mosConfig_live_site."/images/$option/projects/".$ad_id.$ext_name."_t.jpg' align='top' border='0' alt='image$ad_id' />";
    			echo "<input type='checkbox' name='cb_image$i' value='delete' />".ADSMANAGER_AD_DELETE_IMAGE;
    		}
    	}
    	echo "<br />";
    }

    Угадайте что за CMS :)))

    mishanon, 27 Сентября 2014

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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    if (($parents = $method_id->getParentRepeated()) || ($parents = $method_id->getParentGroups())) {
       if(!is_array($parents)){
            $parents=[$parents];
       }
       foreach($parents as $parent){
        ......
       }
    }

    О_____________О
    и после этого ты говоришь что я перемурил с замыканиями? :D

    limitium, 23 Сентября 2014

    Комментарии (1)
  8. JavaScript / Говнокод #16750

    +156

    1. 1
    jQuery("#myId"+object+".myTr"+array.length-1).append(JSON.parse(object));

    Appending result to grid table

    armengabriel, 22 Сентября 2014

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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if (is_array($checked_groups)) {
        if (in_array($group['manager_group_id'], $checked_groups)) {
            $managers = $managerTelephone->getRingsReport($mode_report['report1']['manager_ids'], $date_begin, $current_date, $group['manager_group_id']);
        }
    } else {
        $managers = $managerTelephone->getRingsReport($mode_report['report1']['manager_ids'], $date_begin, $current_date, $group['manager_group_id']);
    }

    из той же серии

    howard, 19 Сентября 2014

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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    $c=count($arr1);
    for($i=0;$i<$c;$i++){
    	$key=key($arr1);
    	ksort($arr1[$key]);
    	next($arr1);
    }

    foreach() чем-то не угодил

    Khvorostin, 16 Сентября 2014

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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $data = $DB->...
    reset($data);
    while (list(, $row) = each($data)) {
        // do somethig with $row
    }

    Я не понимаю, почему просто не использовать foreach?!

    matasar, 13 Сентября 2014

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