Вывод элементов массива?

Question

[VitaliyKaliuzhyn]

есть массив $items не известным количеством элементов

есть вот такой return в котором я для наглядности перечислил для 3 значений массива $items

return
            "<div class='form-group field-users-created_at required'>".
                Html::beginTag('label', $labelOptions) .
                $items[0] .
                Html::Tag('input','',$options).
                "<span></span>".
                Html::endTag('label').
            Html::beginTag('label', $labelOptions) .
            $items[1] .
            Html::Tag('input','',$options).
            "<span></span>".
            Html::endTag('label').
            Html::beginTag('label', $labelOptions) .
            $items[2] .
            Html::Tag('input','',$options).
            "<span></span>".
            Html::endTag('label').
            "<span class='help-block'></span></div>";

данный код возвращает вот такую вот разметку

<div class="form-group field-users-created_at required"><label class="mt-radio">Radio 1<input type="radio" name="test"><span></span></label><label class="mt-radio">Radio 2<input type="radio" name="test"><span></span></label><label class="mt-radio">Radio 3<input type="radio" name="test"><span></span></label><span class="help-block"></span></div>

Вопрос в том как мне без дублирования кода для каждого i-го элемента вывести

return
            "<div class='form-group field-users-created_at required'>".
                Html::beginTag('label', $labelOptions) .
                $items[i] .
                Html::Tag('input','',$options).
                "<span></span>".
                Html::endTag('label').
               "<span class='help-block'></span></div>

Answer 1

[anton_lazarev]

$html = "<div class='form-group field-users-created_at required'>";

foreach ($items as $item) {

$html .= Html::beginTag('label', $labelOptions) .
                $item .
                Html::Tag('input','',$options).
                "<span></span>".
                Html::endTag('label');

}

$html .= "<span class='help-block'></span></div>";

return $html;

Хотя глаза, признаюсь, вытекли...

Comments

[VitaliyKaliuzhyn]

Спасибо Вам огрномное

Answer 2

[Multigame]

Вообще любой цикл.
В теле функции создаете переменную, её заполняете циклом, далее уже в return передаете эту переменную.

$returnHtml =  "<div class='form-group field-users-created_at required'>";
for($i = 0; $i < count($items); $i++){
   $returnHtml .= Html::beginTag('label', $labelOptions).$items[$i].Html::Tag('input','',$options)."<span></span>".Html::endTag('label');
}
$returnHtml .= "<span class='help-block'></span></div>";
return $returnHtml;

Comments

[Camaro67]

Вообще-то для обхода массива есть специальный цикл - foreach. Не нужно учить человека дурному.

[VitaliyKaliuzhyn]

Спасибо Вам огромное.

[Alex]

Camaro67: С чего вдруг то?

[Camaro67]

Alex: "БыдлоГовноНаноКодер" после этого вести дальнейший диалог даже расхотелось, видимо это судьба)

[Multigame]

Alex: А я вот загуглил и Камаро прав, форыч быстрее. https://habrahabr.ru/post/216103/

[Camaro67]

Multigame: Тут дело не только в скорости. Еще много причин по которым стоит использовать foreach, а не for для обхода массива. Например, читабельность кода или ключи могут идти не по порядку и т.д.

[Alex]

Multigame: быстрее не значит правильнее. Вот полез посмотреть ответы Camaro67. https://toster.ru/answer?answer_id=965294#comments...
Он его пихает куда не попадя. У меня от ответа из глаз кровь пошла.

В данном случае да, пройтись проще и быстрее foreach. Но человек должен понимать что можно сделать и так и так и понимать почему тут нужно применять именно тот или иной вариант. А писать комменты аля " Не нужно учить человека дурному. " попахивают.

[Camaro67]

Alex: Вот вы как раз реально не понимаете разницы между for и foreach. И мой ответ который вы привили - там как раз должен быть foreach. Это очевидно из вопроса. Стоит его прочитать, прежде чем писать ересь которую вы выше написали.

[Alex]

Camaro67:
1. Я ваш взял 1й ответ из списка.
2. for там как раз и просится ибо не надо искусственно переменную цикла вводить.
3. Я уже молчу про $k % 4 === 3. зачем начинать цилк с 1 и потом так извращаться? Автор не зря ничего не понял.

[Camaro67]

Alex: А говнокодер это и впрямь судьба)

[Alex]

Camaro67: Т.е. аргументов нет. А моя самокритика в статусе вас зацепила...

[Camaro67]

Alex: Если вы элементарных вещей не понимаете что я могу поделать? Вы даже вопрос не смогли понять. В вопросе у человека ясно говориться, что он выводит записи блога из базы. И ему нужно вывести сообщение после каждой четвертой записи. Т.к. человек не предоставил код, то за МАССИВ записей был взят range( 1, 100 ). А значения в цикле были взяты с 1 для наглядности, что выводится каждая 4я запись. Там значения никакой роли вообще не играли, а были важны ключи. Еще вопросы есть или для вас и это слишком?

[Camaro67]

Alex: И меня не ваша самокритика в статусе задела, а то что пишут говнокод, гордятся этим да еще и пытаются научить других говнокодить. А вы не задумывались, что с этим говнокодом потом, возможно, еще придется кому-то работать после вас и тратить лишнее время на разбор этого г... в место того, что бы потратить это время с большей пользой?

[Alex]

Camaro67: Вопрос по поддержке кода я бы к вам адресовал. А для наглядности начинать цикл с 1 и потом считать остаток от деления равный трем прям очень наглядная вещь. Флейм можно закрывать.

[Camaro67]

Alex: Ясно короче все с вами. Не вижу смысла вести беседу с человеком который даже основ не знает ни в PHP ни в математике. Учитывая, что вы всерьез думаете, что деление по модулю начинается с 1, а не с 0. Можно сделать только 1 вывод: вы даже тот код не поняли как работает.

[Alex]

Camaro67: " модулю начинается с 1, а не с 0 " что?