Как по очереди проверять и по очереди убирать лишнее?

Question

[Yuriy Lyadov]

Всем привет!
Гуру PHP подскажите, пожалуйста...
Не знаю понятно ли объясню, нужно в

foreach ($filters as $keys => $tegi) {
if($tegi[1] == $id['TIP_TOVARA']) {
if($tegi[1] == $id['AFFILIATION']) {

echo $tegi[0];

}
}
}

В $filters выдает массив с названием и id.
Но он при проверке TIP_TOVARA показывает а вот при проверке AFFILIATION уже все, ничего не выдает. Можно ли в этом случае что то сделать или надо до этого массив как то разбирать?

Comments

[AgentSmith72]

foreach ($filters as $keys => $tegi)
Корректно $key, так как у каждого элемента $filters один ключ.
Соответственно, если в одном ключе несколько тегов, то это $tags

В компаниях большое внимание уделяется читаемости кода. Соответственно, не TIP_TOVARA (большими буквами принято в php называть константы!), а просто type.

[Yuriy Lyadov]

AgentSmith72, TIP_TOVARA - это код свойства который создался при выгрузке из 1с.

[AgentSmith72]

mills, предполагаю, что тогда допустимо такое имя ключа в $id.

Вы можете сделать, как вам предложено в ответе, сделать var_dump($id) и соответственно одного из $tegi. Люди подскажут, как лучше реализовать проверку.

У вас сейчас реализовано так, что обе проверки значения $id должны вернуть true.

if($tegi[1] == $id['type'] and $tegi[1] == $id['affiliation']) echo $tegi[0];

[Yuriy Lyadov]

AgentSmith72, Вообще если пойти с самого начала, то я вывожу список элементов где одно из свойств выводит:

/catalog/?set_filter=y&MAX_SMART_FILTER_687_4089903629=Y&MAX_SMART_FILTER_723_703376357=Y&MAX_SMART_FILTER_723_780234748=Y&MAX_SMART_FILTER_723_1501978474=Y&MAX_SMART_FILTER_723_3079159366=Y&MAX_SMART_FILTER_723_1903002705=Y&MAX_SMART_FILTER_723_1345327937=Y

/catalog/?set_filter=y&MAX_SMART_FILTER_687_4089903629=Y&MAX_SMART_FILTER_940_512731071=Y&MAX_SMART_FILTER_723_703376357=Y&MAX_SMART_FILTER_723_780234748=Y&MAX_SMART_FILTER_723_1501978474=Y

/catalog/?set_filter=y&MAX_SMART_FILTER_687_4089903629=Y&MAX_SMART_FILTER_603_3673936931=Y&MAX_SMART_FILTER_605_2680849975=Y&MAX_SMART_FILTER_605_3743148538=Y&MAX_SMART_FILTER_605_2336599219=Y

Потом его разделяю на массив где MAX_SMART_FILTER_687_4089903629 то что мне нужно и уже этот ID я сравниваю с тем что выдается в карточке товара... Скорей всего я не так разместил в массиве и мне сейчас не удобно работать с этим.

Пробовал при помощи strpos в этом то что верху искать и выводить что находит, но выводит и то что нужно и то что не нужно.

[AgentSmith72]

mills, strpos возвращает позицию первого вхождения подстроки в строке.
Возвращает int (Символ от начала строки) или false (если не найдено).

Обычно пригождается для сложных связок, например когда нужно динамически формировать условие для substr.

[Yuriy Lyadov]

AgentSmith72, да в принципе знаю. Вот и думаю как бы нормально эту строку разделить и вытащить ID которое выделил что бы вывести верные элементы))

[AgentSmith72]

mills, рекомендую присмотреться к регулярным выражениям. Сможете решать всевозможные задачи по поиску и проверке. В разы уменьшите свой код и время разработки.

Разбивать строку обычно надо, когда вы используете ее отдельные компоненты в разных операциях или распределяет их по отдельным массивам.

Для поиска в строке, достаточно применить регулярку.

Answer 1

[AgentSmith72]

Вам пригодятся регулярные выражения
Сервис для проверки регулярных выражений

Потом его разделяю на массив где MAX_SMART_FILTER_687_4089903629 то что мне нужно и уже этот ID я сравниваю с тем что выдается в карточке товара

В таком случае достаточно проводить проверку с помощью регулярных выражений:

$regExp = '#' . $id['affiliation'] . '#';
foreach($arr as $key => $string) if(preg_match($regExp, $string)) unset($arr[$key]);

В $regExp размещаете своё условие для поиска.
$arr это ваш массив со строками.

Делаете перебор массива со строками.
При нахождении нужного id, удаляете из массива со строками данный элемент.

Если нужна помощь в составлении регулярного выражения, обращайтесь.

Comments

[Yuriy Lyadov]

while ($arItemTeg = $tegs->GetNext()){
	foreach ($arResult['PROPERTIES'] as $kay => $values) {
		$regExp = '#'.abs(crc32(htmlspecialcharsbx($values['VALUE_ENUM_ID']))).'#';
		if(abs(crc32(htmlspecialcharsbx($values['VALUE_ENUM_ID']))) != 0){
			if (preg_match($regExp, $arItemTeg["PROPERTY_PAGE_URL_VALUE"])) {
				$tegs_massive[$arItemTeg['NAME']] = array($arItemTeg['PROPERTY_GOOD_URL_VALUE'], 
             $arItemTeg['NAME']);
			}
		}
	}
}

Я попробовал preg_match но разницы особо не увидел. Вы написали unset удалять из массива то что нашел, но мне не надо удалять, а наоборот оставить то что нашел.
Он как бы находит все ок, но вот если у 1 тега есть код свойства пол мужской и цвет белый к примеру,а у другого тега есть свойства пол женский и цвет белый и он выдаст мужская футболка белая и женская футболка белая, потому что у того и другого есть код свойства "Белая" ))

while ($arItemTeg = $tegs->GetNext()) - выводит строки которую я показывал выше /catalog/?set_filter=y&MAX_SMART_FILTER_687_4089903629=Y&MAX_SMART_FILTER_603_3673936931=Y&MAX_SMART_FILTER_605_2680849975=Y&MAX_SMART_FILTER_605_3743148538=Y&MAX_SMART_FILTER_605_2336599219=Y

foreach ($arResult['PROPERTIES'] as $kay => $values) - массив с id свойств товара, которые я проверяю со строкой выше и вывожу нужные элементы.

[AgentSmith72]

mills,

if(!preg_match($regExp, $string)) unset()      // ! - чтобы удалить то что не находит
if(preg_match($regExp, $string, $match))      // $match - содержит найденный результат

Я понял ваш код и что вам требуется. Уточните критерии по которым вы хотите отфильтровать. У вас в коде нет того самого критерия, по которому вы бы отделили пол или цвет.

На данный момент у вас реализована проверка, есть ли данный тег в get-параметрах. Нужно продумать дальше, как произвести фильтрацию строк, в которых найдены тэги.

Можно как усложнить регулярное выражение, так и реализовать фильтрации по полу и цвету, внутри конструкции if, если регулярка нашла совпадение.

Попробуйте добавить к тэгу критерий для выбора только белого цвета. Если критерий не однозначен, то создавайте $regExp динамически, через if() else {}

$regExp = $tag . '=white';     // Вместо white используйте ваши обозначения

Внутри конструкции if можно провести дополнительную фильтрацию и сохранить данную строку в новый массив.

[Yuriy Lyadov]

AgentSmith72,

Написал так, $regExp и $regExp2 боле менее выводил, а потом когда уже $regExp3 цвет вставил, то видимо уже не нашел его.

while ($arItemTeg = $tegs->GetNext()){
	$regExp = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TIP_TOVARA']['VALUE_ENUM_ID']))).'#';
	$regExp2 = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['AFFILIATION']['VALUE_ENUM_ID']))).'#';
	$regExp3 = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TSVET_OSNOV']['VALUE_ENUM_ID']))).'#';

	if (preg_match($regExp, $arItemTeg["PROPERTY_PAGE_URL_VALUE"]) && preg_match($regExp2, $arItemTeg["PROPERTY_PAGE_URL_VALUE"]) && preg_match($regExp3, $arItemTeg["PROPERTY_PAGE_URL_VALUE"])) {
		$tegs_massive[$arItemTeg['NAME']] = array($arItemTeg['PROPERTY_GOOD_URL_VALUE'], $arItemTeg['NAME']);
	}
}

[AgentSmith72]

mills, да регулярку бывает нужно тестировать.
Мастер регулярных выражений

Попробуйте добавить одну из строк в сервис и вручную найти каждый из тегов в ней. Так увидите причину, где у вас не те данные, в тэгах или в get-параметрах.

Вам нужно чтобы строго все три тега присутствовали в строке?
Если нужны вхождения тегов, то можно сгруппировать условия

$regExp = '#(' . $tag1 . '|' . $tag2 . '|' . $tag3 . ')#';
if(preg_matchAll($regExp, $string, $matches) {   // $matches содержит все найденные тэги
    if(count($matches) == 3) save();
}

[Yuriy Lyadov]

AgentSmith72, Добрый день. С данным изменением выдает все.

while ($arItemTeg = $tegs->GetNext()){

	$regExp = '#(' . abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TIP_TOVARA']['VALUE_ENUM_ID']))) . '|' . abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['AFFILIATION']['VALUE_ENUM_ID']))) . '|' . abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TSVET_OSNOV']['VALUE_ENUM_ID']))) . ')#';

	if (preg_match_all($regExp, $arItemTeg["PROPERTY_PAGE_URL_VALUE"])) {
		$tegs_massive[$arItemTeg['NAME']] = array($arItemTeg['PROPERTY_GOOD_URL_VALUE'], $arItemTeg['NAME']);
	}

}

Там может быть и 2 и 3 и более тегов выводить. А теги это посадочные страницы другого инфоблока.
К примеру вот каждая эта ссылка это 1 элемент со своим заголовком и ссылкой, это и есть тег:

/catalog/?set_filter=y&MAX_SMART_FILTER_687_499686177=Y&MAX_SMART_FILTER_940_512731071=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_4089903629=Y&MAX_SMART_FILTER_940_1770559273=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_499686177=Y&MAX_SMART_FILTER_940_2162897436=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_499686177=Y&MAX_SMART_FILTER_940_512731071=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_3162991818=Y&MAX_SMART_FILTER_940_1770559273=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_3162991818=Y&MAX_SMART_FILTER_940_512731071=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_630078832=Y&MAX_SMART_FILTER_940_1770559273=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_630078832=Y&MAX_SMART_FILTER_940_512731071=Y
/catalog/?set_filter=y&MAX_SMART_FILTER_687_4089903629=Y&MAX_SMART_FILTER_723_703376357=Y&MAX_SMART_FILTER_723_780234748=Y&MAX_SMART_FILTER_723_1501978474=Y&MAX_SMART_FILTER_723_3079159366=Y&MAX_SMART_FILTER_723_1903002705=Y&MAX_SMART_FILTER_723_1345327937=Y

Таких элементов примерно 30шт на данный момент. В карточке товара где нужно вывести свою ссылку/тег по 3 свойствам поиск надо осуществить, у каждого свойства ID 3162991818 стольки значное число.
На данный момент как бы я не делал поиски по элементам, в карточке где мужская футболка черная, он выводит тег и женскую футболку и белую... Как бы заставить его искать строго по параметрам. И еще такой момент у ссылок есть, что у многих есть тип товара "Футболка" но нет пола и нет цвета... Он как я понял их тоже игнорирует и не выводит

[Yuriy Lyadov]

AgentSmith72, Написал так, но получилось много кода:

while ($arItemTeg = $tegs->GetNext()){
	$regExp = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TIP_TOVARA']['VALUE_ENUM_ID']))).'#';
	$regExp2 = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['AFFILIATION']['VALUE_ENUM_ID']))).'#';
	$regExp3 = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TSVET_OSNOV']['VALUE_ENUM_ID']))).'#';

	$regExp4 = '#'.$arResult['PROPERTIES']['AFFILIATION']['ID'].'#';

	if (preg_match($regExp, $arItemTeg["PROPERTY_PAGE_URL_VALUE"])) {
		$tegs_massive[] = array($arItemTeg["PROPERTY_PAGE_URL_VALUE"], $arItemTeg['PROPERTY_GOOD_URL_VALUE'], $arItemTeg['NAME']);
	}
}

foreach ($tegs_massive as $value) {

	if (!preg_match($regExp4, $value[0]) || preg_match($regExp2, $value[0])) {
		?>
		<a href="<?=$value[1];?>"><?=$value[2];?></a><br>
		<?
	}
}

Сначала я проверяю тип, потом в следующем foreach я делаю проверку по полу $regExp2 и проверку по есть ли в ссылке свойство пол $regExp4 оно в коде трех значное число пример 940... Все бы ничего, но щас цвет суется, в товаре черного цвета, суется белый)) Как бы щас от этого избавиться, но опять же как с полом если свойства нет, то выводить...

[AgentSmith72]

mills, дело за малым, продумать различные инструкции фильтрации.

Для лучшей читаемости, while можно вынести в отдельный метод, который будет возвращать массив тегов. Соответственно перенести из цикла регулярки которые не нужны методу, добавив их перед циклом foreach

[Yuriy Lyadov]

AgentSmith72, Вроде бы сделал и вроде бы все выдает как положено))

while ($arItemTeg = $tegs->GetNext()){
	$tegs_massive[] = array($arItemTeg["PROPERTY_PAGE_URL_VALUE"], $arItemTeg['PROPERTY_GOOD_URL_VALUE'], $arItemTeg['NAME']);
}

$regExp = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TIP_TOVARA']['VALUE_ENUM_ID']))).'#';
$regExp2 = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['AFFILIATION']['VALUE_ENUM_ID']))).'#';
$regExp3 = '#'.abs(crc32(htmlspecialcharsbx($arResult['PROPERTIES']['TSVET_OSNOV']['VALUE_ENUM_ID']))).'#';

$regExp4 = '#'.$arResult['PROPERTIES']['AFFILIATION']['ID'].'#';
$regExp5 = '#'.$arResult['PROPERTIES']['TSVET_OSNOV']['ID'].'#';


foreach ($tegs_massive as $value) {

	if (preg_match($regExp, $value[0])) {
		if (!preg_match($regExp4, $value[0]) || preg_match($regExp2, $value[0])) {
			if (!preg_match($regExp5, $value[0]) || preg_match($regExp3, $value[0])) {
				?>
				<a href="<?=$value[1];?>"><?=$value[2];?></a><br>
				<?
			}
		}
	}
}

Спасибо что меня направили в нужное направление))

[Yuriy Lyadov]

AgentSmith72, Добрый день. Такой вопрос, можно ли в регулярном прописать ИЛИ ? Я пишу #test#|#test2# но что то не выходит. или я не так что то делаю?

[AgentSmith72]

mills, вместо # можно использовать иные, но главное одинаковые символы, например слэши. Они, подобно фигурным скобкам, или тэгам в html, оборачивают регулярное выражение.

А уже сама регулярка пишется внутри них.
#test|test2#

Но если, как в примере выше, с разницей в несколько символов на конце, то можно использовать не | ,а ?
@test2?@

К примеру эта регулярка найдет как test, так и test2. То есть 2 в конце со знаком вопроса и является не обязательным.

Также обратите внимание, на разницу работы preg_match и preg_match_all.
Посмотрите документацию
Краткая справка по возможностям регулярок

[Yuriy Lyadov]

AgentSmith72, Я как пример написал тесты, а на самом деле там цифры разные разница большая

[AgentSmith72]

mills, по скорости как отрабатывает скрипт? Дело в том, что регулярки потребляют много регурсов процессора, и в некоторых случаях компьютер может даже зависнуть.

[Yuriy Lyadov]

AgentSmith72, там сервер мощный. Спасибо большое, снова помогли. Такой вариант #test|test2# мне помог, переделал массив в строку в таком виде и он отфильтровал как надо. Да, регулярка делает)))

[AgentSmith72]

mills, Нужно правильно писать регулярки. Чем длиннее строка, тем точнее нужно указывать последовательность искомых символов.

Катастрофический возврат

[Yuriy Lyadov]

AgentSmith72, Спасибо, постараюсь посмотреть.

Answer 2

[Ипатьев]

А зачем if-ы вкладывать друг в друга?
Как $tegi[1] может быть одновременно равен $id['TIP_TOVARA'] и $id['AFFILIATION']?

Comments

[Yuriy Lyadov]

А например в foreach $tegi[1] что бы строгая проверка была по $id['TIP_TOVARA'] и $id['AFFILIATION'] ?
пробовал $id собрать в массив и сделать проверку in_array но он выдает лишние

[Ипатьев]

КАКАЯ проверка?
В этом коде вообще нет никаких проверок, только вывод.
Мы можете внятно вопрос задать? Приведя здесь примеры массивов $filters и $id и описав здесь конечный результат?
От какого $tegi надо выводить $tegi[0], если совпадут оба? или у $tegi[0] всегда одно и то же значение?

[Yuriy Lyadov]

Вот такой вывод foreach $tegi

Вот выводится $id['TIP_TOVARA'] и $id['AFFILIATION']


И надо что бы он все отсеил и оставил только те id которые в $id['TIP_TOVARA'] и $id['AFFILIATION']

[Ипатьев]

Я у вас просил вывод $tegi? Смотрите, что я писал:
примеры массивов $filters и $id

Если у вас $tegi состоит из одного элемента, то почему вы обращаетесь к $tegi[1]?

[Yuriy Lyadov]

Ипатьев, Я переписал его и убрал от туда $tegi[1]
По сути я кинул то что вы просили

[Ипатьев]

ну по сути я вам ответил

[AUser0]

mills, чего-то вы недоговариваете... Лучше сделайте:
var_dump($filters); var_dump($id);
и покажите сюда результат этих var_dump().