няколко препоръки
От: mitko
На: 7-07-2005@10:15 GMT+2
Оценка: 1/Неутраленинтересен скрипт, в другата версия можеш
да помислиш за следните подобрения
= проверка дали shell_exec функцията е позволена на сървъра
= според мен е несериозно да се пише с error_reporting(0);
[Отговори на този коментар]
Към: няколко препоръки
От: SunHateR <sunhater< at >mail__dot__bg>
На: 7-07-2005@21:16 GMT+2
Оценка: 1/НеутраленАми че то ако не е позволена функцията shell_exec() просто скрипта няма да работи. Без нея просто няма смисъл да се пуска скрипта. Защо трябва да проверявам? След като се установи че не е позволена какво да става?
еrror_reporting(0) съм го сложил за да ги няма warnings. Верно, че и друго число мога да сложа (не 0), ама ме мързеше да провера кое забранява warnings и notices. Все пак всеки, който не го кефи, може да го изтрие този ред :-)
Редактиран на: 7-07-2005@21:47
[Отговори на този коментар]
Към: няколко препоръки
От: mitko
На: 8-07-2005@5:57 GMT+2
Оценка: 1/Неутрален1) Защото така е културно, като не е позволена - die('Няма поддръжка на ...'), не всички, които ползват скрипта ти ще се сетят веднага, че е от това - ще има и такива, които ще си кажат - не работи, бъгава работа, и ще отидат да търсят друг конкурентен скрипт
2) Идеята на еrror_reporting() според мен е следната - докато разработваш - пишеш с еrror_reporting(E_ALL), в продукционна версия - забраняваш грешките, а не да ги криеш и от себе си :)
Това за мен е и показател за качеството на кода, тези които са писали по-големи проекти знаят колко е важен error_reporting-a, и съм сигурен, че по-нататък и ти ще станеш фен на еrror_reporting(E_ALL) :)
[Отговори на този коментар]
Към: Към: няколко препоръки
От: SunHateR <sunhater__at__mail__dot__bg>
На: 8-07-2005@9:38 GMT+2
Оценка: 1/НеутраленХа! културно... Кефи ме определението. По принцип скрипта не е предназначен за толкова големи тъпаци. Скрипта отразява моите виждания как трябва да бъде направен и според мен няма нужда от проверка за наличието на shell_exec(). Пък нека тъпака търси конкурентен скрипт да видим дали и той ще тръгне. Аз не съм виждал други подобни скриптове. Може и да има нищо не казвам.
За error_reporting според мен по-secure е ако е 0, особено за крайни продукти, големи проекти. Така ако се скапе нещо недоброжелателят няма да види грешките и пътищата до скриптовете. Има списъл да не е 0 когато още разработваш проекта за да можеш бързо да се ориентираш да оправиш грешката.
Редактиран на: 8-07-2005@9:54
[Отговори на този коментар]
за тъпаците
От: DIVAK
На: 11-07-2005@8:36 GMT+2
Оценка: 1/Неутраленздравей
направил си нещо хубаво (за автора на скрипта), ама като позачетох коментарите и нещо неможах да разбера, поправи ме ако бъркам ама да пишеш скрипт, за да покажеш своето виждане на нещата, и другите да са тъпаци- е те тва не го разбрах.
Та аз като един голем тъпак да зема да ти дам неколко линка на подобни неща да попогледнеш, и най-важното е че вървът!
http://yola.in-berlin.de/perlwebshell/
http://alfalinux.sourceforge.net/pcom.p... (този може малко да са пипне, щото е малко старичък) и още поне към 50 само на пръв поглед в гугъла.
Та идеята ми беше, не си мисли че който поназнаива PHP, е велик скриптодраскач, а останалите са гламави "тъпаци" и да са спасяват кой ка може.Много е дразнещо автор на скрипт да гледа на потребителите на своя скрипт като на по-низши съшества.
Нищо лично, просто е дразнещо ;) , иначе идеята си я бива.
[Отговори на този коментар]
Към: за тъпаците
От: SunHateR
На: 11-07-2005@11:21 GMT+2
Оценка: 1/НеутраленГрешно си ме разбрал. Аз не казвам че другите са тъпаци. За мен, в нашия случай, тъпак е този, който само взема готови скриптове, ъплоудва ги и се мисли че е PHP/Web мастер. И когато нещо не тръгне ругае автора.
Линковете, които си дал ги погледнах... Няма да спора, само ще кажа общо, че не ми допадат. Признавам си, не съм търсил в гугл за такива скриптове. BTW поназнайвам не само PHP, но стига реклама. И не гледам на всички потребители, като на по-низши същества. Има различни типове потребители. Ако са тъпаци, тогава са по-низши. Извинявам се ако съм засегнал някой.
[Отговори на този коментар]
Към: Към: за тъпаците
От: mitko
На: 11-07-2005@12:43 GMT+2
Оценка: 1/Неутрален"Пък нека тъпака търси конкурентен скрипт да видим дали и той ще тръгне. Аз не съм виждал други подобни скриптове."
Мисълта ми беше не кой е тъпак и кой не, просто ти показах някои проблеми на скрипта, но ти веднага избиваш на "Тъпаци" :)
Кода ти трябва да е чист и дуракоустойчив доколкото е възможно, това ще спести много проблеми на теб самия, така и на колегите ти.
Както и на тъпаците :)
[Отговори на този коментар]