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

[Strange_V]

Вот написал скрипт гостевой книги, хотел бы услышать конструктивную критику. Так как в PHP я не так давно, и еще многого не знаю…



В первую очередь интересует правильность и рациональность кода, потом дизайн и юзабилити…



Пример работающего скрипта

здесь

(только захостился, не знаю когда будет доступен), так же выложил исходники

тут

пароль: php

[kotid]

1. Желательна мультиязычность, лично я украинец и мне побарабану, но некоторые могут не понять

2. Форма доманшей страницы и номер аськи по умолчанию не обязательны, но без них скрипт не добавляет записи.

3. Поле домашней страницы не воспринимает ссылки в виде ip т.е. в небольшой локалке, где нет прописанных соответствий мижду ip и именем хоста будут проблемы. ( я дальше не копал наверно регулярное выражение ?)

4. смайлики не добавляються

Это поверхностный осмотр, будет время покопаю ещё

[Arrest]

У меня почему-то скрипт не добавляет записей из-за e-mail'а. Щас буду смотреть на регулярные выражения.

UPD: скрипт не пущает мейлы типа "login@site.2site.ru".



Пожелания:

1. Языки.

2. Скины.

3. Нормальный валидатор мыл.

4. ICQ бывают от 5-значных до 9-значных.

5. ICQ и сайт - действительно необязательные поля.



Последние 2 проблемы решаются изменением функции validator ($mul, $uri, $asy) на



PHP код:

Код: Выделить всё

function validator ($mul, $uri, $asy) {

//Валідатор validator (мило, урл, аська)

global $error;

$err = '';

if ( strpos ($uri, 'http://') !== FALSE )

    $uri= substr ($uri, 7);

if ( !preg_match ('/^[-_\.a-z\d]{3,15}@[\.a-z\d]{2,10}\.[a-z]{2,4}$/i', $mul) )

    $err= $err." $error[mail]
";

if ( !preg_match ('/^[-_\.a-z\d]{2,20}\.[a-z]{2,4}$/i', $uri))

    if($uri!=='')$err= $err." $error[url]
"; //next 3 lines modified by Lost Soul

if ( !preg_match ('/^\d{5,10}$/', $asy))

    if($asy!=='')$err= $err." $error[icq]
";

if ( '' != $err ){

    $err = $error['validator_titl'].$err;

    return $err;

    }

} 

[Strange_V]

kotid

1. Книг на других языках и так море, хотя… может в следующей версии сделаю

2. Баг…, исправлено

3. Не подумал, сделано

4. Т.е.? не работает JavaScript ввод или вывод php?



Arrest

Пожелания учту, спаисбо за вариант решения проблемы



Для устранения проблем необходимо заменить файлы mygbook/inc/function.php и mygbook/add.php прикрепленными.



хост з гостевой не доступен... не могу пока разобратся

[kotid]

нормально, всё исправлено, а насчет смайликов , я хотел добавить свой набор смайликов, но посмотрел там нужно менять код скриптов для добавления. надо продумать скрипт для изменения смайлов.

[Strange_V]

Спасибо.

Ведуться работы… (85% уже сделал)