По колено в коде — Говнокод.ру

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

+148

1
2
3
4

$("#select_id :selected").attr("selected", false);
$("#select_id option[value='" + new_value + "']").attr("selected", true);
вместо
$("#select_id").val(new_value);

splinter89, 03 Августа 2012

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

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

+56

01
02
03
04
05
06
07
08
09
10
11

$cur_url=$_SERVER['REQUEST_URI'];
if ($cur_url=='/') { 
	$cur_url=$_SERVER['REQUEST_URI']; 
	if ($cur_url=='/') { // Точно-точно адрес не равен слэшу
		echo '';// После всех проверок можно с уверенностью вывести пустую строку
	}
}
$cur_url2=$_SERVER['REQUEST_URI'];// И ещё разок
if ($cur_url2!='/') { 
	echo '';
}

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

RedMonkey, 03 Августа 2012

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

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

+90

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

public enum TimeUnit {
   NANOSECONDS {
        public long toNanos(long d)   { return d; }
        public long toMicros(long d)  { return d/(C1/C0); }
        public long toMillis(long d)  { return d/(C2/C0); }
        public long toSeconds(long d) { return d/(C3/C0); }
        public long toMinutes(long d) { return d/(C4/C0); }
        public long toHours(long d)   { return d/(C5/C0); }
        public long toDays(long d)    { return d/(C6/C0); }
        public long convert(long d, TimeUnit u) { return u.toNanos(d); }
        int excessNanos(long d, long m) { return (int)(d - (m*C2)); }
    }
....
    public long convert(long sourceDuration, TimeUnit sourceUnit) {
        throw new AbstractMethodError();
    }

 
    public long toNanos(long duration) {
        throw new AbstractMethodError();
    }

  
    public long toMicros(long duration) {
        throw new AbstractMethodError();
    }

    public long toMillis(long duration) {
        throw new AbstractMethodError();
    }

 
    public long toSeconds(long duration) {
        throw new AbstractMethodError();
    }

  
    public long toMinutes(long duration) {
        throw new AbstractMethodError();
    }

  
    public long toHours(long duration) {
        throw new AbstractMethodError();
    }

 
    public long toDays(long duration) {
        throw new AbstractMethodError();
    }

  
    abstract int excessNanos(long d, long m);
}

Но зачем?

3.14159265, 02 Августа 2012

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

ActionScript / Говнокод #11522

−92

1
2
3

public function xor(lhs:Boolean, rhs:Boolean):Boolean {
 return !( lhs && rhs ) && ( lhs || rhs );
}

Из http://as3snippets.blogspot.com/2010/09/logical-xor.html

Как известно, в AS3 есть численный оператор XOR ^, а вот для логических значений ^^ нет. Поэтому ребята придумали такую конструкцию (и ещё вариант return Boolean(int(a) ^ int(b)); в комментах), и только через год какой-то чувак догадался что XOR для логических значений всё-таки есть и называется !=

makc3d, 02 Августа 2012

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

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

+141

1
2
3
4
5
6

foreach (var item in text.Split(' ')) {
	if (!string.IsNullOrEmpty(item)) {
		text = item;
		break;
	}
}

переменная text всегда содержит несколько пробелов и затем идентификатор.

paladin80, 02 Августа 2012

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

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

+20

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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
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)

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

+65

1
2
3
4
5
6
7

function draw_text() {  
        // ....
        /* remove background color */
        imagecolortransparent($im_text, $bg_color);
        return $im_text;
        imagedestroy($im_text);
}

Функция вывода текста CAPTCHA в modx Evolution.

MaXL, 02 Августа 2012

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

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

+56

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

if(isset($_POST['btnsubmitup']))
		{
			for ($i = "0"; Arr::get($_POST, 'id'.$i, ''); $i++) {
			if (Arr::get($_POST, 'up'.$i, '') == '1') {
				$p1=-1;
	//			$uploaddir = '/img/brands/';
				$a = Arr::get($_POST, 'id'.$i, '');
	//			$p1 = Upload::save($_FILES['photo'.$i], $uploaddir.$a.'.jpg', './', 0777);

					$rand=rand(1000000,9999999);
				$uploaddir = '/img_carpets/collection/';
				$uploaddir2 = 'img_carpets/collection/';
				$p1 = Upload::save($_FILES['file1'.$i], $uploaddir.'ID-'.$rand.'-1.jpg', './', 0777);
				$p2 = Upload::save($_FILES['file2'.$i], $uploaddir.'ID-'.$rand.'-2.jpg', './', 0777);
				$p3 = Upload::save($_FILES['file3'.$i], $uploaddir.'ID-'.$rand.'-3.jpg', './', 0777);
				$p4 = Upload::save($_FILES['file4'.$i], $uploaddir.'ID-'.$rand.'-4.jpg', './', 0777);
	//			if ($p1!="0") { $p1=$rand; }
	//			if ($p2!="0") { $p2=$rand; }
	//			if ($p3!="0") { $p3=$rand; }
	//			if ($p4!="0") { $p4=$rand; }
					$im2=Image::factory($uploaddir2.'back.png');

// -> и так далее

Начал разбирать библиотеку (фреймворк скорее - kohanaframework) одного сайта, дабы сделать нормальную админку
Дошел до процедуры сохранения картинок. Я посмотрел, по какому же алгоритму сохраняются картинки (формирование имени файла)
И опупел!
** $rand=rand(1000000,9999999); **
В базе поле для сохранения имени картинки - не уникально.
Т.е., разраб решил поиграть в рулетку, анука генератор чисел выберет еще раз одно и то же число, и перезапишет картинку у товара. ))))
А оператор админки будет чесать репу - тут же работало а тут и нет )

topilnik, 01 Августа 2012

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

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

+15

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

//file systeminfo.cpp
#include "../SystemInfoManager.h"
struct : public SystemInfoManager
{
//some atriburtes
//some methods
   void setSomeparametrs()///
{
/*....*/
}
///ect...

} System;


SystemInfoManager * instance()
{
       return &System;
}

Мое имя затрет история, но мои славные дела будут жить, пытайте меня дальше гниды из гестапо!

Psionic, 01 Августа 2012

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

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