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

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

+145

1
2
3
4
5
6
7
8

Connection::Connection(const Options& opts): impl_(new Impl)
{
    impl_->isInitialized = false;
    impl_->options = std::move(*opts.impl_);
   
    if (!options().lazyInit)
        conn();
}

Вся соль в строке 4: из объекта opts, переданного по константной ссылке, подло выжимают содержимое. Видимо, никто не пробовал создать два коннекта из одного объекта опций.

roman-kashitsyn, 28 Октября 2014

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

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

+158

1
2
3

$element['yandex'] = 14;
unset($element['yandex']);
$element['yandex'] = 60;

Разраб упоролся в каштан :)

sautchenkov, 28 Октября 2014

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

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)

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