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

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 / Говнокод #14104

+159

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

$st=$_GET['st'];

// проверяем допустимый номер категории
if ((((((($st != "1")&($st != "2"))&($st != "3"))&($st != "4"))&($st != "5"))&($st != "6")&($st != "7")&($st != "8")&($st != "0"))) {$st="0";};

// Гениальная реализация отношения многие-ко-многим - по флагу на каждый номер категории, проверяем какой флаг установлен
$zapros = "
	SELECT
		product.cod,
		product.`name`,
		sector.s0,
		sector.s1,
		sector.s2,
		sector.s3,
		sector.s4,
		sector.s5,
		sector.s6,
		sector.s7,
		sector.s8
	FROM
		product ,
		sector
	WHERE
		product.cod = sector.cod AND sector.s".$st."=1
	ORDER BY
		product.id ASC
	";

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

mkramer, 16 Ноября 2013

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

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

+158

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

$(document).ready(function () {
		$("#FollowingChoicesWebpart-ProjectsAndCommunities").hide();
		$("#FollowingChoicesWebpart-Sites").hide();
		$("#FollowingChoicesWebpart-News").show();
	});
	$("#FChoices_tab1").click(function () {
		$("#FollowingChoicesWebpart-ProjectsAndCommunities").hide();
		$("#FollowingChoicesWebpart-Sites").hide();
		$("#FollowingChoicesWebpart-News").show();
		$("#FChoices_tab1").addClass("active-button_FChoices");
		$("#FChoices_tab2").removeClass("active-button_FChoices");
		$("#FChoices_tab3").removeClass("active-button_FChoices");
	});
	$("#FChoices_tab2").click(function () {
		$("#FollowingChoicesWebpart-ProjectsAndCommunities").hide();
		$("#FollowingChoicesWebpart-Sites").show();
		$("#FollowingChoicesWebpart-News").hide();
		$("#FChoices_tab1").removeClass("active-button_FChoices");
		$("#FChoices_tab2").addClass("active-button_FChoices");
		$("#FChoices_tab3").removeClass("active-button_FChoices");
	});
	$("#FChoices_tab3").click(function () {
		$("#FollowingChoicesWebpart-ProjectsAndCommunities").show();
		$("#FollowingChoicesWebpart-Sites").hide();
		$("#FollowingChoicesWebpart-News").hide();
		$("#FChoices_tab1").removeClass("active-button_FChoices");
		$("#FChoices_tab2").removeClass("active-button_FChoices");
		$("#FChoices_tab3").addClass("active-button_FChoices");
	});
	$("#FChoices_tab3").click(function () {
		$("#FollowingChoicesWebpart-ProjectsAndCommunities").hide();
		$("#FollowingChoicesWebpart-Sites").hide();
		$("#FollowingChoicesWebpart-News").hide();
		$("#FChoices_tab1").removeClass("active-button_FChoices");
		$("#FChoices_tab2").removeClass("active-button_FChoices");
		$("#FChoices_tab3").addClass("active-button_FChoices");
	});

Голландский джаваскриптик для реализации функционала Tabs. Видимо, написано под кумаром)))

RomashkaPro, 15 Ноября 2013

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

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

+68

01
02
03
04
05
06
07
08
09
10
11

// setting simple fields that couldn't be null
        if (firstOperDay != null) {
            dto.setFirstClosedDay(firstOperDay);
        } else {
            dto.setFirstClosedDay(null);
        }
        if (lastOperDay != null) {
            dto.setLastClosedDay(lastOperDay);
        } else {
            dto.setLastClosedDay(null);
        }

ingenuus, 15 Ноября 2013

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

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

+70

1
2
3
4
5
6
7
8
9

private boolean isVincodeDisabled() {
        if (Long.valueOf(PaymentValidationStatus.vin_code_incorrect.getValue()).equals(paymentDTO.getErrorCode())) {
            return false;
        } else if (paymentDTO.getVincode() == null) {
            return true;
        } else {
            return false;
        }
    }

ingenuus, 15 Ноября 2013

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

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

+115

1
2
3

#define TEN    10
#define TEEN   100
#define TEEEN  1000

glook, 15 Ноября 2013

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

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

+137

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

int memcmp (const void* v1, const void* v2, size_t n)
{
   uint32_t *s1;
   uint32_t *s2;
   size_t  i;

   s1 = (uint32_t*) v1;
   s2 = (uint32_t*) v2;

   for (i = 0; i < n; i++) {
                if (*s1 != *s2) {
                        return *(const uint32_t *)s1 >
                               *(const uint32_t *)s2 ? 1 : -1;
                }
                s1++;
                s2++;
        }
   return 0;
}

Реализация memcmp в библиотеке одной малоизвестной "учебной" ОС реального времени. Учебной в том смысле, что по этой системе разве что доклады, презентации и статьи делали, где-то реально она вряд ли использовалась.
Для интересующихся http://pok.safety-critical.net/

WGH, 14 Ноября 2013

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

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

−122

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

- (void)_syncThread:(NSDictionary*)args
{
	NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
	ERFeedStatusDB *feedStatusDB;
	NSError *error = nil;
	
	[self retain];
	
	// Open FeedStatusDB
	feedStatusDB = [[ERFeedStatusDB alloc] init];
	if (![feedStatusDB openWithPath:[args objectForKey:ERFeedStatusDBSyncFilePathKey] error:&error])
		goto error;
		
	// Perform the sync
	NSAutoreleasePool *pool2 = [[NSAutoreleasePool alloc] init];	// workaround for SQLITE_BUSY issue during -[feedStatusDB close] call
																	// (autoreleased objects are sticking around that keep the DB locked) 
	[self performSyncOnBackgroundThreadWithParams:[args objectForKey:ERFeedStatusDBSyncParamsDictKey] feedStatusDB:feedStatusDB];
	[pool2 release];
	
	[feedStatusDB close];
	[feedStatusDB release];
	feedStatusDB = nil;
	
	[self _setExecuting:NO];
	[_executingCondition lock];
	[_executingCondition signal];
	[_executingCondition unlock];
	
	[self release];
	[pool release];
	return;
	
error:
	[self notifyDelegateOfFailureWithError:error];

	[feedStatusDB close];
	[feedStatusDB release];
	feedStatusDB = nil;

	[self _setExecuting:NO];
	[_executingCondition lock];
	[_executingCondition signal];
	[_executingCondition unlock];
	
	[self release];
	[pool release];
}

Авторелиз пулы. Goto. [self retain] и [self release]. NSLock. Все это в кастомной реализации NSOperation.

Код выносит прогулки по базе в бэкграунд. Другого способа, видать, не нашлось.

Headless, 13 Ноября 2013

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

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

+80

1

new

TarasB, 13 Ноября 2013

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

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

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