1

Estou aprendendo php e estou tentando fazer um combobox, ele está puxando os dados do banco corretamente porém na hora de exibir o resultado apresenta este erro:

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 'and imovel_categoria = '2' and imovel_bairro = '2'' at line 1

PHP

<script src="lightbox/js/jquery-1.11.0.min.js"></script>

 <script type="text/javascript">
 $(document).ready(function(){
  $('#tipo').change(function(){

  $('#cb_categoria').load('combo_categoria.php?cod='+$('#tipo').val());
  });  //fim do change


    //aqui carrega o conteúdo

    $('#cb_categoria').change(function(){

  $('#conteudo').load('resultado.php?cod='+$('#cb_categoria').val());
  }); 

     } );//fim do ready



    </script>

      <?php

       $tipo = new Conexao();
       $tipo->ExecSQL("select * from tipo ");

        ?>



       <!--combo listando os tipos-->
       <label><b>Finalidade:</b></label>
       <select id="tipo">

        <?php

         while($tip = $tipo->ListarDados()){

            ?>

         <option value="<?php echo $tip['tipo_id']?>"><?php echo        $tip['tipo_nome']?></option>

<?php

}//fecho o laço

?>

 </select>
 <!--listando categoria-->
  <label><b>Tipo:</b></label>

    <select id="cb_categoria">

     </select>


      <!--conteudo pesquisado-->
      <div id="conteudo"></div>

PHP combo_categoria:

    <script src="lightbox/js/jquery-1.11.0.min.js"></script>

<script type="text/javascript">
    $(document).ready(function () {
        $('#cb_categoria').change(function () {

            $('#cb_bairro').load('combo_bairro.php?cod='+$('#cb_categoria').val());

        });  //fim do change

        //aqui carrega o conteudo
        $('#cb_bairro').change(function () {

            $('#conteudos').load('resultado.php?cod='+$('#cb_bairro').val());

      });

    });//fim do ready



</script>



<?php
require './app/Config.php';

if (!empty($_GET['cod'])):

    $cod = $_GET['cod'];

else:
    $cod = 0;
endif;

$categoria = new Conexao();
$categoria->ExecSQL("select * from categoria where categoria_tipo = '$cod' ");
?>



<!--combo listando categoria-->


    <?php
    while ($cat = $categoria->ListarDados()) {
        ?>

        <option value="<?php echo $cat['categoria_id'] ?>"><?php echo $cat['categoria_nome'] ?></option>

    <?php
}//fecho o laço
?>



<label><b>Bairro:</b></label>
<select id="cb_bairro">

</select>


<!--conteudo pesquisado-->
<div id="conteudos"></div>

PHP combo_bairro:

<?php

require './app/Config.php';


if (!empty($_GET['cod'])):
    $cod = $_GET['cod'];

else:
    $cod = 0;
endif;


$bairro = new Conexao();
$bairro->ExecSQL("select * from bairro where bairro_categoria = '$cod'");

?>



<!--combo listando Bairro-->


    <?php

    while($bai = $bairro->ListarDados()){

    ?>

    <option value="<?php echo $bai['bairro_id']?>"><?php echo $bai['bairro_nome']?></option>

    <?php

    }//fecho o laço

    ?>

PHP resultado:

<?php
require_once './app/Config.php';


$imovel = new Conexao();

if (!empty($_GET['cod'])):
    $cod = $_GET['cod'];
else:
    $cod = 0;
endif;


$sql = 'select * from imoveis, tipo, categoria, bairro';
$sql .= ' where imovel_categoria = categoria_id and imovel_bairro = bairro_id';
$sql .= ' and imovel_tipo = tipo_id order by imovel_id desc limit 6';


$sql .= " and imovel_categoria = '$cod' ";
$sql .= " and imovel_bairro = '$cod' ";


$imovel->ExecSQL($sql);


echo '<ul>';
while ($imv = $imovel->ListarDados()) {


    //pego a foto
    $i = new Conexao();
    $f = $imv['imovel_id'];
    $i->ExecSQL("select * from imoveis_fotos where foto_imovel = '$f' limit 1");
    $foto = $i->ListarDados();


    //mostrando conteudo
    echo '<li class="listagem_home">';
    echo Fotos::Exibir($foto['foto_nome'], 150, 150);
    echo '<div class="imoveis">';
    echo '<div class="listagem_nome">' . $imv['tipo_nome'] . '</div>';
    echo '<div class="listagem_cat"> ' . $imv['categoria_nome'] . '</div>';
    echo '<div class="listagem_cat">R$ ' . Sistema::GetReal($imv['imovel_valor']) . '</div>';
    echo '<div class="listagem_bai"> ' . $imv['bairro_nome'] . '</div>';
    echo '<div id="botao">';
    echo '<a href="' . Rotas::$detalhe . $imv['imovel_id'] . '" class="btn">saiba mais</a>';
    echo '</div>';
    echo '</div>';
    echo '</li>';
}
echo '</ul>';
?>
3
  • 3
    Do jeito que esta montando seu select, esta ficando assim: select * from imoveis, tipo, categoria, bairro where imovel_categoria = categoria_id and imovel_bairro = bairro_id and imovel_tipo = tipo_id order by imovel_id desc limit 6 and imovel_categoria = '$cod' and imovel_bairro = '$cod'. Percebe onde esta o erro? Commented 9/03/2017 às 17:15
  • Você não pode ter um LIMIT no meio da consulta ele sempre vai no final o mesmo vale para o ORDER BY
    – rray
    Commented 9/03/2017 às 17:16
  • 2
    A estrutura deve sempre seguir mais ou menos essa ordem: SELECT campos FROM tabela JOIN (JOIN, LEFT, RIGTH, OUTER, FULL, ETC) tabela WHERE condição GROUP BY condição ORDER BY campos LIMIT numero podendo possuir todas, mais de uma, ou nenhuma dessas estruturas Commented 9/03/2017 às 17:19

1 Resposta 1

1

No PHP resultado, os filtros por imovel_categoria e imovel_bairro na consulta SQL vêm a seguir ao " order by ", daí o erro Podes fazer assim:

if (!empty($_GET['cod'])):
    $cod = $_GET['cod'];
else:
    $cod = 0;
endif;


$sql = 'select * from imoveis, tipo, categoria, bairro';
$sql .= ' where imovel_categoria = categoria_id and imovel_bairro = bairro_id';
$sql .= ' and imovel_tipo = tipo_id';
$sql .= " and imovel_categoria = '$cod'";
$sql .= " and imovel_bairro = '$cod'";
$sql .= ' order by imovel_id desc limit 6';


$imovel->ExecSQL($sql);

Atenção, aparentemente não estás a limpar os dados enviados pelo utilizador que vêm na variável $_GET ! É mais seguro usar prepared statements, por exemplo. Mais info (em Inglês): https://stackoverflow.com/questions/60174/how-can-i-prevent-sql-injection-in-php

Você deve fazer log-in para responder a esta pergunta.