1С-Битрикс Разработчикам - Можно ли так писать =)

Заглянувший

Сообщений: 29 Баллов: 1 Регистрация: 01.01.2013

29.11.2013 16:45:11

Добрый день подскажите, можно ли так писать с точке зрения оптимального кода. Пытаюсь учиться писать хороший код

Задача была такая: в отдельном компоненте вывести отзывы связанные с услугами которые в свою очередь связанны с мероприятиями.
Что я делаю в этом куске кода:
Беру id услуги из $_GET , затем выбираю все его свойства связанных мероприятий у данной услуги, в свою очередь у этих мероприятий выбираю уже связанные с ними отзывы.

Код

<?if(!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED!==true)die();?>
<?$idEvents = (int)$_GET["ID"]?>// ID текущей услуги
<?$feedback = Array()?>
<?$arSelect = Array("PROPERTY_EVENTS")?>
<?$arFilter = Array("IBLOCK_ID"=>10, "ID"=>$idEvents,"ACTIVE_DATE"=>"Y", "ACTIVE"=>"Y");

$res = CIBlockElement::GetList(Array(), $arFilter, false, false, $arSelect );
   while($ob = $res->GetNextElement()){
      $arFields = $ob->GetFields();      
      if(!empty($arFields["PROPERTY_EVENTS_VALUE"]))
         $arSelectREWIEWS = Array("PROPERTY_REVIEWS");                  
         $arFilterREWIEWS  = Array("IBLOCK_ID"=>11, "ID"=>$arFields["PROPERTY_EVENTS_VALUE"],"ACTIVE_DATE"=>"Y", "ACTIVE"=>"Y");
         $resREWIEWS  = CIBlockElement::GetList(Array(), $arFilterREWIEWS , false, false, $arSelectREWIEWS  );
            while($obREWIEWS  = $resREWIEWS ->GetNextElement()){
               $arFieldsREWIEWS  = $obREWIEWS ->GetFields();
               if(!empty($arFieldsREWIEWS ["PROPERTY_REVIEWS_VALUE"]))
                  $feedback[] = $arFieldsREWIEWS ["PROPERTY_REVIEWS_VALUE"] ;
            }
   }
?>

Пользователь 82983 Постоянный посетитель Сообщений: 508 Баллов: 40 Регистрация: 09.02.2011	#2 29.11.2013 17:01:41 у вас запрос CIBlockElement::GetList. и в цикле вы вызываете ЕЩЕ запросы! CIBlockElement::GetList каждый раз, на каждый итем у вас идет один и тот же запрос, в котором меняется 1 параметр. Это некорректно.

Пользователь 1848 Гуру Сообщений: 1313 Баллов: 284 Регистрация: 15.06.2005 Чтоб жить до глубокой старости, нужно подняться на вершину блаженства бытия	#3 29.11.2013 17:14:20 $_GET["ID"] - Передавать через параметр компонента и не забываем о кешировании. И да, все это должно делаться одним запросом CIBlockElement::GetList Прекрасная жизнь начинается с прекрасных мыслей...

Пользователь 95726

Эксперт

Сообщений: 742 Баллов: 61 Регистрация: 06.07.2011

29.11.2013 17:25:56

Цитата
Julliet пишет: и в цикле вы вызываете ЕЩЕ запросы!

Два чаю этой даме.

ТС, все совсем плохо - так нельзя.
Соберите в цикле все IDшники в один массив, а потом одним GetList'ом выдерните их. И используйте GetNex вместо GetNextElement + GetFields - это быстрее.

Пользователь 157360

Заглянувший

Сообщений: 29 Баллов: 1 Регистрация: 01.01.2013

29.11.2013 17:32:50

ок, тогда не подскажите как одним запросом в битрике мне вывести связанный элемент
Связь примерно такая:
Услуга->мероприятие->отзыв,
как мне из услуги выбрать одним запросом отзыв которые в свою очередь связанны с мероприятиями(связанным при помощи свойства привязка к элементу), по такой связке.
Может sql запрос с join-ом написать, но хочется сделать это api битриксом

Пользователь 1848

Гуру

Сообщений: 1313 Баллов: 284 Регистрация: 15.06.2005

Чтоб жить до глубокой старости, нужно подняться на вершину блаженства бытия

29.11.2013 17:39:47

http://dev.1c-bitrix.ru/api_help/iblock/classes/ciblockelement/getlist.php

Цитата

Можно выбрать и значения свойств элементов по значениям свойства типа "Привязка к элементам". Для этого необходимо указать PROPERTY_<PROPERTY_CODE>. PROPERTY_<PROPERTY_CODE2>, где PROPERTY_CODE - ID или мнемонический код свойства привязки, а PROPERTY_CODE2 - свойство указанного в привязке элемента.

Прекрасная жизнь начинается с прекрасных мыслей...

Пользователь 157360 Заглянувший Сообщений: 29 Баллов: 1 Регистрация: 01.01.2013	#7 29.11.2013 17:52:40 СЕНКС за советы, собрал нужные id в массив, а потом выбрал одним GetlList();

Пользователь 82983 Постоянный посетитель Сообщений: 508 Баллов: 40 Регистрация: 09.02.2011	#8 29.11.2013 18:00:20 в студию

Пользователь 95726

Эксперт

Сообщений: 742 Баллов: 61 Регистрация: 06.07.2011

29.11.2013 18:37:00

Цитата
Сергей Каданцев пишет: СЕНКС за советы, собрал нужные id в массив, а потом выбрал одним GetlList();

Ну можете еще обратить внимание на совет Евгения Малкова - иногда такой вариант может больше подходить, но зависит от случая. Не всегда один большой запрос лучше чем несколько маленьких, хотя я знаю что многие в это верят. Только тестировать и сравнивать. Но это имеет смысл если у вас жесткий хайлоад.

Пользователь 8420

Эксперт

Сообщений: 1231 Баллов: 223 Регистрация: 10.01.2007

#10

30.11.2013 23:34:36

Вместо того, что бы осуждать, надо помогать, либо молчать.

Код

<? if (!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED !== true) {
   die();
}

$eventID = intval($_GET["ID"]);
if (!empty($eventID)) {
   $arReviewsIDs = array();
   $arFeedbacksList = array();

   $rsEventsList = CIBlockElement::GetList(array(), array("IBLOCK_ID" => 10, "ID" => $eventID, "ACTIVE_DATE" => "Y", "ACTIVE" => "Y"), false, false, array("PROPERTY_EVENTS"));
   while ($arEvent = $rsEventsList->Fetch()) {
      if (!empty($arEvent['PROPERTY_EVENTS_VALUE'])) {
         $arReviewsIDs[] = $arEvent['PROPERTY_EVENTS_VALUE'];
      }
   }

   if (!empty($arReviewsIDs)) {
      $rsReviewsList = CIBlockElement::GetList(array(), array("IBLOCK_ID" => 11, "ID" => $arReviewsIDs, "ACTIVE_DATE" => "Y", "ACTIVE" => "Y"), false, false, array("PROPERTY_REVIEWS"));
      while ($arReview = $rsReviewsList->Fecth()) {
         if (!empty($arReview["PROPERTY_REVIEWS_VALUE"])) {
            $arFeedbacksList[] = $arReview["PROPERTY_REVIEWS_VALUE"];
         }
      }
   }
   unset($eventID, $arReviewsIDs, $rsEventsList, $arEvent, $rsReviewsList, $arReview);

   echo '<pre>';
   print_r($arFeedbacksList);
   echo '</pre>';
}

либо, если связка правильно сделана, то хватит и такого

Код

<? if (!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED !== true) {
   die();
}

$eventID = intval($_GET["ID"]);
if (!empty($eventID)) {
   $arReviewsIDs = array();
   $arFeedbacksList = array();

   $rsEventsList = CIBlockElement::GetList(array(), array("IBLOCK_ID" => 10, "ID" => $eventID, "ACTIVE_DATE" => "Y", "ACTIVE" => "Y"), false, false, array("PROPERTY_EVENTS.PROPERTY_REVIEWS"));
   while ($arEvent = $rsEventsList->Fetch()) {
      if(!empty($arEvent['PROPERTY_EVENTS_PROPERTY_REVIEWS_VALUE'])) {
         $arFeedbacksList[] = $arEvent['PROPERTY_EVENTS_PROPERTY_REVIEWS_VALUE'];
      }
   }

   unset($eventID, $arReviewsIDs, $rsEventsList, $arEvent);

   echo '<pre>';
   print_r($arFeedbacksList);
   echo '</pre>';
}

Скорая помощь 1С-Битрикс | Yandex SmartCaptcha | Оплата по ссылке | Умный поиск

Можно ли так писать =)

Продукты

Управление сайтом

Битрикс24

Интернет-магазин + CRM

Решения

Для интернет-магазинов

Каталог готовых решений

Внедрение

Выбрать партнера

Проверить партнера

Стать партнером