Показать сообщение отдельно
  #1 (permalink)  
Старый 23.08.2017, 10:34
Профессор
Отправить личное сообщение для Nexus Посмотреть профиль Найти все сообщения от Nexus
 
Регистрация: 04.12.2012
Сообщений: 3,730

Bad programing style [PHP, example]
Весь backend за исключением ядра CMS в этом стиле.
Метод «remove_debris» класса «registration».

/**

	 * удаление мусора(все в нижний регистра, удаление знаков) из строки

	 * @param unknown $str

	 */

	protected function remove_debris($str)

	{

		$str = strtolower($str);

		$str = preg_replace("#[[:punct:]]#", "", $str);

		$str = str_replace(' ', '', $str);

		

		return mb_strtolower($str);		

	}

Про работоспособность кода ничего писать не буду (он тоже не без упрека, даже этот метод).
Казалось бы ничего страшного в этом стиле нет, но если представить класс целиком...
Руки бы вырвал тому, кто это писал.
Ответить с цитированием