Подскажите в чем ошибка

Wirel

Новичок
Подскажите в чем ошибка

PHP:
1  function write_to_file() {
2  trim ($post['name']);
3  trim ($post['site']);
4  trim ($post['email']);
5  trim ($post['message']);
6  if ($post['name']=="") {
7  $post['ref_page']="";
8  $post['hidd']="ОШИБКА! Введите свое имя!";
9  }     // Конец условия /* if ($post['name']=="") { */
10  if ($post['message']=="") {
11  $post['ref_page']="";
12  $post['hidd']="ОШИБКА! Введите сообщение!";
13  }     // Конец условия /* if ($post['message']=="") { */
14  if ($post['ref_page']=="sign") {
15  $FileName = "msg.txt";
16  $FilePointer = fopen ($FileName, "a");
17  fwrite ($FilePointer, "<div class=\"view_msg\"><strong>Имя:</strong> $post    ['name']&nbsp;&nbsp;|&nbsp;&nbsp;");
18  if ($post['site']=="") {
19  fwrite ($FilePointer, "<strong>Сайт:</strong> не указано&nbsp;&nbsp;|&nbsp;&nbsp;");
20  } else {
fwrite ($FilePointer, "<strong>Сайт:</strong> <a href=\"$post['site']\">$post['site']</a>&nbsp;&nbsp;|&nbsp;&nbsp;");
21  }     // Конец условия /* if ($post['site']=="") { */
22  if ($post['email']=="") {
23  fwrite ($FilePointer, "<strong>E-Mail:</strong> не указано<br>");
24  } else {
25  fwrite ($FilePointer, "<strong>E-Mail:</strong> $post['email'] <a href=\"$post['email']\">[Написать письмо]</a><br>");
26  }     // Конец условия /* if ($post['email']=="") { */
27  fwrite ($FilePointer, "<strong>Сообщение:</strong> $post['message']</div><br>");
28  fclose ($FilePointer);
29  }     // Конец условия /* if ($post['ref_page']=="sign") { */
30  }     // Конец функции write_to_file();
Подскажите в чем ошибка на строке 17
 

DiMA

php.spb.ru
Команда форума
ошибка в апострофах
но вообще, это такой говнокодище, что я за рвотными пакетиками побежал...
 

antson

Новичок
Партнер клуба
$post откуда в функции взялся этот массив ?
суперглобальный у нас $_POST.

сроки 2-5 - не фига не делают

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

для чего используется функция ? гостевая на файлах ?
Если да, то формировать выводной хтмл сразу не есть гуд.

Надеюсь массив проверялся перед передачей в функцию на XSS и другие бяки ?

P.S.писать в файл без lock()'а может только самоубийца или автор хомяка где посещаемость полчеловека в день.

p.p.s. Имхо следующим вопросом будет а как мне mes.txt выводить с разбивкой по страницам. :)
 
Сверху