Почему из foreach выводится только последнее значение массива?

Question

[Aligatro]

Добрый вечер. Сразу прошу простить меня за "новичковый" и глупый вопрос. В общем мне необходимо отсортировать массив, удалив лишнее и передать результат в переменную, с последующей возможность вывести ее из цикла.
Используя следующий код, у меня почему-то отображается только последнее значение массива, потыкался в интернет, но ответа так и не нашел. Посмотрите пожалуйста.

$fields_user_choice = array ( 
'[0] => Красный', 
'[1] => Синий', 
'[2] => Зеленый' 
);
foreach($fields_user_choice AS $e)
{
	$e = preg_replace('(\[.*?\]\s\=\>\s)', $replace, $e);
	$replace = ",";
}
print_r ($e); // выводит только последнее значение "Зеленый"

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

Спасибо.

Answer 1

[VBKesha]

Та вывод стоит за циклом, то есть работает только один раз, и выводит последние что есть в $e а там то что было на последней итерации цикла.

Comments

[Aligatro]

А вывести переменную из цикла вообще возможно, не подключая к этому делу бд?

[VBKesha]

[code]
$fields_user_choice = array (
'[0] => Красный',
'[1] => Синий',
'[2] => Зеленый'
);
$ret = array();
foreach($fields_user_choice AS $e)
{
$ret[] = preg_replace('(\[.*?\]\s\=\>\s)', $replace, $e);
$replace = ",";
}
print_r ($ret);[/code]
Можно где то так, было бы проще если бы вы сказали что хотите сделать?

[Aligatro]

@VBKesha Моя цель это загнать в переменную $e или любую другую отформатированные значения массива $fields_user_choice. Что бы на выходе получилось что-то вроде такого: "Красный, Зеленый, Синий"

[VBKesha]

@Aligatro иногда лучше задавать вопрос напрямую.
Вариант без циклов вообще.

$fields_user_choice = array ( 
'[0] => Красный', 
'[1] => Синий', 
'[2] => Зеленый' 
);
$tmp = implode(', ', $fields_user_choice);
echo $tmp;

php.net/manual/ru/function.implode.php

Вариант с цилком:
$fields_user_choice = array (
'[0] => Красный',
'[1] => Синий',
'[2] => Зеленый'
);
$ret = '';
$replace = ''
foreach($fields_user_choice AS $e)
{
$ret .= $e.$replace;
$replace = ', ';
}
echo($ret);

Тут важно помнить компутер такая вещь котороая выполняет то что ты от неё просишь, а не то что хотелось бы попросить.
Иногда просто надо по шагам продумать что и как будет работать.

[Sergej]

@VBKesha Не понятно только одно, какого марзуха у него не массив а строка.

[VBKesha]

@sayber видимо он только начал осваивать программирование и слабо осознаёт что к чему, многие так начинают ничего необычного.

[Aligatro]

@sayber у меня данные с одного из плагина записываются в сессию из которой я получаю именно такую строку как указанно в примере. Массивом я ее назвал по незнанию.
@VBKesha спасибо, что заступились =) Про implode я почему-то вообще забыл.

Answer 2

[Радость моя]

А мне не понятно как это вообще работает, если использование переменной $replace идет перед ее объявлением :) Я бы на месте интерпретатора сразу ругнулся бы :)

E_NOTICE : type 8 -- Undefined variable: replace -- at line 9
,Зеленый

вот так должно работать:

$replace = ",";

foreach($fields_user_choice AS $e)
{
	$e = preg_replace('(\[.*?\]\s\=\>\s)', $replace, $e);
	
}

Comments

[Aligatro]

Я там немного напортачил, в принципе все нормально выполняется даже при условии, что $replace находится внутри цикла.

[Радость моя]

Оно то будет нормально выполняться если внутри цикла. Проблема была в том что внутри цикла вы объявили что $replace = ',' после того как уже сделали работу по замене с ее участием. Это как требовать от сотрудников результата, не дав им задания. А из цикла я вынес ради экономии памяти, т.к. нет никого смысла объявлять переменную в каждой итерации цикла, если она в этих итерациях не меняется. Правильная практика вообщем ;)

[Aligatro]

@t-alexashka, буду знать, спасибо за пояснение.

Answer 3

[webdisigner]

Будь проще

<?php
   $fields_user_choice = array ( 
     'Красный', 
     'Синий', 
     'Зеленый' 
   );
  $a = implode(', ',$fields_user_choice);
  echo $a; 
?>

Answer 4

[Sergej]

Это вы хотели получить?

$fields_user_choice = array (
            '[0] => Красный',
            '[1] => Синий',
            '[2] => Зеленый'
        );
        $replace = '';
        $result = '';
        $i = 0;
        foreach($fields_user_choice AS $e)
        {
            if ($i != 0) {
                $replace = ', ';
            }
            $result .= preg_replace('(\[.*?\]\s\=\>\s)', $replace, $e);
            $i++;
        }
// Выведет: Красный, Зеленый, Синий
        echo $result

Только не понятно, почему у вас массив в виде строки?
Должно ведь быть :

$fields_user_choice = array (
            [0] => 'Красный',
            [1] => 'Синий',
            [2] => 'Зеленый'
        );

Тогда регулярки не нужны.

Comments

[Радость моя]

ну наверно человек просто с формы напрямую выводит

[Aligatro]

@t-alexashka вы абсолютно правы. Не знал, что достаточно лишь поставить точку, что бы в переменную заносились все данные. Спасибо за помощь.

[Sergej]

@Aligatro Почитайте и попробуйте поработать с массивами, это облегчит вашу жизнь.