Почему неправильно срабатывает запись в БД MySQL PHP?

Question

[VladVysotsky]

Здравствуйте. Я пишу на джаве/си и, в силу обстоятельств, мне пришлось написать один сайт на php. Суть сайта в выборе среди двух людей того, кто более симпатичный ("Социальная сеть", привет). Так вот. Php изучаю прямо по ходу написания.

И вроде все работает (записывает текущий раудн, рейтинг), но при выборе одной записи среди, например, записей с id 2 и 7 запишет в БД 2 и 9 или 3 и 6 (не думаю, что там рандом, но все же), или может вообще только одну запись сделать. Прошу помочь мне разобраться, т.к. в окружении на php никто тоже не пишет (я студент, многого еще не понимаю).
Даю код:

Функция голосования:

function vote($plus, $minus) {
	$winRate = getRatingById($plus);
	$winRounds = getRoundsById($plus);
	$looseRate = getRatingById($minus);
	$looseRounds = getRoundsById($minus);
	$winValue = possibility($winRate - $looseRate);;
	$looseValue = 1 - $posibility;
	$winNewRate = $winRate + coef($winRate,$winRounds)*(1 - $winValue);
	$looseNewRate = $winRate - coef($looseRate,$looseRounds)*($looseValue);
    mysql_query("UPDATE `rating_db`.`girls` SET  `ROUNDS` =  '$winRounds ' + 1, `RATING` = '$winNewRate' WHERE  `girls`.`ID` = '$plus';");
    mysql_query("UPDATE  `rating_db`.`girls` SET  `ROUNDS` =  '$looseRounds' + 1, `RATING` = '$looseNewRate' WHERE  `girls`.`ID` = '$minus';");
}

Из файла index.php:

$id = getRandID(9);
	$id1_img = $source.$id['first'].$format;
	$id2_img = $source.$id['second'].$format;
    if(isset($_POST['first_x']) || isset($_POST['first'])) {
        vote($_POST['first'], $id['second']);
        redirect();
    }
    elseif(isset($_POST['second_x']) || isset($_POST['second'])) {
        vote($_POST['second'], $id['first']);
       	redirect();
    }

Почему так происходит, сижу, ломаю голову. Может, я не до конца понял принцип работы рандома php. В общем, кто может помочь или кто есть в социальных сетях или скайпе, допустим, и может посмотреть, буду признателен.

Comments

[VladVysotsky]

Еще уточню:
Функция генерации рандомного id:

function getRandID($MAX_ID) {
	$id1 = rand(1,$MAX_ID);
	do {
		$id2 = rand(1,$MAX_ID);
	} while ($id2 == $id1);
	return array('first' => $id1, 
				'second' => $id2);
}

Ну и все работает, через POST.

Answer 1

[Алексей]

Если я правильно понял, вы берете 2 рандомных числа. Выводите соответствующие картинки. А потом POST запросом одно значение получаете, а другое вы заново генерируете. Вам нужно оба ID получать через POST запрос.

Comments

[VladVysotsky]

Ой, это я уже экспериментировал. Там раньше было vote($_POST['first'],$_POST['second']); Результат был тот же. Правда, тогда, допустим, $_POST['second'] не было, если кликнуть по первой и наоборот.

[Алексей]

Понятное дело что придет один, а нужен два, по этому нужно добавить в форму скрытые поля ввода которые будут содержать нужные идентификаторы.

[VladVysotsky]

Спасибо огромное!) Вообще не знаю тонкости php и html)

Answer 2

[Павел Волынцев]

Не по теме, но

mysql_query("UPDATE `rating_db`.`girls` SET `ROUNDS` = '$winRounds ' + 1, `RATING` = '$winNewRate' WHERE `girls`.`ID` = '$plus';");

небезопасно

используйте PDO и биндинги параметров

Comments

[VladVysotsky]

Если бы я знал, что это такое :-) Но спасибо, учту, погуглю, изучу. :-)

[Павел Волынцев]

www.php.net/manual/ru/book.pdo.php раздел PDO в документации
и в частности www.php.net/manual/ru/pdostatement.bindparam.php - смотри примеры