Почему текст echo выводиться несколько раз, а не один?

Question

[Илья]

как поправить код, чтобы echo выводилось правильно?

<?
			global $USER;
			$arGroupAvalaible = array(1590,1591, 1592, 1593); 
			$curUserId = $USER->GetID(); // ID текущего пользователя
			$arGroups = CUser::GetUserGroup($curUserId); // ID групп текущего пользователя
			foreach ($arGroups as $curUserGroups) {
				$rsGroup = CGroup::GetByID($curUserGroups); // Получаем названия всех групп
				$arGroup = $rsGroup->Fetch();
				$result_intersect = array_intersect($arGroupAvalaible, $arGroup);
				if(!empty($result_intersect))  {
					echo "<div class='col-md-12 col-sm-12 col-xs-12 manager'>Ваш персональный менеджер "; print_r($arGroup['NAME']); echo "	</div>";
				 }
				else {
					echo "Вам не назначен персональный менеджер.";
				}
			   }
			?>

Comments

[Ярослав Александров]

вы этот код сами писали или это чужой?

[Илья]

Ярослав Александров, взял где-то на просторах интернета и вставил свои данные

[Ярослав Александров]

Илья, тогда закономерно, что код работает не так как вам нужно. Вы не понимая как работает код и что в нём происходит, вам лень разобраться и вы идёте на тостер и просите чтобы кто-то вместо вас сделал.
Сделайте отладку https://dev.1c-bitrix.ru/api_help/main/functions/d...
посмотрите какие данные у вас в коде, почему там такие данные и что нужно изменить.

[Илья]

Ярослав Александров, спасибо, что хоть какой то материал оставили, а не просто обозвали лентяем.

Answer 1

[CarfikDK]

<?
      global $USER;
      $notFound = true;
      $arGroupAvalaible = array(1590,1591, 1592, 1593); 
      $curUserId = $USER->GetID(); // ID текущего пользователя
      $arGroups = CUser::GetUserGroup($curUserId); // ID групп текущего пользователя
      foreach ($arGroups as $curUserGroups) {
        $rsGroup = CGroup::GetByID($curUserGroups); // Получаем названия всех групп
        $arGroup = $rsGroup->Fetch();
        $result_intersect = array_intersect($arGroupAvalaible, $arGroup);
        if(!empty($result_intersect))  {
          echo "<div class='col-md-12 col-sm-12 col-xs-12 manager'>Ваш персональный менеджер "; print_r($arGroup['NAME']); echo "	</div>";
$notFound = false;
         }
         }
if($notFound){
echo "Вам не назначен персональный менеджер.";
}
      ?>

Comments

[Илья]

Спасибо. Это сработало! А в чем суть добавления новой переменной $notFound? Что прочитать можно об этом?

[CarfikDK]

Илья, извини, что отвечаю так поздно)
Суть в том, что это некоторый переключатель, если ты засунешь такой ифик в цикл, то он всегда будет, что-то выводить, а тебе это не нужно, а вот если оно нашло то, что тебе нужно, ты говоришь программе "Вот все есть, не нужно выводить, что нету", ставишь $notFound = false, а если не нашло, то у тебя эта переменная уже выставленна так, что выведет это сообщение

[Илья]

Костя Дехтярук, Спасибо за теорию)

Answer 2

[Mario Kun]

Когда доходит до блока else
вам необходимо выйти из цикла, путем добавления конструкции break после echo

Comments

[Илья]

не там добавил или ещё что-то еще не так сделано? из-за чего три раза сверху выводится echo из else?

<?
			global $USER;
			$arGroupAvalaible = array(1590,1591, 1592, 1593); 
			$curUserId = $USER->GetID(); // ID текущего пользователя
			$arGroups = CUser::GetUserGroup($curUserId); // ID групп текущего пользователя
			foreach ($arGroups as $curUserGroups) {
				$rsGroup = CGroup::GetByID($curUserGroups); // Получаем названия всех групп
				$arGroup = $rsGroup->Fetch();
				$result_intersect = array_intersect($arGroupAvalaible, $arGroup);
				if(!empty($result_intersect)) {
					echo "<div class='col-md-12 col-sm-12 col-xs-12 manager'>Ваш персональный менеджер "; print_r($arGroup['NAME']); echo "	</div>";
				break;
				}
				else {
					echo "Вам не назначен персональный менеджер.";
				}
			   }
			?>

[Mario Kun]

global $USER;
      $arGroupAvalaible = array(1590,1591, 1592, 1593); 
      $curUserId = $USER->GetID(); // ID текущего пользователя
      $arGroups = CUser::GetUserGroup($curUserId); // ID групп текущего пользователя
      foreach ($arGroups as $curUserGroups) {
        $rsGroup = CGroup::GetByID($curUserGroups); // Получаем названия всех групп
        $arGroup = $rsGroup->Fetch();
        $result_intersect = array_intersect($arGroupAvalaible, $arGroup);
        if(!empty($result_intersect)) {
          echo "<div class='col-md-12 col-sm-12 col-xs-12 manager'>Ваш персональный менеджер "; print_r($arGroup['NAME']); echo "	</div>";
        
        }
        else {
          echo "Вам не назначен персональный менеджер.";
break;
        }
         }

[Илья]

Mario Kun, теперь выводится echo из else, но не выводится echo из if

[Mario Kun]

global $USER;
      $arGroupAvalaible = array(1590,1591, 1592, 1593); 
      $curUserId = $USER->GetID(); // ID текущего пользователя
      $arGroups = CUser::GetUserGroup($curUserId); // ID групп текущего пользователя
$message = '';
      foreach ($arGroups as $curUserGroups) {
        $rsGroup = CGroup::GetByID($curUserGroups); // Получаем названия всех групп
        $arGroup = $rsGroup->Fetch();
        $result_intersect = array_intersect($arGroupAvalaible, $arGroup);
        if(!empty($result_intersect)) {
          $message =  "<div class='col-md-12 col-sm-12 col-xs-12 manager'>Ваш персональный менеджер "; print_r($arGroup['NAME']); echo "	</div>";
        
        }
        else {
          $message =  "Вам не назначен персональный менеджер.";
        }
         }
echo $message;

[Илья]

Mario Kun, к сожалению не помогло