deMentoR
Вот кто-нить знает о славе сайта http://manliks.ru/manlix админ которого пишет бесплатные скрипты. Если нет, то зайдите, это пожалуй самый посещаемый из личных сайтов по программированию, а ведь скрипты то там лажа, а код это ваще ужжас, когда я начал разбираться в PHP я понял, что коды там далеки от идеала
Знаешь, скачал я оттуда ради интереса скрипт фотогалереи. Как для бесплатного - более менее (Т.к. бесплатные ставят себе обычно те, кто сам писать не умеет и кому много не надо, и они и делаются максимально просто. А всем высоким индивидуальным требованиям все равно никогда не угодишь).
Посмотрел код - знаешь, довольно ясный и чистый код, можно сказать хороший. Конечно, некоторые (приличные) глупости (неоптимальности) мне лично сразу бросились, но у тебя прав называть этот код "ужас" - ровно столько же, сколько у Доктора называть твой код "говно" (а он, кстати, этого не делал, перечитай что он на самом деле написал).
А вот на твой код я, кстати, не могу посмотреть (вернее, легально не могу ;-) - см. ниже) и сравнить - для скачивания-то твой скрипт не выложен...
deMentoR
я просто пытаюсь понять, чем мой скрипт вам так не угодил
Твой скрипт ничем не не угодил. Не угождает людям твой тон и самоуверенность:
Доктор
Люди, которые думают, что все на свете знают, раздражают нас, людей, которые на самом деле все на свете знают
Ты пришел вопросы задавать, а ведешь себя излишне скандально и самоуверенно. На самом деле твоя квалификация достаточно невысока, что в разговоре хорошо всегда видно тем, у кого она выше. Поэтому твоя самоувереннось типа "я-сам-все-знаю" в ответ на советы или огрызание в ответ на иронию (которая в ответ на "самый лучший в мире скрипт" вполне уместна :-)) - раздражает.
Теперь о твоем скрипте (не поленился я...):
1) >>9. Фильтруютя все специальный символы PHP, т.е. нет бага при введении кавычек и слеша.
"Нет бага" - фича хорошо звучит... а норма значит - "есть баг", так? И вообще - где мог быть этот баг, если ты данные в файле хранишь а не в БД?
2) >>10. Возможность администратору прописывать любое форматирование новости путём добавления html тегов.
Тоже важная фича, явно заслуживает упоминания :-).
3) Ссылку "администрировать" всегда лучше на виду не вешать.
4) Данные не проверяются на соответствие формату - ни мыло, ни дата...
5) О дате отдельный разговор: "не будет работать, если другой формат"... профессионально, блин...
Надо делать какой-нибудь формальный формат и проверку, а не просто так лишь бы что вписать.
6) При редактировании шаблонов: после сохранения шаблона - остается прежний УРЛ + попробуй надавить рефреш. Что это за методы??
7) Гвоздь программы - создание своего файла *.tmp в папке с шаблонами:
http://ruscripts.com/ru/demos/rusnews/admin.php?go=change_incs_form&what=bbbbb- и жми сохранить...
Но это еще цветочки... По предыдущему пункту - видишь что изначально в поле ввода? include, говоришь?
РомикШеф, оказывается, был не прав, что ПХП не надо фильтровать :-). Но он же не знал - где надо и почему именно надо :-)).
Теперь мы можем по желанию - либо в твоем шаблоне делать на серваке что хотим, либо создать свой шаблон. И даже заставить его выполняться не только при его редактировании, а напрямую. Оба варианта:
http://ruscripts.com/ru/demos/rusnews/admin.php?go=change_incs_form&what=aaaaahttp://ruscripts.com/ru/demos/rusnews/templates/aaaaa.tmpА ты говоришь... Вот тебе и чемоданы дыр. А если код внимательно почитать, что там еще будет?
