Говнокод #24448 — PHP — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP / Говнокод #24448

public static function PhpToJSObject($arData, $bWS = false, $bSkipTilda = false, $bExtType = false)
	{
		static $use_bx_encode = null;
		if (!isset($use_bx_encode))
			$use_bx_encode = function_exists('bx_js_encode');
		if ($use_bx_encode)
			return bx_js_encode($arData, $bWS, $bSkipTilda, $bExtType);

		switch(gettype($arData))
		{
		case "string":
			if(preg_match("#['\"\\n\\r<\\\\\x80]#", $arData))
				return "'".CUtil::JSEscape($arData)."'";
			else
				return "'".$arData."'";
		case "array":
			$i = -1;
			$j = -1;
			foreach($arData as $j => $temp)
			{
				$i++;
				if ($j !== $i)
					break;
			}

			if($j === $i)
			{
				$res = '[';
				$first = true;
				foreach($arData as $key => $value)
				{
					if($first)
						$first = false;
					else
						$res .= ',';

					switch(gettype($value))
					{
					case "string":
						if(preg_match("#['\"\\n\\r<\\\\\x80]#", $value))
							$res .= "'".CUtil::JSEscape($value)."'";
						else
							$res .= "'".$value."'";
						break;
					case "array":
						$res .= CUtil::PhpToJSObject($value, $bWS, $bSkipTilda, $bExtType);
						break;
					case "boolean":
						if($value === true)
							$res .= 'true';
						else
							$res .= 'false';
						break;
					case "integer":
					case "double":
						if ($bExtType)
							$res .= $value;
						else
							$res .= "'".$value."'";
						break;
					default:
						if(preg_match("#['\"\\n\\r<\\\\\x80]#", $value))
							$res .= "'".CUtil::JSEscape($value)."'";
						else
							$res .= "'".$value."'";
						break;
					}
				}
				$res .= ']';
				return $res;
			}

			$sWS = ','.($bWS ? "\n" : '');
			$res = ($bWS ? "\n" : '').'{';
			$first = true;
			foreach($arData as $key => $value)
			{
				if ($bSkipTilda && substr($key, 0, 1) == '~')
					continue;

				if($first)
					$first = false;
				else
					$res .= $sWS;

				if(preg_match("#['\"\\n\\r<\\\\\x80]#", $key))
					$res .= "'".CUtil::JSEscape($key)."':";
				else
					$res .= "'".$key."':";

				switch(gettype($value))
				{
				case "string":
					if(preg_match("#['\"\\n\\r<\\\\\x80]#", $value))
						$res .= "'".CUtil::JSEscape($value)."'";
					else
						$res .= "'".$value."'";
					break;
				case "array":
					$res .= CUtil::PhpToJSObject($value, $bWS, $bSkipTilda, $bExtType);

В ядре битрикса в файле bitrix/modules/main/tools.php на 4613 строке, длинной в 141 строку (жаль здесь ограничение на 100 строк), есть замечательный метод который по сути почти повторяет json_encode().

Почти - это потому, что это говно генерит невалидный json (использует одинарные ковычки), да еще не учитывает типы (true превратится в строку 'true').

Битриксоиды этим подосрали даже не веб-студиям, они просто нагенерировали себе много дополнительной работы на долгое время вперед.

Желаю им держать эту планку, и никогда не расти.

Запостил:

gitkv, 05 Июля 2018

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

roskomgovno 05.07.2018 17:53 # +1

Умение изобретать квадратноколесный велосипед это пять.
Теперь пускай XmlToPHP напишут
Ответить
- Geffest 11.09.2018 23:28 # +1
  
  Если Вы считаете, что они не написали свой парсер XML, то вы очень жестко заблуждаетесь))
  Когда-то давно, когда еще только начинал знакомство с битриксом и PHP, решил использовать для решения реальной задачи их XML парсер (потому что не знал, какие вообще есть способы решить задачу). Все настроил, натравили скрипт на реальные данные. И оказалось, что работает очень долго, сервер падает и т.д. Тогда я чуть-чуть погуглил, заменил парсер на SimpleXML. И оказалось, что все работает прекрасно, быстро, никаких проблем больше не возникало. Вот в тот момент я понял, что из себя представляют все эти велосипеды от битрикса.
  А вообще да, PhpToJSObject это тот еще велосипед. Они в своем ядре его используют обычно, чтобы присваивать js переменной и потом использовать. Вообще я так думаю, они написали все эти велосипеды, чтобы битрикс работал на любом окружении, с любыми настройками сервера и т.д. Возможно, свой отпечаток наложил тот факт, что раньше большую долю составляло использование битрикса на кодировке 1251, где есть нюансы использования стандартных функций.
  
  А вообще. Это уже старое ядро. В новом ядре D7 ребята написали много новых велосипедов:
  http://bxapi.ru/src/?module_id=main&name=Json%3A%3Aencode
  http://bxapi.ru/src/?module_id=main&name=Json%3A%3Adecode
  По сути, теперь просто обертка для php функций.
  Ответить
guest8 05.07.2018 22:22 # −999

показать все, что скрытоvanished
Ответить

Говнокод: по колено в коде.

PHP / Говнокод #24448

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

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