es optimo este script ?

[p2pdownloading]

hola, queria saber si el script que acabo de hacer para aumentar un contador en la bd es optimo, o se le puede hacer algo para que consuma menos cpu

Código PHP:

  $id = intval($_GET['down_id']);
if($id > 0) {
include($_SERVER['DOCUMENT_ROOT'].'/wp-config.php');
$con=mysql_connect(DB_HOST, DB_USER, DB_PASSWORD);
mysql_select_db(DB_NAME,$con);
    $file = mysql_query("SELECT id, date, link, permissions FROM $table_prefix"."dm_downloads WHERE id='$id'",$con);
    if(!$file) {
      header('HTTP/1.0 404 Not Found');
      header('Refresh: 4; URL='.$_SERVER['PHP_SELF']);
      exit(__('Invalid File ID.', 'downloads-manager'));
    } else {
    $file = mysql_fetch_array($file);
    mysql_query("UPDATE $table_prefix"."dm_downloads SET clicks = clicks + 1 WHERE id = '$id'",$con);
    header("Location:$file[link]");die();
    }
mysql_close($con);    
} 
   

saludos y gracias

[Triby]

Optimo tal vez si, pero no funcional, porque $files sera falso solo cuando haya error en la consulta, pero verdadero cuando se ejecute bien, independientemente de que se encuentre algun resultado o no.

Código php:

mysql_select_db(DB_NAME,$con);
$file = mysql_query("SELECT id, date, link, permissions FROM $table_prefix"."dm_downloads WHERE id='$id'",$con) or die(mysql_error());
if(mysql_num_rows($file) < 1) {
      mysql_close($con);
      header('HTTP/1.0 404 Not Found');
      header('Refresh: 4; URL='.$_SERVER['PHP_SELF']);
      exit(__('Invalid File ID.', 'downloads-manager'));
}
$datos = mysql_fetch_array($file);
mysql_query("UPDATE $table_prefix"."dm_downloads SET clicks = clicks + 1 WHERE id = '$id'",$con) or die(mysql_error());
mysql_close($con);
header("Location:{$datos['link']}");
exit;