PHP — Говнокод.ру

PHP / Говнокод #17067

+156

1
2
3
4
5
6
7
8

// где то: 
$_SESSION['type_school'] = $db->query("SELECT * FROM ou_type ",array(),'kassoc');

// а потом:
$smarty->assign('type_school',$_SESSION['type_school']);

//и в шаблоне:
<td>{$type_school[$v.base_type].type_name}</td>

$v.base_type - тоже результат запроса. Left/Right/Inner Join - не, не слышал...

А потом: чёта проект тормозит...
Я так подозреваю кто то таким методом оптимизацию и кеширование сделал, ну чтобы бд лишний раз не грузить.. Не, ну, а чё, зачем каждый раз базу данных мучить при открытии одной! странички со списком *****, давайте одну таблицу-справочник из бд откопируем каждому пользователю в его сессию отдельно и будем читать его при каждом session_start даже если на текущей странице нам этот справочник не нужен.

vGhost, 05 Ноября 2014

Комментарии (12)

PHP / Говнокод #17058

+156

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78

<?php
    class view {
        protected $dir; //templates directory
        protected $lang; //language
        protected $authorized;
        protected $user;

        protected function getCache($template) {
            //return false; //uncomment for developing
            if (!isset($_SESSION['cache_' . $template])) return false;
            return $_SESSION['cache_' . $template];
        }

        protected function addCache($template, $content) {
            $_SESSION['cache_' . $template] = $content;
        }

        public function __construct($dir, localization $lang, user $user) {
            $this->dir = $dir; 
            $this->authorized = (bool) $user->authorized;
            $this->user = $user;
            $this->lang = $lang;
        }

        public function invoke($template, $params = [], $return = false, $quests = []) { //can be called w/o params
            $filename = ROOT . '/' . $this->dir . '/tpl/' . $template . '.tpl';
            $lang = $this->lang->getData();
            $content = $this->getCache($template);
            if (!$content) {
                $f = fopen($filename, 'a+');
                $content = fread($f, (filesize($filename) > 0 ? filesize($filename) : 1));
                $this->addCache($template, $content);
            }
            foreach ($params as $key => $value) {
                $content = str_ireplace('{{' . $key . '}}', $value, $content);
            } 
            preg_match_all("@{{:([a-z0-9_]+?)}}@sui", $content, $localization);
            $localization = $localization[1];
            foreach ($localization as $value) {
                $content = str_ireplace('{{:' . $value . '}}', $lang[$value], $content);
            } //applying lang

            foreach ($quests as $key => $value) {
                preg_match_all("@{\?$key=$value\?}(((?!{\?.+\?}).)*?){\?\?}@sui", $content, $matches);
                while (!empty($matches[0])) {
                    $content = str_replace($matches[0][0], $matches[1][0], $content);
                    preg_match_all("@{\?$key=$value\?}(((?!{\?.+\?}).)*?){\?\?}@sui", $content, $matches);
                }
                preg_match_all("@{\?$key=((?!$value).+?)\?}(((?!{\?.+\?}).)*?){\?\?}@sui", $content, $matches);
                while (!empty($matches[0])) {
                    $content = str_replace($matches[0][0], "", $content);
                    preg_match_all("@{\?$key=((?!$value).+?)\?}(((?!{\?.+\?}).)*?){\?\?}@sui", $content, $matches);
                }
            }

            preg_match_all("@{\?access=([a-z0-9]+?)\?}(((?!{\?.+\?}).)*?){\?\?}@sui", $content, $perms);
            while (!empty($perms[0])) {
                foreach ($perms[1] as $value) {
                    if ($this->user->canAccess($value))
                        $content = preg_replace("@{\?access=$value\?}(((?!{\?.+\?}).)*?){\?\?}@sui", "$1", $content);
                    else $content = preg_replace("@{\?access=$value\?}(((?!{\?.+\?}).)*?){\?\?}@sui", "", $content);
                }
                preg_match_all("@{\?access=([a-z0-9]+?)\?}(((?!{\?.+\?}).)*?){\?\?}@sui", $content, $perms);
            }

            $content = preg_replace("@{\?authorized=((?!" . (int) $this->authorized . ").+?)\?}(.+?){\?\?}@sui", "", $content);
            $content = preg_replace("@{\?authorized=" . (int) $this->authorized . "\?}(.+?){\?\?}@sui", "$1", $content);
            $content = preg_replace("@{\?(.+?)\?}(.+?){\?\?}@sui", "", $content);

            $content = str_ireplace('{{DIR}}', '/' . $this->dir, $content); //replacing DIR param
            $content = str_ireplace('{{URI}}', urlencode(other::filter($_SERVER['REQUEST_URI'])), $content); //replacing URI param
            $content = str_ireplace('{{HTTP_HOST}}', $_SERVER['HTTP_HOST'], $content); //replacing HTTP_HOST param
            $content = preg_replace("@{\?((.+?)|(.+?){0})\?}@sui", "", $content);
            if (!$return) echo $content;
            return $content;
        }
    }
?>

Мой шаблонизатор. Детям и беременным женщинам не смотреть.

Efog, 03 Ноября 2014

Комментарии (113)

PHP / Говнокод #17035

+156

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

<?php
class ACL  
{  
    var $perms = array();     // Массив : Содержит привилегия текущего пользователя
    var $userID = 0;          // Целое число : Содержит ID текущего пользователя
    var $userRoles = array(); // Массив : Содержат роли текущего пользователя
  
    function __constructor($userID = '')  
    {  
        if ($userID != '')  
        {  
            $this->userID = floatval($userID);  
        } else {  
            $this->userID = floatval($_SESSION['userID']);  
        }  
        $this->userRoles = $this->getUserRoles('ids');  
        $this->buildACL();  
    }  
    function ACL($userID='')  
    {  
        $this->__constructor($userID);  
    } 
?>

Конструктор
__constructor() предназначен для того, чтобы инициализировать объект при создании экземпляра класса ACL. Он вызывается автоматически после вот этой записи: new ACL();

Не сразу понял, что логика тут есть, но какая-то кривая

DmitryDick, 02 Ноября 2014

Комментарии (9)

PHP / Говнокод #17004

+145

1
2
3
4
5
6
7
8
9

function myscandir($dir){
	if(!file_exists($dir)){
		return false;
	}

    $list = scandir($dir);
    unset($list[0],$list[1]);
    return array_values($list);
}

taras_shs, 31 Октября 2014

Комментарии (26)

PHP / Говнокод #17003

+155

01
02
03
04
05
06
07
08
09
10

function generateSession()
{
    $chars = "qazxswedcvfrtgbnhyujmkiolp1234567890QAZXSWEDCVFRTGBNHYUJMKIOLP";
    $max = rand(20, 32);
    $size = StrLen($chars) - 1;
    $sessionID = null;
    while ($max--)
        $sessionID .= $chars[rand(0, $size)];
    return $sessionID;
}

taras_shs, 31 Октября 2014

Комментарии (14)

PHP / Говнокод #16992

+156

1
2

if($GLOBALS["curLogin"]!='sasha'){ echo '<div class="alert alert-error" style="margin:5px;"><button class="close" data-dismiss="alert" type="button">×</button>
Раздел на реконструкции. Ориентировочно на 20-30 минут. Прошу прощения за неудобства</div>'; exit; }

tcore, 30 Октября 2014

Комментарии (15)

PHP / Говнокод #16988

+145

01
02
03
04
05
06
07
08
09
10

ПРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР
РРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРРР

ПОСРАЛ В ГОВНОКОДАЧОК НЕ СМЫЛ БАЧОК

Stulchak, 30 Октября 2014

Комментарии (0)

PHP / Говнокод #16956

+158

1
2
3

$element['yandex'] = 14;
unset($element['yandex']);
$element['yandex'] = 60;

Разраб упоролся в каштан :)

sautchenkov, 28 Октября 2014

Комментарии (17)

PHP / Говнокод #16955

+158

1
2
3
4

for($i = 0; $i < $db->max; $i++) {
	if($i==1) continue;
	.....
}

Почему бы и нет

UncaughtException, 28 Октября 2014

Комментарии (10)

PHP / Говнокод #16948

+159

1
2
3
4
5

if (AllowOperation(array(1,2,3,4,5,9,10,29), array(), array(), array(), -1, -1)) { 
        } else {
        	//скрывать определённые поля для дизайнера
			$tplCommon->assign("DISPLAY_NONE", "style='display: none;'");  
        }

В AllowOperation указываются идентификаторы групп пользователей, что забиты в БД.
Таких AllowOperation(array(1,2,3,4,5,9,10,29). .), но с разными идентификаторами во всей системе больше 1000.

pavel87, 27 Октября 2014

Комментарии (2)

Говнокод: по колено в коде.

PHP / Говнокод #17067

PHP / Говнокод #17058

PHP / Говнокод #17035

PHP / Говнокод #17004

PHP / Говнокод #17003

PHP / Говнокод #16992

PHP / Говнокод #16988

PHP / Говнокод #16956

PHP / Говнокод #16955

PHP / Говнокод #16948