- 1
- 2
- 3
- 4
function EchoForum( $var ){
return true;
echo $var.'<BR>';
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+66
function EchoForum( $var ){
return true;
echo $var.'<BR>';
}
Из модуля форума...
+63
if (!$dt) $dt = date("Ymd");
if ($dt) {
$dt_week_day = (int)date("w",
mktime(0, 0, 0, substr($dt, 4, 2), substr($dt, 6, 2), substr($dt, 0, 4)));
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$weekday1 && !$weekday2 && !$weekday3 && !$weekday4 && !$weekday5 && !$weekday6 && !$weekday7) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && !$weekday1 && !$weekday2 && !$weekday3 && !$weekday4 && !$weekday5 && !$weekday6 && !$weekday7) $ok = 1;
if ($datefrom && !$dateto && $dt <= $datefrom) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday1 > 0 && $dt_week_day == 1) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday2 > 0 && $dt_week_day == 2) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday3 > 0 && $dt_week_day == 3) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday4 > 0 && $dt_week_day == 4) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday5 > 0 && $dt_week_day == 5) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday6 > 0 && $dt_week_day == 6) $ok = 1;
if (!$datefrom && !$dateto && $permanent == "on" && $weekday7 > 0 && $dt_week_day == 0) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday1 > 0 && $dt_week_day == 1) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday2 > 0 && $dt_week_day == 2) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday3 > 0 && $dt_week_day == 3) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday4 > 0 && $dt_week_day == 4) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday5 > 0 && $dt_week_day == 5) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday6 > 0 && $dt_week_day == 6) $ok = 1;
if ($datefrom && $dateto && $dt >= $datefrom && $dt <= $dateto && !$permanent && $weekday7 > 0 && $dt_week_day == 0) $ok = 1;
// if($ok==0)continue;
}
+71
$separator = '';
for( $i = 0; $i < 136; $i++ ) { $separator .= '—'; }
$sql_compile .= htmlspecialchars( $data ) . "\n" . $separator . "\n";
Для вывода в html документе... Видимо <hr /> вдруг перестал работать.
* Из ряда скорее извращений, нежели — «Г».
+72
//1. файл
...
else if ($elem->tagName == "Error") {
$errorCode = trim($elem->getElementsByTagName("Code")->item(0)->textContent);
$errorText = trim($elem->getElementsByTagName("Text")->item(0)->textContent);
var_dump('Fuck!!! Some Error:',$errorCode,$errorText);
}
...
//2. файл, после получения результатов
...
if(empty($result)){
var_dump($sql);
die('Page fucking dead!!!!!');
}
...
//2. файл,
...
if(!file_exists($filePath)){
var_dump('No file found???!!! Where if fucking CLASS file???!!!', $filePath);
return false;
}
...
//3. файл,
...
if (empty($fields['id'])){
var_dump('AAAAAAAAA. No ID again!!!', $fields, $sql);
return false;
}
...
//4. файл,
...
if (!$error){
if($this->debug)var_dump('Yeeeees!!! Work fine ^_^');
return true;
}
...
Наткнулся на один проектик. Не совсем говнокод, но программер был видать оочень емоциональный :)
+69
//проверка фамилии
if ($tmp['fam'] == ""):
$_SESSION["er_mes_registr"] = "Не заполнено поле `Фамилия`.";
header ("Location: /register/");
die();
endif;
if (strlen($tmp['fam'])>20):
$_SESSION["er_mes_registr"] = "Длина фамилии не может быть более 20 букв.";
header ("Location: /register/");
die();
endif;
//проверка имени
if ($tmp['name'] == ""):
$_SESSION["er_mes_registr"] = "Не заполнено поле `Имя`.";
header ("Location: /register/");
die();
endif;
if (strlen($tmp['name'])>20):
$_SESSION["er_mes_registr"] = "Длина имени не может быть более 20 букв.";
header ("Location: /register/");
die();
endif;
//проверка email
if ($tmp['email'] == ""):
$_SESSION["er_mes_registr"] = "E-mail не может быть пустым!";
header ("Location: ".$_SERVER['HTTP_REFERER']);
die();
endif;
if (!ereg("^([a-z0-9_]|\\-|\\.)+@(([a-z0-9_]|\\-)+\\.)+[a-z]{2,4}\$",$tmp['email'])):
$_SESSION["er_mes_registr"] = "Неверно указан e-mail!";
header ("Location: ".$_SERVER['HTTP_REFERER']);
die();
endif;
Правильная валидация...
+58
<?php if ( $poll ) : ?>
<h2>Опрос: <?php echo $poll->getTitle(); ?></h2>
<?php $count = $poll->votes->count(); ?>
<div>Всего ответивших: <?php echo $count; ?></div>
<?php if ( $sf_user->isAuthenticated() ) : ?>
<?php if ( !$poll->isVote( $sf_user->getGuardUser()->getId() ) ) : ?>
<?php foreach ( $poll->answers as $answer ) : ?>
<div><?php echo link_to( $answer->getTitle(), 'poll_vote', array( 'poll' => $poll->getId(), 'answer' => $answer->getId() ) ); ?></div>
<?php endforeach; ?>
<?php else : ?>
<?php foreach ( $poll->answers as $answer ) : ?>
<div><?php echo $answer->getTitle(); ?> ($count / <?php echo $answer->getPercent( $count ); ?>%)</div>
<?php endforeach; ?>
<?php endif; ?>
<?php else : ?>
<?php foreach ( $poll->answers as $answer ) : ?>
<div><?php echo $answer->getTitle(); ?></div>
<?php endforeach; ?>
<div>Что бы выбрать вариант ответа, необходимо войти.</div>
<?php endif; ?>
<?php endif; ?>
Пал выбор, делать проверку, голосовал пользователь или нет в цикле и выводить соответствующую инфу или сделать проверку и использовать два разных цикла в разных ветках
+139
a = 0;
if (a == 0) {
} else {
b = a;
}
+97
$date_sys=date("20y-m-d");
Код из Марокко. Действительно, зачем юзать Y-m-d, если Y = 20y.
+80
// Для очень важных целей извлекаем текущую дату на MSSQL сервере
$d=mssql_query("SELECT getdate()");
$dattime=mssql_fetch_array($d);
// А теперь внимание. Вставляем полученную в предыдущем запросе дату в новую запись
$result="INSERT INTO lgotrivia_gotrivia_game (dateTime, dateTimeIphone, gamemode, playerid, scores, phoneid, app_id)
VALUES ('".$dattime[0]."','".$datetime_iphone."', '".$gamemode."', '".$playerid."', '".$scores."', '".$phoneid."', '".$appid."')";
$res=mssql_query($result);
По каким-то причинам стало не модно в INSERT запросах использовать вызов GETDATE() или CURRENT_TIMESTAMP. Через PHP прослойку ведь куда круче. Видимо таким ребятам тоже платят за строчки кода.
+74
# Определение IP адреса
static public function ip() {
# Список возможных заголовков для получения ip адреса
$Sip = array();
$Sip['REMOTE_ADDR'] = getenv('REMOTE_ADDR');
$Sip['HTTP_FORWARDED_FOR'] = getenv('HTTP_FORWARDED_FOR');
$Sip['HTTP_X_FORWARDED_FOR'] = getenv('HTTP_X_FORWARDED_FOR');
$Sip['HTTP_X_COMING_FROM'] = getenv('HTTP_X_COMING_FROM');
$Sip['HTTP_VIA'] = getenv('HTTP_VIA');
$Sip['HTTP_XROXY_CONNECTION'] = getenv('HTTP_XROXY_CONNECTION');
$Sip['HTTP_CLIENT_IP'] = getenv('HTTP_CLIENT_IP');
$Sip['Unknow'] = '0.0.0.0';
# Выбор найденного ip адреса
if(isset($Sip['REMOTE_ADDR'])) {
$Rip = $Sip['REMOTE_ADDR'];
} else if(isset($Sip['HTTP_FORWARDED_FOR'])) {
$Rip = $Sip['HTTP_FORWARDED_FOR'];
} else if(isset($Sip['HTTP_X_FORWARDED_FOR'])) {
$Rip = $Sip['HTTP_X_FORWARDED_FOR'];
} else if(isset($Sip['HTTP_X_COMING_FROM'])) {
$Rip = $Sip['HTTP_X_COMING_FROM'];
} else if(isset($Sip['HTTP_VIA'])) {
$Rip = $Sip['HTTP_VIA'];
} else if(isset($Sip['HTTP_XROXY_CONNECTION'])) {
$Rip = $Sip['HTTP_XROXY_CONNECTION'];
} else if(isset($Sip['HTTP_CLIENT_IP'])) {
$Rip = $Sip['HTTP_CLIENT_IP'];
} else {
$Rip = '0.0.0.0';
}
# Валидация ip адреса
# Получение из секций массив
$ARip = explode( '.', $Rip );
# Пересборка ip адреса
$RARip = array(0 => NULL, 1 => NULL, 2 => NULL, 3 => NULL);
for($n = 0; $n < 10; $n++) {
$RARip[0] .= isset( $ARip[0]{$n} ) ? $ARip[0]{$n} : NULL;
$RARip[1] .= isset( $ARip[1]{$n} ) ? $ARip[1]{$n} : NULL;
$RARip[2] .= isset( $ARip[2]{$n} ) ? $ARip[2]{$n} : NULL;
$RARip[3] .= isset( $ARip[3]{$n} ) ? $ARip[3]{$n} : NULL;
}
$BRip = substr( (int) $RARip[0], 0, 3 ) . '.' . substr( (int) $RARip[1], 0, 3 ) . '.' . substr( (int) $RARip[2], 0, 3 ) . '.' . substr( (int) $RARip[3], 0, 3 );
# Возвращение пересобранного ip адреса
return $BRip;
}
Не столько — «Г», сколько — «Извращение»...
Улыбнуло имя переменной — $Rip;
А раньше этот код был суровее — exit( $string ); в случаи не соответствия...