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

Question

[RomyGalaxy]

Есть массив стилей,

$this->_styles = array(
            'body' => 'margin: 0 0 0 0; padding: 10px 10px 10px 10px; background: #ffffff; color: #000000; font-size: 14px; font-family: Arial, Helvetica, sans-serif; line-height: 18px;',
            'a' => 'color: #003399; text-decoration: underline; font-size: 14px; font-family: Arial, Helvetica, sans-serif; line-height: 18px;',
            'table' => 'border: 1px solid #000;',
        );

и функция которая принимает в себе пустой html шаблон и вставляет между стили

public function htmlStyles ($html) {
        foreach ($this->_styles as $tag => $style) {
            $styles = $tag . '{' .$style. '}';
            $html = str_replace('</head>',$styles, $html);
        }
        return $html;
    }

но проблема в том, что проходит только один стиль из массива body, но если вывести их с помощью echo то выводятся все

Comments

[Ипатьев]

Извините, а чего вы хотите добиться строчкой $html = str_replace('</head>',$styles, $html);?

[Ипатьев]

Ну и традиционный вопрос на логиику: а почему вы решили, что "foreach возвращает только один результат"?

[RomyGalaxy]

Ипатьев, эта строчка перед вставляет стили в общем html шаблоне

[RomyGalaxy]

Ипатьев, потому что результатом выводится только

body{margin: 0 0 0 0; padding: 10px 10px 10px 10px; background: #ffffff; color: #000000; font-size: 14px; font-family: Arial, Helvetica, sans-serif; line-height: 18px;}

[Ivan Ustûžanin]

RomyGalaxy,

результатом выводится только

и будет выводиться только это, т.к. такова логика заложенная внутри цикла foreach

[Ипатьев]

и где логика?
если вы утверждаете, что foreach возвращает только один результат, то где вывод тех элементов, которые возвращает foreach? может быть, с foreach и нет никаких проблем? Почему вы утверждаете про "одно значение" не проверив это?

Если же у вас проблемы с заменой текста в цикле, то про это и надо писать.

body{margin: 0 0 0 0; padding: 10px 10px 10px 10px; background: #ffffff; color: #000000; font-size: 14px; font-family: Arial, Helvetica, sans-serif; line-height: 18px;}

Покажите мне в этом выводе строчку </head>, которая должна будет замениться при следующем проходе цикла

[Сергей delphinpro]

RomyGalaxy, просто добавьте точку

public function htmlStyles ($html) {
        foreach ($this->_styles as $tag => $style) {
            $styles = $tag . '{' .$style. '}';
-           $html = str_replace('</head>',$styles, $html);
+           $html .= str_replace('</head>',$styles, $html);
        }
        return $html;
    }

Может быть поможет. "Может быть" потому что я не очень понимаю, что вы делаете.

[Ипатьев]

Сергей delphinpro, понять тут совсем нетрудно. И нет - не поможет :)

Answer 1

[nokimaro]

Заменяя </head> вы как минимум уже получаете невалидный html
Сделать можно так, при условии что у вас в $html есть </style> внутрь которого вы и хотите свои стили дописать

foreach ($this->_styles as $tag => $style) {
    $styles = $tag . '{' .$style. '}'."\n";
    $html = str_replace('</style>', $styles.'</style>', $html);
}

Answer 2

[AUser0]

public function htmlStyles ($html) {
    $add = "";
    foreach ($this->_styles as $tag => $style) {
        $add .= "{$tag} \{{$style}\}\n";
    }
    return str_replace("</head>", "\n<style type='text/css'>\n{$add}</style>\n</head>", $html);
}

Answer 3

[Антон]

$html = str_replace('</head>',$styles, $html);
на первой итерации у вас </head> заменяется, поэтому на следующих заменять уже нечего и ваш сгенерированный код не вставляется

Comments

[RomyGalaxy]

а есть другой метод, чтобы он последовательно добавлял то что мне нужно до ?

[Антон]

RomyGalaxy, заменять не просто на сгенерированные стили, а на стили и </head>
Или сложить все стили в одну переменную и заменить один раз после цикла