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

    В номинации:
    За время:

  2. PHP / Говнокод #15845

    +150

    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
    <?php
	require_once("inc/mysql.php");
	sleep(3); //Для ajax запроса, потом удалить

	// Проверяем, что к нам идёт Ajax запрос
	if (isset($_SERVER['HTTP_X_REQUESTED_WITH']) && !empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
		$city1 = $_POST['city1'];
	}else{
		exit(); // Заканчиваем работу скрипта, если это не ajax запрос
	}

	if (isset($city1)) {
		$query = "SELECT id_city, id_region, id_country FROM cities WHERE city_name_ru LIKE '$city1'";
		$result = mysqli_query($link, $query);// or trigger_error(mysql_error($link)." ".$query);
		$row = mysqli_fetch_array($result);
		$country = $row['id_country'];
		//echo $country;
		//echo '<br>';
		$region = $row['id_region'];
		//echo $region;
		//echo '<br>';
		$query1 = "SELECT country_name_ru FROM countries WHERE id_country = '$country'";
		$result1 = mysqli_query($link, $query1);// or trigger_error(mysql_error($link)." ".$query);
		$row1 = mysqli_fetch_array($result1);
		echo '<p class=\'country\'>Страна '.$row1['0'].'</p>';
		//echo '<br>';
		$query2 = "SELECT region_name_ru FROM regions WHERE id_region = '$region'";
		$result2 = mysqli_query($link, $query2);// or trigger_error(mysql_error($link)." ".$query);
		$row2 = mysqli_fetch_array($result2);
		echo '<p class=\'reqion\'>Область '.$row2['0'].'</p>';
	}
?>

    Прокомментируйте, пожалуйста, как улучшить этот говнокод.

    eprivalov1, 25 Апреля 2014

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

  3. Куча / Говнокод #15792

    +123

    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
    Type win32gui
Create object window
Set window title = "Window"
Set width window = 299
Set height window = 279
S1 = "y"
S2 = "="
S3 = "x^3+1"
Obj = "Graph"
Create object obj
Set obj graph = "y=x^3+1"
Display obj
Rewrite obj

    Название языка ещё не придумал :P

    Данный пример построит график y=x^3+1.
    Картинка тут - http://coun.at.ua/_sf/1/116_Screenshot1.png

    Mobac, 19 Апреля 2014

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

  4. JavaScript / Говнокод #15720

    +153

    1. 1
    $("#basketName").val($(this).parent().parent().find("span").text());

    Достался мне проект хз кем написанный, и я решил поменять кое-где в верстке <span> на другой элемент (из чисто эстетических побуждений). В результате появился баг в js, собственно, вот причина.

    mixalich7b, 10 Апреля 2014

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

  5. JavaScript / Говнокод #15580

    +151

    1. 1
    var $resetBtn = $("<a class=\"filter-reset-btn resetBtn\"/>");

    Это нормально вообще?

    farit_slv, 26 Марта 2014

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

  6. Pascal / Говнокод #15558

    +86

    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
    function RealToStr(X: Real; Count: Integer): String;//Count - количество цифр после запятой
var S: String;
    N: Integer;
begin
  S:=FloatToStr(X); //после запятой - длинная последовательность цифр
//DecimalSeparator - константа, содержащая истинный разделитель целой и дробной частей числа
 N:=Pos(DecimalSeparator, S); //позиция запятой в строке
//вычисляем длину строки с нужным количеством знаков после запятой:
  if N=0//если в строке нет запятой - это целое число, и
    then N:=Length(S)//тогда просто выводим это число
    else N:=N+Count;//иначе вычисляем длину строки
  Result:=Copy(S, 1, N);//копируем часть строки в результат
end;

    Говно на говне сидит и говном погоняет.
    http://www.delphi-manual.ru/work-with-strings.php

    gost, 23 Марта 2014

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

  7. C++ / Говнокод #15545

    +15

    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
    // file: Filter.h

        inline const CString GetField() const
        {
            if (m_strField.compare(MBL_FT_AUTHID_STRING) == 0)
            {
                return CString(MBL_FT_AUTHID_SERVER_STRING);
            }
            else if (m_strField.compare(MBL_FT_CORRELATION_ID_STRING) == 0)
            {
                return CString(MBL_FT_CORRELATION_ID_SERVER_STRING);
            }
            else if (m_strField.compare(MBL_FT_PLAN_STRING) == 0)
            {
                return CString(MBL_FT_PLAN_SERVER_STRING);
            }
            // ... всего 14 веток в таком духе ...
            else if (m_strField.compare(MBL_FT_DDF_ACCOUNTING_STRING_STRING) == 0)
            {
                return CString(MBL_FT_DDF_ACCOUNTING_STRING_SERVER_STRING);
            }
            return m_strField;
        }

    Автор — русский.

    Lavir_the_Whiolet, 21 Марта 2014

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

  8. C# / Говнокод #15514

    +133

    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
    try
            {
                using (var context = new vbEntities())
                {                   
                    
                    var DBF = new DBF();
                    var dt = DBF.GetAll(@"C:\Users\.........\Documents\BNKSEEK.DBF");

                    int i = 0;
                    progressBar1.Maximum = dt.Rows.Count;

                    foreach (DataRow dr in dt.Rows)
                    {
                        var row = new Bank();
                        row.BIK = dr["RKC"].ToString();
                        row.CityName = dr["NNP"].ToString();
                        row.Address = dr["ADR"].ToString();
                        row.Name = dr["NAMEP"].ToString();
                        row.KS = dr["KSNP"].ToString();
                        row.Phone = dr["TELEF"].ToString();
                        row.OKPO = dr["OKPO"].ToString();  
                        context.Bank.Add(row);
                        progressBar1.Value = i++;
                    }
                    context.SaveChanges();
                }
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        }

    orozov, 18 Марта 2014

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

  9. 1C / Говнокод #14550

    −168

    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
    Если СтрНС <> Неопределено Цикл
	Если СтрНС.Значение <> Справочники.Номенклатура.ПустаяСсылка() тогда
		Тост = ОстаткиПоСкладамУпр(СтрНС.Значение);
		Синхро = Истина;
		Если Тост[0].Остаток <> 0 тогда
			НаличиеНаСкладе = Истина;
		Конецесли;
	КонецЕсли;	
КонецЕсли;

//...

Если НаличиеНаСкладе Тогда
	// другой быдлокод
КонецЕсли;

    Переменная НаличиеНаСкладе объявляется только один раз во всем модуле, в приведенном мною куске.
    Угадай, в каком месте выпадает ошибка?

    rull9ss, 12 Февраля 2014

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

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

    +162

    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
    foreach ($foo as $key1=>$value) {
    if ($foo[$key1][parent] == 0 ) {
        echo '<li><a href="/index.php?cPath='.$key1.'">'.$foo[$key1][name].'</a>';
        foreach ($foo as $key2=>$value) { if ($foo[$key2][parent] == $key1){echo '<ul>'; break; }};
        foreach ($foo as $key2=>$value) {
            if ($foo[$key2][parent] == $key1) {
                echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'">'.$foo[$key2][name].'</a>';
                foreach ($foo as $key3=>$value) { if ($foo[$key3][parent] == $key2){echo '<ul>'; break;} };
                foreach ($foo as $key3=>$value) {
                    if ($foo[$key3][parent] == $key2) {
                        echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'">'.$foo[$key3][name].'</a>';
                        foreach ($foo as $key4=>$value) { if ($foo[$key4][parent] == $key3){echo '<ul>'; break;} };
                        foreach ($foo as $key4=>$value) {
                            if ($foo[$key4][parent] == $key3) {
                                echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'_'.$key4.'">'.$foo[$key4][name].'</a>';
                                foreach ($foo as $key5=>$value) { if ($foo[$key5][parent] == $key4){echo '<ul>'; break;} };
                                foreach ($foo as $key5=>$value) {
                                    if ($foo[$key5][parent] == $key4) {
                                        echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'_'.$key4.'_'.$key5.'">'.$foo[$key5][name].'</a>';
                                        foreach ($foo as $key6=>$value) {if ($foo[$key6][parent] == $key5) {echo '<ul>'; break;} };
                                        foreach ($foo as $key6=>$value) {
                                            if ($foo[$key6][parent] == $key5) {
                                                echo '<li><a href="/index.php?cPath='.$key1.'_'.$key2.'_'.$key3.'_'.$key4.'_'.$key5.'_'.$key6.'">'.$foo[$key6][name].'</a>';
                                            }
                                        }
                                        foreach ($foo as $key6=>$value) { if ($foo[$key6][parent] == $key5){echo '</ul>'; break;} };  
                                        echo '</li>';                                      
                                    }
                                } 
                                foreach ($foo as $key5=>$value) { if ($foo[$key5][parent] == $key4){echo '</ul>'; break;} };
                                echo '</li>';
                            }
                        }
                        foreach ($foo as $key4=>$value) {if ($foo[$key4][parent] == $key3) {echo '</ul>'; break;} };
                        echo '</li>';                          
                    }
                } 
                foreach ($foo as $key3=>$value) { if ($foo[$key3][parent] == $key2){echo '</ul>'; break;} }; 
                echo '</li>';              
            }
        }
        foreach ($foo as $key2=>$value) {if ($foo[$key2][parent] == $key1) {echo '</ul>'; break;} };
        echo '</li>';
    }        
}

    Правильное понимание рекурсии, привитое картинками вконтакте.

    Dotterian, 28 Января 2014

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

  11. C# / Говнокод #14411

    +135

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    FileInfo[] fi = new DirectoryInfo(di.FullName).GetFiles();

var creationList = from file in fi
                             where file.CreationTime.CompareTo(new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.Day)) < 0 //This instance is earlier than value. 
                             group file.CreationTime by DateTime.Parse(file.CreationTime.ToString("yyyy-MM-dd")) into ct
                             select ct.Key;

    что делает этот код?

    taburetka, 24 Января 2014

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