- 1
- 2
- 3
- 4
- 5
- 6
- 7
if($atributId){
$sql = "UPDATE directory_atributes SET name = '$newName' WHERE id = $atributId LIMIT 1";
$db-> Query($sql);
die();
} else{
die();
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+57
if($atributId){
$sql = "UPDATE directory_atributes SET name = '$newName' WHERE id = $atributId LIMIT 1";
$db-> Query($sql);
die();
} else{
die();
}
Депрессивное программирование. В любом случае ты умрёшь.
+54
function mail_mime_to_array($imap,$mid,$parse_headers=false)
{
$mail = imap_fetchstructure($imap,$mid);
// ИМЕННО ПОЧИНИЛ
if(!isset($mail->parts)) {
$mail->parts = array();
$mail->parts[0]['type'] = $mail->type;
$mail->parts[0]['encoding'] = $mail->encoding;
$mail->parts[0]['ifsubtype'] = $mail->ifsubtype;
$mail->parts[0]['subtype'] = $mail->subtype;
$mail->parts[0]['ifdescription'] = $mail->ifdescription;
$mail->parts[0]['ifid'] = $mail->ifid;
$mail->parts[0]['bytes'] = $mail->bytes;
$mail->parts[0]['ifdisposition'] = $mail->ifdisposition;
$mail->parts[0]['disposition'] = $mail->disposition;
$mail->parts[0]['ifdparameters'] =$mail->ifdparameters;
$mail->parts[0]['dparameters'] =$mail->dparameters;
$mail->parts[0]['ifparameters'] =$mail->ifparameters;
$mail->parts[0]['parameters'] =$mail->parameters;
}
$mail = mail_get_parts($imap,$mid,$mail,0);
// ИМЕННО ПОЧИНИЛ
if(!isset($mail->parts)) {
$mail[1]['data'] = imap_base64($mail[1]['data']);
}
if ($parse_headers) $mail[0]["parsed"]=mail_parse_headers($mail[0]["data"]);
return($mail);
}
мое говно-детище
АЙФОН к примеру не отправляет заголовки и я не мог правильно под него запарсить вложения к почтовому сообщению.
Решил не париться и именно починил, взял скотч решил эту проблему, просто залепил ее.
Самое обидное, что даже не знаю как ее по нормальному решить.
+66
if($HTTP_SESSION_VARS['username'] == $_SESSION['username']){
include($incl['userrights']);
}
я плакал :(
+50
/**
* Что я курил, когда это писал? Почему нельзя было date('H:i:s', $time) ??? +___=
* @param $seconds
* @return string
*/
private static function prettySecondsPrint($seconds){
return str_pad((int)($seconds / 3600), 2, '0', STR_PAD_LEFT)
.':'.str_pad((int)(($seconds % 3600) / 60), 2, '0', STR_PAD_LEFT)
.':'.str_pad($seconds % 60, 2, '0', STR_PAD_LEFT);
}
А это, к сожалению, мое творение) Комменарий приписал когда позже просматривал код и обнаружил это)
−193
if(!empty($_SESSION['order_id'])&&($_SESSION['order_id'] == $order->id))
;
else{
if(empty($order->user_id))
if($password = $this->request->post('password','string')){
if($url != $this->orders->check_password($url, $password))
return false;
}
else
// Выводим форму запроса пароля
return $this->body = $this->design->fetch('order_login.tpl');
elseif ($order->user_id != $this->user->id)
return false;
}
из разряда "я не умею писать условия"
+50
if($_REQUEST['filter2'] == 'my' OR $_REQUEST['filter2'] == 'all' OR $_REQUEST['filter2'] == 'delegated'){
$filter2 = htmlspecialchars( $_REQUEST['filter2'], ENT_QUOTES );
}else{
$filter2 = 'all';
}
Может тогда уж $filter2 = htmlspecialchars( 'all', ENT_QUOTES ); для полного счастья?
+39
<?php
$pg = $_GET['page']; // получаем номер страницы из URL
$range = 10; // количество записей для одной страницы
$start = ($pg<1)?0:($pg-1)*$range; // получаем номер записи, с которой начнем выборку
$l = mysql_connect("localhost", "root", ""); // соединяемся с сервером БД
$link = mysql_select_db("yiiblog", $l); // выбираем нужную БД
$allpages = mysql_query("select id from tbl_post"); // выбираем все записи из таблицы
$num_pages = mysql_num_rows($allpages); // выбрав, узнаем сколько их штук всего
$cnt = $num_pages;
$res = mysql_query("select * from tbl_post ORDER by id desc limit $start, $range");
// выбираем данные исходя из значения номера страницы
// формируем вывод, разделяя горизонтальной линией
for($i=0; $i<mysql_num_rows($res); $i++)
{
$row = mysql_fetch_array($res);
echo $row['id']."
".$row['title']."<hr>";
}
// ниже считаем сколько у нас получилось страниц, учитывая что в каждой странице по $range записей
$pages = 0;
do
{
$pages++;
$cnt = $cnt - $range;
}
while($cnt>0);
// Сам пагинатор, выводим номера страниц.
for($i=1; $i<=$pages; $i++)
{
$url = $PHP_SELF."?page=".$i;
$link = ($pg != $i)?"<a href=".$url.">".$i."</a> ":"<b>".$i." </b>";
echo $link;
}
echo "<hr>Всего записей: ".$num_pages.", страниц: ".$pages;
?>
Взят с сайта: http://techsmash.ru/php/2012/09/10/primer-paginatora-na-php.html
−203
// Тут начинается сильнейший программинг - формирование GET запроса на основе уже существующего, курите.
private function getFilterURLParams($filter = 0, $option_id, $value_id, $variable = '') {
// При изменении этих параметров, нужно будет поменять соответсвенно их в других файлах. Менять их не советую.
$sep_par = ';'; // разделитель пар опций -> значений: opt1=val1,val2,val3;opt2=val1,val2,val3 ...
$sep_opt = '='; // разделитель внутри пары опция -> значения: opt1=val1,val2,val3 ...
$sep_val = ','; // разделитель для параметров опции: val1,val2,val3 ...
if ($filter) {
$matches = explode($sep_par, $filter);
$options = array();
$values = array();
$parts = array();
foreach ($matches as $option) {
$data = explode($sep_opt, $option);
$parts[] = $option;
$options[] = $data[0];
$values[] = explode($sep_val, $data[1]);
}
if (in_array($option_id, $options)) { // если эта опция уже есть в запросе, то мы не добавляем её
$key = array_keys($options, $option_id); // вычисляем ключ массива для дальнейшей работы с именно этой опцией
if (in_array($value_id, $values[$key[0]])) { // если это значение уже есть в запросе
if (count($values[$key[0]]) == 1) { // и если оно единственное
if (count($matches) == 1) { // еще и с единственной опцией, то удаляем из запроса весь фильтр
$out = '';
} else { // если опция не одна, удаляем только эту опцию с её параметром
$out = '&' . $variable . '=' . str_replace((array_search($parts[$key[0]], $parts) ? $sep_par . $parts[$key[0]] : $parts[$key[0]] . $sep_par), '', $filter);
}
} else { // если значений несколько, удаляем это значение, оставляя другие с опцией
$out = '&' . $variable . '=' . str_replace($parts[$key[0]], $options[$key[0]] . $sep_opt . str_replace((array_search($value_id, $values[$key[0]]) ? $sep_val . $value_id : $value_id . $sep_val), '', implode($sep_val, $values[$key[0]])), $filter);
}
} else { // если значения нет в запросе, то добавляем его к значениям этой опции
$out = '&' . $variable . '=' . str_replace($parts[$key[0]], $options[$key[0]] . $sep_opt . $value_id, $filter);
}
} else { // если этой опции нет в запросе
$out = '&' . $variable . '=' . $filter . $sep_par . $option_id . $sep_opt . $value_id;
}
} else { // если в запросе вообще нет переменной filter
$out = '&' . $variable . '=' . $option_id . $sep_opt . $value_id;
}
return $out; // фух.
}
я после этого долго и нервно курил. чувствовал себя отбросом, а не программистом
+140
$i = 0; // Счетчик
+139
if ((!$zzzzzzzzzzzzz))
{
Если мухи еще не налетели...