Duda con mi script de anuncios ?

[HiToGoRoShi]

Que tal amigos:

Esta es mi duda, mi script para matar los anuncios vencidos por los usuarios se ejecuta desde el INDEX.PHP, pero no estoy seguro pero este podria hacer un poco lento el acceso a mi homepage de mi web, asi que otra opcion recomienda o lo ven bien el codigo ?

Código PHP:

<?php
$mtrAn = mysql_query("select fecha_fin from anuncios",$cnPortal) or die(mysql_error()); 
$fecha = date("d/m/Y");
 
while($matarAnuncio = mysql_fetch_array($mtrAn)){
if($matarAnuncio['fecha_fin']==$fecha){
$id = $matarAnuncio['id'];
mysql_query("update anuncios SET estado = 0 where id = '$id'",$cnPortal) or die(mysql_error()); 
}
}
?>
 

Aunque el script es algo corto, se ejecuta siempre al iniciar al index de mi web, creen que podria generar problemas que otra alternativa podrian darme para matar los anuncios cuando le llegue su fecha

[darkasecas]

Puedes usar un crob job, y/o, (aunque esto no tiene que ver con php) en lugar de seleccionar todos los que vencen y luero recorrer los resultados actualizando uno por uno, solo hacer un update, que la condicion para actualizar sea que la fecha de vencimiento sea igual a la fecha actual >_>

[pateketrueke]

pues de hecho si esta muy mal tu script...

puedes eliminar todos tus registros vencidos con una sola consulta lógica... en tu caso, estas haciendo una consulta general, ciclando, comparando, y haciendo una consulta secundaria por cada elemento...

piensa con calma... ¿acaso esta bien todo esto??

Edito: justo lo que ha dicho darkasecas ... ;)

[cesarpunk]

porque lo recorres? bastaría con un :

mysql_query("update anuncios SET estado = 0 where fecha_fin = '$fecha",$cnPortal);

[HiToGoRoShi]

jjojojo bastante cierto en serio.... error muy noob es que me quede haciendo todo el sistema de anuncio en la madrugada y seguro mi cerebro ya no jlaba xD