Code improvement

[freakunleash]

Hi All,

Im novice and need help in reducing repeating task which are in below code. all PHP code is same except the SQL statement is each block. I will have more of these blocks and don't want to mess with my index file with repeating line of code.

		<div class='wrapper_main1' id='wrapper_Total'>
			<?php
				// sending query
				$result = mysql_query($sql_Total);
				if (!$result) {
					die("Query to show fields from table failed");
				}

			$fields_num = mysql_num_fields($result);

			echo "<h1>Table: </h1>";
			echo "<table border='1'><tr>";
			// printing table headers
			for($i=0; $i<$fields_num; $i++)
			{
				$field = mysql_fetch_field($result);
				echo "<td>{$field->name}</td>";
			}
			echo "</tr>\n";
			// printing table rows
			while($row = mysql_fetch_row($result))
			{
				echo "<tr>";

				// $row is array... foreach( .. ) puts every element
				// of $row to $cell variable
				foreach($row as $cell)
					echo "<td>$cell</td>";

				echo "</tr>\n";
			}
		?>
	</div>
	<div class='wrapper_main1' id='wrapper_Host'>
		<?php
			// sending query
			$result = mysql_query($sql_Host);
			if (!$result) {
				die("Query to show fields from table failed");
			}

			$fields_num = mysql_num_fields($result);

			echo "<h1>Table: </h1>";
			echo "<table border='1'><tr>";
			// printing table headers
			for($i=0; $i<$fields_num; $i++)
			{
				$field = mysql_fetch_field($result);
				echo "<td>{$field->name}</td>";
			}
			echo "</tr>\n";
			// printing table rows
			while($row = mysql_fetch_row($result))
			{
				echo "<tr>";

				// $row is array... foreach( .. ) puts every element
				// of $row to $cell variable
				foreach($row as $cell)
					echo "<td>$cell</td>";

				echo "</tr>\n";
			}
		?>
	</div>
	<div class='wrapper_main1' id='wrapper_Users'>
		<?php
			// sending query
			$result = mysql_query($sql_User);
			if (!$result) {
				die("Query to show fields from table failed");
			}

			$fields_num = mysql_num_fields($result);

			echo "<h1>Table: </h1>";
			echo "<table border='1'><tr>";
			// printing table headers
			for($i=0; $i<$fields_num; $i++)
			{
				$field = mysql_fetch_field($result);
				echo "<td>{$field->name}</td>";
			}
			echo "</tr>\n";
			// printing table rows
			while($row = mysql_fetch_row($result))
			{
				echo "<tr>";

				// $row is array... foreach( .. ) puts every element
				// of $row to $cell variable
				foreach($row as $cell)
					echo "<td>$cell</td>";

				echo "</tr>\n";
			}
		?>
	</div>

[Weedpacket]

function wrapper_block($id, $sql)
{
?>
<div class='wrapper_main1' id='<?php echo $id?>'>
            <?php
                // sending query
                $result = mysql_query($sql);
                if (!$result) {
                    die("Query to show fields from table failed");
                }

            $fields_num = mysql_num_fields($result);

            echo "<h1>Table: </h1>";
            echo "<table border='1'><tr>";
            // printing table headers
            for($i=0; $i<$fields_num; $i++)
            {
                $field = mysql_fetch_field($result);
                echo "<td>{$field->name}</td>";
            }
            echo "</tr>\n";
            // printing table rows
            while($row = mysql_fetch_row($result))
            {
                echo "<tr>";

                // $row is array... foreach( .. ) puts every element
                // of $row to $cell variable
                foreach($row as $cell)
                    echo "<td>$cell</td>";

                echo "</tr>\n";
            }
        ?>
</div> 
<?php
}
?>

.....

<?php
wrapper_block('wrapper_Total', $sql_Total);
wrapper_block('wrapper_Host', $sql_Host);
wrapper_block('wrapper_Users', $sql_Users);
//...
?>

[freakunleash]

Thanks Weedpacket

This really help.