Javascript.RU

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

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);		

	}

Про работоспособность кода ничего писать не буду (он тоже не без упрека, даже этот метод).
Казалось бы ничего страшного в этом стиле нет, но если представить класс целиком...
Руки бы вырвал тому, кто это писал.
Ответить с цитированием
  #2 (permalink)  
Старый 23.08.2017, 17:32
Аватар для Vlasenko Fedor
Профессор
Отправить личное сообщение для Vlasenko Fedor Посмотреть профиль Найти все сообщения от Vlasenko Fedor
 
Регистрация: 13.03.2013
Сообщений: 1,572

Nexus,
явно не на тот форум запостили. Вам сюда по ходу http://govnokod.ru/
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск