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

    +143

    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
    if(!$user['email']) {
     $err = $this->system_message->getLang('empty_email');
} elseif(strlen($user['email']) < self::MIN_USER_NAME) {
      $err = $this->system_message->getLang('small_email');
} elseif(strlen($user['email']) > self::MAX_USER_NAME) {
      $err = $this->system_message->getLang('long_email');
} elseif(!filter_var($user['email'], FILTER_VALIDATE_EMAIL)) {
      $err = $this->system_message->getLang('incorrect_name');
} 

if(!$user['passw']) {
      $err = $this->system_message->getLang('empty_passw');
} elseif(strlen($user['passw']) < self::MIN_PASSWORD_NAME) {
      $err = $this->system_message->getLang('small_passw');
} elseif(strlen($user['passw']) > self::MAX_PASSWORD_NAME) {
      $err = $this->system_message->getLang('long_passw');
}
if($err) {
   //!TODO add a exeption
		
} else {
      $find_email = $this->db_users->prepare("SELECT * FROM `users` WHERE `email` = :email");
      $find_email->execute(array(':email' => $user['email']));

    Мдэээ...

    Запостил: Dev_18, 12 Мая 2015

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

    • ava Arris 12.05.2015 21:04 # 0

      А в чем говнокод то?
      Ответить

      • ava Dev_18 13.05.2015 06:12 # 0

        Мне кажется что он плох
        Ответить

      • ava Stallman 13.05.2015 11:47 # 0

        Не столько говнокод, сколько говноархитектура. Как говнокод, конечно, УГ, ибо такого говна тонны в почти каждом пыхопроекте, но тем не менее вполне достойно быть здесь.
        Ответить

        • ava Arris 13.05.2015 16:11 # 0

          *с улыбкой*
          А в чем архитектура то говно? :)
          Ответить

          • ava Dev_18 13.05.2015 18:18 # 0

            В плане, что нужно использовать готовые классы для проверки пользовательских данных.
            Ответить

            • ava Arris 13.05.2015 18:45 # 0

              Даешь максимальную атомизацию проекта, да? :)
              Ответить

    • ava Dev_18 27.05.2015 20:22 # −1

      Эй говнюки, схрена заминусовали-то?
      Ответить

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