Почему получаю ошибку при попытке получить значение индекса foreach?

Question

[ligisayan]

Всем привет! Есть на сайте wordpress цикл foreach, который выводит перечень <input type="radio">

foreach ( $args['options'] as $option_key => $option_text ) {
				$field .= '<input type="radio"' . checked( $value, esc_attr( $option_text ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $option_text ) . '" /><label>' . esc_html( $option_text ) . '</label>';
			}

Я хочу получить для каждой id с порядковым значениям, чтобы было к ним обращаться через label
Почему при попытке написать так я получаю ошибку? Разве в $option_key не хранится порядковый номер?

foreach ( $args['options'] as $option_key => $option_text ) {
$field .= '<input type="radio" id=""'. echo $option_key . '"' . checked( $value, esc_attr( $option_text ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $option_text ) . '" /><label>' . esc_html( $option_text ) . '</label>';
}

Comments

[morricone85]

какая ошибка?

[ligisayan]

morricone85 ошибка пустой страницы
Ему echo не нравится
echo $option_key
но, в случае такой
esc_html( $option_key )
или такой
$option_key
пишет не то, что нужно

[morricone85]

ошибка пустой страницы - такой не бывает)
включите отображение ошибок

[ligisayan]

morricone85, спасибо, но я уже разобрался без отображения ошибок

Answer 1

[Илья]

'<input type="radio" id=""'
В этом куске кода Вы открываете кавычку для значения атрибута id, и тут же закрываете.
Кроме того, если у Вас нумерованный массив (не ассоциативный), используйте цикл for, а не foreach.

Comments

[ligisayan]

Илья спасибо за кавычку, но не понимаю как здесь переписать цикл под for
И даже без кавычки здесь получаю ошибку

foreach ( $args['options'] as $option_key => $option_text ) {
			$field .= '<input type="radio" id="'. echo $option_key  . '"' . checked( $value, esc_attr( $option_text ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $option_text ) . '" /><label>' . esc_html( $option_text ) . '</label>';
			}

а здесь выводится все тоже самое, что и в value

foreach ( $args['options'] as $option_key => $option_text ) {
			$field .= '<input type="radio" id="'. esc_attr( $option_key ) . '"' . checked( $value, esc_attr( $option_text ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $option_text ) . '" /><label>' . esc_html( $option_text ) . '</label>';
			}

Что-то типа того? Но, теперь получаю id, а значение как получить?

$length = count($args['options']);
			for ($i = 0; $i < $length; $i++) {
				$field .= '<input type="radio" id="'. esc_html( $i ) . '"' . checked( $value, esc_attr( $args[$i] ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $args[i] ) . '" /><label>' . esc_html( $args[$i] ) . '</label>';
			   }

[Илья]

ligisayan, просто уберите echo в этом куске кода:
echo $option_key

[ligisayan]

Илья, да, ошибка исчезает, но выводится не то, что мне нужно.
Пытаюсь сделать, как вы с сказали с помощью for, но $args[$i] не выводится

$length = count($args['options']);
      for ($i = 0; $i < $length; $i++) {
        $field .= '<input type="radio" id="'. esc_html( $i ) . '"' . checked( $value, esc_attr( $args[$i] ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $args[i] ) . '" /><label>' . esc_html( $args[$i] ) . '</label>';
         }

[Илья]

ligisayan, потому что надо писать:
$args['options'][$i]

[ligisayan]

Илья Что-то не то..

for ($i = 0; $i < count($args['options']); $i++) {
			$field .= '<input type="radio" id="'. esc_html( $i ) . '"' . checked( $value, esc_attr( $args['options'][$i] ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $args['options'][$i] ) . '" /><label>' . esc_html( $args['options'][$i] ) . '</label>';
		    }

На выходе получаю value и label пустые, хотя с foreach работает и они выводятся

<input type="radio" id="0" checked="checked" name="order_time" value="">
<label></label>

Сам array вот такой

array(2) { ["Ближайший"]=> string(18) "Ближайший" 
["Удаленный"]=> string(44) "Удаленный" }

[ligisayan]

Илья в итоге решил вот таким образом, но так и не разобрался в чем косяк

$index = 0;
			foreach ( $args['options'] as $option_key => $option_text ) {
			$field .= '<input type="radio" id="'. $index . '"' . checked( $value, esc_attr( $option_text ), false ) . ' name="' . esc_attr( $key ) . '" value="' . esc_attr( $option_text ) . '" /><label>' . esc_html( $option_text ) . '</label>';
			$index++;
			}

[Илья]

ligisayan, проблема была в том, что Вы пытались вывести "индекс" элемента массива и использовать его для атрибутов. Но у Вас массив -- ассоциативный, в нем нет индексов. Более того, ключи массива -- кириллические.
Главное, что получили рабочий вариант ))