problema al concatenar

[morfasto]

Hola, que tal?

Estoy concatenando varias cosas en este codigo:

Código PHP:

  <?php
    $consulta="consulta";
    $consulta= mysql_query($consulta,$con1);
    $i=1;
    while($rt=mysql_fetch_array($consulta)){
        $clientes[$i]="<a href='clientes.php?id=".$rt['usuario_id']."'>".$rt['nombre']." ".$rt['apellido']."</a>";
        $i=$i+1;
    }
    $contar_clientes=count($clientes);
    if($contar_clientes==0){
        $notificacion="";
    }elseif($contar_clientes==1){
        $notificacion=$clientes[1]." ha comentado en la foto <a href='foto.php?id=".$foto_id."' >".$titulo."</a>.";
    }elseif($contar_clientes==2){
        $notificacion=$clientes[1]." y ".$clientes[2]." han comentado en la foto <a href='foto.php?id=".$foto_id."'>".$titulo."</a>.";
    }elseif($contar_clientes>=3){
        for($j=1;$j=($contar_clientes-2);$j++){
            $notificacion=$notificacion.$clientes[$j].", ";
        }
        $notificacion=$notificacion.$clientes[$contar_clientes-1]." y ".$clientes[$contar_clientes]." han comentado en la foto <a href='foto.php?id=".$foto_id."'>".$titulo."</a>.";
    }
?>   

Cuando mando el script me sale este error:

Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'clientes.php?id=6'>Sergio Corcuera ha comentado en la foto

Pero no logro identificar mi error, me parece que el codigo esta bien...

La "consulta" esta correcta, ya la probe en mi sql. $foto_id y $titulo existen, ya los probe tambien.

Alguna idea?

[carlos_belisario]

el problema lo tienes en la consuulta podrías indicarnos cual es tu consulta?? porque el error te lo dice bien claro que estas agregando un texto que no es de la sintaxis de mysql

[morfasto]

Esta es mi consulta:

select clientes.cliente_id, nombre, apellido, fecha from notificaciones, clientes where (clientes.cliente_id=notificaciones.cliente_id) and (fecha>'$rs[fecha]') and (tipo=1) and (tipo_id=$foto_id) order by nombre asc

[carlos_belisario]

pues el poblema que puede estar pasando es que el $foto_id puede estar conteniendo la cadena de texto que te indica el error que no espera, verfica que te traiga lo que tu desees, de todas maneras hasle un echo a tu consulta a ver cuales son los valores de tus variables, saludos

[morfasto]

Encontre que el error estaba en una parte que no habia puesto para que vean, es en el update del final.

Código PHP:

  <?php 
    $consulta="consulta"; 
    $consulta= mysql_query($consulta,$con1); 
    $i=1; 
    while($rt=mysql_fetch_array($consulta)){ 
        $clientes[$i]="<a href='clientes.php?id=".$rt['usuario_id']."'>".$rt['nombre']." ".$rt['apellido']."</a>"; 
        $i=$i+1; 
    } 
    $contar_clientes=count($clientes); 
    if($contar_clientes==0){ 
        $notificacion=""; 
    }elseif($contar_clientes==1){ 
        $notificacion=$clientes[1]." ha comentado en la foto <a href='foto.php?id=".$foto_id."' >".$titulo."</a>."; 
    }elseif($contar_clientes==2){ 
        $notificacion=$clientes[1]." y ".$clientes[2]." han comentado en la foto <a href='foto.php?id=".$foto_id."'>".$titulo."</a>."; 
    }elseif($contar_clientes>=3){ 
        for($j=1;$j=($contar_clientes-2);$j++){ 
            $notificacion=$notificacion.$clientes[$j].", "; 
        } 
        $notificacion=$notificacion.$clientes[$contar_clientes-1]." y ".$clientes[$contar_clientes]." han comentado en la foto <a href='foto.php?id=".$foto_id."'>".$titulo."</a>."; 
    } 
    $update_notificacion="UPDATE notificaciones SET notificacion='$notificacion' where notificacion_id='$notificacion_id'"; 
    if (!mysql_query($update_notificacion,$con1)) 
      { 
      die('Error: ' . mysql_error()); 
      } 
?>   

El problema esta con el $notificacion ya que por alguna extraña razon, la cadena que contiene $notificacion hace alguna interferencia con el update, puede que sea por las comillas, pero nose como corregirlo.

Alguna idea?

[Triby]

Antes de incluir cualquier cadena en una consulta, no olvides que por seguridad y prevencion de errores, debes pasarla por mysql_real_escape_string()... porque?, tan sencillo que si tienes una comilla simple o doble y no la escapas, habra posibilidad de inyeccion SQL y, en el mejor de los casos, el error que tienes ahora.

Por otra parte, la concatenacion es bastante sencilla, la verdad me parece muy complicado usar tanto punto cuando PHP te da la opcion de incluir e interpretar variables en cadenas delimitadas por comillas dobles.

Ejemplo:

Código PHP:

// Concatenando
$texto = "Esto es un texto " . $var1 . " que contiene " . $var2 . " variables y un " . $var3['titulo'] . " elemento de array asociativo.";
 
// Incluyendo las variables directamente
$texto = "Esto es un texto $var1 que contiene $var2 variables y un {$var3['titulo']} elemento de array asociativo.";
 

Siempre que incluyas algun elemento de array en una cadena, encierralo entre llaves y aplica las comillas sencillas cuando el indice sea alfanumerico (no numerico ni variable).

Y bueno, al parecer, el rendimiento de PHP es muy rapido cuando procesa cadenas encerradas entre comillas simples, pero se hace un poco mas lento cuando realizas concatenaciones y, todavia mas lento cuando las concatenaciones son de cadenas entre comillas dobles, lo cual no tiene mucho sentido.

[morfasto]

Muchas gracias por la ayuda! Ahora todo esta funcionando sin errores!