рабочий ли код. в чем ошибки

<?php
require('/home/p-p/req/in.class');
$my = new class_in;
$my->html_headers();
$my->sql_connect();
$error = "";
@$action = $HTTP_POST_VARS['action'];<-!
@$name = substr($_POST['name'], 0, 36);
@$msg = substr($_POST['msg'], 0, 1024);
if(!empty($action))
{
$name = trim($name);
$msg = trim($msg);
if(empty($msg)) { $action = ""; $error = $error.'<li>Вы не ввели сообщение!'; }
if(empty($name)) { $action = ""; $error = $error.'<li>Вы не ввели имя!'; }
if(!empty($email)) { if(!preg_match("/[0-9a-zA-Z_]+@[0-9a-z_^\.]+\.[a-z]{2,3}/i", $email)) { $action = ""; $error = $error.'<li>Неверно введен e-mail. Введите e-mail в виде <i>primer@mail.ru</i>'; }
}
$name = htmlspecialchars(stripslashes($name));
$sity = substr($_POST['sity'],0,32);
$sity = htmlspecialchars(stripslashes($sity));
$email = substr($_POST['email'],0,32);
$email = htmlspecialchars(stripslashes($email));
$msg = htmlspecialchars(stripslashes($msg));
if(empty($error)) {
$my->sql_query = "INSERT INTO tbl_guest(name, msg, time, email, sity) values(".$name.", ".$msg.", ".$my->today_date()." ".$my->today_time().", ".$email.", ".$sity.")";
$my->sql_execute();
include($my->PATH_INC.'/top.inc');
print "Гостевая - Добавление сообщения";
include($my->PATH_INC.'/top2.inc');
print '<META HTTP-ENQUIV="Refresh" CONTENT=0; URL="http://toropcev.ru/guest/">';
include($my->PATH_INC.'/top3.inc');
include($my->PATH_INC.'/navigator.inc');
include($my->PATH_INC.'/bottom.inc');
}
}

!во первых не добавляется запись в базу во вторых в конце скрипта не происходит переадресация

Добавлено через 04:20 сек.
листинг функций базы:

function sql_connect()
{
$this->conn_id = mysql_connect($this->sql_host, $this->sql_login, $this->sql_passwd);
mysql_select_db($this->sql_database);
}
function sql_execute()
{
$this->sql_res = mysql_query($this->sql_query, 
$this->conn_id);
$this->sql_err = mysql_error();
}
function sql_close()
{
mysql_close($this->conn_id);
}

это с какого года код? не бойсь php3?

Parse error: syntax error, unexpected '<' in /usr/home/p-p/htdocs/guest/index.php on line 16
__________
@$action = $HTTP_POST_VARS['action'];<-!
сильно
собаковод?

2. Виталий Степанович G, это я пытался по разному вытаскивать переменные потому что в итоге ничего не получается особенно -без собак извлекать переменные...... потому что если переменная не была передана то хомяк(h2m.ru) ругается

Добавлено через 04:38 сек.
3. ramzes, Parse error: syntax error, unexpected '<' in /usr/home/p-p/htdocs/guest/index.php on line 16 ---------- это фигня ..... ты мне код проонализируй .это ошибка в результате того что пытаюсь понять в чем собака зарыта методом тыка

4. Petr, я тебе уже сказал в чем твоя главная ошика, -> собаковод. лечи это
анализировать я ни чего тебе не буду

Собачки о_0 а фун-ции isset(), empty() на что?)
@$action = $HTTP_POST_VARS['action'];<-!
Зачем же так

Похоже что у вас, автор, ошибка в sql запросе. Вы вводите имена 5 полей, но пишете 6 значений, к тому же забыв запятую. В целом, код довольно нехороший - почему вы не экранируете данные, например?

Добавлено через 02:04 сек.
И еще в проверке e-mail вы обидели домены типа info и mobi