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

    +160

    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
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    <?php

class UserSessionComponent extends Component {
	//...
	protected function loginByUserId($user_id, $is_autologin = false) {
		$list = $this->_loadComponents($this->components, array('UserAuth'));
		$userAuth = $list['UserAuth'];
		$userAuth->startup($this->controller); # !!! WORKFLOW VIOLATION !!!
		$method = $is_autologin ? 'autologin' : 'login';
		if ($userAuth->$method($user_id)) {
			return $this->controller->Session->read('User.data');
		}
	}
	//...
}

class UserAuthComponent extends BaseMagicComponent {
	protected $loginType = 'explicit';

	//...

	public function autologin($params = array()) {
		$this->loginType = 'implicit';
		return $this->_login($params);
	}

	public function login($params = array()) {
		$this->loginType = 'explicit';
		return $this->_login($params);
	}

	//...
	protected function _login($params = array()) {
		switch (true) {
			case (is_array($params) && isset($params['user']) && isset($params['password'])):
				$result = $this->loginByUserAndPassword($params['user'], $params['password']);
				break;
			case (is_numeric($params) && $params > 0):
				$result = $this->loginUser($params);
				break;
			default:
				if (empty($params)) {
					$result = false;
				} else {
					$result = $this->loginUser($params);
				}
		}
		// ...
	}

	// ...
	public function loginUser($user) {
		switch (true) {
			case is_object($user):
				if (isset($user->data)) {
					$a_user = $user->data;
				} else {
					return false;
				}
				break;
			case (is_numeric($user) && ($user > 0)):
				if (!($a_user = $this->User->findById($user))) {
					return false;
				}
				break;
			case (is_array($user) && (isset($user['User']['id']))):
				$a_user = $user;
				break;
			default:
				return false;
		}
		// ...
		CFL::using('log');
		if (empty($GLOBALS['loginIsLogged'])) {
			$loginLog = new cflLoginTrackingLogger();
			if ($this->loginType == 'implicit') {
				$loginLog->implicit_login(array('user_id' => $a_user['User']['id']));
			} else {
				$loginLog->explicit_login(array('user_id' => $a_user['User']['id']));
			}
			$GLOBALS['loginIsLogged'] = true;
		}
		// ...
	}
}

    Сорри за много-много кода, еле-еле вписался в лимит. Но пришлось запостить все. Логическая цепочка совершенно потрясающая (смотрим с самого верхнего метода и идем дальше к самому нижнему методу). Тут у нас кода просто по колено: сумасшедший пахлава-код, использование глобальной переменной флага, особое извращенное использование конструкции switch, фактическое дублирование этого самого switch в двух последних методах, нейминг переменных и методов...
    Казалось бы, как все это вообще могло быть написано? Но использование компонента с говорящим именем BaseMagic все ставит на свои места.

    Запостил: quall, 09 Февраля 2011

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

    • ava quall 09.02.2011 11:16 # −1

      Я вот вообще с трудом понимаю, как можно было собрать столько перлов в одном месте. Программист - настоящий ас своего дела. Я бы даже сказал - асс.
      Ответить

    • ava quall 09.02.2011 11:17 # −1

      На всякий случай - тут используется совершенно феерический фреймворк CakePHP - наверное, худшее, что случалось в моей жизни...
      Ответить

    • ava Lure Of Chaos 09.02.2011 13:32 # +1

      полный WORKFLOW VIOLATION
      Ответить

    • ava gegMOPO4 09.02.2011 13:42 # +2

      Оригинальное использование switch. Никогда такого не видел. Потрясающе!
      Ответить

    • ava Int 09.02.2011 23:03 # +2

      Чем столько всего читать и тратить своё время, я тебе просто поверю, что это говнокод. Этого достаточно?
      Ответить

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