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

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

В номинации:

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

За время:

Ruby / Говнокод #17885

−91

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

def update
    error = false
    if params[:geografic].present? && ((params[:geografic] & IsoCountryCodes.all.map{|c|c.alpha2.downcase}) == params[:geografic])
      current_user.profile.geografic = params[:geografic]
    else
      error = true
      flash[:alert] ||= ''
      flash[:alert] << "Need select countries. "
    end
    begin
      current_user.profile.sectors_of_interests = Sector.find(params[:sectors_of_interests])
    rescue
      error = true
      flash[:alert] ||= ''
      flash[:alert] << 'Can not find sectors. '
    end
    if params[:profession_id].present? && profession = Profession.find(params[:profession_id])
      current_user.profile.update_attribute(:profession_id, params[:profession_id])
    else
      error = true
      flash[:alert] ||= ''
      flash[:alert] << 'Can not find profession. '
    end
    if ['cn', 'gb'].include?(params[:language_iso])
      current_user.profile.update_attribute(:country_iso, params[:language_iso])
    else
      error = true
      flash[:alert] ||= ''
      flash[:alert] << "Need select language. "
    end
    if params[:telephone].present?
      current_user.profile.update_attribute(:phone, params[:telephone])
    else
      error = true
      flash[:alert] ||= ''
      flash[:alert] << "Need input phone number. "
    end

    require_additional_info = (current_user.geografic.blank? || current_user.sectors_of_interests.blank? || current_user.profession.blank? || current_user.phone.blank?)
    if require_additional_info || error
      redirect_to user_path(current_user) and return
    else
      redirect_to root_path
    end
  end

Первый раз я видел такое два года назад, думал, что за это время что-то поменялось в людях, неа. А вы говорите индусы

Ignat_Z, 30 Марта 2015

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

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

+162

1

<td valign="top" class="pt" <?echo "style='width:20%;max-width: 237px;'";?>>

В шаблоне сайта Битрикса.

supercode, 30 Марта 2015

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

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

+158

1
2
3
4
5
6

<?
  include 'const.php';
  PrintUP("404");
  Echo "<img src='https://lh4.googleusercontent.com/-WnF8tpKQA20/AAAAAAAAAAI/AAAAAAAAEEE/IDe9WUshb8c/s120-c/photo.jpg'>Страница не найдена. =(";
  PrintDOWN();
?>

Не спрашивайте, что делают функции PrintUP() и PrintDOWN(). Первая - 600 строк пхп кода со вставками вида echo ' function() { js-код } ' ; , вторая - 140 строк с очень странными манипуляциями с файлами "базы данных".

Arris, 26 Марта 2015

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

Java / Говнокод #17861

+77

1

String value = paramValue; //make new instance

eugene_k, 25 Марта 2015

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

C++ / Говнокод #17811

+54

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

std::list<CCPoint> GameMap::stringToCCPArray(std::string points_string, float _x, float _y)
{
	points_string.push_back(' ');
	std::list<CCPoint> CCPList = list<CCPoint>();
	float x = 0.0f;
	float y = 0.0f;
	unsigned int len = 0;
	bool isCurrentX = true;
	for(std::string::iterator it = points_string.begin(); it != points_string.end(); ++it){
		int q = 0;
		len > 0 ? 
			--len: (isdigit(*it) ? 
				(q = atoi(&(*it)), len = countNumbersInInt(q), 
				isCurrentX ?
					x = atoi(&(*it)): y = atoi(&(*it))) : 
				*it == ',' ? 
					(isCurrentX = false): (CCPList.push_back(ccp(x,(-1 * y))), x = 0.0f, y = 0.0f, isCurrentX = true)) ;
	}

	return CCPList;
}

t0rmund, 18 Марта 2015

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

Objective C / Говнокод #17794

−397

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

[array_static removeAllObjects];
[array_static addObject:@"Files"];
[array addObjectsFromArray:[filemanager contentsOfDirectoryAtPath:DOCUMENTS error:nil]];

for (int l=0; l<array.count; l++) {
	for (int j=0; j<array.count; j++) {
		for (int i=0; i<array.count; i++) {
			if ([[array objectAtIndex:i] rangeOfString:@"."].location != NSNotFound) {
				[array removeObjectAtIndex:i];
			}
		}
	}
}

for (int i = 0; i<array.count; i++) {
	if (![array_static containsObject:[array objectAtIndex:i]]) {
		[array_static addObject:[array objectAtIndex:i]];
	}
}

Дали задание добавить фичу в один старый проект. Открыл проект, сижу и наслаждаюсь видом 8)

OUrsus, 16 Марта 2015

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

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

+156

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

<?php

error_reporting(E_ALL);
require_once('project.php');
$loader = new Twig_Loader_Filesystem('templates');
$twig = new Twig_Environment($loader,
    array(
        'cache' => 'compilation_cache',
        'debug' => true
    )
);
$twig->addExtension(new Twig_Extension_Debug());
$data='';
$data .= summ('summa','zvit');
$payments =getPayments();//tableM(getPayments());


if (array_key_exists('go', $_REQUEST))
    {
    $go=$_REQUEST['go'];
    }
    else
    {
    $go='';
    }
switch ($go) {
    case '':
        echo $twig->render('index.html',array('payments' => $payments)); ); //
        break;
    case 'addData':
        $form = showForm();
        echo "$form";
        break;
    case 'add':
        $data=$_POST['data'];
        $summa=$_POST['summa'];
        addDate($data,$summa);
        redirect('index.php');
        break;
    case 'delete':
        $id = $_GET['id'];
        delete($id);
        redirect('index.php');
        break;
}

vityapro, 13 Марта 2015

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

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

+151

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

<?php
//----------------------------Сначала оригинал который был запосчен--------------------------------
$mas = array("odin", "dva", "tri");

function obm($mas[$i]){
    if $mas[$i] = "odin" $mas[$i] = 1;
        return $mas[$i];
    if $mas[$i] = "dva" $mas[$i] = 2;
        return $mas[$i];    
    if $mas[$i] = "tri" $mas[$i] = 3;
        return $mas[$i];
}
 
  $n = 0;
  $new = array();
    for ($i = 0; $i < 100; $i++) {
      $n++;
        $i = $n * $mas[$i];
    }
//----------------------------- Конец оригинала ---------------------------------------------------------------


//--------------- Потом мне всбрело в голову прокаментить каждую строчку, вот что с этого получилось
$mas = array("odin", "dva", "tri");//Инициализация массива.

//Объявление функции.
function obm($mas[$i]){//Входящий параметр, массив с перемменной? Даже пхп такое не понимает. Ну бум считать что это просто переменная типа $mas_i
    if $mas[$i] = "odin" $mas[$i] = 1;//Сравнение входящей переменной со строкой "odin", зачем было инциализировать массив, если совпадает, то мы массиву который раньше инициализировали присваеваем 1. Но так как это функция, то видимости массива тут нет, то есть основной не иземеняется. Просто создается внутренний и ему присвается значение. При этом у нас нету значения $i. По умолчанию будет $mas[0]. Также сама конструкция if сама по себе не правильная.
        return $mas[$i];//Возвращение значения из массива, присвоенного ранее. Можно было бы в предедущей строчку сделать ретурн. При этом эта строчка будет выполнятся всегда. Так как если бы if был бы правильный, то он сработал бы только на первую операцию. И дальнейший код никогда бы не выполнялся.
    if $mas[$i] = "dva" $mas[$i] = 2; //Ну тут аналогично первым двум строкам.
        return $mas[$i];    
    if $mas[$i] = "tri" $mas[$i] = 3;
        return $mas[$i];
}
//Функция то написана, но блин она же нигде не вызывается!
 
  $n = 0;//Зачем-то инициализируем счетчик.
  $new = array();//Инициализация нового массива.
    for ($i = 0; $i < 100; $i++) {//Стар цикла от 0 до 99 с шагом 1 в переменную $i. Единственная правильная сктрока с осмысленным кодом во всем скрипте.
      $n++;//К счетчику сразу пуляем +1, то есть он у нас является дубликатом переменной $i + 1
        $i = $n * $mas[$i];//Тут я долго думал :-) Допустим первый проход в цикле. $i=0; $n=1; $mas[$i]='odin'. Имеем комбинацию $i = 1 * "odin"; Пхп пытается перевести "odin" в цисловую переменную, получает 1. Итого мы имеем 1 * 1 =1;  Дальше лучше, следующий цикл $i=1; $n=2; $mas[$i]="dva"; $i = 2 * "dva"; То есть получаем в $i значение 2. В следующем цикле аналогично. $i=2;$n=3;$mas[$i]='tri';$i=3*'tri'=3; А вот следующая итерация уже достойна высшей математики :-) $i=3;$n=4; $mas[$i] то есть индекс 3, блин а нет таког, то есть пустая строка, и мы имеем далее $i=4 * "". Пхп интрепретирует пустую как ноль и в переменной $i получаем 0. Что сбрасывает цикл в самое начала. И если бы не растущий счетчик $n, то выхода из цикла бы вообще не было. На следующем шаге получается примерно такое $i=1;$n=5;$mas[$i]='dva';$i=5*"dva"=5. Следующая $i будет 6, потом 0, потом 8, потом 0 и т.д.
    }
    //И в конце концов мы ничего не получаем, так как нигде не использовали массив $new;
//-------------------------------------Конец каментов-----------------------------------------


//-------------Ну и чисто для стеба, решение задачи от меня в три строки---------
$Str = 'odin';

$mas = array("odin", "dva", "tri");
$num = array_search($Str, $mas)+1;
$new = range(0, $num*99, $num);

echo "new='".var_export($new, true)."'\n";
//-------------------------------------DIE() :-)---------------------------------------------------

Нашел на форуме.
Задача. На входе есть строка с содержанием odin, dva или tri. На выходе должен был получится массив со ста числами с шагом указанным в строке, то есть для строки tri, должен быть результат 0, 3, 6, 9, 12 ......294,297
И чел спрашивает, почему у меня код не работает :-) И как его пофиксить.

mrsol, 11 Марта 2015

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

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

+158

1
2
3
4
5
6
7
8
9

<p><!-- Need to do some UI cleanup --><br>
                <script type="text/javascript">// <![CDATA[
                    document.getElementById("badge-images").style.visibility = "hidden";
                    document.getElementById("about-contact").style.visibility = "hidden";
                    document.getElementById("header-menu").style.visibility = "hidden";
                    var hideContactDetails = true;
                    // ]]&gt;</script>
                <br>
            </p>

Недавно один Java-pазработчик не смог разобраться, как пофиксеть баг в WordPress и решил переписать наш корпоративный сайт на Java мотивируя это тем, что будет намного легче поддерживать и развивать новую ситсему. Что из этого получилось видно на наглядном примере наложение стилей на элементы страницы(верстка).

etual, 27 Февраля 2015

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

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