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;