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

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

В номинации:

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

За время:

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

+157

1

var put = $(this).attr("href") + "&sort=" + $(this).parent().parent().children().eq(0).children().val();

It's called inception.

vocal_hydra, 31 Июля 2014

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

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

−151

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

def get_team_choices_with_index
    school_ids = params[:school_id].split(",")
    @index = params[:index]

    @teams_array = Array.new
    school_ids.each do |s|
      next if s.eql? "0"
      School.find(s).teams.each do |t|
        @teams_array << t
      end
    end

    @teams_array.flatten!
    @teams_array
  end

А должно быть что-то вроде
@teams_array = Team.where(school_id: school_ids)

dj_Andreus, 14 Июля 2014

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

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

+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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65

while ($this->tariffs_model->getCarsCheckedByTariff($user_id, $tariff_info[0]['id']) > $tariff_info[0]['cars_count'])
{
	$cars = $this->tariffs_model->getCarsChecked($tariff_info[0]['id'], $user_id);
	if ($cars)
	{
		//удаляем одну объяву
		$this->tariffs_model->delCarChecked($cars[0]);
		$this->sms_model->removeSmsByCarId($cars[0]);
	}
}

/*-=-=-=-=-=-=-=-=-=-=-=-=- реализация ф-ций -=-=-=-=-=-=-=-=-=-=-=-=-*/

/**
 * Получаем кол-во реально отмеченых объявлений
 * @param int $user_id
 * @param int $tariff_id
 */
function getCarsCheckedByTariff($user_id, $tariff_id)
{
	$query = "SELECT COUNT(*) as count FROM tariffs_cars_checked WHERE car_id IN
		(SELECT id FROM a2_cars WHERE user = ? AND expire_date >= ?) AND tariff_id = ?";
	$result = $this->db->query($query, array($user_id, MYSQL_CURDATE, $tariff_id));

	//echo $this->db->last_query();

	if ($result && $result->num_rows() == 1)
	{
		return $result->row()->count;
	}
	else
	{
		return null;
	}
}

/**
 * Список ID отмеченных объявлений по тарифу пользователя
 *
 * @param Int $tariff_id
 * @return Array[]
 * @author КОЕ-КТО 21.12.2009 12:35
 * @uses Controller::Profile
 */
function getCarsChecked($tariff_id, $user_id)
{
	$this->db->select('tariffs_cars_checked.car_id')->from('tariffs_cars_checked')
	->join('a2_cars', 'a2_cars.id = tariffs_cars_checked.car_id', 'inner')
	->where(array('tariffs_cars_checked.tariff_id' => intval($tariff_id), 'a2_cars.user' => intval($user_id)));

	$result = $this->db->get();
	if ($result && $result->num_rows() > 0)
	{
		$cars = array();
		foreach ($result->result_array() as $row)
		{
			$cars[] = $row['car_id'];
		}
		return $cars;
	}
	else
	{
		return null;
	}
}

Удаляем объявления скопом!

smail01, 10 Июля 2014

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

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

+163

01
02
03
04
05
06
07
08
09
10

с сохранением авторского форматирования:

<script type="text/javascript">
                $(function() {
                   
                    if (<%= (orderedPartnerOffer != null).ToString().ToLower() %>) {
                        $('p','#idProjectStepContent<%= stepGuid %>').text('Ваша заявка успешно отправлена.');
         }

</script>

В джаваскрипте на онлоад проверить, есть ли значение у серверной переменной, чтобы с помощью жЭкУэээри обновить текст элемента. Ояебу. Кажется мы наняли сатану.

adoconnection, 09 Июля 2014

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

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

+156

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

<?
$res = CIBlockSection::GetList(array(), array('IBLOCK_ID'=>'3', 'ACTIVE'=>'Y'));
$res1 = CIBlockSection::GetList(array(), array('IBLOCK_ID'=>'3', 'ACTIVE'=>'Y'));
?>
<table class="tableObjects">
    <tr>
        <?while($v = $res->GetNext()){
            ?><td><a href="/projects/list.php?SECTION_ID=<?=$v['ID']?>">
                <?=CFile::ShowImage(CFile::GetPath($v['PICTURE']), 180, 160)?>
            </a></td><?
        }?>
    </tr>
    <tr>
        <?while($v = $res1->GetNext()){
            ?><td><b><a href="/projects/list.php?SECTION_ID=<?=$v['ID']?>"><?=$v['NAME']?></a></b></td><?
        }?>
    </tr>
</table>

Как минимум 2 раза обращаться к базе данных, чтобы разбить данные на 2 строки! Гениально!

dimagyzeev, 09 Июля 2014

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

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

+154

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

<script type="text/javascript">
    if (parent && parent != window && (browser.msie || browser.opera || browser.mozilla || browser.chrome || browser.safari || browser.iphone)) {
      document.getElementsByTagName('body')[0].innerHTML = '';
    } else {
      domReady();
      updateMoney(0);
gSearch.init();
if (window.qArr && qArr[5]) qArr[5] = [5, "by item", "", "goods", 0x00000100];
if (browser.iphone || browser.ipad || browser.ipod) {
  setStyle(bodyNode, {webkitTextSizeAdjust: 'none'});
}
if (0) {
  hide('support_link_td');
}
var ts_input = ge('ts_input'), oldFF = browser.mozilla && parseInt(browser.version) < 8;
if (browser.mozilla && !oldFF) {
  setStyle(ts_input, {padding: (vk.rtl ? '3px 20px 6px 40px' : '3px 41px 6px 20px')});
}
placeholderSetup(ts_input, {back: false, reload: true});
if (browser.opera || browser.msie || browser.mozilla) {
  setStyle(ts_input, {padding: (vk.rtl ? '4px 20px 5px 40px' : '4px 41px 5px 20px')});
} else if (browser.chrome || browser.safari) {
  setStyle(ts_input, {padding: (vk.rtl ? '4px 21px 5px 40px' : '4px 40px 5px 21px')});
}

...

window.tsHintsEnabled = 1;;setTimeout(function() {  try {    var elem = ge(String.fromCharCode.apply(null,[118,107,98,95,97,100,118]));    if (elem) {      ajax.post('/al_profile.php', {act: 'pro', section: 1, hash: '3f8471898e6269d08b'}, {onDone: function() {}, onFail: function() { return true; }});    }  } catch (e) {}}, 5000);
__adsLoaded = false;

...

cur.lang = extend(cur.lang || {}, {
  wall_link_label: 'Link',
  wall_my_friends_only: 'Only your friends can see this post',

Перестала грузиться понятно какая соцсеточка, от нечего делать опять полез в сорцы html. Вперемешку инициализация поиска, битмаски и какой-то qArr[5], уконтрапупивание всего body по непонятному условию, стилизация, if(0), табуляция в соответствии с каким-то особым стандартом, локализация, вторая половина которой зачем-то подтягивается отдельным файлом. Про установку всех переменных прямо в window явно и неявно молчу.

Fike, 08 Июля 2014

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

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

+158

1
2
3

var CLICK = "click";
var CHANGE = "change";
periodTo.addEventListener(CLICK,function(){…});

Когда я спросил «Зачем?!», автор ответил, что так можно же, при необходимости, легко внести изменение типа «var CLICK = "mousedown";»

torbasow, 27 Июня 2014

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

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

+133

1
2

/* All the message are high priority message  */
message_id = message_id | LOW_PRIORITY_MESSAGE_MASK;

Все сообщения с высоким приоритетом. *Да-да, конечно*

sermp, 27 Июня 2014

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

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

+154

1
2
3
4
5
6
7

if($product_info['meta_description']) {
				$this->document->setDescription($product_info['meta_description']);
			} elseif ($pos == true) {
				$this->document->setDescription('Сертифицированный товар (уценка): '.mb_convert_case($product_info['name'], MB_CASE_LOWER, "UTF-8").' за '.(int)$product_info['price'].' грн. Бесплатные доставка по Киеву и Украине от интернет магазина');
			} else {
				$this->document->setDescription('Сертифицированный товар: '.mb_convert_case($product_info['name'], MB_CASE_LOWER, "UTF-8").' за '.(int)$product_info['price'].' грн. Бесплатные доставка по Киеву и Украине от интернет магазина');
			}

Дали сайт на OpenCart на доработку , вот такой код я нашел в контроллере

heetap, 25 Июня 2014

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

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