Как правильно написать if else

[Dude]

Здраствуйте! Подскажите пожалуйста, как правильно написать If - Else в php.

Вобщем, у меня с первой страницы на вторую передается значение переменной cat_id. А на второй, в зависимости от значения, должно подставлятся style="display:block;" или style="display:none;".

Выйти должно:

<div id="qwe" style="display:block;" >
123
</div>

или

<div id="qwe" style="display:none;" >
123
</div>

Я пишу:

<div id="qwe" 
<?
if (cat_id == 1)
{
echo 'style="display:block;" >'; 
}

else
{
echo 'style="display:none;" >';
}
?>
123
</div>

А мне выдает ошибку. Как нужно правильно написать?

Извеняюсь. Несколько часов мучался, в чем проблема? А оказалось, что я забыл написать знак переменной перед cat_id в "if ($cat_id == 1)". Вот за что я и не люблю точные науки. Одна ошибка, и уже ничего не получется))

Edited December 29, 2008 by Dude

[Игорь Ермаков]

echo "style='display:none;'" >;

думаю что так.

Но в любом случае уберите эти > они там ни к селу не к городу

[Иван Шумов]

не, все банально - кто перед cat_id признак переменной ставить будет?

[Dude]

У меня,вроде, все стало работать. С первой страницы мне передается переменная cat_id. А я проверял, ровен ли текст cat_id единице.

Edited December 29, 2008 by Dude

[SelenIT]

Это был не текст, а неопределенная константа. Вот ради того, чтоб не попадаться на таких досадных мелочах, и нужно обязательно выводить (не обязательно на экран, хоть куда-нибудь) все ошибки, включая Notice (т.е. error_reporting(E_ALL) без каких-либо исключений!).

[rus]

Не уверен, но может так:

<div id="qwe" 
<?
if (isset($_GET['cat_id']))
{
$cat_id = $_GET['cat_id'];
if (cat_id == 1)
{
echo 'style="display:block;" >'; 
}

else
{
echo 'style="display:none;" >';
}
?>
123
</div>