Лучший говнокод за всё время

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

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

C# / Говнокод #10319

+98

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50

public class XmlParserClass
    {
        private XmlDocument xmlDoc;
        public static List<Attribute> listOfAttributes;
        public static Dictionary<int, AttributeValue> listOfAttributeValues;
        public static Dictionary<int, File> listOfFiles;        
        public static List<Product> listOfProducts;
        public List<FileCategory> listOfFileCategories;
        public List<Distributor> listOfDistributors;
        public static bool cryptedFiles = false;

        private string xmlfileLocation; 
        private string XmlfileLocation  
        {
            get 
            { 
                if (xmlfileLocation == null)
                    xmlfileLocation = System.IO.Path.GetDirectoryName(Program.GetValue("XMLFileLocation", ""));
                return xmlfileLocation;
            }
        }
	       
	public XmlParserClass(XmlDocument xmlDocument)
        {
            xmlDoc = xmlDocument;
            listOfAttributes = new List<Attribute>();
            listOfAttributeValues = new Dictionary<int, AttributeValue>();
            listOfProducts = new List<Product>();
            listOfFileCategories = new List<FileCategory>();
            listOfFiles = new Dictionary<int, File>();
            listOfDistributors = new List<Distributor>();

			.......
			
               //Читаем а с XML ....
                foreach (XmlNode xmlValue in attr.SelectSingleNode("Values").ChildNodes)
                {
                   ................
                }

                listOfAttributes.Add(a);

			
			.........
			//Читаем fc с XML....
			listOfFileCategories.Add(fc);
			
			.........
		}
	}

Создали экземпляр чтобы прочитать данные с XML. Куда бы нам их положить? Конечно в статические поля!

dreamer, 18 Мая 2012

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

1C / Говнокод #10297

−102

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87

Процедура СортировкаСписка(Список) 
	
	Перем лСредний;  
	
	лЛевый= СоздатьОбъект("СписокЗначений");
	лПравый = СоздатьОбъект("СписокЗначений");
	
	Если Список.РазмерСписка()>1 Тогда  
		
		лРазмерСписка = Список.РазмерСписка();
		лПозицияСреднего = Цел(лРазмерСписка/2);
		лСреднее = Список.ПолучитьЗначение(лПозицияСреднего);
		
		Для й = 1 По лРазмерСписка Цикл
			
			Если й<>лПозицияСреднего Тогда
				
				лПроверяемоеЗначение =  Список.ПолучитьЗначение(й);
				
				Если лПроверяемоеЗначение<лСреднее Тогда
					
					лЛевый.ДобавитьЗначение(лПроверяемоеЗначение);
					
				ИначеЕсли лПроверяемоеЗначение>лСреднее Тогда
					
					лПравый.ДобавитьЗначение(лПроверяемоеЗначение);
					
				КонецЕсли;
				
			КонецЕсли;
			
		КонецЦикла;
		
		СортировкаСписка(лЛевый);
		СортировкаСписка(лПравый);
		
		Список.УдалитьВсе();
		
		Для й = 1 По лЛевый.РазмерСписка() Цикл
			Список.ДобавитьЗначение(лЛевый.ПолучитьЗначение(й));
		КонецЦикла; 
		
		Список.ДобавитьЗначение(лСреднее); 
		
		Для й = 1 По лПравый.РазмерСписка() Цикл
			Список.ДобавитьЗначение(лПравый.ПолучитьЗначение(й));
		КонецЦикла;
		
	КонецЕсли;
	
	
КонецПроцедуры

Процедура Выполнить()
	
	ОтсортированныйСписок = СоздатьОбъект("СписокЗначений");
	
	СписокЗначений = СоздатьОбъект("СписокЗначений");   
	
	СписокЗначений.ДобавитьЗначение(2);
	СписокЗначений.ДобавитьЗначение(7);
	СписокЗначений.ДобавитьЗначение(15);
	СписокЗначений.ДобавитьЗначение(16);
	СписокЗначений.ДобавитьЗначение(20);
	СписокЗначений.ДобавитьЗначение(21);
	СписокЗначений.ДобавитьЗначение(3);
	СписокЗначений.ДобавитьЗначение(5);
	СписокЗначений.ДобавитьЗначение(4);
	СписокЗначений.ДобавитьЗначение(14);
	СписокЗначений.ДобавитьЗначение(12);
	СписокЗначений.ДобавитьЗначение(13);
	СписокЗначений.ДобавитьЗначение(11);
	СписокЗначений.ДобавитьЗначение(19);
	СписокЗначений.ДобавитьЗначение(1);
	СписокЗначений.ДобавитьЗначение(8);
	СписокЗначений.ДобавитьЗначение(6);
	СписокЗначений.ДобавитьЗначение(17);
	СписокЗначений.ДобавитьЗначение(9);
	СписокЗначений.ДобавитьЗначение(10);
	СписокЗначений.ДобавитьЗначение(18);
	
	СортировкаСписка(СписокЗначений);
	
	ВыбранноеЗначение = 0;
	СписокЗначений.ВыбратьЗначение(ВыбранноеЗначение);
	
КонецПроцедуры

informatikmarazmatik, 16 Мая 2012

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

1C / Говнокод #10292

−115

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

Тз = СоздатьОбъект("ТаблицаЗначений");
	Тз.НоваяКолонка("Число");
	Для й=1 По 100 Цикл
		Тз.НоваяСтрока();
		Тз.Число = й;
	КонецЦикла;
	КоличествоСтрокТз = Тз.КоличествоСтрок();
	Для й=-КоличествоСтрокТз По -1 Цикл
		ЗначениеИзТз = Тз.ПолучитьЗначение(-й,"Число");
		ПроверяемоеЗначение = ЗначениеИзТз/2-Цел(ЗначениеИзТз/2);
		Если ПроверяемоеЗначение <> 0 Тогда
			Тз.УдалитьСтроку(-й);
		КонецЕсли;
	КонецЦикла;
	Тз.ВыбратьСтроку();

Четные числа до 100

informatikmarazmatik, 16 Мая 2012

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

PHP / Говнокод #10279

+58

01
02
03
04
05
06
07
08
09
10
11
12

if ($who == 'student') $answerArray['student_id'] = $this->people->id;
if ($who == 'teacher') $answerArray['teacher_id'] = $this->people->id;
if ($who == 'student') $answerArray['student_text'] = $post_array['message'];
if ($who == 'teacher') $answerArray['teacher_text'] = $post_array['message'];
if ($who == 'student') $answerArray['student_date'] = date('Y-m-d H:i:s');
if ($who == 'teacher') $answerArray['teacher_date'] = date('Y-m-d H:i:s');
if ($who == 'student' && $files) $answerArray['student_file'] = $files[0]['name'];
if ($who == 'teacher' && $files) $answerArray['teacher_file'] = $files[0]['name'];
if ($who == 'student' && $files) $answerArray['student_file_orig_name'] = $files[0]['orig_name'];
if ($who == 'teacher' && $files) $answerArray['teacher_file_orig_name'] = $files[0]['orig_name'];
if ($who == 'student') $answerArray['status'] = 0;
if ($who == 'teacher') $answerArray['status'] = 1;

Обнаружил в корпоративном коде. )

MikleSmart, 15 Мая 2012

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

PHP / Говнокод #10264

+78

01
02
03
04
05
06
07
08
09
10
11
12
13
14

class uagent_info
{
   //standardized values for true and false.
   var $true = 1;
   var $false = 0;

   function XXX() {
      if (...) {
      	return $this->true;
      } else {
      	return $this->false;
      }
   }
}

http://code.google.com/p/mobileesp/source/browse/PHP/mdetect.php

neanton, 14 Мая 2012

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

JavaScript / Говнокод #10260

+157

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30

$(document).click(function (e) {
    if ($(e.target).parents().filter('#idAddItTopPopUpMenu:visible').length != 1) {
        $('#idAddItTopPopUpMenu').hide();
    }
    if ($(e.target).parents().filter('#idAddItProductDetailPopUpMenu:visible').length != 1) {
        $('#idAddItProductDetailPopUpMenu').hide();
    }
    if ($(e.target).parents().filter('#idPampItProductDetailPopUpMenu:visible').length != 1) {
        $('#idPampItProductDetailPopUpMenu').hide();
    }
    if ($(e.target).parents().filter('#idPrivacyProductPopUpMenu:visible').length != 1) {
        $('#idPrivacyProductPopUpMenu').hide();
    }
    if ($(e.target).parents().filter('#idSocialNetworkMenu:visible').length != 1) {
        $('#idSocialNetworkMenu').hide();
    }
    if ($(e.target).parents().filter('#idPampItTopPopUpMenu:visible').length != 1) {
        $('#idPampItTopPopUpMenu').hide();
    }
. . .
    if ($(e.target).parents().filter('div[id=idAreaLastMessage]:visible').length != 1 && $(e.target).parents().filter('div[id=idbtnMessages]').length != 1) {
        $('div[id=idListLastMessage]').hide();
    }
    if ($(e.target).parents().filter('div[id=idAreaLastAlert]:visible').length != 1 && $(e.target).parents().filter('div[id=idbtnAlert]').length != 1) {
        $('div[id=idListLastAlert]').hide();
    }
    if ($(e.target).parents().filter('div[id=idAreaNearestCalendar]:visible').length != 1 && $(e.target).parents().filter('div[id=idbtnNearestCalendar]').length != 1) {
        $('div[id=idListNearestCalendar]').hide();
    }
});

нет слов o_O

DarkThinker, 14 Мая 2012

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

PHP / Говнокод #10254

+68

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

<span class="realty_price"><?php echo $ad['price']?>
                    <?php if(! empty($ad['addinfo']['unit'])): ?>
                      <?php echo $ad['addinfo']['unit']['value'] ?>
                      <?php
                      unset($ad['addinfo']['unit']);
                      else:
                      ?>
                      <?php
					  // Говнокод
					  if($ad['addinfo']['type_service']['value'] == 'аренда'):
					  ?>
                      руб.мес.
                      <?php else: ?>
                      тыс.руб.
                      <?php endif; ?>
                    <?php endif; ?>
                  </span>

Хороший человек, искренний...

nethak, 12 Мая 2012

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

C# / Говнокод #10244

+96

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

List<LocalBusiness> sortedLocalBusinesses = new List<LocalBusiness>();
foreach (LocalBusiness business in foundLocalBusinesses)
{
	if (!business.Logo.UrlSmall.Equals("/images/local_business_default_logo.png"))
	{
		sortedLocalBusinesses.Add(business);
	}
}
foreach (LocalBusiness business in foundLocalBusinesses)
{
	if (business.Logo.UrlSmall.Equals("/images/local_business_default_logo.png"))
	{
		sortedLocalBusinesses.Add(business);
	}
}

Сортировка - первыми должны быть объекты с не дефолтными картинками.

mastermindCsharp, 11 Мая 2012

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

PHP / Говнокод #10224

+74

1
2
3
4
5
6
7

if($this->getRequest()->isPost()) {
			$month = $this->getRequest()->getPost('month', null);
			$year  = $this->getRequest()->getPost('year', null);
		} else {
			$month = $this->getRequest()->getParam('month', null);
			$year  = $this->getRequest()->getParam('year', null);
		}

Индусы, такие индусы.

anycolor, 09 Мая 2012

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

Куча / Говнокод #10207

+123

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

<table border="0" id="table" width="800" align="center">
<tr><td colspan="5" align="center">Ваш заказ:</td></tr>

<? $query=mysql_query("Select * from `$name`");
for ($c=0; $c<mysql_num_rows($query);$c++){
?><?
$result=mysql_fetch_array($query);
$a=$result['menu_id'];
$query1=mysql_query("SELECT * FROM menu WHERE id=$a");
$result1=mysql_fetch_array($query1);
?>

<tr><td width="50"><? echo $c+'1'; ?></td><td width="50" align="center"><img src="img/tira.png" style="height:20px; width:20px;"/></td><td><? echo $result1['name'];?></td><td><a href="#">-</a><input type="text" min="1" value="<? echo $result['count'];?>" max="40" size="2" /><a href="#" onclick="up()">+</a></td><td>удалить</td></tr>
<script language="javascript" type="application/javascript">
var a=parseInt("<? echo $result['count'];?>");
function up(){
var b = a + 1;
document.write(b);
}
</script>
<?
}
?>

"Создаю корзину покупателя! Формирую таблицу выбранных пользователем товаров и его кол-во в цикле! добавил функцию на OnClick с выводом значение в ячейке! при нажатии выводит только значение последнего товара из цикла." Слов нет.

zloirock, 06 Мая 2012

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

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