PHP - цикл For возвращает только последнюю переменную в массиве

Question

У меня странная проблема, когда цикл for в PHP возвращает только последний элемент в моем массиве.

Массив создается с помощью SimpleXML из файла XML.

Код должен вернуть это:

<tags><tag value="Tag1" /><tag value="Tag2" /><tag value="Tag3" /></tags>

Но вместо этого я просто получаю:

<tags><tag value="Tag3" /></tags>

Так что он игнорирует все, кроме последнего элемента в массиве, независимо от того, сколько я там попал.

Кто-нибудь может увидеть, что я делаю не так?

Вот код:

<?php

function gettags($xml)
{
    $xmltags = $xml->xpath('//var[@name="infocodes"]/string');
    return $xmltags[0];
}

//Path to the XML files on the server
$path = "/xmlfiles/";

//Create an array with all the XML files
$files = glob("$path/*.xml");

foreach($files as $file)
{
    $xml = simplexml_load_file($file);
    $xmltags = gettags($xml);

//Using the , character split the values coming from the $xmltags into an array
$rawtags = explode(',', $xmltags);

//Loop through the tags and add to a variable. Each tag will be inside an XML element - <tag value="tagname" />
for ($i = 0; $i <count($rawtags); $i++){
    $tags = '<tag value="' . $rawtags[$i] . '" />';
}

//Replace HTML escaped characters (ä, å, ö, Å, Ä, Ö) and the | character with normal characters in the tags variable
$tagsunwantedchars = array("&Ouml;", "&Auml;", "&Aring;", "&ouml;", "&auml;", "&aring;", "|");
$tagsreplacewith = array("Ö", "Ä", "Å", "ö", "ä", "å", " - ");
$tagsclean = str_replace($tagsunwantedchars, $tagsreplacewith, $tags);

//Create the full tag list and store in a variable
$taglist = "<tags>$tagsclean</tags>";

}

echo $taglist;

?>

Вот файл XML:

<wddxPacket version='1.0'>
    <header/>
    <data>
        <struct>
            <var name='infocodes'>
                <string>Tag1,Tag2,Tag3</string>
            </var>
        </struct>
    </data>
</wddxPacket>

Answer 1

Простая ошибка: используйте $tags .= вместо $tags = в цикле:

$tags = '';
for ($i = 0; $i <count($rawtags); $i++){
    $tags .= '<tag value="' . $rawtags[$i] . '" />';
}

Answer 2

for ($i = 0; $i <count($rawtags); $i++){
    $tags = '<tag value="' . $rawtags[$i] . '" />';
}

Вы просто перезаписываете переменную $tags на каждой итерации. Попробуйте:

$tags = '';
foreach ($rawtags as $rawtag) {
    $tags .= '<tag value="' . $rawtag . '" />';
}

Обратите внимание, что вы должны инициализировать $tags перед добавлением к нему (в противном случае это сгенерирует предупреждение PHP).

Кроме того, использование foreach вместо цикла for делает ваш код более простым и более читабельным. Такие простые ошибки легче обнаружить, если они не окружены шумом.

Answer 3

$tags .= '<tag value="' . $rawtags[$i] . '" />';

Это может решить вашу проблему.