I want to include 2 files based on the following conditional statement
<?php
if (isset($_SESSION['name']) && $_SESSION['name'] == true) {
include 'file_a.php';
}else{
include 'file_b.php';
}
?>
Is this the correct way?
-
1this should help stackoverflowuser3616528– user36165282014-05-11 12:25:18 +00:00Commented May 11, 2014 at 12:25
-
If you are getting what you want, then there is nothing wrong with this method, I think.Abhishek Goyal– Abhishek Goyal2014-05-11 12:25:22 +00:00Commented May 11, 2014 at 12:25
-
You can also use ternary operator to change in single line statmentpoush– poush2014-05-11 12:25:30 +00:00Commented May 11, 2014 at 12:25
-
Using ternary operation in this case won't be rather good choice. It would make code illegibleMarcin Nabiałek– Marcin Nabiałek2014-05-11 12:37:59 +00:00Commented May 11, 2014 at 12:37
Add a comment
|
2 Answers
You can try next code:
if(session_id() == '') {
session_start();
include !empty($_SESSION['name']) ? 'file_a.php' : 'file_b.php';
}
1 Comment
Klanto Aguntuk
I don't know why these conditional satements aren't following the logic. I used mysql num rows to compare and call the include files seperately. Howeve I shall get it working. Thannks,
It depends what you mean by 'correct':
If you mean 'is it valid' then yes, your code will work and there are no syntax errors.
If you mean 'can it be prettier', then perhaps it could, but it's personal preference whether you'd want to use ternary operators or not:
$toShow = isset($_SESSION['name']) && $_SESSION['name'] ? "file_a.php" : "file_b.php";
include $toShow;
Again, whether or not this is better or worse than your previous code, is down to your personal opinion.
TL;DR: Yes, your code is correct.
