- 1
Перейти на страницу <span onmouseover="this.className='menu_over'" onmouseout="this.className='menu'" class="menu" onClick="window.location='<?echo $arItem["DETAIL_PAGE_URL"]?>'">--></span>
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+167
Перейти на страницу <span onmouseover="this.className='menu_over'" onmouseout="this.className='menu'" class="menu" onClick="window.location='<?echo $arItem["DETAIL_PAGE_URL"]?>'">--></span>
тег "a" видимо был незнаком.
+161
function get_section(){
if(ifGet('section')){
if($this->get_file('./applications/app.'.$this->section.'.php')){
switch (Get('section')){
case ''.$this->section.'':
$m = new $this->section();
break;
}
} else {
switch (Get('section'))
{
default:
include_once('./applications/sys.pages.php');
$m = new pages();
break;
}
}
}else{
switch (Get('section'))
{
default:
include_once('./applications/sys.home.php');
$m = new home();
break;
}
}
if(isset($m) && is_object($m)) {
$this->output = $m->output;
$this->title = $m->title;
$this->crumbs = $m->crumbs;
}
}
я сам в шоке как такое мог написать...
+155
insert_image.php:
<?php
require_once('../config.inc.php');
if ((!isset($_SESSION['user_type'])) || ($_SESSION['user_type'] != 0)) {
header('Location: /');
die;
}
require('admin_image.inc.php');
settitle.php:
<?php
require_once('../config.inc.php');
if ((!isset($_SESSION['user_type'])) || ($_SESSION['user_type'] != 0)) {
header('Location: /');
die;
}
$id = intval($_POST['image']);
$title = $_POST['imgtitle'];
mysql_query("UPDATE images SET title='$title' WHERE id=$id");
require('admin_image.inc.php');
upload_image.php:
<?php
require_once('../config.inc.php');
if ((!isset($_SESSION['user_type'])) || ($_SESSION['user_type'] != 0)) {
header('Location: /');
die;
}
if ((isset($_POST['upload'])) && (isset($_FILES['newimage']))) {
$title = (isset($_POST['title'])) ? $_POST['title'] : '';
$file = $_FILES['newimage'];
if ($file['error'] != 0) {
$msg = 'An error occured during uploading file. (Error code:' . $file['error'] . ')';
} else {
$type = $file['type'];
$tmp_name = $file['tmp_name'];
//check if we are uploading image or not
if (!(((preg_match('/\.gif/i', $file['name'])) || (preg_match('/\.jpg/i', $file['name'])) ||
(preg_match('/\.jpeg/i', $file['name'])) || (preg_match('/\.bmp/i', $file['name'])) ||
(preg_match('/\.png/i', $file['name'])))
&& ((preg_match('/gif/i', $file['type'])) || (preg_match('/jpg/i', $file['type'])) ||
(preg_match('/jpeg/i', $file['type'])) || (preg_match('/bmp/i', $file['type'])) ||
(preg_match('/png/i', $file['type']))))) {
$msg = 'You are trying to upload a non-image file.';
} elseif (filesize($tmp_name) <= 0) {
$msg = 'You are trying to upload file which size is 0 bytes.';
} else {
$img_data = fread(fopen($tmp_name, 'r'), filesize($tmp_name));
if (mysql_query("INSERT INTO images (id, image, type, title)"
. " VALUES ('',"
. " '" . mysql_escape_string($img_data) . "',"
. " '" . mysql_escape_string($type) . "',"
. " '" . mysql_escape_string($title) . "')"))
$msg = 'Image uploaded.';
else
$msg = 'An error occured during inserting image in DB.';
}
}
}
if (isset($msg))
$MyPage->assign('msg', $msg);
require('admin_image.inc.php');
Какая экспрессия, какое необычное именование файлов... lower_case_with_underscores + просто текст
А главное какое необычное представление о модульности: 2 файла по 10 строк, и один на 40, в каждом из которых прописана авторизация и которые инклюдят главный файл
+166
$expired = (time() > 1280241436) ? true : false;
if ($expired) { return; }
В начале кэш-файлов phpBB3.
+172
//Взятые из разных файлов варианты обращений к файлу конфигураций характеризуют о гибкости подхода к разработке приложения
require_once '../../../config.php';
require_once("../config.php");
require_once("../../../../config.php");
require_once(dirname(dirname(__FILE__)) . '/config.php');
require('../config.php');
require_once(dirname(__FILE__) . '/../../config.php');
require_once(dirname(dirname(dirname(dirname(__FILE__)))) . '/config.php'); //included from messagelib (how to fix?)
Предлагаю обратить внимание на http://moodle.org/ -- это Система Управления Обучением.
Продукт поражает своей архитектурой. Предлагаю взглянуть одним глазком.
+152
if ($UsingComponents){
// если используем компоненты 2.0 - очищаем буфер
ob_end_clean();
$APPLICATION->AddBufferContent('empty');
...
} else {
// иначе выводим всё, что у нас накопилось
ob_end_flush();
$APPLICATION->AddBufferContent('empty');
...
}
Не совсем говнокод, но комментарий на строке 8, имхо, забавный...
+155
private $data = array(
'name' => '',
'organization' => '',
'mail' => '',
'phone' => '',
'comment' => '',
'model' => '',
);
Так адекватные люди создают асоциативные массивы? Или по другому?
+149
$txt['aeva_gallery'] = isset($txt['aeva_gallery']) ? $txt['aeva_gallery'] : 'Media';
Кусочек языкового файла модуля медиагалереи
+149
// get info from db
$results = $this->datadb->dataForIndex();
$data_['mainResult'] = $results['mainResult'];
$data_['pics'] = $results['pics'];
$data_['com'] = $results['com'];
$data_['huuInvert'] = $results['huuInvert'];
$data_['dateInvert'] = $results['dateInvert'];
$data_['type'] = $results['type'];
Кусок со своего CMS. Да я знаю что можно объединить массивы, но иногда не все требуется из функции.
+168
function UploadAvatar($FILE_tmp, $FILE_name){
/*куча тупого кода*/
if($width >= $height)
$kf = (float) $width/$height;
if($width < $height)
$kf = (float) $height/$width;
//Проверка на квадратность! ппц идиотизм, идея не моя.
if($kf>1.1){
$error = "Аватар неквадратен. Пожалуйста выберите квадратный или обрежьте текущий в графическом редакторе.";
return 0;
}
/*еще куча тупого кода*/
}
Вырезал кусок из собственного старого сайта. Комменты оставил уникальными.
Когда дизайнер потребовал чтоб аватары были только квадратны, я наваял такое.
Допуск в 1,1 - это поблажка юзерам)))