- 1
- 2
- 3
if (strlen($arResult["ERROR_MESSAGE"]) <= 0){
// любая ахинея
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 4
+157
if (strlen($arResult["ERROR_MESSAGE"]) <= 0){
// любая ахинея
}
2014 год. Битрикс все так же терпеливо ожидает патент на строки отрицательной длины.
Полагаю, такой икспрешн используется не только в компоненте корзины, учитывая страсть авторов к копипасте.
+154
if ($_SERVER["REQUEST_METHOD"] == "POST" && strlen($_REQUEST["backButton"]) > 0 && ($arParams["DELIVERY_NO_SESSION"] == "N" || check_bitrix_sessid()))
{
if($arResult["POST"]["CurrentStep"] == 6 && $arResult["SKIP_FORTH_STEP"] == "Y")
$arResult["CurrentStepTmp"] = 3;
if($arResult["POST"]["CurrentStepTmp"] <= 5 && $arResult["SKIP_THIRD_STEP"] == "Y")
$arResult["CurrentStepTmp"] = 2;
if($arResult["POST"]["CurrentStepTmp"] <= 3 && $arResult["SKIP_SECOND_STEP"] == "Y")
$arResult["CurrentStepTmp"] = 1;
if(IntVal($arResult["CurrentStepTmp"])>0)
$arResult["CurrentStep"] = $arResult["CurrentStepTmp"];
else
$arResult["CurrentStep"] = $arResult["CurrentStep"] - 2;
$arResult["BACK"] = "Y";
}
if ($arResult["CurrentStep"] <= 0)
$arResult["CurrentStep"] = 1;
$arResult["ERROR_MESSAGE"] = "";
Компонент страницы заказа в магазине Битрикса, 14я версия.
Тут вам и отсутствие констант для осмысленных шагов, и отсутствие конфигурирования этих шагов, как они i18n-ятся, вообще не представляю.
КАК ЭТО ВСЁ РАБОТАЕТ??!!!
ЗЫ мало того, тут же в компонент вкопипащен (или переговнокоден заново, не сравнивал) процесс залогинивания/регистрации нового юзера, елси он не залогинен. Не смог перекопипастить эту копипасту лишь потому, что она содержит 310 строк. Неслабый DRY.
+172
$var=trim($var);
$var=str_replace("\t", ' ',$var);
$var=str_replace("\r", ' ',$var);
$var=str_replace("\n", ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
$var=str_replace(' ', ' ',$var);
Надежно, чо.
+159
##############################################
# Bitrix: SiteManager #
# Copyright (c) 2002-2006 Bitrix #
# http://www.bitrixsoft.com #
# mailto:[email protected] #
##############################################
if (!class_exists("CCaptcha"))
{
class CCaptcha
{
var $imageWidth = 180;
var $imageHeight = 40;
// ...etc
Это Битрикс. Опять. bitrix\modules\main\classes\general\capt cha.php
Определение нативной капчи.
В строке 8 создатели сего как бы задаются воспросом «А вдруг еще никто не писал до нас капчи?».
Или перестраховываются — «а вдруг require() уже вызывался? И что такое require_once(), про который все так много говорят?»
Добротный, защищенный на все сто, класс капчи. Невозможно сломать, уже просто потому, что невозможно понять...