Форум администраторов игровых серверов - Веб обвязка от команды ZG-Team

На дворе было 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";

}

Давай теперь посмотрим что там:

PHP код:


		
			
Header();

exit;

Вот скажите человек написавший это умеет думать?

Зачем писать пустой 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 будет куда лучше нежели которая она сейчас в разработке.

Скриншот сайта:

Скриншот галереи:

Скриншот форума: