Хочу сделать что-то типа голосования. Есть файл имеющий тип:
Ответ1:
Ответ2:
Ответ3:(кол-во ответивших так)
Есть форма:

Код

<form action="vote.php" method="GET"> <b>ВопросиК(например Оцените дизайн сайта)<br> <input type=hidden name=id value=1> Хорошо: <input type="radio" name="ans" value="0"><br> Средне: <input type="radio" name="ans" value="1"><br> Плохо:  <input type="radio" name="ans" value="2"><br> <input type="submit" name="submit" value="Vote"> </form>

И есть скрипт который должен просто записать то что выбрал:

Код

<? include("config.php"); //Тама настройки типа $base_dir="votes" if (isset($_REQUEST["id"])) //ID голосования {  chdir($base_dir); //перехдим в папку  $file=file($id.".dat"); //1.dat в массив  $f=fopen($id.".dat", "w+"); //перезаписываю 1.dat  for ($i=0; $i<count($file); $i++) //3 раза проходит?  {   list($q,$a) = explode("::", $file[$i]);  //разбиваю вопрос и ответ   if ($ans==$i) { $a=$a+1;} //$a++ вообще вроде не работало ну это роли не играет.   fputs($f,$q."::".$a."\r\n");//а почему он тут не правильно записывает?  } fclose($f); } ?>

Я долго мучался незнаю может просто туплю.. мало спал сегодня =) Если в массив идут строки \r\n\ то почему он записывает без перехода если я не указываю их сам?
А если указываю получается пробелы и другой бред. Если выбрал 1 вариант
Хорошо::1
Норма:
--а тут пустая строка появляется--
Плохо:
А если выбераю последнее или среднее я непомню то тама вообще что то типа
Хорошо::1
-пустая
Норма:
-пустая
Плохо
:

Простите если че нить глупое допустил но ни кто не может дать готовую номальную функцию для голосования?? Или скажите где ошибка....

Слухай! вопервых нах проходить 3 раза?
для разбиения на массив используй регекспы!
preg_match_all('/<div class="text">(.*)<\/div>/Uims', $f, $text);
вовторых нафига тебе текстовой файл? пользуй мускул! быстрее и круче! 2 запроса и всё в порядке!

проще исеть такую структуру файла:

Цитата

11:22:15

Соответствено ответ1:11, ответ2:22,ответ3:15
читаешь весь файл в переменную, затем разрезаешь строку $mas=explode(':',$file_content) затем $mas[$nomer_otveta]++; затем склеиваешь $file_content=join(":",$mas) и пишешь обратно в файл

Ripper, а теперь подумай, что будет с твоим траффиком, если я сделаю так:

HTML

<form action="vote.php" method="GET"> <b>ВопросиК(например Оцените дизайн сайта)<br> <input type=hidden name=id value="http://mysite.com/video700megabytes"> Хорошо: <input type="radio" name="ans" value="0"><br> Средне: <input type="radio" name="ans" value="1"><br> Плохо: <input type="radio" name="ans" value="2"><br> <input type="submit" name="submit" value="Vote"> </form>

Кажется, будет не очень хорошо

dm9 Ты это к тому что можно вставить скрипт любой допустим? Ну в обрабатывабщем скрипте можно проверить если ID есть то если есть файл $id.".dat" то работать дальше?

Secandr
Спасибо попробую

Ripper Передовать имя файла из формы - секуритибаг. Так делать нельзя.

Если уж сильно хочется, то делай проверку:

Код

$id+=0; // теперь $id точно число. if (($id<1)and($id>999)){echo"Ну ты и сволочь! взломать меня захотел!!!!!";exit;}

Secandr насчет файла... а как я выведу вопрос? Я знаю только их номера ведь?
И насчет Id непойму. у меня есть куча файлов. 1.dat ,2.dat ,3.dat, и вывожу все в зависимости от айди... разве нельзя проверить есть ли такой файл у меня в папке ?

Ripper
ты будешь делать проверку на существование, а я в поле id введу вместо цифры другую информацию,например имя одного из твоих скриптов, а в конце пробел, и затрёшь ты свой файл. Мелкое но западло!

Из формы лучше передовать только цифры, как ты и делаешь, но ты не проверяешь корректность информации при открытии файла, а этим может воспользоваться злоумышленик.

$id.",dat" - не защита от взломщика, а защита от сбоя и дурака.

Цитата

dm9 Ты это к тому что можно вставить скрипт любой допустим? Ну в обрабатывабщем скрипте можно проверить если ID есть то если есть файл $id.".dat" то работать дальше?

Secandr многое сказал, но добавлю ещё.

Просто file() работает так, что если ты передашь ему url, он его будет пытаться скачать. В этом сучае:
1. Ты тратишь много траффика.
2. Процесс php занимает до фига памяти и очень сильно грузит систему (сам проверял, на Apache под Windows).

Это отключается в настройках php, но по умолчанию, насколько я знаю, включено... Почитай статьи по безопсности. Я давал пару интересных ссылок в соседнем топике.

PS Как раз большие файлы c расширением dat в инете найти несложно, вот можно и устроить западло. Конечно, этот баг ещё надо будет найти, ведь php-код не выдаётся сервером, но всё равно такие дыры не есть гуд.

Ну а как мне вопрос вывести? (secandr) И потом все равно не догоняю... я же могу проверить чтобы айди тока из цифр состояло. Или например если запрос не с формы то остоновить действия? И вообще как лучше делать голосование т.е. строктура файлов... чтобы можно было делать их много и вставлять в любое место сайта... как я определю какое голосование я вставил? тока через hidden??

Я и говорю, что нужно проверять чтобы id состояло только из цифр. $id+=0; - точно переведёт id в числовой вид, а затем ифом проверяешь не фыходит ли id за рамки <1 и >9999

А еще лучше установить точный путь до базы. И читать так: file('basedir/'.intval($id).'.dat');, максимум защиты, минимум кода. Единственное что тебе светит, это ошибка интерпретатора что файл не существует.