Xwab
Форумыnavigate_nextИстория

Топ: Небольшие вопросы по PHP
Сообщения
КрошкаБуги

#8911
Ну у тебя так
Генерация
if(){
Обработчик
}else{
Форма
}
Идёт интерпретация,переменной сессии присваиваеться рандом.В форме выводиться,передаеться в ПОСТ.Опять интерпретация, переменной сессии присваиваеться новый рандом сверяеться со старым с ПОСТ,естественно не равно.
добавлено спустя 2 минуты:
Сделай так
if(){
Обработчик
}else{
Генерация
Форма
}

11 Мар 2011, 21:27
LooK

Как правильно кол-во прочтений считать чтоб накрутить нельзя было?

11 Мар 2011, 21:42
КрошкаБуги

LooK, проверяй по ip. Одному ip не больше одного прочтения. Накрутить то смогут,но не тупо обновляя страницу. Можеш ещё куку устанавливать.Получилось,знач считаем.
Или писать чё нибудь в сессию на странице ранее а в прочтении проверять записанно ли?
Усложнит достаточно.Че сильно так крутят?

11 Мар 2011, 21:54
aBSuRD

Slan1, убери в начале скрипта эту строчку:
$_SESSION['code']=rand(1000, 9999);
и поставь её перед выводом:
$_SESSION['code']=rand(1000, 9999);
echo 'Описание: блаблабла Проверочный код: '.$_SESSION['code'].' блаблабла';
Как то так)

11 Мар 2011, 21:55
Slan1

КрошкаБуги,aBSuRD спасибо!Оказывается я забыл в форме актион сменить т.е. данные передавались с reg.php к reg1.php..Короче налагал.Ещё раз спасибо!

11 Мар 2011, 22:13
КрошкаБуги

aBSuRD, предложенный тобой вариант

}Slan1, и вместо этого:if(!empty($_POST['name']) and !empty($_POST['cat_id']) ...и еще куча POST данных...) {Лучше просто кнопке назначить имя, и проверять, нажата ли она: ...

косяковый. Так не будет происходить проверка данных на пустоту.
Предлагаю оставить код в исходнике.

11 Мар 2011, 22:13
aBSuRD

КрошкаБуги, че ты предрался то?гг Я не предложил все изменить, просто показал, как удобнее

11 Мар 2011, 22:17
КрошкаБуги

aBSuRD, упороть косяк это не совсем удобно. -))) Это не придирание а поучительные полезные истории по оптимизации. (Бгг)
Например меня ещё смущает использование функции obr() в коде автора в фильтрации на право и налево. В целочисленных она точно лишняя,intval() достаточно с головой,т.е имеем транжирство ресурсами,а вот в стрингах то хотелось бы её видеть.
добавлено спустя 13 минут:
О.о
Далее идет еще и preg_match() по целочисленным с патерном 0-9 .
Это ваще лишнее после intval() то.Не говоря что такие регулярки зло,ибо аналогично ctype_digit() только работает в раз 50 медленее пожирая ресурсы. :nono:

11 Мар 2011, 22:47
I_am_Silence

Ребят, подскажите пожалуйста, можно какой нiбудь функцией спарсить все jpg изображения.. Если они лежат в директории такого вида - сайт.ru/includes/img/chat/12345.jpg .. ? Если можно, подскажите плиз, какуЮ функцию юзать, или какие маны почитать

12 Мар 2011, 10:14
Fashion

I_am_Silence, preg_match_all('#site\.ru/includes/img/chat/\d+\.jpg#i',$a,$b);
print_r($b);

12 Мар 2011, 11:27
Эта тема закрыта, вы не можете писать ответы и редактировать сообщения.