В php получаю разную область видимости для переменной внутри условия — как исправить?

Question

[ligisayan]

Всем привет!
В интернет-магазине есть поля в которые приходят значения из формы $_POST, среди которых billing_email1, billing_email2, billing_email3 и т.д. в зависимости от заданного количества. (другие поля тоже есть, но нас здесь не интересуют)

Моя задача состоит в том, чтобы приходящие значения записать в поля:

В billing_email первое значение billing_email1, а в billing_email_dop уже все последующие billing_email2, billing_email3 и т.д. через точку с запятой.

Уже день бьюсь, никак не могу разобраться в чем причина, почему $key вне и внутри условия оказываются различными..

<?php	
    $fields = $checkout->get_checkout_fields( 'billing' );

	foreach ( $fields as $key => $field ) {
		$i = 1;
		$postdata = "";
		$postdop = "";
		$postemail = "";
		var_dump($key); // $key = billing_email_dop есть
		if($_POST["$key"."$i"]) {
			var_dump($key); // $key = billing_email_dop уже нет, вместо него $key = billing_email - почему?
			while($_POST["$key"."$i"]) { 
				
				if($key==="billing_email") {
					if(("$key"."$i")=="billing_email1") {
						$postemail = $_POST['billing_email1'];
					}
					else {
						$postdop .= $_POST["$key"."$i"]."; "; //в $postdop значение записывается корректно
					}
				}
				else {
					$postdata .= $_POST["$key"."$i"]."; ";
				}
				$i++;
			}
		}
			if($key=="billing_email") {
				woocommerce_form_field( $key, $field, $postemail );
			}
			else if($key=="billing_emaildop") {
				woocommerce_form_field( $key, $field, $postdop );
			}
			else {
				woocommerce_form_field( $key, $field, $postdata );
			}
	} 
?>

Comments

[Максим Тимофеев]

А Вас не смущает, что у Вас $postdop = ""; перезаписывается постоянно при каждой итерации цикла?

[ligisayan]

Максим Тимофеев, смущает, но меня смущает и вот это участок кода

var_dump($key); // $key = billing_email_dop есть
    if($_POST["$key"."$i"]) {
      var_dump($key); // $key = billing_email_dop уже нет, вместо него $key = billing_email - почему
   }

Что ни крутил - не знаю как это решить

Answer 1

[ThunderCat]

сделайте вардамп $fields после $checkout->get_checkout_fields( 'billing' ); а то не понятно что вы туда получаете, может это вообще ресурс...

PS: добавьте if($_POST["$key"."$i"]) {...} else{var_dump($key);} и все встанет на сво места.

PPS: вообще код конечно кроме того что построен ногами вперед, пестрит как явными ошибками, так и неявными(для новичков) глюками... например billing_email_dop в итоге становится billing_emaildop, про пропуски итераций внутри условных операторов я молчу, и как указал Максим Тимофеев - циклически затираемые данные
Короче классический пример как делать НЕ НАДО.

Как нужно переделать:
1) Нет смысла строить цикл по $fields, вы же четко знаете его структуру, в отличие от данных из $_POST.
2) В цикле по $_POST ищете подходящие по шаблону ключи(например проверка на подстроку billing_email или регулярка billing_email\d{1,2}). Если точно billing_email1 - пишете в $postemail, иначе $postdop[] = $val; По уму на клиенте поля формы должны называться более внятно, например billing_email[], после чего сразу можно получить ВЕСЬ массив емэйлов.
3) в конце если $postdop не пустой - имплодите запятыми значения $postdop, получите вашу строку с емэйлами через запятую.
4) присваиваете все в нужные поля вашего $fields.
5) Profit.

Comments

[ligisayan]

ThunderCat Вот: (остальные поля не выкладывал, они не представляют интереса, т.к. заполняются верно. Единственная проблема с полем billing_emaildop, которое пустое. billing_email заполняется тоже верно)

["billing_email"]=>
  array(11) {
    ["label"]=>
    string(6) "E-MAIL"
    ["required"]=>
    bool(true)
    ["type"]=>
    string(4) "text"
    ["class"]=>
    array(1) {
      [0]=>
      string(13) "form-row-last"
    }
    ["validate"]=>
    array(2) {
      [0]=>
      string(8) "required"
      [1]=>
      string(5) "email"
    }
    ["autocomplete"]=>
    string(14) "email username"
    ["priority"]=>
    int(70)
    ["options"]=>
    array(0) {
    }
    ["placeholder"]=>
    string(0) ""
    ["enabled"]=>
    bool(true)
    ["custom"]=>
    bool(false)
  }
  ["billing_emaildop"]=>
  array(11) {
    ["type"]=>
    string(4) "text"
    ["label"]=>
    string(0) ""
    ["options"]=>
    array(0) {
    }
    ["placeholder"]=>
    string(0) ""
    ["priority"]=>
    int(80)
    ["enabled"]=>
    bool(true)
    ["validate"]=>
    array(0) {
    }
    ["required"]=>
    bool(false)
    ["custom"]=>
    bool(true)
    ["display_options"]=>
    array(2) {
      [0]=>
      string(6) "emails"
      [1]=>
      string(10) "view_order"
    }
    ["class"]=>
    array(1) {
      [0]=>
      string(13) "form-row-wide"
    }
  }

[ThunderCat]

sandbox.onlinephpfunctions.com/code/c2f7e5e1606887...
если код будет срабатывать как предполагается(а не как вы думаете что он работает) то все будет ок.

[ligisayan]

ThunderCat, не очень понимаю - вы мне предлагаете потестировать через песочницу? А как я получу значения $_POST ?

[ligisayan]

ThunderCat оно-то да,
if($_POST["$key"."$i"]) {..} не подходит для $key=billing_emaildop, но это особо ничего не меняет, т.к. мне нужно получить значения billing_email2, billing_email3 приходящие с $_POST["$key"."$i"] и они проходят проверку if($key==="billing_email") {..}
Ну т.е. я по-прежнему не знаю как решить задачу

[ThunderCat]

ligisayan, рекомендую мысленное упражнение: внимательно читайте код построчно, на каждой строке мысленно подставляйте имеющиеся переменные (для этого все изначально известные переменные вардампом выведите), можете даже на бумажке разложить все циклы, после этого все у вас получится. У вас там код много чего не учитывает, например не происходит инкрементация если не срабатывает иф(и вы это никак не учитываете, соответственно по кругу мотая неверные проверки), еще есть глюки. По уму стоило бы переписать, но для именно поднятия скила сделайте такой разбор, вы сами все поймете на каком-то этапе...

[ligisayan]

ThunderCat, уже больше суток сижу с дампом и бумажками - в целом понимаю как что куда записывается, но решить задачку ну никак не могу. На первый взгляд кажется просто и так бы и было, если бы в $_POST приходил $key = billing_emaildop или хотя бы не проходил проверку
if($key==="billing_email") {..}
но $postdop в цикле перезаписывается и становится равным пустоте

[ThunderCat]

ligisayan,

if($_POST["$key"."$i"]) {..} не подходит для $key=billing_emaildop, но это особо ничего не меняет, т.к.

угу, в частности не меняет $i, который теперь снова "не подходит" и снова не меняется )
Я же говорил - выпишите переменные пошагово руками )

[ThunderCat]

ligisayan, очевидно же что ваш код делает совершенно не то что вы предполагаете, неужели сложно натыкать вардампов через строчку, если уж мысленно построить выполнение у вас не получается. Дампите через строчку ВСЕ ВСТРЕЧАЮЩИЕСЯ В ЭТОМ МЕСТЕ В КОДЕ ПЕРЕМЕННЫЕ, и рядом печатайте метку места вывода дампа, увидите как код идет по циклам и какие значения там реально получаются.

[ligisayan]

ThunderCat, писал, что только так и делаю var_dump вторые сутки везде где только можно, поэтому навряд ли сложно.
я вижу, что были в $postdop данные, а затем затерлись в цикле - как это обойти не знаю.
Об этом уже говорил

[ThunderCat]

ligisayan,

я вижу, что были в $postdop данные, а затем затерлись в цикле - как это обойти не знаю.

Максим Тимофеев :

А Вас не смущает, что у Вас $postdop = ""; перезаписывается постоянно при каждой итерации цикла?

+ у вас есть условие когда $postdop .= $_POST["$key"."$i"]; НЕ ВЫПОЛНЯЕТСЯ по ифу после $postdop = ""; я не знаю как это можно пропустить если вы действительно что-то там выводили дампом.

[ThunderCat]

ligisayan, Дополнил ответ, надеюсь поможет.

[ligisayan]

ThunderCat, аа, понял!
Нужно было

$postdop = "";
$postemail = "";

вынести за пределы над foreach Бывают затупы..
Спасибо за помощь! Включите это в ваш ответ и я его отмечу выполненным.

Answer 2

[Moses Fender]

А как выглядит woocommerce_form_field(…)?
Видимо, там первый параметр по ссылке передаётся, и там изменяется. Потому он в цикле разный.

Comments

[ligisayan]

Moses Fender https://wp-kama.ru/function/woocommerce_form_field

[Moses Fender]

Ну стало быть смотришь не так. $key всегда будет разным в каждой итерации, потому что не может быть в ассоциативном массиве элементов с одинаковым ключом. Попробуй протрассировать дебагером эти подробности. Скорее всего сразу поймёшь в чём беда.

[ligisayan]

Moses Fender, $key понятно, что разный - я это прекрасно понимаю, но вам не странный вот этот кусок кода, где среди перечня вариаций $key до условия значение billing_email_dop есть, а уже внутри нет этого значения

var_dump($key); // $key = billing_email_dop есть
    if($_POST["$key"."$i"]) {
      var_dump($key); // $key = billing_email_dop уже нет, вместо него $key = billing_email - почему?
    }

Вот здесь я и завис как раз-таки

каким образом через протрассировать дебагером?

[ThunderCat]

ligisayan, очевидно же что на одной из итераций в это условие код не попадает, читайте мой ответ

[ligisayan]

ThunderCat, да, спасибо. Заменил на такое условие

if($_POST["$key"."$i"] || $key == 'billing_emaildop') {..}

и теперь уже попадает внутрь, но $postdop по-прежнему циклически перезаписывается и к финишу приходит пустым и соответственно в поле billing_emaildop записывается тоже пустым