quiero saber si este codigo donde esta mal

[rikichin]

ya que no me funciona

Código PHP:

  //********** 
function assignreferrals($acountid,$refid,$status,$level) { 
include "config.php"; 
 
if($status==0) { 
$rs=mysql_query("Update members set ref_by=".$refid." where ID=".$acountid); 
} 
 
if($level < ($levels+1)) { 
$referralid=0; 
$rs=mysql_query("Select * from members where ID=".$refid); 
if(mysql_num_rows($rs)>0) 
{ 
$arr=mysql_fetch_array($rs); 
if($level==1) { 
$rs=mysql_query("Update members set Level1=Level1+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level1." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level1." where ID=".$refid); 
} 
elseif($level==2) { 
$rs=mysql_query("Update members set Level2=Level2+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level2." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level2." where ID=".$refid); 
} 
elseif($level==3) { 
$rs=mysql_query("Update members set Level3=Level3+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level3." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level3." where ID=".$refid); 
} 
elseif($level==4) { 
$rs=mysql_query("Update members set Level4=Level4+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level4." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level4." where ID=".$refid); 
} 
elseif($level==5) { 
$rs=mysql_query("Update members set Level5=Level5+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level5." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level5." where ID=".$refid); 
} 
elseif($level==6) { 
$rs=mysql_query("Update members set Level6=Level6+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level6." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level6." where ID=".$refid); 
} 
elseif($level==7) { 
$rs=mysql_query("Update members set Level7=Level7+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level7." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level7." where ID=".$refid); 
} 
elseif($level==8) { 
$rs=mysql_query("Update members set Level8=Level8+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level8." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level8." where ID=".$refid); 
} 
elseif($level==9) { 
$rs=mysql_query("Update members set Level9=Level9+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level9." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level9." where ID=".$refid); 
} 
elseif($level==10) { 
$rs=mysql_query("Update members set Level10=Level10+1 where ID=".$refid); 
$rs=mysql_query("Update members set Total=Total+".$level10." where ID=".$refid); 
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level10." where ID=".$refid); 
} 
 
 
 
if($arr[11]!=0) { 
$referralid=$arr[11]; 
} 
 
if($referralid!=0) { 
assignreferrals($acountid,$referralid,1,$level+1); 
} 
 
} 
} 
 
}//end of function 
 
 
//********** 
function assignreferralss($acountid,$refid,$status,$level) { 
include "config.php"; 
 
if($status==0) { 
$rs=mysql_query("Update members set ref_by=".$refid." where ID=".$acountid); 
} 
 
if($level < ($levels+1)) { 
$referralid=0; 
$rs=mysql_query("Select * from members where ID=".$refid); 
if(mysql_num_rows($rs)>0) 
{ 
$arr=mysql_fetch_array($rs); 
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); 
} 
 
if($arr[11]!=0) { 
$referralid=$arr[11]; 
} 
 
if($referralid!=0) { 
assignreferralss($acountid,$referralid,1,$level+1); 
} 
 
} 
} 
 
}//end of function  
   

[temaqueja]

Bueno, podemos comenzar por sacar el include de las funciones (preferible) ó utilizar include_once en su lugar (http://www.php.net/manual/es/function.include-once.php)

[gnzsoloyo]

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>=1 AND $level<=10 ) {
$qry = "Update members set Level".trim($level)."=Level".trim($level)."+1 where ID=".$refid;
$rs=mysql_query($qry);
}  
   

[gnzsoloyo]

Ah!
Y esta cosa:

Código PHP:

  if($level==1) {
$rs=mysql_query("Update members set Level1=Level1+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level1." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level1." where ID=".$refid);
}
elseif($level==2) {
$rs=mysql_query("Update members set Level2=Level2+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level2." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level2." where ID=".$refid);
}
elseif($level==3) {
$rs=mysql_query("Update members set Level3=Level3+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level3." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level3." where ID=".$refid);
}
elseif($level==4) {
$rs=mysql_query("Update members set Level4=Level4+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level4." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level4." where ID=".$refid);
}
elseif($level==5) {
$rs=mysql_query("Update members set Level5=Level5+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level5." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level5." where ID=".$refid);
}
elseif($level==6) {
$rs=mysql_query("Update members set Level6=Level6+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level6." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level6." where ID=".$refid);
}
elseif($level==7) {
$rs=mysql_query("Update members set Level7=Level7+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level7." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level7." where ID=".$refid);
}
elseif($level==8) {
$rs=mysql_query("Update members set Level8=Level8+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level8." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level8." where ID=".$refid);
}
elseif($level==9) {
$rs=mysql_query("Update members set Level9=Level9+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level9." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level9." where ID=".$refid);
}
elseif($level==10) {
$rs=mysql_query("Update members set Level10=Level10+1 where ID=".$refid);
$rs=mysql_query("Update members set Total=Total+".$level10." where ID=".$refid);
$rs=mysql_query("Update members set Unpaid=Unpaid+".$level10." where ID=".$refid);
}  
   

bien podrías cambiala por esto:

Código PHP:

  if($level>=1 and $level<=10 ) {
$qry = "Update members set Level".trim($level)."=Level".trim($level)."+1, Total=Total+".$level.", Unpaid=Unpaid+".$level." where ID=".$refid;
$rs=mysql_query($qry);
}  
   

En serio, deberías estudiar un poco de SQL antes de ponerte a meterlo en el PHP...

Nota: Tener diez campos para almacenar diez valores diferentes de diez niveles, es una muestra de que ea base está desnormalizada, y por tanto es bastante ineficiente. Eso más allá de que te funcione o no. Una base desnormalizada implica que a corto plazo tendrás consultas de baja performance, y problemas de optimización. Pero ese es tema de otro foro (Bases de Datos).

[marlanga]

Es con AND, si pones un OR, siempre será TRUE xd.

[gnzsoloyo]

Iniciado por marlanga

Es con AND, si pones un OR, siempre será TRUE xd.

Cierto...


Es que no me acostumbro a que no exista un BETWEEN para usar en programación, como en SQL...

[marlanga]

Yo sólo digo verdades, incluso cuando insulto.