porque no me funciona esta concatenacion

[pithon]

Hola la consulta esta bien porque la compruebo con valores fijos y funciona pero al hacer la concatenacion me da error muestro la linea de error:

Código PHP:

<?php
$sql="SELECT ciudad.ciudades
FROM ciudad
WHERE  ciudad.ciudad_id=".$_GET["id"]."
AND ciudad.ciudades='".$_GET["ciudades"]."'";
?>
 

El id es un numero y ciudades letras, alguna ayuda, saludos.

[Ronruby]

Haz verificado los valores que traen las variables ID y ciudades? Trata imprimir la variable $sql antes de hacer la consulta y verifica si tiene algun error.

[Sourcegeek]

Se supone que debemos adivinar el error?

[pithon]

comprobe la consuta con valores fijos con phpmyadmin y me dio ok, y tambien mostre en pantalla los datos por $_GET y tambien me dio correcto.
Tengo bien la concatenacion?

[pithon]

perdonar muestro las lineas de error:

Undefined index: id in C:\wamp\www\clientes\form.php on line 50

Undefined index: ciudades in C:\wamp\www\clientes\form.php on line 51

[pithon]

Hola bueno resulta que la concatenacion esta bien y el codigo me funciona bien lo que pasa que al cargar la pagina me salen estos errores hasta que no elijo una opcion del formulario no se quitan. Como puedo hacer que no salgan estos errores?

[andresdzphp]

Está claro que no se están pasando correctamente esas variables vía $_GET verifica el formulario.

[jamesjara]

Iniciado por pithon

Hola bueno resulta que la concatenacion esta bien y el codigo me funciona bien lo que pasa que al cargar la pagina me salen estos errores hasta que no elijo una opcion del formulario no se quitan. Como puedo hacer que no salgan estos errores?

Amigo lo que pasa es que tienes MODO ESTRICTO tu configuracion de php , lo que ves son NOTICE no ERROR asi que son noticias de que no tienes definido tus variables.. Y aqui hablamos de la variable en si..

Solucion MALA 1: agrega @ antes de la variable para no mostrar mensajes de erores o noticas No la utilices aqui , te cuento para que aprendas

Código PHP:

     <?php
    $sql="SELECT ciudad.ciudades
    FROM ciudad
    WHERE  ciudad.ciudad_id=".@$_GET["id"]."
    AND ciudad.ciudades='".@$_GET["ciudades"]."'";
    ?>   

Solucion CORRECTA 2: define las variables en null primero.. pero como en este caso estas llamando una variable GET debes no puedes definirla pero si comprobarla con ISSET http://php.net/manual/es/function.isset.php

Código PHP:

     <?php
if (isset($_GET["id"])and($_GET["ciudades"])){
    $sql="SELECT ciudad.ciudades
    FROM ciudad
    WHERE  ciudad.ciudad_id=".$_GET["id"]."
    AND ciudad.ciudades='".$_GET["ciudades"]."'";
} else {
die('faltan parameetros');
}
    ?>   

Si lo quieres mas elegante:

Código PHP:

     <?php
     $continue = true;            
     $var1 = isset($_GET['id'])     ? $_GET['id'] : $continue = false ;    
     $var2 = isset($_GET['ciudades'])     ? $_GET['ciudades'] : $continue = false ;    
     
      if ($continue==true){

    $sql="SELECT ciudad.ciudades FROM ciudad   WHERE  ciudad.ciudad_id=".$var1"  AND ciudad.ciudades='".$var2."'";

      }else {
      die('faltan parameetros');
      }

    ?>   

pd: informate de SQL INJECTION

[pithon]

jamesjara gracias era eso que no existen esos valores a cargar la pagina lo solucione como me dijistes, saludos.