Thread: HTML Within PHP

I have the following code that is inserted into a Wordpress theme file. It's pretty slick! It guesses at the visitor's country, based on a database of IP addresses.

As you can imagine, it's not outputting correctly. I am no coder so I need some help with this. The following code should (hopefully) determine the country a visitor is from and display the appropriate banner to them.

PHP Code:

<?php
$country = wp_ozh_getCountryName(0) ;
switch ($country) {
   case "Canada" :
<a target="_blank" href="http://www.affiliate.com/fs-bin/click?id=referrer_ID"><IMG alt="Canadian Banner" border="0" src="http://images.affiliate.com/outsidebanners/Homepage-120X600.gif"></a><IMG border="0" width="1" height="1" src="http://ad.affiliate.com/fs-bin/show?id=referrer_ID">;
        break;
   default :
<a target="_blank" href="http://www.affiliate.com/fs-bin/click?id=referrer_ID"><IMG alt="American Banner" border="0" src="http://images.affiliate.com/outsidebanners/Homepage-120X600.gif"></a><IMG border="0" width="1" height="1" src="http://ad.affiliate.com/fs-bin/show?id=referrer_ID">;
}
?>

The code that determines the country is working fine when I substitute the html banner links with the echo command. I just need someone to tell me how to include the html code with the correct syntax inside the php tags.

Thanks!

Try this

Code:

 <?php 
$country = wp_ozh_getCountryName(0) ; 
switch ($country) { 
   case "Canada" : 
?>
<a target="_blank" href="http://www.affiliate.com/fs-bin/click?id=referrer_ID"><IMG alt="Canadian Banner" border="0" src="http://images.affiliate.com/outsidebanners/Homepage-120X600.gif"></a><IMG border="0" width="1" height="1" src="http://ad.affiliate.com/fs-bin/show?id=referrer_ID">; 
<?
        break; 
   default : 
?>
<a target="_blank" href="http://www.affiliate.com/fs-bin/click?id=referrer_ID"><IMG alt="American Banner" border="0" src="http://images.affiliate.com/outsidebanners/Homepage-120X600.gif"></a><IMG border="0" width="1" height="1" src="http://ad.affiliate.com/fs-bin/show?id=referrer_ID">; 
<?
} 
?> 

When you have more that can be quickly printed or echoed, switch back and forth between code mode and html mode <? ?>. It makes it so much easier to read.


Cyberbite added 1 Minutes and 54 Seconds later...

And if you need to get a variable into the HTML you can do this

Code:

<?
//code
?>
<a href="http://blah><?=$variable?></a>
<?
//code
?>

See the <?=$variable?>, use that to echo php code into html.

Thanks Dude!


Worked like a charm!

PHP is not the prettiest looking language but its the easiest I've ever used. Typically I follow Cyberbyte's logic, but if you ever do go the other way, get a good PHP IDE that show's you where you have errors, or you will be looking for a missing backslash all day.

The caveat of the above code is that it uses short tags and IIRC the <?=$foo?> is also being deprecated.

You should always use long tags <?php and use a full echo <?php echo $foo; ?> to be sure your code won't break in the near future.

It's easy house keeping to avoid house cleaning in the future.

Originally Posted by jronmo

PHP is not the prettiest looking language but its the easiest I've ever used. Typically I follow Cyberbyte's logic, but if you ever do go the other way, get a good PHP IDE that show's you where you have errors, or you will be looking for a missing backslash all day.

Use error_reporting() properly and your scripts will tell you everything that is wrong from your server without an IDE.