Which is the Right way

Hello

which is the Right way of the two Codes.

if (HtmlSpecialchars($_GET['op']) == "news"){

or

if(isset($_GET['op']) == 'news' ? htmlspecialchars($_GET['op']) == 'news' : (isset($_GET['op']) == 'news' ? htmlspecialchars($_GET['op']) == 'news' : '')){

if (isset($_GET['op']) && trim($_GET['op']) == 'news')
{
[code here]
}

I think that first one is the right code,you can use it and show result.

Advanced Acai (http://www.articlesbase.com/health-articles/advanced-acai-review-does-advanced-acai-really-work-1778693.html)

I'm going to take an educated guess and say your 2nd example is the correct way, because it looks like you have a default value for $_GET['op'] in case it's not set.

If a variable has no value you get those undefined errors with PHP 5.3 I think. I'm not 100% sure on this though.


-----------------

It depends what he's doing with the GET var. If that if clause is the only place it's used, there's no need to set a default value. If he's using its value in code later on, then checking and setting is required.

The Get is to load a Post Form . I belive my way right but if we Need More Security i think the right way is MATTF way.

if (isset($_GET['op']) && trim($_GET['op']) == 'news')
{
[code here]
}