- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
if (empty($_GET["page"])) {
$page_get = 1;
} else {
$page_get = "$_GET[page]";
$page_get = trim($page_get);
$page_get = stripslashes($page_get);
$page_get = htmlspecialchars($page_get);
$page_get = intval($page_get);
$page_get = "$page_get";
}
mrbig66 22.04.2010 10:48 # +9
DanxilLs 25.04.2010 23:04 # +1
leprosus 22.04.2010 11:02 # +3
Это просто классический пример говнокода!
Для администрации сайта: предлагаю открыть раздел эталонного говнокода.
mrbig66 22.04.2010 11:24 # +2
mrbig66 22.04.2010 11:25 # +2
КЭП негодуэ.
Spider 23.04.2010 00:24 # 0
C++0x 23.04.2010 00:34 # 0
Death 22.04.2010 12:23 # +4
Но решение автора - это 100% шедевр говнокода, в избранное
paranoid 22.04.2010 17:16 # 0
Death 22.04.2010 18:52 # −4
mrbig66 22.04.2010 21:34 # +1
Поэтому нужен иссет.
murz 22.04.2010 21:37 # 0
Death 22.04.2010 22:23 # −7
error_reporting = E_ALL & ~E_NOTICE
Spider 23.04.2010 00:26 # +4
Отключить вывод ошибок != избавиться от ошибок. Просто надо правильно писать код, чтобы не было никаких нотисов. paranoid прав
C++0x 23.04.2010 00:32 # −2
C++0x 23.04.2010 00:33 # −1
C++0x 23.04.2010 00:33 # −1
xXx_totalwar 23.04.2010 11:54 # 0
murz 23.04.2010 07:38 # −4
turdman 23.04.2010 09:44 # 0
murz 23.04.2010 09:53 # 0
Какая разница, если я сделаю $a = intval($_GET['id']); с отключенным нотисом и $a= isset($_GET['id']) ? intval($_GET['id']) : 0; с включенным. На выполнение это никак не повлияет.
И пример Ваш не уместен, нотисы пользователь не видит, т.е. ширинка всегда застегнута, а если Вы какую переменную забыли проверить сразу будет вывод (при соответствующих настройках). Нотисы это ширинка, а варниги - вообще без штанов ходить
Death 23.04.2010 10:21 # −3
вопрос только нафик такое писать всё время???
если можно короче и красивше.
paranoid 23.04.2010 12:49 # +1
2. если хотя бы в жопе мозги есть то поверяй как я предложил
3. ну а если головной мозг хоть немного работает - напишешь класс (ну или хотя бы ф-ю), которая это всё будет каждый раз делать за тебя. и будет тебе убогому радость Request::get('olololol')
Spider 23.04.2010 11:43 # 0
Death 23.04.2010 10:19 # −3
тем более ни разу не встречал хостинга, в котором бы вообще вывод ошыбок был включён. А локально они не мешают.
WARNING конечно нужен, но не NOTICE...
Spider 23.04.2010 11:42 # +3
P.S. Вопрос для размышления: если нотисы бесполезны - зачем разработчики PHP их ввели?
Death 23.04.2010 13:30 # −3
а нехер забывать...
murz 23.04.2010 14:31 # 0
P.S. Сам я в подавляющем большинстве случаев проверяю существование POST и GET параметров, но не понимаю тех, кто пытается доказать, что отключение NOTICE это зло.
turdman 23.04.2010 16:36 # +3
Вообще-то в приличном обществе за большую часть нотисов дают по морде exception'ом, segfault'ом или вообще не компилируют.
Раз. $items[i] = something($other, $i);
Два. something_other(SOME_CONSTANTT);
Три. $countr++;
Все три без мелких бредовых предупреждений тихо проглатываются.
paranoid 23.04.2010 12:43 # +1
paranoid 23.04.2010 12:43 # +1
а потом мля а чтож оно всё не работает... а через пол дня оказывается, что где-то в массиве нет какого-то элемента.
Death 23.04.2010 13:30 # −3
Death 22.04.2010 22:45 # −1
UncleAli 22.04.2010 22:21 # 0
Death 22.04.2010 22:29 # −2
хотя по поводу "практики" -- это скорее дело вкуса, а не практики.
лично я спокойно использую хрени типа:
$query = "SELECT id,text_name FROM {$prefix}_{$table} WHERE id='$id' AND active=1 ORDER BY $order LIMIT $limit";
при этом естественно $id приведён к int, $ord и $limit -- имеют корректную структуру. $prefix и $table задаются программно...
Вариант №2
$query = 'SELECT id,text_name FROM '.$prefix.'_'.$table.' WHERE id="'.$id."' AND active=1 ORDER BY '.$order.' LIMIT '.$limit; мне просто не нравится (и пофик на потери милисекунд в скорости)
paranoid 23.04.2010 12:51 # +1
xXx_totalwar 23.04.2010 13:30 # 0
Death 23.04.2010 13:31 # −3
Каждый пишет так, как ему удобнее ))
xXx_totalwar 23.04.2010 13:38 # +2
стандартная отмазка говнокодера №1
Death 23.04.2010 15:40 # −3
lugaro 22.04.2010 12:32 # +1
cfdev 22.04.2010 12:50 # 0
Полноценный движок для wap-сайта.
Вот некоторые его особенности:
-> Движок построен целиком на файлах
-> Имеется мощная админпанель.
-> Сайт, созданный на движке будет доступен не только с мобильного телефона, но и с компьютера.
-> У админа есть возможность менять цветовую схему сайта и его разметку. Есть поддержка скинов.
-> Управлять сайтом сможет любой человек, даже не знающий языка программирования PHP, тем не менее знания html понадобятся.
Эта система - лучший выбор для того, кто решил создать wap-сайт.
nikita2206 22.04.2010 13:16 # 0
murz 22.04.2010 13:26 # +1
lugaro 22.04.2010 13:33 # 0
nikita2206 22.04.2010 13:47 # 0
Это деградация называется. Сам не пользуюсь и не знаю ни одного человека, который пользуется. Во время 3Г, вай-фай, Ваймакс пользоваться вапом - это ужас.
ПС Может вы путаете просто мобильную версию сайта с wap-сайтом?(который кстати использует WML)
ППС А вообще давай еще ГК по этому движку, веселые там разрабы
lugaro 22.04.2010 13:53 # +1
ЗЫ http://ru.wikipedia.org/wiki/WAP
mrbig66 22.04.2010 13:41 # +1
С ума сойти. Это как?
murz 22.04.2010 14:48 # 0
striker 22.04.2010 15:54 # +7
Как говорится, при разработке ни одна папка не пострадала
cfdev 22.04.2010 15:58 # +1
ещё встречал вики-движок, который тоже чисто на файлах.
ну а вообще в 95% случаев авторы просто не осилили БД :)
unfunk 22.04.2010 16:27 # +5
UncleAli 22.04.2010 22:25 # 0
cfdev 22.04.2010 23:01 # +2
BigTable — высокопроизводительная проприетарная поддерживающая сжатие база данных построенная на основе Google File System (GFS), Chubby Lock Service и некоторых других продуктах Google.
Работа над BigTable была начата в 2004 году
:)
C++0x 23.04.2010 00:09 # −3