Re: better way to mix html and php code?

[Curt Zirzow <czirzow@xxxxxxxxx>]

On Wed, Nov 16, 2005 at 10:12:53AM -0600, afan@xxxxxxxx wrote:
> Hi to all,
> always wondered what's better way to mix html and php code. Here are 
> three "ways" of the same code. Which one you prefer? (And why, of caurse :))
> 
> Solution 2a:
> ############################################################################################
<?php
require 'includes/header.php';
 
$cat_parent = (int)$_GET['cat_parent'];
$cat_id = (int)$_GET['cat_id'];

$cat_names = $_SESSION['CATEGORIES']['name'];

?>

<table class="table-heading">
	<tr>
		<td>
			<?php echo htmlentities($cat_names[$cat_parent])?> ::
			<?php echo htmlentities($cat_names[$cat_id])?>
		</td>
		<td>
			<a href="new_product.php?cat_id=<?php echo $cat_id ?>&cat_parent=<?php echo $cat_parent?>">[ Add New Product ]</a>
		</td>
	</tr>
</table>

<table class="table-results">
<?php

$sql = "SELECT  p.prod_id, p.prod_name, p.prod_no, p.prod_status
          FROM categories_has_products as chp, products as p
         WHERE chp.categories_cat_id = :cat_id 
           AND chp.products_prod_id = p.prod_id";

$stmt = $dbh->prepare($sql);
$stmt->bindParam(':cat_id', $cat_id);
$stmt->execute();

$query_cat_id_parent = "products.php?cat_id=$cat_id&cat_parent=$cat_parent"; 
while($result = $stmt->fetch())
{
	$query_product = "$query_cat_id_parent&prod_id={$result['prod_id']}";

	$del_link = "$query_product&action=delete";

	switch($result['prod_status'])
	{
		case 'live':
			$status_link = $query_product ."&new_status=hidden";
			$status_img  = 'live';
			break;

		case 'hidden':
			$status_link = $query_product ."&new_status=live";
			$status_img  = 'hidden';
			break;

		default:
		case 'temp':
			$status_link = '';
			$status_img  = 'temp';
	}

?>

	<tr>
		<td class="prod_name">&raquo; <?php echo htmlentities($result['prod_name']); ?></td>
		<td class="prod_no">[ <?php echo $result['prod_no'] ?> ]</td>
		<td class="prod_action">
			<img src="../images/icon_edit2.gif" alt="EDIT" width="14" height="14">

			<a onclick="return confirm('Do you really want to delete this product?');"
				href="<?php echo $del_link?>">
				<img src="../images/icon_delete.gif" alt="DELETE" width="14" height="14">
			</a>

			<?php if($status_link) { ?>
				<a href="<?php echo $status_link?>">
					<img src="../images/status_<?php echo $status_img?>.gif" 
						alt="<?php echo $status_img?>" width="13" height="13">
				</a>
			<?php } else { ?>
				<img src="../images/status_<?php echo $status_img?>.gif" 
					alt="<?php echo $status_img?>" width="13" height="13">
			<?php } ?>

		</td>
	</tr>

<?php

} // $stmt->fetch()

?>
</table>

<?php

include 'includes/footer.php';
?>

Curt.
-- 

-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php