Почему капча не обновляется? Если она статическая то смысла от неё - 0
При ошибке открывает новую страниц с белым фоном и какой-то надписью - не хорошо. Намного лучше когда например под/над формой отображаются ошибки.

Спасибо, за конструктивную и полезную критику. Постараюсь исправить.

Также комменты можно добавлять для не существующих материалов - это не гуд.
http://www.clubcrimea.com/view_post.php?id=-2
http://www.clubcrimea.com/view_post.php?id=1000

Также из-за того, что у тебя капча не обновляется, мне не составит труда написать бота, который тебе вся зафлудит
Вот пример
http://www.clubcrimea.com/view_post.php?id=1001

И ещё было бы не плохо выводить время добавления комментария.

---

Спустя 4 минуты, 39 секунд Winston написал(а):

---

Кнопка налезла на футер

ОГО)))) Спасибо, Winston, еще раз. ФОрму для комментариев я пока уберу подальше. Доделаю, а потом уже поставлю)))

Спасибо!

Да зачем убирать форму? Я не думаю, что твой сайт настолько популярный и на нем хранится настолько ценная инфа, что кто-то будет специально, что-то ломать или флудить. Потому форму оставь, пусть будет.

После того как форму убрал, вот как теперь отображается

Форму убрал, ну а флудить-то все равно можно
http://www.clubcrimea.com/view_post.php?id=1001

Н-да

буду работать дальше. Рано показывать полез.


Спасибо, что показали мои бока - лучше сейчас, чем когда там будет много текстов.

Может еще какие-то мои тупки есть?

http://www.clubcrimea.com/files/
Как не хорошо, позволять всем заглядывать в каталоги сайта

"Помедленней, пожалуйста, я записываю))) "

Закрыл каталоги, снял скрипт, обрабатывающий форму

Опять кучу спама в базу прилетит)))

А еще что-то нашли?

гыгыгы - осталось спросить "Ну как вам дизайн" ыыыыыы

Форму убрал, не дал мне поиграться с различными штучками. А я бы хорошие ролики бы выставил... эх!
Движок свой - это плюс.
Движок слабый - это минус.
Уязвимости были - это минус.
Уязвимости скрыл путём убирания функционала - это минус.
Попов - минус.
Дизайн - на троечку, но огромный плюс в том, что почти ненапряжный дизайн.
Даты америкосовские - это минус
SQL-inj мне не удалось из-за редиректа на главную страницу, скорее всего его и нет.

Проблема в том, что код Поповский, а значит внутренности самого кода страшные, а внешняя оболочка симпатична более-менее.

Поповский код был изначально, учился по нему - потом долго и мучительно с криками "ПОПОВ , я люблю твороженные булочки!!" я его переписывал. Сделал защиту от SQl и XSS - у Попова ее нету в принципе, убрал все его говно do-while, printf и прочие коды - стало намного быстрее работать.

Оставил Поповскую капчу, и маленький кусок скрипта для комментариев - ну вот и получил)))

Буду дальше работать. А мой пример - это пример того, что Поповского кода на сайте не должно быть не строчки)))

В любом случае старт дан вполне нормальный, главное на этом не останавливаться.

А что значит "Даты америкосовские - это минус"?

год-месяц-день

наша:
день-месяц-год
Или даже "3 декабря 2011г." такой вид.

Понял, спасибо, за подсказку. Это не сложно будет поправить в коде)))

В запросе. DATE_FORMAT.

Угу - сообразил))) Спасибо.

Сейчас ищу что-нибудь толковое почитать по капче - как ее утсановить. Ну и пытаюсь закрыть id для несущствующих постов. то есть, чтобы шел запрос к базе, если там нет такого id - редиректило на главную.

Позаботься чтоб в адресе картинки были нормальные альты и мета, это поможет поисковикам.

при нажатии на меню кнопок новигации
http://www.clubcrimea.com/view_cat.php?cat=1
и
http://www.clubcrimea.com/view_cat.php?cat=2
футер куда-то девается
наверное шаблон не общий или ещё как

а сам код страницы шаблона кончается так

</table> <br/><p align="center"> Крым Советский. Клуб влюбленных в Крым! </p>

и для того чтоб не было подобного что описал Winston
в своем сообщении а именно вот это

min-height: 1000px;// это как пример , свою высоту сам придумывай(1000px)  

Раздел "Советские купальники: от мешка к обнаженке". "Мешок" есть, а где "ображёнка"? Фоты надо бы Иначе раздел как неполноценный воспринимается. (ну это так просто, не воспринимай всерьёз)

В этом же разделе. Уже более серьёзно. В самом низу вижу "Читайте также"... А там - ссылка на эту же статью. Хрень какая-то... И это касается всех подобных ссылок, все указывают сами на себя, т.е. на ту же статью, где они находятся.
И еще - эту ссылку хорошо бы как-нибудь выделить. Цветом, размером, жирностью - чем угодно, чтобы она была понятна издалека, что ссылка.

Да и вообще хочется побольше фоток.
Например, те же санатории советские. Написано хорошо, хоть и совсем мало. И фоток нету совсем. Сравнить не с чем. Если человек молодой, ездил отдыхать в Турцию-Египет, то вообще мало поймет, о чем идет речь.

Раздел "Создатели сайта"... Это первое, что надо было заполнить!!!

Фон, наверное, лучше сделать непрокручиваемым, чтобы только основная, информационная часть могла пролистываться. Но это уже "моя ИМХА"

YVSIK, sergeiss, спасибо за комменты и полезные замечания - всё по делу и в точку. Буду исправляться. Когда сам смотришь, глаз замыливается, уже не замечаешь - тем более, что это первый мой сайт)))

Фоток - будет больше, ну и материалов, конечно, тоже. Это так - пробные закинул, чтобы посмотреть как будет на хостинге.

Спасибо еще раз всем, кто посоветовал, обнаружил косяки.

kir25 а в чем прикол когда нажимаешь кнопки "искать", и оставляешь пустым

<input type="text" maxlength="40" size="20" name="search">
<br>

Выбивает на белый лист и пишет Поисковый запрос не введен, либо он менее 4-х символов.
Не практично!!!
придеться нажимать кнопку назад на браузере

Спасибо еще раз, ошибки понял - поработаю над ними

Закройте тему, пожалуйста