Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +157

   1. 1
   2. 2
   3. 3
   4. 4

   if ( !empty($vendor) and strtolower(substr($record_title, 0, strlen($vendor)+1)) == strtolower("$vendor ")
   ) {
   $record_title = substr($record_title, strlen($vendor)+1);
   }

   Вырзаем из названия товара, компанию производителя.

   0r10n, 06 Октября 2014

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

2. 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)

3. PHP / Говнокод #16800

   +160

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8

   mysql_query('select UNIX_TIMESTAMP(DATE_FORMAT(FROM_UNIXTIME(UNIX_TIMESTAMP(Event.Time) - DATE_FORMAT(Event.Time,"%w") * 60 * 60 * 24), "%Y-%m-%d 00:00:00")) as GroupDate from ...');
   
   // и контрольный:
   $dateFormated = strftime($format, $GroupDate);
   
   // расшифровка (SPOILER):
   // автор пытался найти воскресенье предшествующее дате из Event.Time
   // и вывести его в формате YYYY-MM-DD

   нашел в процессе code review

   pronin, 06 Октября 2014

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

4. PHP / Говнокод #16795

   +157

   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

   // экшен произвольного контроллера, они там все такие
   
   $data['heading_title'] = $this->language->get('heading_title');
   
   $data['text_my_account'] = $this->language->get('text_my_account');
   $data['text_my_orders'] = $this->language->get('text_my_orders');
   $data['text_my_newsletter'] = $this->language->get('text_my_newsletter');
   $data['text_edit'] = $this->language->get('text_edit');
   $data['text_password'] = $this->language->get('text_password');
   
   // еще 20-30 аналогичных строчек
   
   $data['column_left'] = $this->load->controller('common/column_left');
   $data['column_right'] = $this->load->controller('common/column_right');
   $data['content_top'] = $this->load->controller('common/content_top');
   $data['content_bottom'] = $this->load->controller('common/content_bottom');
   $data['footer'] = $this->load->controller('common/footer');
   $data['header'] = $this->load->controller('common/header');
   
   if (file_exists(DIR_TEMPLATE . $this->config->get('config_template') . '/template/account/account.tpl')) {
   	$this->response->setOutput($this->load->view($this->config->get('config_template') . '/template/account/account.tpl', $data));
   } else {
   	$this->response->setOutput($this->load->view('default/template/account/account.tpl', $data));
   }

   теги: Опенкарт, вчера релизнули, модный паттерн MVC-L, тонкие контроллеры, классика, табы для того и сделаны, чтобы ими отбивать
   
   https://github.com/opencart/opencart/tree/master/upload/catalog/controller/account

   Fike, 03 Октября 2014

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

5. PHP / Говнокод #16786

   +155

   1. 1
   2. 2
   3. 3
   4. 4

   $logics = $elementORM->getLogics();
   					if (count($logics > 0)) {
   						foreach($logics as $kLogic => $logic) { // logic
   							foreach($logic["conditions"] as $kCondition => $condition) { // condition

   limitium, 02 Октября 2014

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

6. PHP / Говнокод #16785

   +161

   1. 1
   2. 2
   3. 3
   4. 4

   try {
    Transform_Excel::$zohoFilterEbanyKostyl2 = true;
    $importService->import($id);
    Helpers_Response::json([

   limitium, 02 Октября 2014

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

7. PHP / Говнокод #16783

   +160

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6

   static public function dateDb2Human($dbDate) {
   
   	$date = strptime(explode('.', $dbDate)[0], '%Y-%m-%d %H:%M:%S');
   	return $date['tm_mday'].'.'.$date['tm_mon'].'.'.(1900+$date['tm_year']).' ('.($date['tm_hour'] < 10 ? '0' : '').$date['tm_hour'].':'.($date['tm_min'] < 10 ? '0' : '').$date['tm_min'].')';
   
   } //static public function dateDb2Human($dbDate)

   Это функция для преобразования даты из формата ГГГГ-ММ-ДД ЧЧ:ММ:CC в ДД.ММ.ГГГГ (ЧЧ:ММ)

   alxkolm, 01 Октября 2014

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

8. PHP / Говнокод #16779

   +162

   1. 1

   $cityName_UTF8 = iconv ( 'UTF-8' , 'Windows-1251',   $cityName_Win1251 );

   ShopCMS

   www, 30 Сентября 2014

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

9. 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)

10. PHP / Говнокод #16760

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8

    $q=  mysql_query("select * from BillingParam where Name like ('%  %')");
    
    while($r=mysql_fetch_object($q)){
        $name=str_replace("  ", " ", $r->Name);
        $que="update BillingParam set Name='".$name."' where ID='".$r->ID."'";
        echo $que."<br>";
    
    $qq=mysql_query($que)or die(mysql_error());

    Нужно больше, ещё больше запросов к базе!

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

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

• ← влево
• вправо →

• 571
• …
• 451
• 450
• 449
• 448
• 447
• 446
• 445
• 444
• 443
• …
• 1