проверка значений радиокнопок

[apexis]

на странице index.php есть форма с двумя радиокнопками

<form method="post" action="shout.php" >
			 <input type="text" id="name" class="formfields" placeholder="Имя"/> <input type="radio" name="yesorno" value="1">Положительный <input type="radio" name="yesorno" value="2">Отрицательный<br />
		 <textarea id="message" cols="85" rows="4" class="formfields" placeholder="Ваш отзыв"></textarea><br />
<input type="submit" id="submit" value="Оставить отзыв" class="sendbtn" />

часть кода обработчика shout.php

function populate_shoutbox() {
global $dbh;
$sql = "select * from shoutbox order by date_time desc limit 20";
echo '<ol class="list">';

if (isset($_POST['yesorno']) == "1") {$made = "plus";}
else {$made == "minus";}

foreach ($dbh->query($sql) as $row) {
	 echo '<li>';
echo '<div class="choise '.$made.'">';
	 echo '<span class="name">'.$row['name'].'</span><br />';
	 echo '<span class="message">'.$row['message'].'</span></div>';
	 echo '</li>';
}
echo '</ol>';
}
?>

Переменная made должна принимать значение plus или minus, но видно я что-то напутал в этой строке

if (isset($_POST['yesorno']) == "1") {$made = "plus";}
else {$made == "minus";}

Где ошибка?

[Int]

Сделай var_dump($_POST) и погляди сам

[rus]

в if вы присваиваете значение, а в else сравниваете.

[McLotos]

кстати да, rus прав

в идеале код должен выглядеть так

if (
(!empty($_POST['yesorno'])) //$_POST['yesorno'] не пустое
&&($_POST['yesorno'] == "1")) // и равно 1
{$made = "plus";} //тогда присвоить плюс
else //или
{$made = "minus";} //ПРИСВОИТЬ минус

PS почему именно так:

1. когда вы используете isset($a), php на него реагирует примерно так:

да. есть тут такая переменная, всё ОК

а когда вы используете !empty($a) - именно с восклицательным знаком, т.е. НЕ пустая

php будет реагировать немного по-другому:

да есть тут такая переменная $a, но я не могу с ней работать потому-что !empty($a) даёт результат FALSE, т.е. переменная существует но ей не задали никакого значения, т.е. она empty - пустая

Edited February 4, 2013 by McLotos

[apexis]

Вставил этот код

if (
(!empty($_POST['yesorno'])) //$_POST['yesorno'] не пустое
&&($_POST['yesorno'] == "1")) // и равно 1
{$made = "plus";} //тогда присвоить плюс
else //или
{$made = "minus";} //ПРИСВОИТЬ минус

Проверил с помощью var_dump($_POST['yesorno']); - результат null. http://lidomed.com.ua/test/.

Может javascript на странице с формой мешает?

<script type="text/javascript">
$(function() {
    refresh_shoutbox();
    setInterval("refresh_shoutbox()", 15000);

    $("#submit").click(function() {
	    var name    = $("#name").val();
	    var message = $("#message").val();
	    var data    = 'name='+ name +'&message='+ message;
if (document.getElementById('name').value == "" || document.getElementById('message').value == "") {document.getElementById('error').style.display = "block";
return false;}
if (document.getElementById('name').value != "" || document.getElementById('message').value != "") {document.getElementById('error').style.display = "none";}

	    $.ajax({
		    type: "POST",
		    url: "shout.php",
		    data: data,
		    success: function(html){
			    $("#shout").slideToggle(500, function(){
				    $(this).html(html).slideToggle(500);
				    $("#message").val("");
			    });
		  }
	    });    
	    return false;
    });
});

function refresh_shoutbox() {
    var data = 'refresh=1';

    $.ajax({
		    type: "POST",
		    url: "shout.php",
		    data: data,
		    success: function(html){
			    $("#shout").html(html);
		    }
	    });
}

</script>

[McLotos]

Честно, никогда так не извращался. и вообще считаю это плохим тоном запихивать всё подряд в один файл. такую кашу PHP+JS+CSS+HTML не поймёт ни php программист, ни html-верстальщик, на css-дизайнер, или я не прав?

как вариант, можете сделать по-другому

if (empty($_POST['yesno']))
{die('переменная пустая');}
else
{
switch($_POST['yesno'])
{
	 case '0':$made='minus';break;
	 case '1':$made='plus';break;
}
}

Edited February 5, 2013 by McLotos