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

    • ava santa_microbe 15.01.2015 17:34 # 0

      А оно вообще правильно работает ?
      Получается, что если я пытаюсь залить и я авторизован, то я владелец
      Ответить

    • ava Stallman 15.01.2015 17:49 # +1 

      $_GET('action');

      А оно вообще работает? Или где-то выше:
      $_GET = function($key) { /* ... */ };
      Ответить

      • ava Stallman 15.01.2015 18:19 # 0

        Или с использованием ООП руками
        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);
        Ответить

        • ava Анонимус 15.01.2015 18:27 # 0

          class TrolleybusIzBukhankiHleba

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

          • ava Stallman 15.01.2015 18:40 # 0

            Это был сарказм.

            А вообще, пыхобляди почему-то любят оборачивать глобальные массивы говностатикой. Видел такое не на одном проекте.
            Ответить

            • ava Анонимус 15.01.2015 20:01 # +1

              Наверное чтобы потом писать в резюме "использую объектно-ориентированный подход к разработке"
              Ответить

      • ava Gkoder_ 15.01.2015 18:26 # 0

        Почти... там было
        function get($key) { /* ... */ };

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

    • ava Gkoder_ 15.01.2015 18:32 # 0

      Это первый говнокод, поэтому не судите строго))
      Ответить

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