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

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

В номинации:

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

За время:

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

+159

1
2
3
4
5
6
7
8

$value = "";
$content = $content;
$val = "\$content->" . $field->name;
$fieldname = $field->name;

eval("if (isset($val)) \$value = \"$val\";");
$value = JText::_($value);
$value = stripslashes($value);

Joomla. Adsmanager

Jetti, 10 Мая 2011

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

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

+161

1
2
3
4
5
6
7

function get_phrase()
{
	global $words;
	global $phrase;
	$phrase = implode(" ", $words);
	if(strlen($phrase) < 4) return;
}

I was amazed to find this!?! The last line of the function rationalized my day...

wyand, 05 Мая 2011

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

Си / Говнокод #6537

+140

1
2
3
4

if ((u[i]!=100500)&&(v[j]==100500))
    v[j]=matr[i+1][j+1].cost-u[i];
if ((u[i]==100500)&&(v[j]!=100500))
    u[i]=matr[i+1][j+1].cost-v[j];

пересчет платежей в ТЗЛП

resettik, 04 Мая 2011

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

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

+161

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

if($age != '') {
			$d_m_y = explode('-', $age); // Разделяем на day(0) month(1) year(2)
		// ГЕНЕРАЦИЯ ФОРМЫ С ДНЕМ РОЖДЕНИЯ
		$vars['bday'] = "<select name=\"b_day\">\n<option>-</option>\n";
		$i = 1;
		while ($i <= 31) {
			if($d_m_y[0] == $i) {
				$sel = ' selected';
			} else {
				$sel = '';
			}
			$vars['bday'] .= '<option'.$sel.'>'.$i."</option>\n";
			$i++;
		}
		$vars['bday'] .= "</select>\n";
		// ГЕНЕРАЦИЯ СПИСКА С МЕСЯЦАМИ
		$i = 1;
		$vars['bmonth'] = "<select name=\"b_month\">\n<option>-</option>\n";
		while ($i <= 12) {
			if($d_m_y[1] == $i) {
				$sel = ' selected';
			} else {
				$sel = '';
			}
			$vars['bmonth'] .= '<option'.$sel.'>'.$i."</option>\n";
			$i++;
		}
		$vars['bmonth'] .= "</select>\n"; // Месяц ДР при ред
		// ГЕНЕРАЦИЯ СПИСКА С ГОДАМИ
		$i = 1940;
		$vars['byear'] = "<select name=\"b_year\"\n<option>-</option>\n"; // Год ДР при ред
		while ($i <= 2003) {
			if($d_m_y[2] == $i) {
				$sel = ' selected';
			} else {
				$sel = '';
			}
			$vars['byear'] .= '<option'.$sel.'>'.$i."</option>\n";
			$i++;
		}
		$vars['byear'] .= "</select>\n";
		$vars['use_age'] = false; // ОТКЛЮЧАЕМ ПОКАЗ ПУСТОГО СПИСКА
		} else {
		$vars['use_age'] = true;
		$vars['bday'] = false;
		$vars['bmonth'] = false;
		$vars['byear'] = false;
		}

Генерация SELECT'ов для выбора даты рождения

Мартин, 02 Мая 2011

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

SQL / Говнокод #6503

−864

1
2

comment on column XXX.status
  is '0 -- вопрос включен, 1-- выкл';

Duke_Raven, 28 Апреля 2011

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

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

+124

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

private void TextBox_KeyDown(object sender, KeyEventArgs e)
        {
            if (e.Key == Key.Space)
            {
                TextBox box = sender as TextBox;

                int start = box.SelectionStart;
                int length = box.SelectionLength;

                if (length == 0)
                {
                    box.Text = box.Text.Insert(start + length, " ");
                    box.SelectionStart = start + 1;
                }
                else
                {
                    string str1 = box.Text.Substring(0, start);
                    string str2 = box.Text.Substring(start + length);

                    box.Text = str1 + " " + str2;
                    box.SelectionStart = start + 1;
                }

                e.Handled = true;
            }
        }

Отборный говнокод выращенный на территории Индии.
Видимо им зарплату действительно за строчки кода платят.

strato, 26 Апреля 2011

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

SQL / Говнокод #6448

−860

1
2
3
4
5
6
7
8

-- Из простого...
select kdet, cexnum, cex, last_value(cex) over(partition by kdet) as final_cex
from xxtpa_rout_aztpa_v;

-- Сделаем сложное...
select t.*, max(max_cex) over(partition by kdet, rout_num) max_cex
from (select t.*, case cex_num when max(cex_num) over(partition by t.kdet, t.rout_num) then t.cex else null end max_cex
from xxtpa_rout_aztpa_v t) t;

Пример того, как НЕ применять KISS-принцип.

dwinner, 22 Апреля 2011

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

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

+158

01
02
03
04
05
06
07
08
09
10

foreach ($firstQueryRes as $row)
			{
				$K = $this->ccnm;
				$U = $this->getFncByCategory($row->category);  // compile material href //
				$R = $row->category;
				$W = $row->date;
				$A = $row->huu_1;
				$materialHref[$row->id_all] = site_url("$K/$U/$R/$W/$A");
				$materialTitle[$row->id_all] = $row->name;
			}

Задание: расшифруйте какое польское плохое слово здесь инкапсулировано.

increazon, 21 Апреля 2011

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

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

+72

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

public class Tm_SP_RP extends Tm_Service implements IObject{
	private static final String m_MsgProfileStore =
		"UPDATE TM_SP_RP SET strMsgProfile=? WHERE nServiceID=?";
	private static final String m_WorkaroundHack =
		"SELECT data_type FROM user_tab_columns WHERE table_name='TM_SP_RP' AND column_name='STRMSGPROFILE'";

	private String strMsgProfile = null;

/* Здесь ещё разные всякие методы */

        private static String getWorkaroundType(Connection conn) throws SQLException
    {
    	PreparedStatement stmt = conn.prepareStatement(m_WorkaroundHack);
    	try
    	{
    		ResultSet rset = stmt.executeQuery();
    		try
    		{
    			if (!rset.next())
    				return "VARCHAR2";
    			return rset.getString(1);
    		}
    		finally
    		{
    			if (rset != null)
    				rset.close();
    		}
    	}
    	finally
    	{
    		if (stmt!=null)
    			stmt.close();
    	}
    }

	public void storeMsgProfile(Connection conn) throws SQLException
	{
		String w_around = getWorkaroundType(conn); 
		
    	PreparedStatement stmt = conn.prepareStatement(m_MsgProfileStore);
    	try
    	{
    		if (w_around.equalsIgnoreCase("VARCHAR2") ||
    			w_around.equalsIgnoreCase("VARCHAR"))
    		{
    			if (strMsgProfile == null)
    				stmt.setNull(1, Types.NULL);
    			else
    				stmt.setString(1, strMsgProfile);
    		}
    		else
    		{
    			byte []data = (strMsgProfile == null) ?
	    				new byte[0] : strMsgProfile.getBytes();
	    		stmt.setBytes(1, data);
    		}
    		
    		stmt.setLong(2, this.getId());
    		
    		stmt.executeUpdate();
    	}
    	finally
    	{
    		if (stmt!=null)
    			stmt.close();
    	}
	}
}

Комбинация из багованных JDBC-дров Oracle и работающего с ним Hibernate (чтоб он сдох) иногда заставляет рождать вот такие хитрые workaround-хаки. Несколько баз, в одной тип поля - LONG, в другой - VARCHAR2.

SadKo, 21 Апреля 2011

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

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