[X] Помощник

[tro9an Topic Starter]

ZiTosS, посоветовал мне написать форум, так сказать чтобы поднабраться опыта, вот что получилось:

 forum.rar ( 17,17 килобайт ) Кол-во скачиваний: 100


прошу (кто не сильно занят) посмотреть чё я тут написал, и указать на ошибки и недостатки!

З.Ы сильно не ругайтесь т.к ето мои самые первые скрипты на php
З.Ы.Ы если БД не работает (найдёте её в архиве), то скажите я перезалью!
З.Ы.Ы.Ы подскажите чё ещё мо;но написать кроме этого Задачи для начинающих в PHP

[gaaarfild]

Базу данных-то я не вижу в архиве. =)

[gaaarfild]

index.php
[php]$result=mysql_query ("SELECT * FROM form");
$myrow=mysql_fetch_array($result);
do {
printf ("<p><a href='cat.php?cat=%s'>%s</a></p>",$myrow['cat'],$myrow['title']);
}
while ($myrow=mysql_fetch_array($result));[/php]

Здесь вместо do~while разумнее было бы использовать while. Ну или хотя бы проверить, а вдруг категорий вообще нет. А если нет, то вывести соответствующее сообщение.

vhod.php
[php]if ((!empty($_POST['login']))and(!empty($_POST['pass']))) //Тут empty() рекомендую заменить на isset();

....

header('Refresh: 2; URL=index.php'); //Тут зачем пауза эта в 2 секунды? Лучше использовать header('Location: index.php');[/php]

cat.php
[php]printf ("<a href='add_tema.php?cat=%s&id=%s'>Новая тема</a></p>",$_SESSION['cat'],$id); // А тут, я считаю, что проще и удобнее пользоваться echo. Приверно так:
echo "<a href='add_tema.php?cat=".$_SESSION['cat']."&id=".$id."'>Новая тема</a></p>";[/php]

И так же, зачем категорию помещать в сессию??? Если вы в данной категории, а категория выбирается из массива $_GET, то и используйте его. Иначе это получается бесполезное перекладывание переменных.

Тут же, функцию admin(); удобней поместить в другом файле, а здесь пользоваться. Зачем захламлять.

И все примерно в таком духе. В общем, очень даже неплохо. Единственное, ЗарегИстрироваться, а не зарегЕстрироваться. =)

Моя главная рекомендация, не создавать лишних переменных, если в этом нет необходимости. =)

Ждем дальнейших работ. А еще саму БД. =) Хочется в действии посмотреть а не в коде. =)

[tro9an Topic Starter]

ну вот собственна БД:  forum.sql.zip ( 966 байт ) Кол-во скачиваний: 103

[НЕПЛОХОЙ]

1) не работает редактирование темы
2) Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in Z:\home\forum\www\reg.php on line 4 (почти на каждой странице)
3) если человек не модератор и даже не зарегистрирован, то он не должен видеть "Удалить тему, редактировать сообщение"
4) поработайте над дизайном, очень не удобно. Если даже вы не дизайнер ниразу, посмотрите как выглядят форумы и сделайте хотя бы примерно так же.

а вообще, форум довольно сложная штука, я так понял вы недавно начали изучать пхп? можно написать срипты вроде Голосования, Новостей, гостевой книги, фотогалереи

[tro9an Topic Starter]

Евгений, этот форум я делал чтобы набраться хоть какого-то опыта, сейчас начал писать чат, где постараюсь сделать Голосования, Новости, гостевую книгу, фотогалерею

[bestolloch]

чат как отдельная страница будет? или мини-чат?
просто не представляю как можно туда добавить голосование (а если голосование то чего?) и тд))) ждем результатов

[gaaarfild]

Для чата надо как минимум AJAX. Иначе чат получится очень грузный.

[tro9an Topic Starter]

ну тогда: "Forum_v_2.0" :)

[НЕПЛОХОЙ]

"Для чата надо как минимум AJAX. Иначе чат получится очень грузный."
это в идеале.
раньше небыло AJAX, а чаты делали на фреймах.