Better listing of products

Here is the code from the site if anyone want to have it on there site. It's a bid hardcoded, so it will need some kind of finishing.

<?php
function generaterandomproduct($itemsubclassid) {
  $random_nid = db_result(db_query("SELECT nid FROM {ec_product} WHERE itemsubclassid = %d AND uservisiblestatus = 0 ORDER BY RAND() LIMIT 1", $itemsubclassid));
  $result = db_query("SELECT * FROM {node} WHERE nid = %d", $random_nid);
  while ($data = db_fetch_object($result)) {
    $result2 = db_result(db_query("SELECT body FROM {node_revisions} WHERE nid = %d AND vid = %d", $random_nid, $data->vid));
    $startpos = strpos($result2, 'files/productpictures/');
    $startpos += 22;
    $newstring = substr($result2, $startpos);
    $endpos = strpos($newstring, 'jpg') +3;
    $body = substr($result2, $startpos, $endpos);
    if (!strstr($body, '.jpg')) {
		$body .= '.jpg';
	}
    $price = db_result(db_query("SELECT price FROM {ec_product} WHERE nid = %d", $random_nid));
    $output123 .= '      <table width="100%" cellpadding="2" cellspacing="0" border="0">';
    $output123 .= '        <tr class="messages status">';
    $output123 .= '          <td align="right" colspan="2">';
    $output123 .= '            <a href="/node/'.$random_nid.'">';
    $output123 .= '            <strong>'.payment_format($price).'</strong> ('.payment_format($price *1.25).')</a>';
    $output123 .= '          </td>';
    $output123 .= '        </tr>';
    $output123 .= '        <tr>';
    $output123 .= '          <td>';
    $output123 .= '            '.$data->title.'<br />';
    $output123 .= '          </td>';
    $output123 .= '          <td>';
    $output123 .= '            <a href="/node/'.$random_nid.'"><img border="0" height="150" width="200" src="http://www.schultzconsult.dk/files/productpictures/'.$body.'" alt="'.$body.'"/>';
    $output123 .= '          </td>';
    $output123 .= '        </tr>';
    $output123 .= '      </table>';
}
  return $output123;
}
$output123 = '<div id="randomproducts"><table border="0" cellpadding="5" cellspacing="5">';
$output123 .= '<TR>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(70);

$output123 .= '    </TD>';
$output123 .= '    <TD>';

$output123 .= generaterandomproduct(71);

$output123 .= '    </TD>';
$output123 .= '    <TD>';

$output123 .= generaterandomproduct(70);

$output123 .= '    </TD>';
$output123 .= '  </TR>';
$output123 .= '  <TR>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(71);

$output123 .= '    </TD>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(70);

$output123 .= '    </TD>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(71);

$output123 .= '    </TD>';
$output123 .= '  </TR>';
$output123 .= '  <TR>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(70);

$output123 .= '    </TD>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(71);

$output123 .= '    </TD>';
$output123 .= '    <TD class="frontpage33">';

$output123 .= generaterandomproduct(70);

$output123 .= '    </TD>';
$output123 .= '  </TR>';
$output123 .= '</table></div>';

print $output123;
?>