Как поправить функцию?

Question

[Вечно Крайний]

Здравствуйте, я продолжаю изучить функции, задовал уже подобный вопрос ранее
Почему не работает пользовательская функция?
Столкнулся с тем, что не всегда она подходит.
Решил написать слегка по другому

function wrAr (array($start) = false, array($val)){
      global $xml; // это xmlwriter
  
      if(empty(array_column($val, '1'))) {
          return NULL;
      }
      if (!empty($start)) { 
          foreach($start as $st) {
              return $xml->startElement($st['0']);
          }
      }
      foreach($val as $value) {
          if (empty($value)) {
              continue;
          }
          return $xml->writeElement($value['0'], !$value['2'] ? $value['1'] : $value['2'][$value['1']]);
      }
      $start = array_reverse($start);
      foreach($start as $st) {
          return $xml->endElement();
      }
   
  }

Что я хочу в итоге
допустим в $video какая -то ссылка на видео

wrAr(false,['video', $video]);
wrAr(false,[['video', $video],['video', video1]]);

И получить в XML файле

<videol>https://expamle.com/111111</video>

<videol>https://expamle.com/111111</video>
<videol>https://expamle.com/222222</video>

или

wrAr([videos],[['video', $video],['video', video1]]);

<videos>
<videol>https://expamle.com/111111</video>
<videol>https://expamle.com/222222</video>
</videos>

или если есть какой-то справочник

$book = [ 'red' => 'красный', 'blue' => 'синий'];
$color = 'red';
wrAr(false,['colorist', $color, $book]);

Нужный результат
<colorist>красный</colorist>
Подскажите, что я сделал не так
Проверка не пустое поле или нет перед записью в XML обязательно(

Answer 1

[Вечно Крайний]

После пары выдернутых мною колочков волос. я разобрался. Теперь только вопрос есть ли способ ее оптимизировать))

function wrAr ($start = false, $val){
      global $xml;
      $data = []; 
      $rv = array_filter($val, 'is_array');
      if (count($rv)==0) {
          $val[] = $val;
      }
      if(empty(array_column($val, '1'))) {
          return NULL;
      }   
      if (!empty($start)) {
          foreach($start as $st) {
              $data[] = $xml->startElement($st);
          }   
      }   
      foreach($val as $value) {
          if (empty($value)) {
              continue;
          }       
          $data[] =  $xml->writeElement($value['0'], !$value['2'] ? $value['1'] : $value['2'][$value['1']]);
      }               
      $start = array_reverse($start);  
      foreach($start as $st) {
          $data[] =  $xml->endElement();
      }
      return $data;
 }

Comments

[Spartak (Web-StyleStudio)]

$book = [ 'red' => 'красный', 'blue' => 'синий'];
$color = 'red';
wrAr(false,['colorist', $color, $book])

Вот с этим оно работать не будет, из-за этой проверки

if(empty(array_column($val, '1'))) {
          return NULL;
}

И если $start = false, то будет ошибка в этой строке, Invalid argument supplied for foreach() on line

foreach($start as $st)

Answer 2

[Spartak (Web-StyleStudio)]

function wrAr (array($start) = false, array($val))

Потому что ошибка в этой строке!

Правильно так
function wrAr ($start = false, $val)

P/S: включите отображение ошибок на экран для отладки скрипта

<?php
ini_set('error_reporting', E_ALL);
ini_set('display_errors', 1);

/*далее ваш код*/

Comments

[Вечно Крайний]

Я так пробовал изначально. Все равно не работает(

[Spartak (Web-StyleStudio)]

Вечно Крайний,
И поэтому решили сделать ещё более не рабочей))

[Вечно Крайний]

Spartak (Web-StyleStudio), я нашел косяк!)

[Вечно Крайний]

кстати включение ошибок ничего не выводит(

[Spartak (Web-StyleStudio)]

Вечно Крайний,

я нашел косяк!)

Да там он не один, а целая стая))

[Spartak (Web-StyleStudio)]

Вечно Крайний,

кстати включение ошибок ничего не выводит(

Это не означает что их нет, у вас либо в .htaccess отключены ошибки, либо в скрипте где-то, либо вы файл подключаете через @ оператор подавления ошибок!

[Spartak (Web-StyleStudio)]

Вечно Крайний,

$value['0'], $value['1'], $value['2'] итд

P/S: нет необходимости числовые индексы заключать в кавычки

[Вечно Крайний]

Spartak (Web-StyleStudio), я понимаю, что в данной ситуации заключать в кавычки нет необходимости, но насколько я знаю, что если без ковычек, то php сначала ищет константу( или переменную, не помню точно, но что-то ищет), а если не находит, то только потом берет как индекс.