1. Список говнокодов пользователя domaster

    Всего: 19

  2. PHP / Говнокод #11267

    +56

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if ('cardiscon' == $key) {
	echo '<td class="even cardiscon">'.$row['name'].':</td>';
} elseif ('cardprice' == $key) {
	echo '<td class="even cardiscon">'.$row['name'].':</td>';
} else {
	echo '<td class="even">'.$row['name'].':</td>';
}

    Странная конструкция

    domaster, 21 Июня 2012

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

  3. PHP / Говнокод #11250

    +66

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (mysql_num_rows(mysql_query("SELECT * FROM `usermail` WHERE `email` LIKE '%".$_POST['email']."%' "))) {
	# email уже есть
} else {
	# новый email
}

    Проверка на наличие email в таблице. Через SELECT * и LIKE !!!

    domaster, 20 Июня 2012

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

  4. PHP / Говнокод #10966

    +61

    1. 1
    $istmount = mysql_num_rows(mysql_query("SELECT `cid` FROM `usermail` WHERE `date` >='".date('Y-m-d', strtotime('-1 month'))."'"));

    Подсчет количества подписчиков за прошлый месяц. Про count(*) автор, похоже, не в курсе...
    И, чтобы не было вопросов про формат поля даты:
    date - это varchar(11) :)

    domaster, 15 Июня 2012

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

  5. PHP / Говнокод #10954

    +59

    1. 1
    2. 2
    3. 3
    4. 4
    <table>
	<tr><th colspan="2">Если вы еще не зарегистрированы:</th></tr>
	<tr><td colspan="2"><a href="/registration.php">Перейдите к регистрации на сайте</a></td></tr>
</table>

    Глубокий смысл в наличии colspan="2" постичь никак не получается...

    domaster, 14 Июня 2012

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

  6. PHP / Говнокод #10939

    +55

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    $i = 0;
foreach($_POST['union'] as $key=>$value) {
	$i++;
	if ($i == 1) {
		$fid = $key;
	}
	if ($i > 1) {
		[код поскипан]
	}
}

    Ну почему не используется if { } else { }?

    domaster, 13 Июня 2012

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

  7. PHP / Говнокод #10931

    +69

    1. 1
    $file[] = $_FILES[''.$key.''];

    Подобное "приведение типов (?)" для $key просто убивает. Размазано по всему проекту...

    domaster, 13 Июня 2012

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

  8. PHP / Говнокод #10856

    +63

    1. 1
    2. 2
    $ToName = "$user_iname";
$ToEmail = "$user_email";

    Кавычки-то зачем?

    domaster, 09 Июня 2012

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

  9. PHP / Говнокод #10619

    +57

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    $format   = $_[3];           # Format object

# Check for a format object
if (isset($_[3])) {
    $ixfe = $format->get_xf_index();
} else {
    $ixfe = 0x0F;
}

    Сначала присвоить, а потом проверить наличие - это сильно!
    Найдено в class.writeexcel_worksheet.inc.php - классе для заботы с .xls

    domaster, 08 Июня 2012

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

  10. PHP / Говнокод #10401

    +67

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    preg_match('/(\d+)-(\d+)-(\d+)/i', date('d-m-Y'), $match);
ob_start();
echo (($match[1] == 30 || $match[1] == 31)?'01':(($match[2] <= 10)?'0':'').''.(++$match[1])).'.'; 
echo ((!$match[2][1])?'0':''); 
echo (($match[1] == 30 || $match[1] == 31)?++$match[2]:$match[2]); 
echo '.'.$match[3];
$rdate = ob_get_contents();
ob_end_clean();

    Досталось в наследство...
    По алгоритму $rdate должна содержать текущее число следующего месяца в формате date('d.m.Y'). Мало того, что способ означивания переменной чрезвычайно необычен, код оформлен безобразно, так оно еще и не работает!
    На 29.05.2012 выдает "030.6.2012"

    domaster, 29 Мая 2012

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