Обзор чужого php кода

Наткнулся на код видео галереи от Цыбулевского В.И.
Причем это не левый скрипт с форума, а седьмая версия с readme на 116 строк!
Открыл код и прокомментировал какие ошибки допустил автор этого говнокода… Они в каждой третей строке. Читайте если интересно…
Мои комменты заключены в /* слеш со звездочкой*/ , остальное писал Цыбулевский. Ссылка на исходный скрипт http://megaice.net/click.html?http://megaice.net/downloads/e107/vg/utf8.zip

Наслаждайтесь:

<?php
/*подключаем класс с загадочным названием class2 - видимо у автора этот класс был вторым в жизни */

require_once("class2.php");
/*при подключении языков в начале стоит @ - это значит, что если пользователь потеряет или переименует эти файлы, то ошибку придется искать очень долго */

@include_once(e_PLUGIN.'video_gallery/languages/'.e_LANGUAGE.'.php');

@include_once(e_PLUGIN.'video_gallery/languages/English.php');

include_once(e_PLUGIN.'video_gallery/vg_sql.php');
/*переменные с загадочными названиями... Тсссс это магия! */

$textx=VG_128;

$ut=VG_129;

$un=VG_130;
/*внимание! подключен шаманский класс, тру название детектед "vg_class" (имя идентификатора в стиле Кернигана и Ричи) */

$videogallery = new vg_class;

//Присланные данные
/*Да Вы что? а откуда Вы знаете, что $_GET["q"] вообще есть?*/

$q=$_GET["q"];

/////////////////////////////

// Добавление в избранное
/*ооо еще куча GET, это уже не "Присланные данные"? */

$usersid=$_GET["uid"];

$vg_id =$_GET["act"];

$usersnames=$_GET["un"];

$arrct=$_GET["q"];

if ($q) 

{

  $tmp = explode(",", $q);
/*Рульные комменты "Действие"! */

  $action = $tmp[0];                    // Действие

  $act_id1 = varset($tmp[1],'');// ID элемента

  $act_id2 = varset($tmp[2],'');//на  сколько прибавить

  unset($tmp);

}

///////////////////////////////////////////////Рейтинг/////////////////////////////////////////////////
/*дальше два раза подряд оператор if написан If, от чего в моем редакторе gedit он не выделяется подсветкой кода)) */

If ($action=="plus"){
/*держите меня! одной проверки мало, условие ($action=="plus") повторяется дважды подряд */

        If ($action=="plus"){ // Прибавим к рейтингу

                $errxvg=false;
/* "or" так никто не пишет, да и можно было просто диапазон проверить*/

                if (($act_id2==1) or ($act_id2==2) or ($act_id2==3) or ($act_id2==4) or ($act_id2==5) or ($act_id2==10)){

                        $videogallery->vg_rate_add($act_id1,USERID,$act_id2);

                } else {

                        $errxvg=true;

                }

        }

        // Вывод  результатов голосования!

        $videogallery->get_vg_el($act_id1);

        //параметры в массив

        $rowmedia = $sql->db_Fetch();
/* вот это стиль! правильно - весь код надо писать в одну строку! тогда код будут мерять не в строках, а в символах */

        if ($errxvg==true){$texyx=VG_278;} else {$texyx='';}

/* автор не знает разницу между " и ' */
/* раз уж ", то можно было писать $rat="<RATE>$rowmedia['vg_rate']</RATE>"; и тд. */

        $rat="<RATE>".$rowmedia['vg_rate']."</RATE>";

        $clikr="<CLIK>".$rowmedia['vg_rate_clic']."</CLIK>";

        

        $knc="<RATETEXT>".VG_128_1."</RATETEXT>".$rat." "."<CLIKTEXT>".VG_128_2."</CLIKTEXT>".$clikr.$texyx; //  Если уже голосовал

        if ($errxvg<>true){

                $resx =$knc.$ut; 
/*опять наш шаманский стиль 1 строки*/

        } else {$resx =$knc;}

}
/*еще одна строка занятая //// не помешает... так будет казаться что кода больше)) */

//////////////////////////////////////////////////////////////////////////////////////////////////////

///////////////////////////////////////////////Избранное пользователя/////////////////////////////////////////////////
/* еще одно бесполезное условие и if написан с большой буквы - ЭГО */

If (($arrct=="del") or ($arrct=="add")){

        If ($arrct=="del"){ // Прибавим к рейтингу

                $message=$videogallery -> vg_rem_user_top($usersid,$vg_id,$usersnames); 

                $arrct2='add';
/* это просто шедевр двойных кавычек! религия запрещает юзать одинарные */

                $resx=$message."<br><input class='button' type=\"button\" name=\"vg_user_fav_add\" id=\"txt2\" onclick=\"set_user_fav('$usersid','$vg_id','$usersnames','$arrct2')\" value='".VG_226."' /> ";;

        }

        If ($arrct=="add"){ // Удалим из  рейтинга

                $message=$videogallery -> vg_add_user_top($usersid,$vg_id,$usersnames);

                $arrct2='del';

                $resx=$message."<br><input class='button' type=\"button\" name=\"vg_user_fav_del\" id=\"txt2\" onclick=\"set_user_fav('$usersid','$vg_id','$usersnames','$arrct2')\" value='".VG_227."' /> ";

        }



}
/*еще одна строка занятая //// не помешает... так будет казаться что кода больше)) */

/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

if ($resx == "")

  {
/*мы веселые пацаны - на Error покажем смайл - ведь ошибок у нас много, если расстраиваться можно и дипресняк уйти */

  $response="Error :) (".$q.")";

  }

else

  {
/*оправляем первые заголовки браузеру, хотя вывод результатов голосования по словам автора был еще в 46 строке */

  header("Content-type: text/html; charset=utf-8");

  $response=$resx;

  }

/*что интересно, header с указанием кодировки мы в случае ошибки не отправляем... если все разваливается, то пусть разваливается полностью...*/

echo $response;

?>