На дворе было 5часов утра, зная что будет выходной, я решил "немного" посидеть ночью за компом. Т.к я любитель онлайн игр, в это время играл в вов, и как многие Русские играл я на фри серверах.
Какой сервер без бага? Баги есть везде, и этот сервер где я играю есть баги, и один из них я знал на деньги. Вот сидел и смотрел как самописный аддон делает мне голду, только изредко приходилось мне продовать вещи которые делал аддон.
К чему это я? Ах да, смотреть то нудно и решил полазить я как то по инету. Помимо игр в онлайн игры я так же являюсь программистом в веб. Да да иммено веб программист, это моя профессия которая мне даёт хлеб, не много но на жизнь хватает. Так вот заходя на разные сайты где лежат всякие сборки серваков, а т.к zone-game один из моих любимых сайтов как и л-а-2база, то первым зашёл на zone-game.
И я видел потрясающию новость: "Веб обвязка от команды ZG-Team"
Зная что на данном сайте меньше людей "ламеров" нежели на другом моём любимом сайте. Я сразу же принялся читать новость.
Зная какая тут граммотная администрация а PROGRAMMATOR`y вообще респект, я первым же делом скачал данную "Веб обвязку".
Но, как я уже говорил я работаю да да иммено веб программистом, и php один из спец языков на котором я специализируюсь. Я не говорю что он единственный, но как многие уже знают что он самый популярный и в тоже время простой и удобный.
Но не будем отходить от темы.
Значит скачал я "Веб обвязку" и ожидал увидеть что то нечно, ну или хотяб прилично хорошее. Но не хотя пишу это увидел я "кучу навоза" мягко вырожаясь.
Можно вырозить это так: Дай обезьяне клаву да мышку, и вы увидите "Веб обвязка от команды ZG-Team".
Нет не подумайте, я вас не оскорбляю.
Можно выразиться что это я вас дразню, может возмётесь за голову. Или хотябы сами увидите что написал(и).
Много я болтаю да?
)
Я уверен что не все это прочли.
Ну да ладно, из болтавни к практике.
Чесно могу сказать, жалко что выходной, ибо эту работу я бы показал в офисе и мы бы громко смеялись.
Всё всё теперь только о скрипте
Эх начнём
PHP код:
if (isset($_GET['k']) || isset($_POST['k'])) {
$k = trim(isset($_POST['k']) ? $_POST['k'] : $_GET['k']);
if (preg_match("/[^a-zA-Z0-9_]/", $k)) {Header();exit;}}
if (isset($_GET['id']) || isset($_POST['id'])) {
$id = trim(isset($_POST['id']) ? $_POST['id'] : $_GET['id']);
if (preg_match("/[^a-zA-Z0-9_]/", $id)) {
Header("Location: index.php?id=start");
exit;
}
} else {
$id = "start";
}
Вот скажите это читабельно?
Где начала выражения а где конец?
Чесно увидев это, я даже не стал смотреть другие фалы, а это только index.php
Ладно не поленился сделал читабельно:
PHP код:
if (isset($_GET['k']) || isset($_POST['k']))
{
$k = trim(isset($_POST['k']) ? $_POST['k'] : $_GET['k']);
if (preg_match("/[^a-zA-Z0-9_]/", $k))
{
Header();
exit;
}
}
if (isset($_GET['id']) || isset($_POST['id']))
{
$id = trim(isset($_POST['id']) ? $_POST['id'] : $_GET['id']);
if (preg_match("/[^a-zA-Z0-9_]/", $id))
{
Header("Location: index.php?id=start");
exit;
}
}
else
{
$id = "start";
}
Давай теперь посмотрим что там:
Вот скажите человек написавший это умеет думать?
Зачем писать пустой
Header, когда мы всё равно потом завершаем скрипт???
PHP код:
Header("Location: index.php?id=start");
exit;
Теперь всё наоборот? Зачем завершать скрипт если нас в любом случае сразу перекинет на другой?
Идём дальше:
PHP код:
switch ($id):
case 'change_skin':
$skin_name = $_POST["skin"];
setcookie("skin_name", $skin_name, time()+2592000);
quickrefresh('index.php?id=start');
break;
case 'logout':
resetcookies();
quickrefresh('index.php?id=start');
break;
endswitch;
Эмм, программист писавший "Веб обвязка от команды ZG-Team", ты наверное перепутал, это PHP а не паскаль подобный язык
)
Ладно переписываем делаем читабельно:
PHP код:
switch ($id)
{
case 'change_skin':
$skin_name = $_POST["skin"];
setcookie("skin_name", $skin_name, time()+2592000);
quickrefresh('index.php?id=start');
break;
case 'logout':
resetcookies();
quickrefresh('index.php?id=start');
break;
}
Думаете тут нет недочётов?
Сколько не искал мой notepad++ по всем директориям скрипта функцию
quickrefresh её нигде нет! Думаю ну мож я не знал о такой функции в php.
Залез на сайт php.net ввёл в поиск функцию, и её там тоже нет.
http://ru.php.net/manual-lookup.php?...n=quickrefresh
Отюсда делаю вывод, данный участок кода просто "ворованый".
Кстате может слышал про такую недо обьявязку как "Moy-Web" ?
Так вот главный файл ну уж очень похож от туда.
Ладно лезем дальше:
resetcookies, как вы думаете что с этой функцией? Да да тоже самоё её нигде нет!
PHP код:
$skin_name = $_COOKIE['skin_name'];
Программист "Веб обвязка от команды ZG-Team", ответь мне что делает этот участок кода?
)
Ты слышал о ошибках в PHP?
Мне кажется если написать в вверх "твоего" "скрипта":
PHP код:
error_reporting(E_ALL);
Ты узнаешь много интересного
)
PHP код:
$skin_name = $_COOKIE['skin_name'];
if ($skin_name == '') $skin_name = $skins[1][1];
$skin_name="skins/".$skin_name."/".$skin_name.".php";
if (file_exists($skin_name))
{
include($skin_name);
}
else
{
include ('skins/'.$skin.'/'.$skin.'.php');
}
if (!file_exists('skins/'.$skin.'/'.$skin.'.php'))
Эх, как жалко что выходной
я так жажду поделится этим index.php с коллегами, и мы будем долго смеяться
Ладно переписываем на читабельный код:
PHP код:
$skin_name = $_COOKIE['skin_name'];
if ($skin_name == '')
$skin_name = $skins[1][1];
$skin_name = "skins/".$skin_name."/".$skin_name.".php";
if (file_exists($skin_name))
{
include($skin_name);
}
else
{
include ('skins/'.$skin.'/'.$skin.'.php');
}
if (!file_exists('skins/'.$skin.'/'.$skin.'.php'))
if ($skin_name == '')
Программист "Веб обвязка от команды ZG-Team" почитайте по ссылке:
http://ru2.php.net/manual/en/function.empty.php
if (!file_exists('skins/'.$skin.'/'.$skin.'.php'))
Чесно сказать, долго думал зачем это.
Так и не понял, может Программист "Веб обвязка от команды ZG-Team" делает новый подход к программированию, делая пустые выражения?
)
И это только один файл index.php
А сколько настрочил то
Жду сообщений типо:
Цитата:
|
Если ты шибоко умный, напиши что нить своё "крутое".
|
Отвечу сразу таким людям, я не хаю скрипт.
Он работает, люди радуются.
Я против такого кода.
Меня учили писать красивый и понятный код. Чтобы он не только работал но и подходил по нормам этики программирования.
Да я тоже учился, и я писал тоже плохо, но меня критиковали ещё хуже.
Кто нить здавал на права? Может многим так же повезло как и мне когда инструктор на вас орёт и т.д
Зато теперь вы ему благодарны, как и я своему, и своим учителям в программировании.
Я не говорю выбрасывать данный скрипт в помойку.
Я лиш указываю разработчикам их ошибки и смотрю их реакцию. Как быстро они решают проблему.
Ведь пользователи видят это сообщения и думаю некоторые из них призадумаются.
Я могу долго писать про этот скрипт. Достать чуть ли не каждый файл, ну не трогая форум phpbb.
Но если судить по 5 бальной шкале, то я бы поставил 2-.
Написал бы это какой то другой человек, о котором я не слышал бы, то я бы поставил 3-. Потому что данный "скрипт" не достоен носить такое название как "Веб обвязка от команды ZG-Team", а точнее быть продуктом такого хорошего сайта как zone-game.
Если кому интересно могу дальше продолжить писать про скрипт достовая их ошибки, и глядишь версия 2.0 будет куда лучше нежели которая она сейчас в разработке.