Ver Mensaje Individual
  #3 (permalink)  
Antiguo 20/12/2013, 12:01
Avatar de gnzsoloyo
gnzsoloyo
Moderador criollo
 
Fecha de Ingreso: noviembre-2007
Ubicación: Actualmente en Buenos Aires (el enemigo ancestral)
Mensajes: 23.324
Antigüedad: 16 años, 5 meses
Puntos: 2658
Respuesta: quiero saber si este codigo donde esta mal

La verdad es que yo lo veo demasiado largo y complicado, lleno de UPDATEs que podrían hacerse de un modo más dinámico (sin contar conque sin concoer la estructura de tablas es dificil opinar bien).

Piensalo así, si esos son los nombres de los campos, yo esto:
Código PHP:
if($level==1) {
$rs=mysql_query("Update members set Level1=Level1+1 where ID=".$refid);
}
elseif(
$level==2) {
$rs=mysql_query("Update members set Level2=Level2+1 where ID=".$refid);
}
elseif(
$level==3) {
$rs=mysql_query("Update members set Level3=Level3+1 where ID=".$refid);
}
elseif(
$level==4) {
$rs=mysql_query("Update members set Level4=Level4+1 where ID=".$refid);
}
elseif(
$level==5) {
$rs=mysql_query("Update members set Level5=Level5+1 where ID=".$refid);
}
elseif(
$level==6) {
$rs=mysql_query("Update members set Level6=Level6+1 where ID=".$refid);
}
elseif(
$level==7) {
$rs=mysql_query("Update members set Level7=Level7+1 where ID=".$refid);
}
elseif(
$level==8) {
$rs=mysql_query("Update members set Level8=Level8+1 where ID=".$refid);
}
elseif(
$level==9) {
$rs=mysql_query("Update members set Level9=Level9+1 where ID=".$refid);
}
elseif(
$level==10) {
$rs=mysql_query("Update members set Level10=Level10+1 where ID=".$refid);

bien podría reemplazarlo por esto:
Código PHP:
if($level>=AND $level<=10 ) {
$qry "Update members set Level".trim($level)."=Level".trim($level)."+1 where ID=".$refid;
$rs=mysql_query($qry);

__________________
¿A quién le enseñan sus aciertos?, si yo aprendo de mis errores constantemente...
"El problema es la interfase silla-teclado." (Gillermo Luque)

Última edición por gnzsoloyo; 20/12/2013 a las 12:14