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

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

0

1
2
3
4
5
6
7
8

QMutableListIterator<AISObjectQML *> it(m_visibleList);
if (it.hasNext()) {
    AISObjectQML *v = it.next();
    if (v) { // здесь должен быть while, но, признаться, на это уже посрать
        delete v;
    }
}
m_visibleList.clear();

Есть люди, которые придумали концепцию итераторов для реализации generic алгоритмов. А есть люди, которые любят итераторы, но не любят generic алгоритмы. Ведь с ними код не внушает ужаса и рвотных позывов

Antervis, 15 Марта 2016

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

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

+3

1
2
3
4
5
6
7
8
9

JSON объект:
{ 
   "Tickets":[100999,100998,100997],
   "Price":[5,5,5],
   "TicketCount":3
}

Вместо положенных:
{"Tickets":[ ["100999","5"],["100998","5"],["100997","5"]]}

Разработчик высылает JSON объект где описывает массив, вместо того чтобы просто выслать массив)))))

mypaxa, 15 Марта 2016

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

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

+3

1
2

var listModuleId = dbContext.MODULES.Where(n => n.IFDELETE == 0 && n.DEVICETYPE == 1).Select(m => m.IDMODULE).ToList();
var ListMeasure = dbContext.MEASURE.Where(n => listModuleId.Contains(n.IDMODULES)).GroupBy(m => m.IDMODULES).ToList();

Если в MODULES несколько сотен датчиков, то вторая строка разворачивается в sql-запрос на 22 килобайта, а если в MEASURE 13 тысяч записей, то этот запрос выполняется полторы минуты.
Молчу уже о том, что listModuleId нигде, кроме второй строки, не используется.

avialaynen, 15 Марта 2016

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

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

0

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

static void Main(string[] args)
{
	try
	{
		watcher.Path = TargetPath;
		watcher.Created += Watcher_Created;
		watcher.Renamed += Watcher_Created;
		watcher.EnableRaisingEvents = true;
		logger.Info("Сервис запущен.");
		System.Threading.Thread.Sleep(System.Threading.Timeout.Infinite);
	}
	catch(Exception ex)
	{
		logger.Error(ex, "Global error", null);
		Main(null);
	}
}

хуита, 15 Марта 2016

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

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

−1

1
2
3
4
5
6

private static async Task<DataTable> GetWaitingBetsHandlingHistory(int customerID, int betID, DateTime date)
       {
           // copied from: CBets.GetWaitingBetsHandlingHistory
           string connectionString = AppConfig.MachineAppSettings("connectionString");
           SqlConnection conn = new SqlConnection(connectionString);
           ...

Copy-paste-driven development confession

pro687, 14 Марта 2016

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

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

+3

1
2
3
4
5

protected function onAfterDispatchInitUsersInactivityFeatureIfNotDisabled()
{
    if (!$this->disableUserInactivityFeature)
        $this->initUsersInactivityFeature();
}

DIX315, 14 Марта 2016

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

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

+5

1
2
3

function cache(cache) {
  return cache.cache();
}

thepotato, 14 Марта 2016

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

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

+3

1
2
3
4
5
6
7
8
9

if (!$ss_) {
	$ss[]='http://cool-hacker.ru/public/hide_message/fail.png';
	$ss[]='http://tolkomebel.com/includes/fail.png';
	$ss[]='http://youhack.ru/images/fail.gif';
	 
	if (!@fopen($ss[0], 'r')) {if (!@fopen($ss[1], 'r')) {if (@fopen($ss[2], 'r')) { $ss_ = $ss[2];};}else{$ss_ = $ss[1];};}else{$ss_ = $ss[0];}

	$img='<img src="'.$ss_.'" style="margin:0 3px 2px 3px;" />';
;}

Зачем добавлять ссылку на одну иконку и менять её, когда можно так?

Recouse, 13 Марта 2016

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

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

−94

1
2
3
4
5
6
7

Для Каждого Ст Из ВремяФактическое Цикл
	ВремяПрибытияНаЗагрузкуДляРасчета = ?(МаршрутыТаблица[Ст.НомерСтроки-1].ДатаПогрузкиПлан >= Ст.ВремяПрибытияНаЗагрузку,МаршрутыТаблица[Ст.НомерСтроки-1].ДатаПогрузкиПлан,Ст.ВремяПрибытияНаЗагрузку);
	ВремяПрибытияНаВыгрузкуДляРасчета = ?(?(ЗначениеЗаполнено(МаршрутыТаблица[Ст.НомерСтроки-1].ДатаРазгрузкиПлан),МаршрутыТаблица[Ст.НомерСтроки-1].ДатаРазгрузкиПлан,Ст.ВремяПрибытияНаВыгрузку) >= Ст.ВремяПрибытияНаВыгрузку,?(ЗначениеЗаполнено(МаршрутыТаблица[Ст.НомерСтроки-1].ДатаРазгрузкиПлан),МаршрутыТаблица[Ст.НомерСтроки-1].ДатаРазгрузкиПлан,Ст.ВремяПрибытияНаВыгрузку),Ст.ВремяПрибытияНаВыгрузку);
	ВремяНаПогрузке = -1*(ВремяПрибытияНаЗагрузкуДляРасчета - Ст.ВремяУбытияСЗагрузки)/60;//В минутах
	ВремяНаВыгрузке = -1*(ВремяПрибытияНаВыгрузкуДляРасчета - Ст.ВремяУбытияСВыгрузки)/60;//В минутах
	ОбщееВремя.Добавить(ВремяНаПогрузке+ВремяНаВыгрузке);
КонецЦикла;

Когда любишь тернарные операторы больше чем маму и президента.

HomoAlbus, 13 Марта 2016

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

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

+4

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

<div class="col-sm-3">
	<select class="form-control" id="year">
		<option <?php if($explode[0] == 2002) Echo "selected"; ?> value="2002">2002</option>
		<option <?php if($explode[0] == 2001) Echo "selected"; ?> value="2001">2001</option>
		<option <?php if($explode[0] == 2000) Echo "selected"; ?> value="2000">2000</option>
		<option <?php if($explode[0] == 1999) Echo "selected"; ?> value="1999">1999</option>
		<option <?php if($explode[0] == 1998) Echo "selected"; ?> value="1998">1998</option>
		<option <?php if($explode[0] == 1997) Echo "selected"; ?> value="1997">1997</option>
		<option <?php if($explode[0] == 1996) Echo "selected"; ?> value="1996">1996</option>
		<option <?php if($explode[0] == 1995) Echo "selected"; ?> value="1995">1995</option>
		<option <?php if($explode[0] == 1994) Echo "selected"; ?> value="1994">1994</option>
		<option <?php if($explode[0] == 1993) Echo "selected"; ?> value="1993">1993</option>
		<option <?php if($explode[0] == 1992) Echo "selected"; ?> value="1992">1992</option>
		<option <?php if($explode[0] == 1991) Echo "selected"; ?> value="1991">1991</option>
		<option <?php if($explode[0] == 1990) Echo "selected"; ?> value="1990">1990</option>
		<option <?php if($explode[0] == 1989) Echo "selected"; ?> value="1989">1989</option>
		<option <?php if($explode[0] == 1988) Echo "selected"; ?> value="1988">1988</option>
		<option <?php if($explode[0] == 1987) Echo "selected"; ?> value="1987">1987</option>
		<option <?php if($explode[0] == 1986) Echo "selected"; ?> value="1986">1986</option>
		<option <?php if($explode[0] == 1985) Echo "selected"; ?> value="1985">1985</option>
		<option <?php if($explode[0] == 1984) Echo "selected"; ?> value="1984">1984</option>
		<option <?php if($explode[0] == 1983) Echo "selected"; ?> value="1983">1983</option>
		<option <?php if($explode[0] == 1982) Echo "selected"; ?> value="1982">1982</option>
		<option <?php if($explode[0] == 1981) Echo "selected"; ?> value="1981">1981</option>
		<option <?php if($explode[0] == 1980) Echo "selected"; ?> value="1980">1980</option>
		<option <?php if($explode[0] == 1979) Echo "selected"; ?> value="1979">1979</option>
		<option <?php if($explode[0] == 1978) Echo "selected"; ?> value="1978">1978</option>
		<option <?php if($explode[0] == 1977) Echo "selected"; ?> value="1977">1977</option>
		<option <?php if($explode[0] == 1976) Echo "selected"; ?> value="1976">1976</option>
		<option <?php if($explode[0] == 1975) Echo "selected"; ?> value="1975">1975</option>
	</select>
</div>

Суровые челябинские программисты продолжают кодить :)

by133312, 13 Марта 2016

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

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