Почему не работает пользовательская функция?

Question

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

Здравствуйте! Мне нужно создать XML - файл согласно определенным требованиям. Наполняю я его из базы. ($dada)
Иногда, для некоторых площадок нужно нашу информацию передать слегка в другом формате. Я называю это справочник)
Например:

$colorist = array(
  'зеленый' => 'greeb',
  'красный' => 'red',
  'синий' => 'blue',
);

Функции необходимо проверить если информация в базе, т.к. не все сотрудники заполняют всю информацию и если нет, то не записывать это поле в файл. Если есть то записывать.
Я написал, но не могу понять почему не работает.

// справочник
$colorist = array(
  'зеленый' => 'greeb',
  'красный' => 'red',
  'синий' => 'blue',
);
//массив полученный из базы данных
$data[];

// тег для XML-файла ($teg)
// справочник, но он не всегда нужен ($book)
// поле из базы данных которое я должен записать ($value)
// т.к. поле может множественным, например картинки, а их придется выводить циклом использовал count()

function write ($teg, $book, $value) {
	if ($value == false || count($value)<1) { 
		return NULL;
	} 
	if ($book == false) {
		return $xml->writeElement("color", $value);
	} else {
		return $xml->writeElement("color", $colorist[$value]);
	}
}

Comments

[Алексей Уколов]

Я написал, но не могу понять почему не работает.

Давайте, прежде чем выяснять почему, поймём что.

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

Алексей Уколов, точно забыл написать. она вообще ничего не записывает. Даже если есть все три элемента.

// если нет справочника я хотел бы так записывать
write("Price", $data['price']);
// пробовал так
write("Price", false, $data['price']);

[ivashjke]

для начала в функции write - $xml - у вас не определена
а $value - выходит всегда массив и вы пытаетесь вызвать у неизвестного $xml метод
+ к этому вместо строки пытаетесь записать массив
или вообще в массиве $colorist подставляете значение по ключу $value - который какбы массив на входе

[Spartak (Web-StyleStudio)]

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

// если нет справочника я хотел бы так записывать
write("Price", $data['price']);

Если нет справочника, Тогда вам ещё надо немного изменить функцию, чтобы так её использовать
function write ($teg, $value, $book = false)

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

Spartak (Web-StyleStudio), тоже не заработала.Если через функции он вообще не записывает

function wr ($teg, $value,  $book=false) {
  global $book, $value, $xml, $teg;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
    return $xml->writeElement($teg, $value);
  } else {
    return $xml->writeElement($teg, $colorist[$value]);
  }
}

Answer 1

[DevMan]

потому что переменные вне функции не видны внутри функции.
https://www.php.net/manual/ru/language.variables.s...

Comments

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

Spartak (Web-StyleStudio), тоже не заработала.Если через функции он вообще не записывает

function wr ($teg, $value,  $book=false) {
  global $book, $value, $xml, $teg;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
    return $xml->writeElement($teg, $value);
  } else {
    return $xml->writeElement($teg, $colorist[$value]);
  }
}

Answer 2

[Spartak (Web-StyleStudio)]

Добавьте в функцию

function write ($teg, $book, $value) {
    $global $colorist, $xml;
     ... 
}

И включите отображение ошибок на экран

Comments

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

Spartak (Web-StyleStudio), тоже не заработала.Если через функции он вообще не записывает

function wr ($teg, $value,  $book=false) {
  global $book, $value, $xml, $teg;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
    return $xml->writeElement($teg, $value);
  } else {
    return $xml->writeElement($teg, $colorist[$value]);
  }
}

[Spartak (Web-StyleStudio)]

Вечно Крайний,
Это не правильно
global $book, $value, $xml, $teg;
Бук, валуе, тег вы передаёте в саму функцию

Так надо

function write ($teg, $value,  $book=false) {
  $global $colorist, $xml;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
    return $xml->writeElement($teg, $value);
  } else {
    return $xml->writeElement($teg, $colorist[$value]);
  }
}

Использование

// без справочника 
write("Price", $data['price']);
// с справочником
write("Price", $data['price'], true);

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

Spartak (Web-StyleStudio), так справочников же много. Не обезательно только цвет. Это может быть материал например.
У нас в базе кирпич, а в файле нужен кирпичный

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

Spartak (Web-StyleStudio), вот так заработало! Спасибо)

function wr ($teg, $value,  $book=false) {
  global $xml;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
    return $xml->writeElement($teg, $value);
  } else {
    return $xml->writeElement($teg, $colorist[$value]);
  }
}

[Spartak (Web-StyleStudio)]

Использование

// без справочника 
write("Price", $data['price']);
// с справочником
write("Price", $data['price'], true);

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

Spartak (Web-StyleStudio), тогда он не знает к какому спрвочнику обращаться.
write("Price", $data['price'], $colorist);
Вот так долдно быть поидее

[Spartak (Web-StyleStudio)]

вот так заработало! Спасибо)
function wr ($teg, $value, $book=false) {
global $xml;
if ($value == false) {
return NULL;
}
if ($book == false) {
return $xml->writeElement($teg, $value);
} else {
return $xml->writeElement($teg, $colorist[$value]);
}
}

Будет работать до тех пор пока
$book == false
Потому что $colorist вне поле видимости функции

[Spartak (Web-StyleStudio)]

$book и $colorist
Это один и тот же справочник?
Если, да, то это

return $xml->writeElement("color", $colorist[$value]);

Надо заменить на
return $xml->writeElement("color", $book[$value]);

И если нужно использовать функцию с справочником:
write("Price", $data['price'], $colorist);

[Spartak (Web-StyleStudio)]

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

function write ($teg, $value,  $book = false) {
  $global $xml;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
    return $xml->writeElement($teg, $value);
  } else {
    return $xml->writeElement($teg, $book[$value]);
  }
}

// Использование без справочника 
write("Price", $data['price']);
// с справочником цветов 
write("Price", $data['price'], $colorist);

Третьим параметром $book в функцию вы передаёте уже нужный вам справочник, например $colorist

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

Spartak (Web-StyleStudio), да согласен. Я как раз написал туже самую функцию только выше! В ответ на Ваш комментарий. Где благодарил за помощь)

[Spartak (Web-StyleStudio)]

Я как раз написал туже самую функцию только выше

Да, но только с ошибкой, посмотрите внимательно

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

Spartak (Web-StyleStudio), да опечатка с $colorist. Спасибо еще раз)
Правда я пошел чуть дальще! Теперь она имеет такой вид

function wr ($teg, $value,  $book=false, $tegStart=false, $teg2 =false, $teg3 = false) {
  global $xml;
  if ($value == false) { 
    return NULL;
  } 
  if ($book == false) {
	if ($teg2 == false) {
		$xml->writeElement($teg, $value);
	} else {
		return array (
		$xml->startElement($tegStart),
		$xml->writeElement($teg, $value),	
		$xml->writeElement($teg2, $teg3),
		$xml->endElement(),
		);
	}
} else {
	if ($teg2 == false) {
    	return $xml->writeElement($teg, $book[$value]);
	} else {
		return array (
		$xml->startElement($tegStart),
    	$xml->writeElement($teg, $book[$value]),
		$xml->writeElement($teg2, $teg3),
		$xml->endElement(),
		);
  }
}   
}

Возможно ее нужно как-то сократить
это я ее изменил не много чтобы можно было получать такой результат

$xml->startElement("price"); //цена
    $xml->writeElement("value",$aRes['price']));
    $xml->writeElement("currency","RUR");
  $xml -> endElement();

[Spartak (Web-StyleStudio)]

как-то сократить

Можно так перезаписать:

function wr ($teg, $value, $book = false, $tegStart = false, $teg2 = false, $teg3 = false) {
  global $xml;
  if (!$value) { 
    return NULL;
  }
  if ($tegStart && $teg2 && $teg3) {
    return array (
    	$xml->startElement($tegStart),
    	$xml->writeElement($teg, !$book ? $value : $book[$value]),
    	$xml->writeElement($teg2, $teg3),
    	$xml->endElement(),
    );
  } else {
    	return $xml->writeElement($teg, !$book ? $value : $book[$value]);
  }
}