1. PHP / Говнокод #17450

    +165

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    public static function isOwner()
    {
            $user = app::getSession();
            if ($user->isAuth()){
    
                switch (true){
                    case ($_GET('action') == 'upload'):
                    case ($_GET('login') == $user->getUserLogin()):
                        return true;
                        break;
                    default:
                        return false;
                        break;
                }
    
            }
            return false;
        }

    Проверка является ли пользователь владельцем фотогалереи)))

    Запостил: Gkoder_, 15 Января 2015

    Комментарии (8) RSS

    • А оно вообще правильно работает ?
      Получается, что если я пытаюсь залить и я авторизован, то я владелец
      Ответить
    • $_GET('action');

      А оно вообще работает? Или где-то выше:
      $_GET = function($key) { /* ... */ };
      Ответить
      • Или с использованием ООП руками
        class Get
        {
        	private $_params;
        	
        	public function __construct(array $params)
        	{
        		$this->_params = $params;
        	}
        	
        	public function __invoke($key)
        	{
        		return isset($this->_params[$key]) ? $this->_params[$key] : null;
        	}
        }
        
        $_GET = new Get($_GET);
        Ответить
        • class TrolleybusIzBukhankiHleba

          Зачем глобальный ассоциативный массив оборачивать?
          Ответить
          • Это был сарказм.

            А вообще, пыхобляди почему-то любят оборачивать глобальные массивы говностатикой. Видел такое не на одном проекте.
            Ответить
            • Наверное чтобы потом писать в резюме "использую объектно-ориентированный подход к разработке"
              Ответить
      • Почти... там было
        function get($key) { /* ... */ };

        но для удобности(понятности) исправил на $_GET, а скобки поменять забыл((
        Ответить
    • Это первый говнокод, поэтому не судите строго))
      Ответить

    Добавить комментарий