[Resolvido] @header ("location:

[Carlos Louvato 0]

Olá Pessoal. Seguindo as dicas , abri um novo post para tirar uma dúvida de @header ("location: .  Meu formulário inseri normalmente no banco , mais não mostrar nem uma mensagem de erro. Separei o PHP do Html.  Ele retorna a mesma na página porem  SEM O ERRO. Segui meu codigo:

 

$usuario_cad = mysqli_query($conecta, "SELECT usuario FROM usuario WHERE usuario = '$usuario'")
              or die(mysql_error());
if(@mysqli_num_rows($usuario_cad) >= '1'){
	$erro = "Usuário já cadastrado no sistema, escolha outro.";
}else{

$cadastra_users = mysqli_query($conecta, "INSERT INTO usuario(nome, email, cidade, usuario, senha, data)
                          VALUES('$nome', '$email', '$cidade', '$usuario', '$senha', '$data')")
			 or die(mysql_error());
$resposta = ("$cadastra_users");
if($resposta){
	$erro = "Usuário cadastrado com sucesso!";
}else{
	$erro = "Erro ao cadstrar usuário!";
}
}
@header ("location:cadastra.php?&sim=$erro");

Se puderem me auxilar agradeço de já. Carlos Louvato

[Jhess 46]

Tá... mas qual é o erro?

[leo.sena777 1]

Opa mano,

Blz ?

 

Você está enviando a mensagem de erro por GET. Você tem domínio sobre isto ?

Sugiro que faça as seguintes modificações:

 

$resposta = ("$cadastra_users");

Troque para, pois isto não é uma string e sim uma variável 

$resposta = ($cadastra_users);

Troque isto

if($resposta){
	$erro = "Usuário cadastrado com sucesso!";
}else{
	$erro = "Erro ao cadstrar usuário!";
}
}
@header ("location:cadastra.php?&sim=$erro");

Por isto

if($resposta){
	$mensagem = "Usuário cadastrado com sucesso!"; // aqui deu ok e tem uma mensagem de ok
}else{
	$mensagem = "Erro ao cadstrar usuário!"; // aqui deu erro e tem uma mensagem de erro
}
}
@header ("location:cadastra.php?mensagem=$mensagem"); //você não estava passando as query strings corretamente

Pronto. Eu n testei mas deve estar tudo ok. Agora você precisa exibir a mensagem. Coloque o seguinte código em algum lugar que queira que apareça e estilize

 

 if(isset( $_GET['mensagem'] )){ echo $_GET['mensagem']; }

eu n testei mas deve estar ok. Se der erro vai ser alguma coisa simples pra resolver.Deve estar funcionando, mas essa não é uma opção segura, você precisa tratar as query strings para ter segurança no código. Leia sobre como utilizar melhor os $_GET.

 

Espero ter ajudado, qualquer coisa me fala.

 

 

 

[Carlos Louvato 0]

Obrigado pela ajuda. Fiz as devidas mudanças como você me indicou..  Já oonsegui ficar mais atento ao código,

 

Nesta base de estudo observei que utilizava : if(isset($_POST (retirei do código) e agora passamos a utilizar: if(isset( $_ GET .

Já estou estudando mais sobre esta segurança .

 

Aparece um erro: Parse error: syntax error, unexpected '@' in /home/copam957/public_html/copa1/cadastra.php on line 36

}	
@header ("location:cadastra.php?mensagem=$mensagem"); //você não estava passando as query strings corretamente

 

 

Tentei algumas mudanças mais não consegui funcionar. 

 

.Carlos Louvato

[Carlos Louvato 0]

10 horas atrás, Jhess disse:

Tá... mas qual é o erro?

Olá. Jhess... o erro de não aparecer as mensagens na parte superior do formulario.

Grato Carlos.

[leo.sena777 1]

Olá amigo,

O problema persiste ? 

Eu só conseguiria ver o erro se você postasse o php da página cadastro.php,

entretanto parece que é apenas um erro de sintaxe, ou seja, alguma coisa que ficou aberta ou não está sendo tratada corretamente, da uma revisada no código que deve ser simples de corrigir. 

 

Outra sugestão que te dou é de n passar as mensagens por get como está fazendo, sugiro que altere da seguinte forma:

 

desta forma você não passa as mensagens pela url e consegue tratar separadamente as mensagens dos erros verificando se essas variáveis foram criadas, podendo ter várias mensagens 1,2,3

if($resposta){
        header("location:cadastra.php?msg=1");
}else{
        header("location:cadastra.php?erro=1");
}

 

 

Na página cadastro você trata o aviso

 

if(isset($_GET['msg'])){
   
  if($_GET['msg'] = 1){
    echo "mensagem de ok";
  }
 
}

if(isset($_GET['erro'])){
  
  if($_GET['erro'] = 1){
    echo "mensagem de erro";
  }

}

 

[Carlos Louvato 0]

Olá Amigo

Realmente estou um pouco perdido para fazer funcionar.

 

Depois das mudanças foi acontecendo uma série de erros. Cada hora que corrige ele aparece um novo. rsss

Não entendi onde é o melhor local para inserir  a linha:

if(isset( $_GET['mensagem'] )){ echo $_GET['mensagem']; }

No meu estudo ele utilizou :

if(isset($_POST['cad_users']) && $_POST['cad_users'] == 'cad'){

Com ECHO no final do formulário:

<?php echo @$_GET["mensagem"];

 

No momento meu código está assim:

<?php
$hostname_conecta = "localhost";
$database_conecta = "meubanco";
$username_conecta = "meuusuario";
$password_conecta = "123456";
$conecta = mysqli_connect($hostname_conecta, $username_conecta, $password_conecta) or trigger_error(mysql_error(),E_USER_ERROR);
$database = mysqli_select_db($conecta, $database_conecta);
mysqli_set_charset($conecta, "utf8");

if(isset($_POST['cad_users']) && $_POST['cad_users'] == 'cad'){

$nome    = $_POST["nome"];
$email   = $_POST["email"];
$cidade  = $_POST["cidade"];
$usuario = $_POST["usuario"];
$senha   = $_POST["senha"];
$data    = date('d/m/Y');

$usuario_cad = mysqli_query($conecta, "SELECT usuario FROM usuario WHERE usuario = '$usuario'")
              or die(mysql_error());
if(@mysqli_num_rows($usuario_cad) >= '1'){
	$mensagem = "Atletas já cadastrado no sistema, escolha outro.";
}else{

$cadastra_users = mysqli_query($conecta, "INSERT INTO usuario(nome, email, cidade, usuario, senha, data)
                          VALUES('$nome', '$email', '$cidade', '$usuario', '$senha', '$data')")
			 or die(mysql_error());
$resposta = ($cadastra_users);
if($resposta){
	$mensagem = "Usuário cadastrado com sucesso!"; // aqui deu ok e tem uma mensagem de ok
}else{
	$mensagem = "Erro ao cadstrar usuário!"; // aqui deu erro e tem uma mensagem de erro
}
@header ("location:cadastra.php?mensagem=$mensagem"); //você não estava passando as query strings corretamente
}
?>

<link href="estilos.css" rel="stylesheet" type="text/css">
<div align="center"><iframe src="topointerno.html" width="800" height="400" align="middle" scrolling="no" frameborder="0"></iframe>
      </div>
<script type="text/javascript" src="jquery/jquery.js"></script>
<script type="text/javascript" src="jquery/jquery.validate.js"></script>
<script type="text/javascript" src="jquery/funcao.js"></script>

<div id="cadastro">

  <form name="cadastra" id="cadastra" method="post" action="">
    <fieldset>
     <legend>Cadastrar usuário</legend>
     
     <label>
      <span>Nome</span>
      <input type="text" name="nome">
     </label>
     
     <label>
      <span>E-mail</span>
      <input type="text" name="email">
     </label>
     
     <label>
      <span>Cidade</span>
      <input type="text" name="cidade">
     </label>
     
     <label>
      <span>Usuário</span>
      <input type="text" name="usuario">
     </label>
     
     <label>
      <span>Senha</span>
      <input type="password" name="senha">
     </label>
     <input type="hidden" name="cad_users" value="cad" />
     <input type="submit" name="cadastrar" value="Cadastrar" class="btn">
    </fieldset>
    <div class="voltar"><a href="index.php">Voltar</a></div>
    <p align="center">
	
	<?php echo @$_GET["mensagem"];
	?></p>
  </form>
</div>

Com erro de Sintaxe:

Parse error: syntax error, unexpected end of file in /home/copam957/public_html/copa1/cadastra.php on line 85

 

Agradeço muito pela ajuda.

Carlos Louvato

 

[Gabriel Heming 766]

Está faltando fechar uma chaves, veja:

if(isset($_POST['cad_users']) && $_POST['cad_users'] == 'cad') { // <-- abriu A

    $nome    = $_POST["nome"];
    $email   = $_POST["email"];
    $cidade  = $_POST["cidade"];
    $usuario = $_POST["usuario"];
    $senha   = $_POST["senha"];
    $data    = date('d/m/Y');

    $usuario_cad = mysqli_query($conecta, "SELECT usuario FROM usuario WHERE usuario = '$usuario'") or die(mysql_error());
    if(@mysqli_num_rows($usuario_cad) >= '1') { // <-- abriu A
        $mensagem = "Atletas já cadastrado no sistema, escolha outro.";
    } // <-- fechou B
    else 
    { // <-- abriu C

        $cadastra_users = mysqli_query($conecta, "INSERT INTO usuario(nome, email, cidade, usuario, senha, data)
                              VALUES('$nome', '$email', '$cidade', '$usuario', '$senha', '$data')")
                 or die(mysql_error());
        $resposta = ($cadastra_users);
        
        if($resposta) { // <-- abriu D
            $mensagem = "Usuário cadastrado com sucesso!"; // aqui deu ok e tem uma mensagem de ok
        } // <-- fechou D
        else 
        { // <-- abriu E
            $mensagem = "Erro ao cadstrar usuário!"; // aqui deu erro e tem uma mensagem de erro
        } // <-- fechou E
        @header ("location:cadastra.php?mensagem=$mensagem"); //você não estava passando as query strings corretamente
    } // <-- fechou C

Na lógica, faltou fechar A.

[Carlos Louvato 0]

Tudo certo e funcionando! Obrigado pela ajuda.

Carlos Louvato