What's wrong with this code?

[-Adam]

}if ($arr['rankpos'] == "1" || $arr['rankpos'] == "2" ||$arr['rankpos'] == "3" ||$arr['rankpos'] == "4" ||$arr['rankpos'] == "5" ||$arr['rankpos'] == "6" ||$arr['rankpos'] == "7" ||$arr['rankpos'] == "8" ||$arr['rankpos'] == "9" ||$arr['rankpos'] == "10" ||$arr['rankpos'] == "11"){ ?>
<table align="center" width="100%" class="tbl">
<tr><td align="center" class="hdr">.::Unlucky::.</td></tr>
<tr><td align="left" class="tbl">
GOO Away.<br />
</td></tr>
</table>
<br />

Whats wrong with this? Im trying to make it so that if your rankpos isnt at least 12, it says go away.

[jackass]

Why don't you just do... :S

| $arr['rankpos'] >= "12" |

[eLv]

OR

<?php if ($arr['rankpos'] != "12") {
<table align="center" width="100%" class="tbl">
<tr><td align="center" class="hdr">.::Unlucky::.</td></tr>
<tr><td align="left" class="tbl">
GOO Away.<br />
</td></tr>
</table>
<br />
<?php } ?>

[Jamesy]

OR

<?php if ($arr['rankpos'] != "12") {
<table align="center" width="100%" class="tbl">
<tr><td align="center" class="hdr">.::Unlucky::.</td></tr>
<tr><td align="left" class="tbl">
GOO Away.<br />
</td></tr>
</table>
<br />
<?php } ?>

That wont do what he wants if the rank is above 12 though

[ThisNameWillDo!]

<?
if ($arr['rankpos'] >= "12")
echo "Go away?";
?>

[Source]

I don't quite get why people are putting integers in strings?

<?php

if( $arr['rankpos'] < 12 ) echo 'Go Away!';

?>

Also I am pretty sure everyone has been using the equal to or more than operator, when he clearly stated he wanted it to show go away if the level wasn't at least 12.

[Blob]

I don't quite get why people are putting integers in strings?

<?php

if( $arr['rankpos'] < 12 ) echo 'Go Away!';

?>

Also I am pretty sure everyone has been using the equal to or more than operator, when he clearly stated he wanted it to show go away if the level wasn't at least 12.

No need for if there, you could easily do

<?php

echo ( $arr['rankpos'] < 12 ) ? "Go Away": "Hello";

?>

Where hello is the content

[Source]

That is really extremely illogical blob, apart from the fact you may have wanted to show off you can do some extremely basic PHP work.

Those statements ( : ? ) should only be used for setting variables in my personal opinion as what you have done would look messy and unorganised if it was a big amount of content. He wanted a simple if, else statement that would do it neatly and without the need to over complicate things.

[Blob]

That is really extremely illogical blob, apart from the fact you may have wanted to show off you can do some extremely basic PHP work.

Those statements ( : ? ) should only be used for setting variables in my personal opinion as what you have done would look messy and unorganised if it was a big amount of content. He wanted a simple if, else statement that would do it neatly and without the need to over complicate things.

Not really, I was showing a different way to do things, that doesn't look messy at all...

[Invent]

Not really, I was showing a different way to do things, that doesn't look messy at all...

If you've got a huge block of HTML in the echo then the ternary operator is not a smart choice.