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

    • ava guest 02.06.2010 20:12 # −9

      показать все, что скрыто
      Logger::log("ПОНИ")
      Ответить

      • ava mrbig66 02.06.2010 22:43 # +4

        Вместо С++ теперь пони.
        Скоро Страйкеру придется опять коммендантскую неделю вводить.
        Провеку регистрации по месту жительства и т.д.
        Ответить

    • ava This is obvious 02.06.2010 20:42 # +1

      возьми и уеби (с)
      Ответить

    • ava guest 02.06.2010 21:22 # −3

      взъеби коня!
      Ответить

    • ava da4ever 02.06.2010 21:43 # +4

      Не беспокоят ли вас гесты?
      Ответить

    • ava Lure Of Chaos 03.06.2010 18:33 # 0

      мухахаха, $_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'] - мая ржоть......
      $_SERVER['PHP_SELF'] нигадица?
      Ответить

      • ava xXx_totalwar 05.06.2010 09:00 # −1

        нигадица.
        оба наговнякали: аффтар дублирует GET-данные, а ты их игноришь. $_SERVER['REQUEST_URI'] уже достаточно.
        Ответить

        • ava cr0t 05.06.2010 11:48 # 0

          В общем-то, не достаточно, т.к. REQUEST_URI - это одно, а запускаемый PHP-скрипт сам по себе - это другое. Роутинг, понимаете ли...
          Но показанном мной случае все равно на лицо полное не знание внутренней структуры фреймворка и попытка использования только стандартных PHP-средств (хотя фреймворк сам может прекрасно "сказать", какой экшн в каком контроллере был вызыван, чтобы записать это в лог "двумя словами" :)
          Ответить

          • ava xXx_totalwar 05.06.2010 12:01 # 0

            а ну канеш) как это я не заметил в гомне роутинга))
            на кой болт держат этих выродков, две строчки связать не могут, лучше б дворниками въябывали
            Ответить

          • ava Lure Of Chaos 05.06.2010 12:51 # 0

            о май гад, пхп-фреймворк? какой же, позвольте полюбопытствовать?

            а вообще при отладке фреймворка мне тож приходилось брать и гет, и пост, и сессион
            Ответить

    • ava xXx_totalwar 05.06.2010 12:03 # 0 

      Logger::log_exception($_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'], $this->log_file_name);
	return true;

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

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