- 1
- 2
- 3
- 4
- 5
- 6
- 7
char *z="group=";
char *x="end group\n";
char *text=new char[sizeof(z)+sizeof(x)+sizeof(groupName)+1];
strcat(text, z);
strcat(text, groupName);
strcat(text, "\n");
strcat(text, x);
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+899.2
char *z="group=";
char *x="end group\n";
char *text=new char[sizeof(z)+sizeof(x)+sizeof(groupName)+1];
strcat(text, z);
strcat(text, groupName);
strcat(text, "\n");
strcat(text, x);
Нашел в исходниках одного менеджера паролей. Вот как надо формировать строки.
−100.2
def ordinalize_ru integer
postfix = case integer
when 0 then 'ой'
when 2 then 'ой'
when 3 then 'ий'
when 6 then 'ой'
when 7 then 'ой'
when 8 then 'ой'
else 'й'
end
[ integer, postfix ].join '-'
end
Когда нет времени вникать в правила родного языка и русской речи. Суровый уральский говнокод.
+905.2
if ( tmp - res == 11 )
{
tmp = tmp;
}
Нашел в проэкте сегодня. Кто отгадает, зачем этот код был нужен - тому ничего не будет.
+92.8
// Я приводить пример кода не буду, т.к. он давно исправлен и остался только в памяти и SVN-е,
// но вполне реальный человек захуячил такое:
// В событии OnPaint таблицы происходил запрос к БД, чтоб узнать, какой именно текст надо отрисовывать.
// Очень весело было изменять размеры формы и наблюдать, с какой скоростью увеличивается лог-файл xDDD
+156.8
if (ereg("[0-9]+")) $id = (int)$_GET['id'];
elseif (ereg("[0-9\.]+")) $id = (float)$_GET['id'];
else $id = $_GET['id'];
Не то, чтоб жуткий говнокод, но как-то глуповато.
+162.6
<?php
if($_GET['id'] == 10 ){
$location = "/articles/lyustry/lustra-na-kuhne/";
}elseif($_GET['id'] == 13 and $_GET['id_type'] == 17){
$location = "/market/interernye_svetilniki/lyustry/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 22 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bolshie_lyustry/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 34 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/lyustry/bolee_6_rozhkov/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 2 ){
$location = "/market/interernye_svetilniki/podvesnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 40 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/podvesnye/odinochnyj_podves/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 41 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/podvesnye/trojnoj_i_bolee/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 5 ){
$location = "/market/interernye_svetilniki/nastenno-potolochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 35 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/potolochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 52 ){
$location = "/market/interernye_svetilniki/vlagozawitnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 51 ){
$location = "/market/interernye_svetilniki/podsvetka_kartin/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 7 ){
$location = "/market/interernye_svetilniki/bra/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 37 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/odnorozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 38 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/dvuhrozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 39 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/bra/treh_i_bolee_rozhkovye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 1 ){
$location = "/market/interernye_svetilniki/tochechnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 42 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/odinochnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 43 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/dvojnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 44 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/tochechnye/tri_i_bolee/";
}
elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 8){
$location = "/market/interernye_svetilniki/nastolnye_lampy/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 46 and $_GET['act'] == 'sub_type'){
$location = "/market/interernye_svetilniki/nastolnye_lampy/ofisnye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 47 and $_GET['act'] == 'sub_
type'){
$location = "/market/interernye_svetilniki/nastolnye_lampy/prikrovatnye_domashnie_ili_interernye/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 6 ){
$location = "/market/interernye_svetilniki/torshery/";
}elseif(
$_GET['id'] == 13 and $_GET['id_type'] == 9 ){
$location = "/market/ulichnye_svetilniki/";
}...
И так далее..
Клиент поставил нашу CMS, на старой были ссылки типа /?id=10 и т.д. Сделал ему для примера скрипт редиректа с использованием switch case и вот что я увидел через две недели 0_0 Неприкрытый говнокод.
+170.2
...
<title><?php echo iconv('utf-8', 'windows-1251', 'Какой-то текст'); ?>
...
и такие извращения для каждого куска текста в шаблоне
вот такой вот оригинальный способ перевести шаблон в utf-8
подозреваю, что человеку очень мешали BOM, от которых он не мог избавится, других причин не вижу
+144
if (mysql_num_rows($res)) {while($row = mysql_fetch_assoc($res)) {
$aSingStat[] = $row;
}
}
толи строчек жалко, толи решил обфусцыровать код для меньшей узнаваемости.
написал уже б так: if (mysql_num_rows($res)) while($row = mysql_fetch_assoc($res)) $aSingStat[] = $row;
Тогда б я вообще не парился и дропнул нафиг строчку.
Андрюха, не пиши больше так!
Вот пример как надо писать http://www.sergunik.name/?p=178
+160
# сегодняшнее число
$date2 = date(
"d.m.Y",
mktime(0, 0, 0, date("m") , date("d"), date("Y"))
);
+144.5
define('MYSQL_PASS', 'пароль');
нет, ну вот прям так вот и лежит посреди кода :)