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

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

  2. Java / Говнокод #13054

    +115

    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
    /**
 * Converts the Accpac fields to names that do not
 * require Sherlock Holmes to decipher.
 * 
 * @param arcus Accpac customer object
 */
public Store(final ARCus arcus) {
	name = trim(arcus.getIdcust());
	description = trim(arcus.getNamecust());
	addressLine1 = trim(arcus.getTextstre1());
	addressLine2 = trim(arcus.getTextstre2());
	addressLine3 = trim(arcus.getTextstre3());
	addressLine4 = trim(arcus.getTextstre4());
	suburb = trim(arcus.getNamecity());
	state = trim(arcus.getCodestte());
	postalCode = trim(arcus.getCodepstl());
	country = trim(arcus.getCodectry());
	contactName = trim(arcus.getNamectac());
	phone1 = trim(arcus.getTextphon1());
	phone2 = trim(arcus.getTextphon2());
	email = trim(arcus.getEmail2());
	department = arcus.getAudtorg();
}

    someone, 27 Мая 2013

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

  3. C++ / Говнокод #12523

    +14

    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
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    #define MAX_MONSTER_ID			600
#define MAX_ITEM_FOR_MONSTER	40

	for (int j=0; j < 1000; j++)
	{
		AllMobItemsDrop[j].MMap			= 0;
		AllMobItemsDrop[j].MMinLvl		= 0;
		AllMobItemsDrop[j].MMaxLvl		= 0;
		AllMobItemsDrop[j].IDropRate	= 0;
		AllMobItemsDrop[j].IGroup		= 0;
		AllMobItemsDrop[j].IIndex		= 0;
		AllMobItemsDrop[j].IMinLvl		= 0;
		AllMobItemsDrop[j].IMaxLvl		= 0;
		AllMobItemsDrop[j].ILvlRate		= 0;
		AllMobItemsDrop[j].IMinOpt		= 0;
		AllMobItemsDrop[j].IMaxOpt		= 0;
		AllMobItemsDrop[j].IOptRate		= 0;
		AllMobItemsDrop[j].ISkill		= 0;
		AllMobItemsDrop[j].ISkillRate	= 0;
		AllMobItemsDrop[j].ILuck		= 0;
		AllMobItemsDrop[j].ILuckRate	= 0;
		AllMobItemsDrop[j].IMinExc		= 0;
		AllMobItemsDrop[j].IMaxExc		= 0;
		AllMobItemsDrop[j].IExcRate		= 0;
		AllMobItemsDrop[j].IAnc			= 0;
		AllMobItemsDrop[j].IAncRate		= 0;
	}

	AllMobArrayMaxItem = 0;

	for (int i=0; i < MAX_MONSTER_ID; i++)
	{
		for (int j=0; j < MAX_ITEM_FOR_MONSTER; j++)
		{
			ItemsDrop[i][j].MMap		= 0;
			ItemsDrop[i][j].MMinLvl		= 0;
			ItemsDrop[i][j].MMaxLvl		= 0;
			ItemsDrop[i][j].IDropRate	= 0;
			ItemsDrop[i][j].IGroup		= 0;
			ItemsDrop[i][j].IIndex		= 0;
			ItemsDrop[i][j].IMinLvl		= 0;
			ItemsDrop[i][j].IMaxLvl		= 0;
			ItemsDrop[i][j].ILvlRate	= 0;
			ItemsDrop[i][j].IMinOpt		= 0;
			ItemsDrop[i][j].IMaxOpt		= 0;
			ItemsDrop[i][j].IOptRate	= 0;
			ItemsDrop[i][j].ISkill		= 0;
			ItemsDrop[i][j].ISkillRate	= 0;
			ItemsDrop[i][j].ILuck		= 0;
			ItemsDrop[i][j].ILuckRate	= 0;
			ItemsDrop[i][j].IMinExc		= 0;
			ItemsDrop[i][j].IMaxExc		= 0;
			ItemsDrop[i][j].IExcRate	= 0;
			ItemsDrop[i][j].IAnc		= 0;
			ItemsDrop[i][j].IAncRate	= 0;
		}
		ArrayMaxItem[i] = 0;
	}

    Рабочий код с одного сервера. Код инициализации класа с заполнением структуры. А ведь это можно было уместить в:
    memset(&AllMobItemsDrop, 0, sizeof(AllMobItemsDrop));
    memset(&ItemsDrop, 0, sizeofe(ItemsDrop));

    Killbrum, 31 Января 2013

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

  4. Pascal / Говнокод #12097

    +81

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    Если ты имеешь в виду, что появляются полосы прокрутки и 
уродуют окно или stringgrid, то просто оставляй "запас" в
несколько пикселей, т.к. на разных системах может быть разная
ширина краев формы и прочих элементов.
А что, я еще и код писать должен чтобы какие-то экзотичные
темы винды поддерживать? Нет уж, я лучше несколько
пикселей оставлю запаса.

    Вся суть дельфинов
    http://www.gamedev.ru/code/forum/?id=168899

    Topacer, 11 Ноября 2012

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

  5. PHP / Говнокод #12091

    +54

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    $res = mysql_query("SELECT * FROM `users`");
while ($arr = mysql_fetch_assoc($res))
{
	$res1 = mysql_query("SELECT * FROM `cabinet_users` WHERE `nick`='$arr[user]'");
	$arr1 = mysql_fetch_assoc($res1);
        ...
}

    JOIN'ы для слабаков.

    UnnamedUser, 10 Ноября 2012

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

  6. Haskell / Говнокод #11976

    −82

    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
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    {-# LANGUAGE ExistentialQuantification,
             DeriveDataTypeable,
             PatternSignatures #-}

import Data.Typeable
import Control.Concurrent
import Control.Concurrent.MVar
import Control.Concurrent.Chan

-- Core data types

data Message = forall t . Typeable t => Message t | StopMessage
    deriving Typeable

data Handler = forall t . Typeable t => Handler (t -> IO ())


-- Worker thread

data Worker = Worker (Chan Message) (MVar ())

workerThread :: [Handler] -> Chan Message -> MVar () -> IO ()
workerThread handlers chan finish = loop where
    loop = do
        message <- readChan chan
        case message of
            StopMessage -> putMVar finish ()
            Message val -> do
                foldr (tryHandler val) (putStrLn "Unhandled message") handlers
                loop
    tryHandler val (Handler h) rest = maybe rest h (cast val)

startWorker :: [Handler] -> IO Worker
startWorker handlers = do
    chan <- newChan
    finish <- newEmptyMVar
    forkIO (workerThread handlers chan finish)
    return $ Worker chan finish

send :: Typeable m => Worker -> m -> IO ()
send (Worker chan _) message = do
    writeChan chan $ Message message

stopWorker :: Worker -> IO ()
stopWorker (Worker chan finish) = do
    writeChan chan $ StopMessage
    takeMVar finish


-- Some tests

data Test = Test Bool String deriving Typeable

intHandler :: Int -> IO ()
intHandler val = putStrLn $ "Int: " ++ show (val * 2)

strHandler :: String -> IO ()
strHandler val = putStrLn $ "String: " ++ reverse val

testHandler :: Test -> IO ()
testHandler (Test b s) = putStrLn $ "Test: " ++ show b ++ " " ++ show s

main = do
    w <- startWorker [
        Handler intHandler,
        Handler (\(val::Char) -> putStrLn $ "Char: " ++ show val),
        Handler strHandler,
        Handler testHandler]
    send w (5::Int)
    send w False
    send w 'a'
    send w "foo"
    send w (Test True "bar")
    stopWorker w
    putStrLn "Finished!"

    Вот такая вот портянка была написана под влиянием дискуссии с HaskellGovno http://govnokod.ru/11968, и недавней его просьбой рассказать об общении потоков в хаскеле.

    Код запускает тред, в который можно передавать различные сообщения (ограничение только одно - тип сообщения должен быть инстансом тайпкласса Typeable). В треде исполняются указанные хендлеры, каждый из которых ловит свой тип сообщений.

    P.S. Для неимеющих хаскеля, но желающих посмотреть на работу кода: http://ideone.com/OMVamc.

    bormand, 22 Октября 2012

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

  7. Куча / Говнокод #11968

    +128

    1. 1
    f = expr `catches` [Handler (\ (ex :: ArithException) -> handleArith ex), Handler (\ (ex :: IOException)    -> handleIO    ex)]

    Собственно это пример как в хаскеле перехватывать исключения из одного выражения expr.
    Выглядит конечно отстойно. Очень многословно и судя по всему метода проще нет. И понятное дело, в чистых ленивых функциях это не работает. Ловить исключения можно только в грязных IO-функциях, тк сигнатура такова: catches :: IO a -> [Handler a] -> IO a

    HaskellGovno, 21 Октября 2012

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

  8. Си / Говнокод #11670

    +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
    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
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    int closeestsockptr( SOCKET* pSocket )
{
	char chBuf[ 100 ];
	fd_set rdevents, exevents;
	struct timeval tv;
	int iRetVal = 1;
	int optlen;
	SOCKET s = INVALID_SOCKET;
	BOOL bLinger;
	LINGER lng;
	if( NULL == pSocket || INVALID_SOCKET == *pSocket )
		return SOCKET_ERROR;
	s = *pSocket;
	optlen = sizeof( bLinger );
	iRetVal = getsockopt( s, SOL_SOCKET, SO_DONTLINGER, ( char* )&bLinger, &optlen );
	if( 0 == iRetVal && TRUE == bLinger ) //linger is off
	{
		lng.l_onoff = 1; //set linger on
		lng.l_linger = 1; //set linger timeout to 1 second
		iRetVal = setsockopt( s, SOL_SOCKET, SO_LINGER, ( char* )&lng, sizeof( lng ) );
		if( 0 == iRetVal )
		{
			if( 0 == shutdown( s, SD_SEND ) )
			{
				tv.tv_sec = 3; //seconds
				tv.tv_usec = 0; //microseconds
				while( 1 )
				{
					FD_ZERO( &rdevents );
					FD_ZERO( &exevents );
					addsock2fd( &rdevents, NULL, &exevents, s ); //FD_SET( s, &rdevents ), FD_SET( s, &exevents );
					iRetVal = select( 1, &rdevents, NULL, &exevents, &tv );
					if( SOCKET_ERROR != iRetVal && 0 != iRetVal && FD_ISSET ( s, &rdevents ) && !FD_ISSET ( s, &exevents ) )
					{
						iRetVal = recv( s, chBuf, sizeof( chBuf ) / sizeof( *chBuf ), 0 );
						if( iRetVal > 0 ) //Some data received
							continue;
						if( 0 == iRetVal ) //Receive FD_CLOSE
							break;
						else //SOCKET_ERROR returned
							break;
					}
					else if( 0 == iRetVal ) //exceeded the timeout
					{
						WSASetLastError( WSAETIMEDOUT );
						break;
					}
					else //SOCKET_ERROR returned
						break;
				}
			}
		}
	}

	if( NULL == pSocket || INVALID_SOCKET == *pSocket )
		return SOCKET_ERROR;
	iRetVal = closesocket( *pSocket );
	*pSocket = INVALID_SOCKET;
	return iRetVal;
}

    Пуресишник наваял

    sokol, 29 Августа 2012

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

  9. Java / Говнокод #11640

    +60

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    class Test {
    private int m_field;

    int getField() {
        return m_field;
    }

    void setField(int field) {
        m_field = field;
    }
}

    Дискасс.

    bormand, 23 Августа 2012

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

  10. C++ / Говнокод #11520

    +20

    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
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    96. 96
    class TCP1251ToUTF16StringConverter
{
public:
	static WideChar convert(Char Source)
	{
		WideChar Result=static_cast<unsigned char>(Source);
		const WideChar Russian_YO=static_cast<unsigned char>('Ё');
		const WideChar Russian_yo=static_cast<unsigned char>('ё');
		const WideChar RussianWide_YO=L'Ё';
		const WideChar RussianWide_yo=L'ё';
		const WideChar Russian_A=static_cast<unsigned char>('А');
		const WideChar RussianWide_A=L'А';
		const unsigned int AmountOfSymbols=0x40;
		if(Result==Russian_YO)
			return RussianWide_YO;
		if(Result==Russian_yo)
			return RussianWide_yo;
		if(Result>=Russian_A&&Result<Russian_A+AmountOfSymbols)
			return (Result-Russian_A+RussianWide_A);
		return Result;
	};
	
	static void convert(PwideChar UTF16StringDestination, PChar CP1251WinEngRusStringSource, const size_t TextLength)
	{
		assert(CP1251WinEngRusStringSource!=NULL);
		size_t i=0;
		for(;;)
		{
			if(i>=TextLength)
				break;
			assert(i<TextLength);
			Char CP1251SourceChar=CP1251WinEngRusStringSource[i];
			if(CP1251SourceChar=='\0')
				break;
			UTF16StringDestination[i]=convert(CP1251SourceChar);
			++i;			
		};
		UTF16StringDestination[i]=L'\0';
		assert(i<=TextLength);
	};
	
	static std::wstring convert(const std::string& CP1251WinEngRusStringSource)
	{
		assert(CP1251WinEngRusStringSource.c_str()!=NULL);
		std::wstring UTF16StringDestination;
		std::transform(CP1251WinEngRusStringSource.begin(), CP1251WinEngRusStringSource.end(), std::inserter(UTF16StringDestination, UTF16StringDestination.end())/*std::back_inserter(UTF16StringDestination)*//*VC 6.0 compatible*/, makePointerToFunction(convertChar));
		return UTF16StringDestination;
	};
	
private:
	static WideChar convertChar(char Source)
	{
		return convert(Source);
	};
};

template<const size_t MaxAmountOfChar>
class TCP1251ToUTF16StringInPlaceConverter
{
public:
	TCP1251ToUTF16StringInPlaceConverter(PChar CP1251WinEngRusStringSource)
	{
		STATIC_ASSERT(MaxAmountOfChar>0, MaxAmountOfChar_must_be_above_zero);
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, MaxAmountOfChar);
	};
	
	TCP1251ToUTF16StringInPlaceConverter(PChar CP1251WinEngRusStringSource, const size_t TextLength)
	{
		STATIC_ASSERT(MaxAmountOfChar>0, MaxAmountOfChar_must_be_above_zero);
		assert(TextLength<=MaxAmountOfChar);
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, TextLength);
	};
	
	void convert(PChar CP1251WinEngRusStringSource)
	{
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, MaxAmountOfChar);
	};
	
	void convert(PChar CP1251WinEngRusStringSource, const size_t TextLength)
	{
		assert(TextLength<=MaxAmountOfChar);
		TCP1251ToUTF16StringConverter::convert(&(_buffer[0]), CP1251WinEngRusStringSource, TextLength);
	};
	
	PWideChar Get(void) const
	{
		return &(_buffer[0]);
	};
	
	PwideChar Get(void)
	{
		return &(_buffer[0]);
	};
	
	wideChar _buffer[MaxAmountOfChar+1];
};

    USB, 02 Августа 2012

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

  11. Си / Говнокод #11347

    +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
    33. 33
    int * lPtr=item;
int * lPtrEnd = item + count;

for (lPtr = item ; lPtr < lPtrEnd && *lPtr<fid ; lPtr+=4);
if( lPtr >= lPtrEnd ) lPtr=lPtrEnd-1; // last element
if(*lPtr != fid)
{
        if(--lPtr >=item)
        {   
                if(*lPtr!= fid)
                {   
                        if(--lPtr >=item)
                        {   
                                if(*lPtr!= fid)
                                {   
                                        if(--lPtr >=item)
                                        {   
                                                if(*lPtr!= fid)
                                                {   
                                                        break; // value not found
                                                }   
                                        } else break;
                                }   
                        }  else break;
                }   
        } else break;
}


if( lPtr > item)
{
        while ( *(lPtr-1) == fid && lPtr > item) lPtr--; // first ocurence of fid
}

    Быстрый алгоритм поиска

    wtfcode, 03 Июля 2012

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