"PHP" / Говнокод #20135 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
public function selectValueType($value)
    {
        $type = null;
        if (App::helperPhone()->parseMobile($value)) {
            $type = 1; //телефон    
        }
        if (App::helperEmail()->validate($value)) {
            $type = 2; //email
        }
        return $type;
    }



А потом  в десятке методов:

 public function getLogin($value)
    {
        $type = $this->selectValueType($value);
        switch ($type) {
            case 1:
                $type = 'phone';
                break;
            case 2:
                $type = 'email';
                break;
        }
        if ($type) {
            $value = $type == 'phone' ? App::helperPhone()->parseMobile($value) : $value;
           // ... прочий говнокод
        }
}

Вот такие перлы достались в наследство. Приходится проводить полный рефакторинг :((

Запостил: Panda Panda, (Updated )

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

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?


    8