O que tem de errado?

[Kakaroto1309 0]

Não consigo achar o erro, ele me retorna o erro

Parse error: syntax error, unexpected T_ELSE in /home/meusite/public_html/inserir_clientes.php on line 37

 

if ($nome!="" and $rg!="" and $cidade!="" and $cidade!="Selecione")
{
$inserir = mysql_query("INSERT INTO clientes (nome,rg,cidade) VALUES ('$nome','$rg','$cidade')");
$procura=mysql_query("SELECT nome FROM clientes WHERE nome='$nome'");
if ($procura == false)
	{
	if ($inserir)
		{
		echo "Cadastro realizado com Sucesso!<br>";
		echo "<a href='clientes.php'>Voltar</a>";
		}
		else
		{
		echo "Não cadastrado. Login Existente.";
		echo "<a href='javascript:history.back(1);'>Voltar</a>";
		}
	else  ------------------ LINHA 37 É ESSA ----------------------
	{
	echo "Nome já cadastrado.";
	}
}
else
{
echo "Não pode haver campos em branco ou deselecionados";
}

[Vinicius Borsato 0]

Tem dois "else" ai, troca por "if else"

[Quelipe 15]

if ($nome!="" and $rg!="" and $cidade!="" and $cidade!="Selecione")
{
$inserir = mysql_query("INSERT INTO clientes (nome,rg,cidade) VALUES ('$nome','$rg','$cidade')");
$procura=mysql_query("SELECT nome FROM clientes WHERE nome='$nome'");
if ($procura == false)
	{
	if ($inserir)
		{
		echo "Cadastro realizado com Sucesso!<br>";
		echo "<a href='clientes.php'>Voltar</a>";
		}
		else
		{
		echo "Não cadastrado. Login Existente.";
		echo "<a href='javascript:history.back(1);'>Voltar</a>";
		}
               } // <-- Faltou fechar o ($procura == false)
	else  ------------------ LINHA 37 É ESSA ----------------------
	{
	echo "Nome já cadastrado.";
	}
}
else
{
echo "Não pode haver campos em branco ou deselecionados";
}

 

At+

[Kakaroto1309 0]

Valeus Vinicius

 

Deu certo Quelipe, porém toda vez que eu digito um nome e do em enviar, ele da o erro NOME JA EXISTENTE e cadastra do mesmo jeito no bd, sabe como posso resolver?

[Vinicius Borsato 0]

Ops, é isso aí, a indentação me confundiu, tenho certeza que assim como você também Kakaroto1309.

Sempre indente seu código, você acha o erro na hora!

[Quelipe 15]

Faça sua query de validação do nome primeiro e a de gravação coloque dentro da condição verdadeira.

 

At+

[Vinicius Borsato 0]

Sim independente de existir ou não você está inserindo no banco e só depois que consulta.

 

Faz um select procurando o registro, se encontrar envia o erro, se não, cadastra o nome!

 

=)

[Kakaroto1309 0]

Mais se eu coloca a condição de Gravação dentro da opção verdadeira, ele vai cadastrar do mesmo jeito. Pois

$procura será TRUE, então é sinal de que achou no bd.

[Vinicius Borsato 0]

Troque

 

$procura=mysql_query("SELECT nome FROM clientes WHERE nome='$nome'");
if ($procura == false)

 

por:

 

$procura=mysql_query("SELECT nome FROM clientes WHERE nome='$nome'");
$existe = mysql_num_rows($procura);

if ($existe == 0) // Nenhum registro encontrado
{
// grava
}
else
{
// ja existe o nome
}

[Kakaroto1309 0]

agora deu ...

 

qual a diferença dos dois?

[Quelipe 15]

if(!= "") {
  consulta
  if(consulta) {
     já cadastrado
  }else {
     inseri
     if(inseri) {
        cadastrado con sucesso
     } else {
        não cadastrado
     }
  }
} else {
  não pode branco
}

 

a lógica é essa.

 

At+

[Vinicius Borsato 0]

o mysql_query() executa uma query =)

o mysql_num_rows() verifica a quantidade de linhas que sua consulta retornou

 

ou seja da forma que você estava fazendo ele retornava TRUE porque a query foi executada com sucesso! Sempre iria retornar TRUE nesse caso