PHP表单需要两次提交按钮单击才能消失

I am using session variable to hide a log in form after user logs in. Here are first two lines from the PHP webpage

<?php
session_start();

Now,this is the logout button which appears after a user has logged in but take disappears only after two clicks

<div class="col-lg-4 col-md-4 col-sm-6 col-xs-12" <?php if(!isset($_SESSION['login'])) echo 'style="display:none;"'; else echo 'style="display:block;"';?>/>
<form method="post" action="">
<p>Hey, <?php echo ''.$_SESSION['user'].''; ?></p>
<input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875;" type="submit" name="submit" value="Logout">
</form>
<?php
if ($_POST['submit'] == 'Logout'){
session_destroy();
}?>

Here is code snippet from login form

<div class="col-lg-4 col-md-4 col-sm-6 col-xs-12" <?php if(isset($_SESSION['login'])) echo 'style="display:none;"'; else echo 'style="display:block;"';?>/>
<form method="post" action="">

After succesful login I set $_SESSION['login'] = true; $_SESSION['user'] = $memberinfo['USER'];

Why do I have to click two times to keep get the forms work as expected?

EDIT Here is the complete code with HTML and Javascript stripped out

    <?php
    session_start();
    $mysqli = new mysqli($db_hostname,$db_username,$db_password,$db_database);
    if (mysqli_connect_errno()) {
    printf("Connect failed: %s
", mysqli_connect_error());
    exit();
    }
    if($passkey=$_GET['passkey']){
    $result=$mysqli->query("SELECT * FROM unactivated WHERE code ='$passkey'");
    if($result){
    $count=mysqli_num_rows($result);
    if($count==1){
    $rows=mysqli_fetch_array($result);
    $user=$rows['USER'];
    $email=$rows['EMAIL'];
    $password=$rows['PASSWORD'];
    $password=$mysqli->real_escape_string(hash_hmac('sha256',$password, 'c#haRl891', true));
    if($mysqli->query("INSERT INTO member ( USER, EMAIL, PASSWORD) VALUES ('$user', '$email', '$password')"))
    {
    echo "Your account has been activated";
    $mysqli->query("DELETE FROM unactivated WHERE code = '$passkey'");
    }}}
    else {
    echo "Wrong Confirmation code";
    }}?>
    <div class="col-lg-4 col-md-4 col-sm-6 col-xs-12" <?php
if(isset($_SESSION['login'])) echo 'style="display:none;"'; else echo 'style="display:block;"';?>/>
    <form method="post" action="">
    <input class="form-control" type="text" name="user" placeholder="Alex Bruno (for Sign Up)" />
    <input class="form-control" type="email" name="email" placeholder="your@email.com" required />
    <input class="form-control" type="password" name="password" value="" placeholder="Strong Password" required />
    <p>Enter valid email to get a login link.</p>
    <input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875;" type="submit" name="submit" value="Register">
    <input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875; float:right;" type="submit" name="submit" value="Login">
    </form>
    <?php
    if ($_POST['submit'] == 'Register'){
    if (!empty($_REQUEST['user'])&&!empty($_REQUEST['email'])&&!empty($_REQUEST['password'])) {
    $user = $_POST['user'];
    $email = $_POST['email'];
    $password = $_POST['password'];
    $user = $mysqli->real_escape_string($user);
    $email = $mysqli->real_escape_string($email);
    $password = $mysqli->real_escape_string($password);
    $query = $mysqli->query("SELECT * from member WHERE EMAIL ='$email'");
    $rows = mysqli_num_rows($query);
    if ($rows == 0)
    {
    $code=md5(uniqid(rand()));
    $mysqli->query("INSERT INTO unactivated (CODE, USER, EMAIL, PASSWORD) VALUES ('$code', '$user', '$email', '$password')");
    $to=$email;
    $subject="Please verify your Sign Up";
    $header="from: 8mags <contact@8mags.com>";
    $message="You or someone else used this email address to sign up for 8mags. If you did not do it ignore the message.
";
    $message.="To verify your account please click on the link below
";
    $url="http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'].'?passkey='.$code.'';
    $message.=$url;
    $sentmail = mail($to,$subject,$message,$header);
    if($sentmail){
    echo 'Your Confirmation link Has Been Sent To Your Email Address.';
    }}
    echo 'Email already registered!';
    }}
    if ($_POST['submit'] == 'Login'){
    $password=$mysqli->real_escape_string(hash_hmac('sha256',$password, 'c#haRl891', true));
    $query = $mysqli->query("SELECT * from member WHERE EMAIL = '$email' AND PASSWORD = '$password'");
    $rows = mysqli_num_rows($query);
    if($rows==1)
    {
    $memberinfo = mysqli_fetch_array($query);
    $_SESSION['login'] = true;
    $_SESSION['user'] = $memberinfo['USER'];
    }
    else{
    echo 'Entered Password and Email Combination is wrong!';
    }}?>
    <div class="col-lg-4 col-md-4 col-sm-6 col-xs-12" <?php if(!isset($_SESSION['login'])) echo 'style="display:none;"'; else echo 'style="display:block;"';?>/>
    <form method="post" action="">
    <p>Hey, <?php echo ''.$_SESSION['user'].''; ?></p>
    <input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875;" type="submit" name="submit" value="Logout">
    </form>
    <?php
    if ($_POST['submit'] == 'Logout'){
    }?>

写回答
好问题 0 提建议
追加酬金
关注问题
分享
邀请回答
编辑收藏删除结题
收藏举报

1条回答默认最新

duanhunlou7051 2015-01-17 08:20

关注

Destroy session should come at the very top of the page [EDIT] followed by a redirect with a header() to reload the page (as suggested by, and who reminded me of..) by DarkBee.

Also, here is what I would recommend to you for your page. Everything is notated:

functions.php (new page containing your main functions)

<?php
    // You should make these functions and on
    // a separate page that you include
    function Login($checkpass, $checkemail, $mysqli)
        {
                $email      =   $mysqli->real_escape_string($checkemail);
                $password   =   $mysqli->real_escape_string(hash_hmac('sha256',$checkpass, 'c#haRl891', true));
                $query      =   $mysqli->query("SELECT * from member WHERE EMAIL = '$email' AND PASSWORD = '$password'");
                $rows       =   mysqli_num_rows($query);

                if($rows == 1) {
                    $memberinfo = mysqli_fetch_array($query);
                    $_SESSION['login']  =   true;
                    $_SESSION['user']   =   $memberinfo['USER'];
                    return true;
                }
        // Return false if failed
        return false;
    }

    function Register($mysqli)
        {
            if (!empty($_REQUEST['user'])&&!empty($_REQUEST['email'])&&!empty($_REQUEST['password'])) {
                $user       =   $_POST['user'];
                $email      =   $_POST['email'];
                $password   =   $_POST['password'];
                $user       =   $mysqli->real_escape_string($user);
                $email      =   $mysqli->real_escape_string($email);
                $password   =   $mysqli->real_escape_string($password);
                $query      =   $mysqli->query("SELECT * from member WHERE EMAIL ='$email'");
                $rows       =   mysqli_num_rows($query);

                if ($rows == 0) {
                    $code       =   md5(uniqid(rand()));
                    $mysqli->query("INSERT INTO unactivated (CODE, USER, EMAIL, PASSWORD) VALUES ('$code', '$user', '$email', '$password')");
                    $to         =   $email;
                    $subject    =   "Please verify your Sign Up";
                    $header     =   "from: 8mags <contact@8mags.com>";
                    $message    =   "You or someone else used this email address to sign up for 8mags. If you did not do it ignore the message.
";
                    $message    .=  "To verify your account please click on the link below
";
                    $url        =   "http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI'].'?passkey='.$code.'';
                    $message    .=  $url;

                    // Return success
                    if(mail($to,$subject,$message,$header)){
                        return true;
                    }
                }
            }

            // Return fail by default
            return false;
        }

    function FetchPassKey($mysqli)
        {
            // This part is a bit scary, you are not binding or sanitizing this.
            // It's open to sql injection attacks
            $passkey    =   $_GET['passkey'];
            $result     =   $mysqli->query("SELECT * FROM unactivated WHERE code ='$passkey'");
            if($result) {
                    $count  =   mysqli_num_rows($result);
                    if($count==1){
                        $rows       =   mysqli_fetch_array($result);
                        $user       =   $rows['USER'];
                        $email      =   $rows['EMAIL'];
                        $password   =   $rows['PASSWORD'];
                        $password   =   $mysqli->real_escape_string(hash_hmac('sha256',$password, 'c#haRl891', true));

                        if($mysqli->query("INSERT INTO member ( USER, EMAIL, PASSWORD) VALUES ('$user', '$email', '$password')")) {
                            $mysqli->query("DELETE FROM unactivated WHERE code = '$passkey'");
                            return true;
                        }
                    }
                }

            return false;
        }
    ?>

config.php

<?php
    function DBConnect($db_hostname = 'host',$db_username = 'username',$db_password = 'password',$db_database = 'dbname')
        {
            // Database 
            $mysqli = new mysqli($db_hostname,$db_username,$db_password,$db_database);
            if(mysqli_connect_errno()) {
                printf("Connect failed: %s
", mysqli_connect_error());
                exit();
            }

            return $mysqli;
        }

    // Start session
    session_start();
    $mysqli =   DBConnect(); ?>

index.php (Whatever name this page is, I picked index.php)

    // Make a db function
    include_once('config.php');
    // Include the functions that run this page actions
    include_once('functions.php');
    // If logging out, destroy session
    if(isset($_POST['submit'])) {
        // Logout script
        if($_POST['submit'] == 'Logout') {
                session_destroy();
                header("Location: ".$_SERVER['PHP_SELF']);
                exit;
            }
        // Run your login script
        elseif($_POST['submit'] == 'Login')
            $valid  =   Login($_POST['password'],$_POST['email'],$mysqli);
        elseif($_POST['submit'] = 'Register')
            $registered =   Register($mysqli);
    }      

    // You are doing an equals here, should be double equal ==
    // or it will always be true
    if(isset($_GET['passkey']) && !empty($_GET['passkey'])){
        echo (FetchPassKey($mysqli) == true)? "Your account has been activated":"Wrong Confirmation code";
    }

    // Just don't print it to page
    if(!isset($_SESSION['login'])) { ?>
    <div style=" display: inline-block;" class="col-lg-4 col-md-4 col-sm-6 col-xs-12">
        <form method="post" action="">
            <input class="form-control" type="text" name="user" placeholder="Alex Bruno (for Sign Up)" />
            <input class="form-control" type="email" name="email" placeholder="your@email.com" required />
            <input class="form-control" type="password" name="password" value="" placeholder="Strong Password" required />
            <p>Enter valid email to get a login link.</p>
            <input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875;" type="submit" name="submit" value="Register">
            <input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875; float:right;" type="submit" name="submit" value="Login">
        </form>
    </div>
    <?php }

    // Just echo the results of your register validate function
    if (isset($_POST['submit']) && $_POST['submit'] == 'Register')
        echo ($registered == true)?  'Your Confirmation link Has Been Sent To Your Email Address.':'Email already registered!';

    // Just echo the results of your validation here but assign and check at top.
    if(isset($valid) && $valid == false)
        echo 'Entered Password and Email Combination is wrong!';

    // Just display if logged in
    if(isset($_SESSION['login'])) { ?>
    <div class="col-lg-4 col-md-4 col-sm-6 col-xs-12" style="display:block;">
        <form method="post" action="">
            <p>Hey, <?php echo ''.$_SESSION['user'].''; ?></p>
            <input style="border: none; padding: 8px 15px; background: #BAFF00; box-shadow: 0px 0px 3px #757875;" type="submit" name="submit" value="Logout">
        </form>
    </div>
    <?php } ?>

本回答被题主选为最佳回答 , 对您是否有帮助呢?

报告相同问题？

关注问题

PHP表单需要两次提交按钮单击才能消失 database mysql php
2015-01-17 08:16

回答 1 已采纳 Destroy session should come at the very top of the page [EDIT] followed by a redirect with a heade
在正确的信息通过之前需要提交两次PHP表单 javascript mysql php
2018-07-12 18:35

回答 1 已采纳 Sal, you mustn't rush into this. Let's first understand what your setcookie function actually does
PHP表单数据检查提交按钮名称 php
2018-05-25 11:53

回答 3 已采纳 If you want, you can alternatively use hidden inputs. You can access hidden inputs from php just l
PHP面试题（含答案），持续更新
2022-08-25 14:50

ypa-达瓦里希的博客 PHP面试题（含答案），持续更新（会有重复）
jQuery Ajax在一个表单中形成两个提交按钮 ajax javascript jquery php
2017-07-15 01:43

回答 1 已采纳 After trial and errors I found the script work if you change this line : data: new FormData($("#a
表单需要在PHP中处理两个提交按钮 php
2014-01-27 05:38

回答 2 已采纳 Create another script register_or_login.php, let that script handle the form submission and make i
单击按钮提交表单并禁用按钮 html javascript php symfony
2016-07-13 11:03

回答 3 已采纳 I have create one demo. It is working fine. HTML <form> <input type="submit" value=
分步表单如何实现 html_HTML表单入门的分步指南
2020-07-30 08:43

cumi7754的博客分步表单如何实现 htmlby Abhishek Jakhar 通过阿比舍克·贾卡(Abhishek Jakhar) HTML表单入门的分步指南 (A step-by-step guide to getting started with HTML forms) 总览 (Overview) ...
PHP表单 - 单击单选按钮提交表单 javascript php
2014-08-13 14:47

回答 5 已采纳 Simply add an onchange attribute to your radios: $qA='<input type="radio" onchange="this.form.
JavaScript表单验证问题 - 必须单击提交两次 ajax javascript jquery php
2014-08-04 12:55

回答 1 已采纳 Instead of calling Ajax validations on focustout event, you can call it on click of your button an
使用php的HTML表单不能使用单个提交按钮。 html5 mysql php
2018-07-25 10:54

回答 1 已采纳 Your submit button is outside of form that's why its not working. Refer below code to resolve your
mysql 中计数器_PHP和MySQL中的简单投票计数
2020-07-20 01:47

cunchi8090的博客我们需要做的第一件事是创建两个表。一张桌子将包含颜色选择。每种颜色将有一行。另一个表将包含投票。每一票将有一行。您可能想知道为什么我们不只是在颜色表中添加一个计数器并直接在颜色行中累积票数之和。...
PHP表单需要提交两次以设置会话 php
2014-03-25 04:14

回答 4 已采纳 Since you have all your code for login, processing and showing the user is logged in at the same p
（附源码）php柘城农产品销售系统毕业设计020832
2022-09-06 16:30

WeiXin_DZbishe的博客用户想进入系统，首先进入系统登录界面，通过正确的用户名、密码，选择登录类型登录，... 用户选择要删除的信息并单击Delete按钮。系统提示是否删除信息。如果用户想要删除信息，系统将删除信息。系统数据库删除信息
php初中历史专题教学网站毕业设计-附源码100623
2023-03-22 09:00

weixin_61498557的博客初中历史专题教学网站主要功能模块包括课程资料管理、历史科普、科普朝代、学生管理、老师管理、学生作业、作业提交等，采取面对对象的开发模式进行软件的开发和硬体的架设，能很好的满足实际使用的需求，完善了对应...
没有解决我的问题, 去提问

悬赏问题

¥15 安卓adb backup备份应用数据失败
¥15 eclipse运行项目时遇到的问题
¥15 关于#c##的问题：最近需要用CAT工具Trados进行一些开发
¥15 南大pa1 小游戏没有界面，并且报了如下错误，尝试过换显卡驱动，但是好像不行
¥15 没有证书，nginx怎么反向代理到只能接受https的公网网站
¥50 成都蓉城足球俱乐部小程序抢票
¥15 yolov7训练自己的数据集
¥15 esp8266与51单片机连接问题(标签-单片机|关键词-串口)（相关搜索：51单片机|单片机|测试代码）
¥15 电力市场出清matlab yalmip kkt 双层优化问题
¥30 ros小车路径规划实现不了，如何解决？(操作系统-ubuntu)

码龄粉丝数原力等级 --

PHP表单需要两次提交按钮单击才能消失

1条回答默认最新

码龄粉丝数原力等级 --

悬赏问题

PHP表单需要两次提交按钮单击才能消失

1条回答 默认 最新

悬赏问题

1条回答默认最新