Оцените сайт, сайт написан на PHP, с нуля, уже много чего было переписано/переработано, вот теперь хотелось бы услышать как вам сайт, что еще нужно сделать/переделать/доработать. Зарание спасибо
ссылка на GitHub: https://github[dot]com/Seriyyy95/pcpost
1. LinuxUser - 03 Июня, 2013 - 14:58:55 - перейти к сообщению
2. EuGen - 03 Июня, 2013 - 15:08:43 - перейти к сообщению
Стандартно
- как обычно, проверки на int не везде достаточна;
- мало того, что удаётся вызвать листинг всех материалов (привет, DoS на страницу), так ещё и 400 от хостинга выдаётся при использовании пагинатора.
- здесь комментировать излишне
- выдаёт 400 bad request (правда, похитрее оформленный). Введена была тарабарщина, но я ожидал увидеть, где ошибся.
Это всё бегло просмотренные ошибки. Вероятно, есть и более серьёзные недочёты.
Ну и клон habrahabr, думаю, не будет особенно популярным на фоне оригинала.
Спойлер (Отобразить)
- как обычно, проверки на int не везде достаточна;
Спойлер (Отобразить)
- мало того, что удаётся вызвать листинг всех материалов (привет, DoS на страницу), так ещё и 400 от хостинга выдаётся при использовании пагинатора.
Спойлер (Отобразить)
- здесь комментировать излишне
Спойлер (Отобразить)
- выдаёт 400 bad request (правда, похитрее оформленный). Введена была тарабарщина, но я ожидал увидеть, где ошибся.
Это всё бегло просмотренные ошибки. Вероятно, есть и более серьёзные недочёты.
Ну и клон habrahabr, думаю, не будет особенно популярным на фоне оригинала.
3. LinuxUser - 03 Июня, 2013 - 15:17:16 - перейти к сообщению
EuGen пишет:
Спасибо, буду исправлять
Стандартно
Спойлер (Отобразить)
- как обычно, проверки на int не везде достаточна;
Спойлер (Отобразить)
- мало того, что удаётся вызвать листинг всех материалов (привет, DoS на страницу), так ещё и 400 от хостинга выдаётся при использовании пагинатора.
Спойлер (Отобразить)
- как обычно, проверки на int не везде достаточна;
Спойлер (Отобразить)
- мало того, что удаётся вызвать листинг всех материалов (привет, DoS на страницу), так ещё и 400 от хостинга выдаётся при использовании пагинатора.
4. esterio - 03 Июня, 2013 - 15:43:38 - перейти к сообщению
ну скиньте уже и мне в личку урл. уж больно интересно стало после поста EuGen. И если несекрет Z_LoadClassException я так понял ето Zend Framework?
5. Crate - 03 Июня, 2013 - 15:45:04 - перейти к сообщению
Ну тогда и мне ссылку.
6. LinuxUser - 03 Июня, 2013 - 15:48:24 - перейти к сообщению
esterio пишет:
ну скиньте уже и мне в личку урл. уж больно интересно стало после поста EuGen. И если несекрет Z_LoadClassException я так понял ето Zend Framework?
Сейчас скину, Нет это не Zend, это мой фреймворк (Я понимаю что велосипеды это не очень хорошо, но мне было интересно сделать все самому)
(Добавление)
Crate пишет:
Ну тогда и мне ссылку.
отправил по email
7. Crate - 03 Июня, 2013 - 16:13:35 - перейти к сообщению
Ну чтож, по коду вам уже все сказали, а дизайн остался в районе веб 0.1
8. esterio - 03 Июня, 2013 - 16:18:12 - перейти к сообщению
почему мне дизайн тянет так на твердую 3+. Может просто я не дизайнер и чаще всего оцениваю код. А так кода нет, то и оценивать нечего. А багы искать щс как-то не охота
9. LinuxUser - 03 Июня, 2013 - 16:20:21 - перейти к сообщению
Crate пишет:
- значит еще поработаю и над дизайном, спасибо
Ну чтож, по коду вам уже все сказали, а дизайн остался в районе веб 0.1
10. DeepVarvar - 03 Июня, 2013 - 16:30:26 - перейти к сообщению
Код выложи - люди просят.
На гитхаб например.
Видно же что ты зарегался только ради этого раздела ))
На гитхаб например.
Видно же что ты зарегался только ради этого раздела ))
11. LinuxUser - 03 Июня, 2013 - 16:36:03 - перейти к сообщению
DeepVarvar пишет:
Окей, сделаю
Код выложи
12. LinuxUser - 03 Июня, 2013 - 20:08:38 - перейти к сообщению
esterio пишет:
почему мне дизайн тянет так на твердую 3+. Может просто я не дизайнер и чаще всего оцениваю код. А так кода нет, то и оценивать нечего. А багы искать щс как-то не охота
DeepVarvar пишет:
Код выложи - люди просят.
На гитхаб например.
На гитхаб например.
Ссылку на Github скинул по емейл, потому что не уверен можно ли ее размещать здесь)
13. DeepVarvar - 03 Июня, 2013 - 21:29:17 - перейти к сообщению
1) sitemap.xml - лишние указатели на шемы, почитай за формат.
2) /Settings/site.ini - считай что в базу тебе уже зашли, т.к. конфиг доступен по прямой ссылке.
3) Все пароли пользаков в БД хранятся в открытом виде - считай что их ВСЕХ у тебя тоже слили.
По коду все нормально, обычно, не сложно, понятно.
Если и искать косяки - то надо вникать.
Ссылку с гитхаба сюда положить конечно можно.
Может кто-то еще найдет что-то.
2) /Settings/site.ini - считай что в базу тебе уже зашли, т.к. конфиг доступен по прямой ссылке.
3) Все пароли пользаков в БД хранятся в открытом виде - считай что их ВСЕХ у тебя тоже слили.
По коду все нормально, обычно, не сложно, понятно.
Если и искать косяки - то надо вникать.
Ссылку с гитхаба сюда положить конечно можно.
Может кто-то еще найдет что-то.
14. LinuxUser - 04 Июня, 2013 - 12:37:09 - перейти к сообщению
DeepVarvar пишет:
1) sitemap.xml - лишние указатели на шемы, почитай за формат.
2) /Settings/site.ini - считай что в базу тебе уже зашли, т.к. конфиг доступен по прямой ссылке.
3) Все пароли пользаков в БД хранятся в открытом виде - считай что их ВСЕХ у тебя тоже слили.
По коду все нормально, обычно, не сложно, понятно.
Если и искать косяки - то надо вникать.
Ссылку с гитхаба сюда положить конечно можно.
Может кто-то еще найдет что-то.
2) /Settings/site.ini - считай что в базу тебе уже зашли, т.к. конфиг доступен по прямой ссылке.
3) Все пароли пользаков в БД хранятся в открытом виде - считай что их ВСЕХ у тебя тоже слили.
По коду все нормально, обычно, не сложно, понятно.
Если и искать косяки - то надо вникать.
Ссылку с гитхаба сюда положить конечно можно.
Может кто-то еще найдет что-то.
спасибо большое