If else statement for login and logout [closed]

Question

Closed. This question does not meet Stack Overflow guidelines. It is not currently accepting answers.

---

Closed 8 years ago.

• This question appears to be off-topic because it lacks sufficient information to diagnose the problem. Describe your problem in more detail or include a minimal example in the question itself.
• Questions concerning problems with code you've written must describe the specific problem — and include valid code to reproduce it — in the question itself. See SSCCE.org for guidance.

Improve this question

i am using a script at front end, if a user logs in, it should render username in place of login button else it should remain the same. but my script is not not working....

<?php 
    if(!isset($_SESSION['txt_username']) || !isset($_SESSION['txt_pwd'])){
    echo "<a href='login.php'>Log in</a>"; 
    header("Location:login.php");
    else {echo $_SESSION['txt_username']}
    }
?>

Answer 1

There are a couple of syntax errors. Properly formatting your code will help dramatically when trying to diagnose errors:

<?php 
if(!isset($_SESSION['txt_username']) || !isset($_SESSION['txt_pwd'])){
    //echo "<a href='login.php'>Log in</a>";                                          
    header("Location:login.php");
}else{
    echo $_SESSION['txt_username'];
}

Answer 2

The header function of php does the redirect immediatly when it finds that line. (it does other stuff also :) ) Read at: http://php.net/manual/en/function.header.php .

try this:

if(!isset($_SESSION['txt_username']) || !isset($_SESSION['txt_pwd'])){
    echo "<a href='login.php'>Log in</a>";                                          
else {
    echo $_SESSION['txt_username'];
}

In the ending: DO NOT KEEP THE PASSWORD IN $_SESSION!!!! . At least use md5() or some encryption on that password.