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

    +158

    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
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    /**
     * Метод проверяет права доступа на использование этого модуля, возвращает результат проверки.
     * Кроме этого, происходит запись в лог
     *
     * @return boolean
     */
    private function check_access() {
    	Logger::log_exception($_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'], $this->log_file_name);
    	return true;
    }
    
    // [...]
    
    /**
     * Попытка переключить рабочую базу данных
     */
    public function actionChange_db() {
    	if (!$this->check_access()) {
    		echo strval($this->err_failure_access);
    		return;
    	}
    
    // [...]

    Вот такое иногда встречается в коде коллеги, про название метода "log_exception" я промолчу, т.к. это тоже его рук дело, а общий для всех метод Logger::log() ему почему-то не понравился? Я уже просто устал ругаться с ним...

    Запостил: cr0t, 02 Июня 2010

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

    • показать все, что скрыто
      Logger::log("ПОНИ")
      Ответить
      • Вместо С++ теперь пони.
        Скоро Страйкеру придется опять коммендантскую неделю вводить.
        Провеку регистрации по месту жительства и т.д.
        Ответить
    • возьми и уеби (с)
      Ответить
    • взъеби коня!
      Ответить
    • Не беспокоят ли вас гесты?
      Ответить
    • мухахаха, $_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'] - мая ржоть......
      $_SERVER['PHP_SELF'] нигадица?
      Ответить
      • нигадица.
        оба наговнякали: аффтар дублирует GET-данные, а ты их игноришь. $_SERVER['REQUEST_URI'] уже достаточно.
        Ответить
        • В общем-то, не достаточно, т.к. REQUEST_URI - это одно, а запускаемый PHP-скрипт сам по себе - это другое. Роутинг, понимаете ли...
          Но показанном мной случае все равно на лицо полное не знание внутренней структуры фреймворка и попытка использования только стандартных PHP-средств (хотя фреймворк сам может прекрасно "сказать", какой экшн в каком контроллере был вызыван, чтобы записать это в лог "двумя словами" :)
          Ответить
          • а ну канеш) как это я не заметил в гомне роутинга))
            на кой болт держат этих выродков, две строчки связать не могут, лучше б дворниками въябывали
            Ответить
          • о май гад, пхп-фреймворк? какой же, позвольте полюбопытствовать?

            а вообще при отладке фреймворка мне тож приходилось брать и гет, и пост, и сессион
            Ответить
    • Logger::log_exception($_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'], $this->log_file_name);
      	return true;

      похуй на ошибки при логгировании - возвращаем true и ниибет
      Ответить

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