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

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

+109

1
2
3
4
5

try
            {
                splash.Close(new TimeSpan(0, 0, 0, 3, 0));
            }
            catch (Exception ex) { System.Diagnostics.Debug.WriteIf(false, ex.ToString()); }

Вот так вот эксепшн обработал. Помню, недавно сказал такую вещь "В нашем игровом движке содержится лучший код, который я когда-либо писал" про свой игровой движок. Казалось бы, удобная имплементация, структура и принципы ООП правильно используются. Но потом я увидел ЭТО (см. выше). Надо больше спать, видимо... %)

RaZeR, 16 Февраля 2011

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

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

+115

1
2
3
4

public bool SameTerritoryAsEditor(string terrName)
{
   return _personService.GetTerritoryByUserIdentity(Thread.CurrentPrincipal.Identity.Name).Name.Equals(terrName);
}

Определяем принадлежность пользователя к определенной территории по её названию.

St0rm, 16 Февраля 2011

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

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

+128

1
2
3
4
5

public static Boolean GetAttBool(XmlNode node, string attName)
 {
     long value = GetAttLong(node, attName);
      return value == 1 ? true: false;
 }

uaavg, 16 Февраля 2011

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

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

+144

1

Вопрос...

Ребят, у меня такой вопрос, тут или посмейтесь, или скажите как правельней, сайт не для вопросов согласен, но помогите не говнакодить)))
Ядро системы у меня, сканирует директории модулей и плагинов таким образом что у имени деректории есть префикс mod and plg где если это модуль то вот имя директории mod[news], нет оно всё не в куче разумеется, порядок соблюдаю разумеется, но и исправить метод можно не проблема, но я пишу это к чему, недавно готовый сайт выложил на хостинг на котором ранее небыл, и всё конечно идеально, за исключением одного нюанса, FTP менеджеры не могут сканировать директории в имене которых находятся знаки [ ] и только единственный менеджер может сканировать отлично это FAR... И у меня просьба, посмеятся какой я дурак если это реально проблема, или это плюсом, что важные директории проблемно прочитать? Работа системы проходит на ура хоть где не в этом вопрос, а именно имена директорий с знакоми [ ], и проблема их сканировать из "ВНЕ"... Стоит ли их переименовывать?

nethak, 16 Февраля 2011

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

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

−241

1
2
3
4
5
6
7

private function fürDieLuLz(event:String, params:StringHash):void
{
	if (ExternalInterface.available)
		ExternalInterface.call(
			"function(){window.fireEvent('" + event + 
			"',JSON.stringify(" + params + "));}");
}

Я долго пытался объяснить человеку пишущему JS часть проекта, что не нужно перекодировать объекты каждый раз из JSON и обратно при общении с Флешем, если они все равно проходят через eval(). Но у человека ЧСВ победил все. В итоге пришел "главный инженер" и "решил", что "если мы раньше так делали, то мы будем так делать и дальше, и не волнует". В итоге родился этот шедевр. Я сегодня с утра для поднятия настроения показал этот код JS кодеру. Долго объяснял, что он делает. После чего он сказал, что больше не будет со мной разговаривать :)
Я счастлив

wvxvw, 16 Февраля 2011

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

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

+122

1
2
3
4
5
6
7
8
9

catch (TargetInvocationException ex) {
     try {
          throw ex.InnerException;
     }
     catch (EndpointNotFoundException innerEx) {
          factory.Abort();
          throw new InvalidOperationException("Service unreachable", innerEx);
     }
}

Обработчик исключений %)

Guid, 16 Февраля 2011

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

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

+163

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

<?php

if($_SERVER['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest') {

sleep(3);

if($_POST[parent_id]) $parent_id = preg_replace('/\D+/i','', $_POST[parent_id]);
else $parent_id = 0;

$author = trim($_POST[author]);
$comment = trim($_POST[comment]);



if(!$author) $error[author] = 'Введите имя!';
if(!$comment) $error[comment] = 'Напишите комментарий!';

if($error)
exit(json_encode($error));


require_once 'blocks/bd.php';


$sql = "INSERT INTO comments (parent_id, name, comment, date_add) VALUES ($parent_id, '$author', '$comment', NOW())";
$result = mysql_query($sql);
if(!$result)
{
$error[] = 'Произошла ошибка, комментарий не сохранен';
exit(json_encode($error));
}
exit();
}

?>

qbasic, 16 Февраля 2011

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

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

+175

1

$("input[id='yearB']")

striker, 16 Февраля 2011

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

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

+158

1
2
3
4
5
6
7
8
9

public function search() {
    $ret = array();
    foreach($this->dir() as $file)
        if($file->folder())
            $ret = array_merge($ret,$file->search()->asArray());
        else
            $ret[] = $file;
    return new mod_file_list($ret);
}

Красота, в результате работы функции мы получаем объект, состоящий из массива объектов класса file (этот метод в этом классе)... "чудо-поиска" (хотя это вовсе и не поиск, а банальный список файлов всех дочерних папок), особенно будет забавно, если в списке будет несколько тысяч файлов...
В этом классе в принципе много мест, которые наровят насоздавать кучу экземпляров этого же класса, вместо замены одной приватной переменной...

GhOsTMZ, 15 Февраля 2011

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

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

+162

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

public static function perform($table, $data, $action = 'insert', $parameters = '') {
		reset($data);
		if ($action == 'insert') {
			$query = 'INSERT INTO ' . $table . ' (';
			while (list($columns, ) = each($data)) {
				$query .= $columns . ', ';
			}
			$query = substr($query, 0, -2) . ') values (';
			reset($data);
			while (list(, $value) = each($data)) {
				switch ((string)$value) {
					case 'now()':
						$query .= 'NOW(), ';
						break;
					case 'null':
						$query .= 'NULL, ';
						break;
					default:
						$query .= '\'' . mysql_escape_string($value) . '\', ';
						break;
				}
			}
			$query = substr($query, 0, -2) . ')';
		} elseif ($action == 'update') {
			$query = 'UPDATE ' . $table . ' SET ';
			while (list($columns, $value) = each($data)) {
				switch ((string)$value) {
					case 'now()':
						$query .= $columns . ' = NOW(), ';
						break;
					case 'null':
						$query .= $columns .= ' = NULL, ';
						break;
					default:
						$query .= $columns . ' = \'' . mysql_escape_string($value) . '\', ';
						break;
				}
			}
			$query = substr($query, 0, -2) . ' WHERE ' . $parameters;
		}

		self::request($query);
	}

Вот так у нас вставляют в базу :)

wmmorgun, 15 Февраля 2011

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

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

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

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

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

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

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

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

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

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

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

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