вот еще момент... Меня как-то совершенно убивают куски:
Код:
header("Location: http://".$_SERVER['HTTP_HOST']."
и лучше бы без них обойтись. Мне кажется, что вместо (например)
Код:
header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&news=1");
достаточно будет писать
Код:
header("Location: /".$lk_dir."/?m=message&news=1");
и это будет гораздо предпочтительнее по многим причинам.
Еще вот такие циклы:
Код:
102 //---------
103 if (isset($_POST['lotsDEL'])){
104 if ($auc->ControlGuid_User($_POST['lotsDEL'])){
105 if ($auc->DEL_Auction($_POST['lotsDEL'])){
106 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=1");
107 } else {
108 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=0");
109 }
110 } else {
111 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=0");
112 }
113 }
114 //---------
115 if (isset($_POST['lotsSHOP_DEL'])){
116 if (!$auc->ControlGuid_User($_POST['lotsSHOP_DEL'])){
117 if ($auc->DEL_Shop($_POST['lotsSHOP_DEL'])){
118 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=2");
119 } else {
120 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=3");
121 }
122 } else {
123 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=0");
124 }
125 }
Вызывают стойкое ощущение дежавю. Может быть их как-то можно сократить в одну функцию с подстановкой переменных? (сорри, я в терминах не менделеев, может это и не "функции" с "переменными" называется)