Как найти и исправить ошибку в записи текста между тегами?

Question

[Петр Смирнов]

Мне нужно перезаписать текст между тегами div в текущем файле.
Помогите найти ошибку и исправить мой код.
Один файл:

<?php
$x = $_SERVER[PHP_SELF];
$y = $x;
$file = basename($y);//Имя файла

$name = 'Masha';//Новый текст между тегами

if( isset( $_POST['nazvanie_knopki'] ) )
{
$html = file_get_contents($fl);
preg_match_all( '#<div>(.+?)</div>#is', $html, $matches );
foreach ( $matches[1] as $value ) echo '<br>';

$filename = $fl;
$file2 = file_get_contents($filename);
$title_pattern='#<div>(.*)?</div>#s';
$test= preg_replace($title_pattern, "<div><header>$name</header>$value</div>", $file2);//$value - старый текст между тегами
file_put_contents($filename, $test);
echo "<script language='JavaScript' type='text/javascript'>window.location.replace(window.location.href)</script>";
}
?>

<div></div>//теги, между которыми надо вставить текст
<br>
<form method="POST">
<input type="submit" name="nazvanie_knopki" value="popdpisatsya">
</form>

Comments

[FeST1VaL]

А ошибка, то какая?

[Петр Смирнов]

При нажатии на кнопку перезапись не срабатывает

[FeST1VaL]

Петр Смирнов, ну как минимум у вас не понятно что за переменная $fl, предполагаю что она $file

[FeST1VaL]

Петр Смирнов, Плюс, вы же берете тот же файл, вы в нем и меняете... а открываете файл как просто текст, т.е он и PHP код переписывает.

[Дмитрий]

Тут как минимум лишнее preg_match_all
Цикл с перебором

Делается все на много проще, возможно даже в одну строку

file_put_contents(preg_replace(..., file_get_contents($file)), $file);

Answer 1

[Евгений Матвеев]

рекомендую воспользоваться Html-парсером, например DOM