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

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

+158

1
2
3
4

for($i = 0; $i < $db->max; $i++) {
	if($i==1) continue;
	.....
}

Почему бы и нет

UncaughtException, 28 Октября 2014

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

Python / Говнокод #16954

−99

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

@login_required
def datadelivery_stats_report(request, campaign_id):

    try:
        start_date = extract_date_to_default_timezone(request, 'start_date')
    except ValidationError:
        return HttpResponseServerError("The %s parameter is invalid." % 'start_date')
    except AttributeError:
        return HttpResponseServerError("The %s parameter is invalid." % 'start_date')
    except KeyError:
        return HttpResponseServerError("The %s parameter is missing." % 'start_date')

    try:
        end_date = extract_date_to_default_timezone(request, 'end_date')
    except ValidationError:
        return HttpResponseServerError("The %s parameter is invalid." % 'end_date')
    except AttributeError:
        return HttpResponseServerError("The %s parameter is invalid." % 'end_date')
    except KeyError:
        return HttpResponseServerError("The %s parameter is missing." % 'end_date')

Джанга такая джанга... Почему же нельзя выбросить ошибку валидации? 404 можно...

kyzi007, 28 Октября 2014

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

1C / Говнокод #16952

−124

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

Результат = Запрос.Выполнить();
Если Результат.Выгрузить().Количество() <> 0 Тогда
	Выборка = Результат.Выбрать();
	Пока Выборка.Следующий() Цикл
		... тут некий код... и внутри контрольный в голову...
		Если ТЗ.Количество() <> 0 Тогда
			Для Каждого Строка Из ТЗ Цикл
				...
			КонецЦикла;	
		КонецЕсли;	
		...
	КонецЦикла;
КонецЕсли;

Видимо это такая "оптимизация", другого объяснения подобным проверкам (это не единичный случай) не нашел.

Twilight_Sun, 27 Октября 2014

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

Pascal / Говнокод #16949

+88

1
2
3
4
5
6
7
8
9

with dm.Query do
    begin
      if Active then Close;
      //вот так оно работало до ввода параметров
      //dm.QStat.SQL.Text := 'select * from T_ORDER where (date between ''' + IntToStr(ys) + '-' + IntToStr(ms) + '-' + IntToStr(ds) + ''' and ''' + IntToStr(yf) + '-' + IntToStr(mf) + '-' + IntToStr(df) + ''')' ;

      SQL.Text := 'select * from T_ORDER where (date between :StartDate  and :FinishDate)';
      Parameters.ParamByName('StartDate').Value := IntToStr(ys) + '-' + IntToStr(ms) + '-' + IntToStr(ds);
      Parameters.ParamByName('FinishDate').Value := IntToStr(yf) + '-' + IntToStr(mf) + '-' + IntToStr(df);

в комменте - код, ниже - его вменяемая замена

knowy, 27 Октября 2014

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

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

+159

1
2
3
4
5

if (AllowOperation(array(1,2,3,4,5,9,10,29), array(), array(), array(), -1, -1)) { 
        } else {
        	//скрывать определённые поля для дизайнера
			$tplCommon->assign("DISPLAY_NONE", "style='display: none;'");  
        }

В AllowOperation указываются идентификаторы групп пользователей, что забиты в БД.
Таких AllowOperation(array(1,2,3,4,5,9,10,29). .), но с разными идентификаторами во всей системе больше 1000.

pavel87, 27 Октября 2014

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

Pascal / Говнокод #16947

+94

1
2

fisher(Sender);//fisher
macd(Sender);//macd

Буква "С" - Содержательные комментарии.

Практически единственные комменты в программе на 7000 строк.

hardreset, 27 Октября 2014

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

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

+129

1
2
3
4

# Превратим входную строку в вектор, и отсортируем  
used_letter <- sort(strsplit(s, "")[[1]])
# Находим самый большой используемый индекс(у нас уже есть гарантии, что в строке остались только допустимые символы)
big_letter <- used_letter[length(used_letter)]

Оригинальный способ поиска максимального элемента в массиве

alineman, 27 Октября 2014

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

Python / Говнокод #16945

−106

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

import pygame

window = pygame.display.set_mode((600, 600))
pygame.display.set_caption("GAME")
screen = pygame.Surface((600, 600))

class Sprite:
    def __init__(self, xpos, ypos, filename):
        self.x=xpos
        self.y=ypos
        self.bitmap=pygame.image.load(filename)
        self.bitmap.set_colorkey((0,0,0))
    def render(self):
        screen.blit(self.bitmap, (self.x,self.y))

laser = Sprite(0, 0, 'laser.png')

done = True
while done:
    window.fill((50,50,50))
    for e in pygame.event.get():
        if e.type == pygame.QUIT:
            done = False

    screen.fill((50,50,50))

    laser.render()
    window.blit(screen, (0,0))
    pygame.display.flip()

картинка на черном фоне

archiwise, 27 Октября 2014

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

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

+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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86

// Активация заявки (мало ли, пригодиться =) )
if($_GET['o'] == 'activate') {

	if($zayavka = $GLOBALS['db']->GetRow("SELECT * FROM `".DB_PREFIX."_pan_zayavki` WHERE `id` = '".intval($_GET['zid'])."'")) {
		
		// Если заявка на админку
		if($zayavka['type'] !== 'unban') {
		
			$type = $GLOBALS['db']->GetRow("SELECT
													*
												FROM
													`".DB_PREFIX."_pan_types`
												WHERE
													`srvgid`= '".$zayavka['type']."'");
			
			$period = time() + ($zayavka['period'] * 86400);
				
			$aid = $userbank->AddAdmin($zayavka['nick'], $zayavka['steamid'], $zayavka['password'], $zayavka['e-mail'], '-1', '', $type['srvgid'], '', '', '', $period);

			// Удаляем левые админки
			$GLOBALS['db']->Execute("DELETE FROM `".DB_PREFIX."_admins_servers_groups` WHERE `admin_id` = '".$aid."'");
			
			// И задаём ему новые админки на выбранных серверах
			$zservers = explode(',',$zayavka['server_id']);
			
			foreach($zservers AS $s)
			{
				if($s == '') { continue; }
				$pre = $GLOBALS['db']->Prepare("INSERT INTO ".DB_PREFIX."_admins_servers_groups (admin_id,group_id,srv_group_id,server_id) VALUES (?,?,?,?)");
				$GLOBALS['db']->Execute($pre,array($aid,
												   0,
												   -1,
												   (int)$s));
			}
			
			if(isset($GLOBALS['config']['config.enableadminrehashing']) && $GLOBALS['config']['config.enableadminrehashing'] == 1)
			{
			// Отправляем на сервера sm_rehash
			echo '<script>ShowRehashBox("'.$zayavka['server_id'].'", "Добавление админа", "<b>Пожалуйста, не закрывайте и не обновляйте страницу</b><br />Обновление админов на серверах", "green", "index.php?p=admin&c=zayavki");TableReload();</script>';
			}
			else 
			{
				echo '<script>ShowBox("Админ добавлен", "Админ сервера успешно добавлен", "green", "index.php?p=admin&c=zayavki");TableReload();</script>';
			}
		}
		
		// Если же покупает разбан
		else 
		{
			$unban = true;
			
			// Ищем бан
			if($ban = $GLOBALS['db'] -> GetRow("SELECT bid FROM `".DB_PREFIX."_bans` WHERE `bid` = '".$zayavka['password']."' AND `RemoveType` = 'U'"))
			{
				echo '<script>ShowBox("Игрок разбанен", "Игрок '.$ban['name'].' разбанен ранее", "green", "index.php?p=admin&c=zayavki");TableReload();</script>';
			}
			else
			{
				// Разбаниваем игрока
				$GLOBALS['db'] -> Execute("UPDATE `".DB_PREFIX."_bans` SET `RemovedBy` = '0', `RemoveType` = 'U' WHERE `bid` = '".$zayavka['password']."'");

				echo '<script>ShowBox("Игрок разбанен", "Игрок успешно разбанен", "green", "index.php?p=admin&c=zayavki");TableReload();</script>';
			}
		}
		
		// Обновляем статус заявки на payd
		$GLOBALS['db']->Execute("UPDATE `".DB_PREFIX."_pan_zayavki` SET `payd` = '1' WHERE `id` = '".intval($_GET['zid'])."'");
	}
}

// Логфайл
if(isset($_GET['logfile'])) {
	if(is_file(INCLUDES_PATH . '/robo1/logs/log-ID-'.intval($_GET['logfile']).'.txt')) {
		$filename = file_get_contents(INCLUDES_PATH . '/robo1/logs/log-ID-'.intval($_GET['logfile']).'.txt');
		echo nl2br($filename);
		echo '<br />';
		echo '<div style="width:100%; text-align:center;">
			<input type="button" class="btn ok" onClick="document.location.href=\'index.php?p=admin&c=zayavki\'" value="Назад" />
		</div>';
		PageDie();
	}
	else
	{
		echo '<script>ShowBox("Ошибка", "Лог файл не найден", "red", "");</script>';
	}
}

Equality, 27 Октября 2014

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

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

+131

1
2
3
4
5
6
7
8
9

Dictionary<string, string> Users = new Dictionary<string, string>();
//somecode
foreach (string key in Users.Keys)
{
      string str = Users[key];
      m_LookUpProjectSupervisorFilter.Text = str;
      m_LookUpProjectSupervisorFilter.Value = key;
      break;
}

Такое часто в рабочем проекте.

r1nk, 26 Октября 2014