PHP storing session data, storing userid in session

Question

Sorry but I'm new to this. The following code only displays 'Please login'. Code does work but it's not doing what I want it to, I'm trying to store userid in a session. Here is my code

<?php

    if(!isset($_SESSION)) session_start();


    if(isset($_POST['userid'])){
         $userid = $_POST['userid']; 
         $_SESSION['userid']=$userid;
         echo "Welcome $_SESSION[userid]";
    }

    if (!isset($_SESSION['userid'])){
        echo "Please login";
        exit;
    }
    ?>

Typo echo "Welcome $_SESSION[userid]"; must be echo "Welcome $_SESSION['userid']"; it won't solve your problem — Narendrasingh Sisodia
– Narendrasingh Sisodia, Commented May 6, 2015 at 8:51
Hei @r33drum why you changed the correct answer to another answer after some time and down voted my answer since it is working? — Sanjay Kumar N S
– Sanjay Kumar N S, Commented May 6, 2015 at 9:11
@Sanjay Kumar N S I cant vote, I don't have 15 rep, I used your code and it worked, I appreciate it, I gave you the correct answer. — r33drum
– r33drum, Commented May 6, 2015 at 9:13

Sanjay Kumar N S · Accepted Answer · 2015-05-06 08:57:36Z

2

session_start() should be at the beginning without any condition. And also one extra closing bracket ) in the second if condition statement. Try this:

 <?php
   session_start();


        if(isset($_POST['userid'])){
            $_SESSION['userid']= $_POST['userid'];
            echo "Welcome {$_SESSION['userid']}";
        }

        if (!$_SESSION['userid']){
            echo "Please login";
            exit;
        }

answered May 6, 2015 at 8:57

Sanjay Kumar N S

4,7595 gold badges25 silver badges41 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

xNeyte · Accepted Answer · 2015-05-06 08:59:49Z

0

session_start(); shouldn't be in your if condition, it has to be at the beginning of the script and always launched

<?php

    session_start();


    if(isset($_POST['userid'])){
        $_SESSION['userid']= $_POST['userid'];
        echo "Welcome ".$_SESSION['userid'];
    }

    if (!$_SESSION['userid']){
        echo "Please login";
        exit;
    }

edited May 6, 2015 at 8:59

answered May 6, 2015 at 8:54

xNeyte

6224 silver badges18 bronze badges

Comments

Antonisin Maxim · Accepted Answer · 2015-05-06 09:13:47Z

0

session_start() need to be before you start to work with session.
I suggest you to replace isset() by empty(), because empty() checked for:
- "" (an empty string)
- 0 (0 as an integer)
- 0.0 (0 as a float)
- "0" (0 as a string)
- NULL
- FALSE
- array() (an empty array)
- $var; (a variable declared, but without a value)

answered May 6, 2015 at 9:13

Antonisin Maxim

365 bronze badges

Collectives™ on Stack Overflow

PHP storing session data, storing userid in session

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related