Skip to main content
adicionou 162 caracteres ao conteúdo
Fonte Link
Largato
  • 92,9mil
  • 18
  • 190
  • 261

Duas coisas. Primeiro que você está fazendo uma série de comparações separadas. Segundo que o ideal seria verificar se está vazio, e usar o else para ir eliminando condições:

if ( empty( $login ) && empty( $senha ) ){
    echo"Por favor preencha o nome do usuário e senha" ;
} elseif ( empty( $login ) ) {
    echo "Por favor preencha o campo do usuário";
} elseif ( empty( $senha ) ) {
    echo "Por favor preencha o campo senha";
} elseif ( $_SESSION['login'] == $login ) {
    unset ($_SESSION['senha']);
    echo "Senha inválida";      
} else {
    unset ($_SESSION['login']);
    echo "Usúario inválido";
}

Pra complementar, faltou o die() (ou exit() ) no redirect:

header('location:editar-excluir.php');
die();

Vale dizer que trabalhar com senha em aberto como você está fazendo é inadmissível em sistemas para uso real, mas se for só um exercício, "até passa".

Além disso, como você não está sanitizando os valores do POST, grande chance de um curioso com um mínimo de conhecimento apagar todo o seu DB remotamente.

Outra coisa estranha é o uso que você está fazendo do unset(). Como não entendi direito qual é o objetivo real destas linhas, não tenho como dar detalhes se deveriam ser como estão.

Pode ter outros problemas, pois só olhando o trecho do código postado, não dá pra ter certeza de como realmente é pra funcionar.

Duas coisas. Primeiro que você está fazendo uma série de comparações separadas. Segundo que o ideal seria verificar se está vazio, e usar o else para ir eliminando condições:

if ( empty( $login ) && empty( $senha ) ){
    echo"Por favor preencha o nome do usuário e senha" ;
} elseif ( empty( $login ) ) {
    echo "Por favor preencha o campo do usuário";
} elseif ( empty( $senha ) ) {
    echo "Por favor preencha o campo senha";
} elseif ( $_SESSION['login'] == $login ) {
    unset ($_SESSION['senha']);
    echo "Senha inválida";      
} else {
    unset ($_SESSION['login']);
    echo "Usúario inválido";
}

Pra complementar, faltou o die() (ou exit() ) no redirect:

header('location:editar-excluir.php');
die();

Vale dizer que trabalhar com senha em aberto como você está fazendo é inadmissível em sistemas para uso real, mas se for só um exercício, "até passa".

Duas coisas. Primeiro que você está fazendo uma série de comparações separadas. Segundo que o ideal seria verificar se está vazio, e usar o else para ir eliminando condições:

if ( empty( $login ) && empty( $senha ) ){
    echo"Por favor preencha o nome do usuário e senha" ;
} elseif ( empty( $login ) ) {
    echo "Por favor preencha o campo do usuário";
} elseif ( empty( $senha ) ) {
    echo "Por favor preencha o campo senha";
} elseif ( $_SESSION['login'] == $login ) {
    unset ($_SESSION['senha']);
    echo "Senha inválida";      
} else {
    unset ($_SESSION['login']);
    echo "Usúario inválido";
}

Pra complementar, faltou o die() (ou exit() ) no redirect:

header('location:editar-excluir.php');
die();

Vale dizer que trabalhar com senha em aberto como você está fazendo é inadmissível em sistemas para uso real, mas se for só um exercício, "até passa".

Além disso, como você não está sanitizando os valores do POST, grande chance de um curioso com um mínimo de conhecimento apagar todo o seu DB remotamente.

Outra coisa estranha é o uso que você está fazendo do unset(). Como não entendi direito qual é o objetivo real destas linhas, não tenho como dar detalhes se deveriam ser como estão.

Pode ter outros problemas, pois só olhando o trecho do código postado, não dá pra ter certeza de como realmente é pra funcionar.

Fonte Link
Largato
  • 92,9mil
  • 18
  • 190
  • 261

Duas coisas. Primeiro que você está fazendo uma série de comparações separadas. Segundo que o ideal seria verificar se está vazio, e usar o else para ir eliminando condições:

if ( empty( $login ) && empty( $senha ) ){
    echo"Por favor preencha o nome do usuário e senha" ;
} elseif ( empty( $login ) ) {
    echo "Por favor preencha o campo do usuário";
} elseif ( empty( $senha ) ) {
    echo "Por favor preencha o campo senha";
} elseif ( $_SESSION['login'] == $login ) {
    unset ($_SESSION['senha']);
    echo "Senha inválida";      
} else {
    unset ($_SESSION['login']);
    echo "Usúario inválido";
}

Pra complementar, faltou o die() (ou exit() ) no redirect:

header('location:editar-excluir.php');
die();

Vale dizer que trabalhar com senha em aberto como você está fazendo é inadmissível em sistemas para uso real, mas se for só um exercício, "até passa".