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

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

В номинации:

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

За время:

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

−167

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

-- =============================================
--          Create procedure basic template
-- Предназначена - проставить номер идентичного счета по данной АК
-- =============================================

CREATE PROCEDURE Num_List_of_Flights
	
AS
-- Результаты группировки - Временная таблица
SELECT DISTINCT List_of_Flights, Count(List_of_Flights) AS Kol_fo_Flights
INTO Vrem_Table_Flights
FROM PPLS2BILLS_OBUF a
WHERE Flag_Bill=0 and  List_of_Flights Is not Null
GROUP BY List_of_Flights
-- Копирую перечень сфомированных счетов в Steck_Table_Flights
INSERT INTO Steck_Table_Flights 
SELECT    
  List_of_Flights = S.List_of_Flights,
  Kol_fo_Flights  = S.Kol_fo_Flights 
FROM dbo.Vrem_Table_Flights AS S

-- Удаляю временную таблицу
DROP TABLE Vrem_Table_Flights

-- Создаю временную таблицу группировки по счетам (сколько одинаковых счетов )
SELECT DISTINCT List_of_Flights, Count(List_of_Flights) AS Kol_Flights
INTO Vrem_Steck_Table_Flights
FROM Steck_Table_Flights
GROUP BY List_of_Flights

-- Корректирую номера Flights  и Сбрасываю флаг Flag_Bill в 1 (как обработано)
UPDATE a
SET Num_Flights  = b.Kol_Flights,
    Flag_Bill    = 1
FROM PPLS2BILLS_OBUF a INNER JOIN Vrem_Steck_Table_Flights b
              ON a.List_of_Flights = b.List_of_Flights
WHERE Flag_Bill = 0
-- Удаляю временную таблицу Vrem_Steck_Table_Flights
DROP TABLE  Vrem_Steck_Table_Flights

GO

-- EXEC Num_List_of_Flights

никому нельзя верить, а особенно собственным группировкам

bahamot, 29 Декабря 2013

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

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

+153

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

<?php
namespace Custom\Entity;

class slideshow{
    const IBLOCK_ID = SLIDER_ON_INDEX_IBLOCK_ID;

    static function getLabel($type){
        $label = "";
        switch($type){
            case NEW_OF_DAY_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker news_of_day" src="/static/img/labels/news_of_day_l.png"><img class="sticker news_of_day_r" src="/static/img/labels/news_of_day_r.png">';
                break;
            case HOT_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker hot infoBlock rsABlock rsNoDrag" src="/static/img/labels/hot.png">';
                break;
            case SLUH_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker infoBlock infoBlockLeftBlack rsABlock rsNoDrag" src="/static/img/labels/gossip.png">';
                break;
            case ITEM_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker thing_of_week infoBlock rsABlock rsNoDrag" src="/static/img/labels/thing_of_week.png">';
                break;
            case POPULAR_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker most_popular infoBlock rsABlock rsNoDrag" src="/static/img/labels/most_popular.png">';
                break;
            case CHOOSE_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker editors_choice infoBlock" src="/static/img/labels/editors_choice.png">';
                break;
            case BEST_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker the_best infoBlock rsABlock rsNoDrag" src="/static/img/labels/the_best.png">';
                break;
            case NEW_ENUM_ID_PROPERTY_MAIN_ARTICLES_IBLOCK_ID:
                $label = '<img class="sticker new infoBlock rsABlock rsNoDrag" src="/static/img/labels/new.png">';
                break;
        }
        return $label;
    }
}
?>

И еще один замечательный пример ChinaCode от "грамотных разработчиков на битриксе, которые знают неймспейсы". Особенно порадовал css-класс rsNoDrag, хотя по большому счету надо было бы данный класс назвать "NoDrugs" и взять на вооружение данным "пейсателям".

sturm, 26 Декабря 2013

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

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

+162

01
02
03
04
05
06
07
08
09
10

$.validator.addMethod("phoneNumber",
        function(value, element) {
             var letterNumber = /^[a-zA-Z]+$/;
            if (value.match(letterNumber)) {
                return false;
            }

            var phoneString = value.replace(/\D/g, '');
            return phoneString.length == 0 || phoneString.length >= 11;
        });

данная кучка "валидировала" телефонный номер

Tessen, 13 Декабря 2013

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

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

+148

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

<?php

/* Почему статические методы — говно */

abstract class BaseNewsPeer {
…
	/**
	 * Method to select one object from the DB.
	 *
	 * @param      Criteria $criteria object used to create the SELECT statement.
	 * @param      PropelPDO $con
	 * @return     News
	 * @throws     PropelException Any exceptions caught during processing will be
	 *		 rethrown wrapped into a PropelException.
	 */
	public static function doSelectOne(Criteria $criteria, PropelPDO $con = null)
	{
		$critcopy = clone $criteria;
		$critcopy->setLimit(1);
		$objects = NewsPeer::doSelect($critcopy, $con);
		if ($objects) {
			return $objects[0];
		}
		return null;
	}
…

class NewsPeer extends BaseNewsPeer
{
	…
}

kix, 13 Декабря 2013

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

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

+6

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

class Log
{

	ReverseStruct<std::list<std::string> > _logList;
	semafor semafor;
	Log()
	{
		std::list<std::string> right , left;
		std::shared_ptr<std::list<std::string> > ptrRight=std::shared_ptr<std::list<std::string> >(*right);
		std::shared_ptr<std::list<std::string> > ptrLeft=std::shared_ptr<std::list<std::string> >(*left);
		_logList=TReverseStruct<std::list<std::string> >(ptrRight,ptrLeft);
	}


	void writeMessage(std::string message, Level level ){_logList.getWriteStorage.push_back( currentTime+level+messge); semafor.signal();}
	void body()
	{
		try
		{
			semafor.wait();
			_logList.revers();
			for (std::list<std::string>::iterator i =_logList.getReadStorage->begin(); i != _logList.getReadStorage->end(); ++i)
			{
			fixedBufferString<7000, '\0'> stringLog;
			stringLog.append(*i);
			FileDevice flashInternal(FLASHINTERNAL);
			const DriveErrors::E WriteResult = flashInternal.writeFile("log.iso", (byte*)stringLog.content(), stringLog.length());
			}
		}
		catch()
		{}
	}
}

laMer007, 11 Декабря 2013

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

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

−115

1
2
3
4
5

- (NSUInteger)application:(UIApplication *)application supportedInterfaceOrientationsForWindow:(UIWindow *)window {

return (NSUInteger)[application supportedInterfaceOrientationsForWindow:window] | (1<<UIInterfaceOrientationPortrait);

}

Самый укуренный способ определить флаги UIInterfaceOrientation, который я видел.
http://stackoverflow.com/questions/14533521/game-center-causing-uiapplicationinvalidinterfaceorientation ?answertab=votes#tab-top

krypt, 06 Декабря 2013

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

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

+124

1
2
3
4
5
6
7

S consume (S) (ref S data, Regex ! ( Unqual!(typeof(S.init[0])) ) rg)
{
    writeln (typeid(Unqual!(typeof(S.init[0]))));

    auto m = match(data, rg);
    return m.hit;
}

LispGovno, 03 Декабря 2013

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

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

+154

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

class Icon extends IconBuilder
{
	public static function create($type, $white = false)
	{
		$class = __CLASS__;
		return new $class($type, $white);
	}

	public function getType()
	{
		return $this->_type;
	}

	public function getWhite()
	{
		return $this->_white;
	}
}

Код выдернут из Bootstrap.PHP http://allking.ru/bootstrap.php/
Особое внимание уделить функции create и константе __CLASS__
Вопрос: почему не обойтись просто self? В чем магия?

З.Ы.: Пока писал, нашел в доках, что "волшебная" константа содержит так же пространство имен.
Извиняюсь, что не узнал об этом раньше.
Но я все же наложу сюда, с вопросом, даже учитывая всю эту информацию, почему мы не можем обойтись способами, исключающими использование константы?
(все равно получается говнокод, хоть и работающий)

oooZinka, 18 Ноября 2013

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

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

+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

function date_to_text($date){
	

	
	$now=time();
	if ($s=strtotime($date)) //the function returns false is $date is not a string (as already in numeric unix date format)
		$date = $s;
	$d=round(($now-$date)); //s

	//echo $d;
	if($d>60){
		$d=$d/60;//m	
		if($d>60){
			$d=$d/60;//h
			if($d>24){
				$d=$d/24;//d
				if($d>7){
					$d=$d/7;//w						
					if($d>(52/12)){
						$d=$d/52*12;//m
						if($d>12){
							$d=$d/12;//y
							$d=floor($d);			
							$str=$d.' year'.(($d>1)?'s':'');								
						}else{
							$d=floor($d);			
							$str=$d.' month'.(($d>1)?'s':'');		
						}
					}else{
						$d=floor($d);			
						$str=$d.' wk'.($d>1?'s':'');									
					}
				}else{					
					$d=floor($d);
					$str=$d.' day'.($d>1?'s':'');											
				}
			}else{			
				$d=floor($d);			
				$str=$d.' hr'.(($d>1)?'s':'');		
			}
		}else{
			$d=floor($d);			
			$str=$d.' min'.(($d>1)?'s':'');			
		}
	}elseif ($d>0) {
		$d=floor($d);			
		$str=$d.' sec'.(($d>1)?'s':'');		
	}
	if ($d>0){
		$str.=' ago';
	} else {
		$str = '';
	}
	return $str;
}

неопiсуемо

rombik, 22 Октября 2013

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

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

+140

1

Никому не двигаться! Мы реквизировали, реквизировали этот сайт, смекаешь?

PragramistOtBoga, 22 Октября 2013

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

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