Скрипт гостевой книги

Печать RSS
326

N
Автор
Пришелец
0
Вспоминаю азы пэхэпэ, которые я когда то учил
_______________________________________
Скрипт гостевой книги, написан на php + mysql
функционал:
Добавление сообщений (удивительно, правда?) при добавлении сообщения ник Admin запрещен
Капча при добавлении сообщения
Постраничная навигация
Смайлы (15шт)
ББкоды
Авторизация админа
Админка:
Создание постов от ника Admin
Ответ на любой пост
Удаление постов

Вроде бы все описал
Просьба оценить, если что не так подсказать как исправить
ПРОДАЖА ЗАПРЕЩЕНА и присваивание тоже =)

Добавлено через 04:18 сек.
скрин http://rghost.ru/40111676
архив http://upwap.ru/2818248 (на ргхост не захотел)

Землянин
0
выложил бы сюда в загруз центр
N
Автор
Пришелец
0
сейчас закину, не знал, что можно

Землянин
0
на скрине скрипт вроде ничего) исходники не смотрел
N
Автор
Пришелец
0
4. fL1pE, ну дак я ж и дизайнить могу =)
мне больше интересны оценки кода
Изменил: No-tactic (31.08.2012 / 11:27)
А

Оранжевые штаны
0
Автор, не проще эти две строчки:
<?php
$act = isset($_REQUEST['act']) ? trim(htmlspecialchars($_REQUEST['act'])) : false;
...
if($act != false){ ...

заменить на:
<?php
...
if(isset($_REQUEST['act'])){ ...
Не знаю используется ли переменная act еще где то, но если нет, то код попахивает
Изменил: Артур (31.08.2012 / 11:43)
N
Автор
Пришелец
0
точно, не поправил, у меня там просто изначально switch был, я использовал чтоб notice не выскакивал
$act еще в admin.php используется
А

Оранжевые штаны
0
Ну так $act тот же, или другая переменная с тем же именем? ))
Много там такого автор, долго смотреть
N
Автор
Пришелец
0
8. Башка,

$act = isset($_REQUEST['act']) ? trim(htmlspecialchars($_REQUEST['act'])) : false;
...
if($act == 'add')
...
if($act == 'del')
...
if($act == 'ans')
Изменил: No-tactic (31.08.2012 / 11:52)
А

Оранжевые штаны
0
Зачем там вся эта проверка если стоят жесткие условия?
Стикеры / Теги / Правила / Топ тем / Топ постов / Поиск