- 1
- 2
- 3
- 4
- 5
<select>
<option class="a2" value="0" selected="selected">любой возраст</option>
<option class="a2" value="1">0 мес - 6 мес</option>
<option class="a2" value="2">0 мес - 12 мес</option>
</select>
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 19
+141
<select>
<option class="a2" value="0" selected="selected">любой возраст</option>
<option class="a2" value="1">0 мес - 6 мес</option>
<option class="a2" value="2">0 мес - 12 мес</option>
</select>
selected="selected" для элемента, выбираемого по дефолту - зачем?
ps. Код - кусок статичного html.
+75
if ( !empty($special['catalog']) ) {
if (count($special['catalog'])>0) {
if (count($special['catalog'])>1) {
...
}
}
}
Интересная конструкция...
+62
$ar = array(5050,5370,6285,48100,6730,6730,4700,6500,3450,7260,2500,1370,4288,9620);
$cpr = 0;
foreach($ar as $k=>$v)
{
$cpr += $v;
}
echo $cpr;
Нестандартность мышления в действии
+52
$Qock = mysql_query("SELECT ...");
$Cock = mysql_num_rows($Qock);
Переменная $Cock - это ж надо так назвать!
+67
if ($_GET['search'] == ' Поиск' || $_GET['search'] == 'Поиск') {
$_GET['search'] = '';
}
if ($_GET['date'] == ' Дата' || $_GET['date'] == 'Дата') {
$_GET['date'] = '';
} else {
$_GET['date'] = preg_replace('/(\d+).(\d+).(\d+)/i','$3-$2-$1', $_GET['date']);
}
Уж не знаю, что и сказать... А вдруг перед текстом попадутся 2 пробела? Или 3?
+65
if ($_GET['where'] != '') {
if ($_GET['where'] != '') {
die ("<meta http-equiv='refresh' content='0; url=".$page."?where=".$_GET['where']."'>");
} else {
die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
}
} else {
die ("<meta http-equiv='refresh' content='0; url=".$page."?pid=".$_GET['pid']."'>");
}
Интересно, в каком случае выполнится второй die()?
+61
if (is_page('orders_item')) {
die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
} elseif (is_page('orders_garant')) {
die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_garant.php?pid=".$_GET['pid']."'>");
} else {
die ("<meta http-equiv='refresh' content='0; url=".$base_url."/orders_item.php?pid=".$_GET['pid']."'>");
}
exit();
Хоть опросник пиши:
- чем первый die() отличается от последнего?
- почему их 3?
- кто будет экранировать $_GET?
- зачем в конце exit() подпирает die()?
+45
$ApsolutePath = $_SERVER['DOCUMENT_ROOT']; # Абсолютный путь до сайта (автоматическое определение корневой директории)
$ApsolutePath - Ну как? КАК?
+66
// Масив с выбором
if ($type=="textAr") {
foreach ($stext as $key=>$row) {
if ($key==$data) {
return $row;
break;
}
}
}
Кто-то тролит хостинг...
Заменено на if ('textAr'==$type && isset($stext[$data]) ) return $stext[$data];
+66
// text
if ($type=="text") {
[skip]
}
// order
if ($type=="order") {
[skip]
}
// select
if ($type=="select") {
[skip]
}
// chekbox
if ($type=="chekbox") {
[skip]
}
Таких проверок штук 20. Коментарии на всех соответствующие.