Форум: Форум PHPФорум ApacheФорум Регулярные ВыраженияФорум MySQLHTML+CSS+JavaScriptФорум FlashРазное
Новые темы: 0000000
Объектно-ориентированное программирование на PHP. Авторы: Кузнецов М.В., Симдянов И.В. PHP. Практика создания Web-сайтов (второе издание). Авторы: Кузнецов М.В., Симдянов И.В. PHP Puzzles. Авторы: Кузнецов М.В., Симдянов И.В. Самоучитель MySQL 5. Авторы: Кузнецов М.В., Симдянов И.В. Социальная инженерия и социальные хакеры. Авторы: Кузнецов М.В., Симдянов И.В.
ВСЕ НАШИ КНИГИ
Консультационный центр SoftTime

Форум PHP

Выбрать другой форум

 

Здравствуйте, Посетитель!

вид форума:
Линейный форум Структурный форум

тема: Моя новая Гостевуха
 
 автор: ZC   (24.03.2005 в 17:44)   письмо автору
 
 

Вот посмотрите, покритикуйте и воще обсудите моюгостевую книгу.

   
 
 автор: cheops   (25.03.2005 в 00:01)   письмо автору
 
   для: ZC   (24.03.2005 в 17:44)
 

В дампе базы данных последняя таблица приведена не полностью - присутствует только её окончание
  'post_answer' text NOT NULL,
  'post_hide' enum('show','hide') NOT NULL default 'show',
  'post_putdate' datetime NOT NULL default '0000-00-00 00:00:00',
  PRIMARY KEY  ('id_post')
) TYPE=MyISAM PACK_KEYS=0 AUTO_INCREMENT=64 ;

   
 
 автор: ZC   (25.03.2005 в 11:15)   письмо автору
 
   для: cheops   (25.03.2005 в 00:01)
 

Гостевая книга работает только с двумя таблицами.
Только вот PHPMyAdmin что-то не то добавил в конец файла.
Вот держите нормальный дамп таблиц.

   
 
 автор: cheops   (26.03.2005 в 13:20)   письмо автору
 
   для: ZC   (25.03.2005 в 11:15)
 

Гостевая хорошая и ценная, так как написана с использованием объектов, но имеются недочёты.
Так в файле addrec.php имеются строки
<?php
            $author_name 
convert_cyr_string($author_name);
            
$post_body convert_cyr_string($post_body);
?>

В функции convert_cyr_string отустувуют два обязательных параметра. Кроме того, по умолчнанию включено отпавление письма с уведомлением, причём в коде жёстко забит ваш email - это не очень хорошо и будет настораживать пользователей - лучше вынести email в параметры в utils/kernel.php, а так же ввести дополнительный параметр, который включал бы и выключал отправку почтового уведомления (по умолчанию луче выключить, так как тестирование часто проходит на Windows-машинах с отсутствующим транспортным почтовым агентом).

   
 
 автор: cheops   (26.03.2005 в 13:22)   письмо автору
 
   для: ZC   (25.03.2005 в 11:15)
 

Кроме того, если отсутствует ответ Администратора, может не выводить строку
Администратор -

Когда их много - не очень смотрится и создаётся впечатление, что администратор демонстративно не хочет отвечать на сообщения посетителей.

   
Rambler's Top100
вверх

Rambler's Top100 Яндекс.Метрика Яндекс цитирования