Displaying logged in users username

[Fehm]

Hi

Developing my PHP knowledge im making a radio panel, like i've said in another post, and ive looked for tutorials but cant find any on how to display a users username..

What i though it would be (And im a terrible coder so it wont work!, im coding to gain experience)

<?php
 
session_start();
 
if(!isset($_SESSION['logged_in']) 
}
 
$username = $_SESSION['username']; 
?>
 
Hello, <?= $username ?>! Welcome to my website.

Any help would be greatly appreciated.. Thank you
+rep if you can help...

[Excellent2]

echo "$_SESSION['username']";

[Dentafrice]

echo "$_SESSION['username']";

You don't need quotes around it..

Hello, <?php echo $_SESSION['username']; ?>! Welcome to our website.

[Fehm]

You don't need quotes around it..

Hello, <?php echo $_SESSION['username']; ?>! Welcome to our website.

Thank you lots !

Now, if i where to put that in a seperate file and then added <? include(loggedin.php)?> it would work wouldnt it??

Sorry im still learning ^^

[Dentafrice]

Yes it would. Please quit using short-tags though (<?) and use (<?php).

<?php include "loggedin.php"; ?>

[Excellent2]

You don't need quotes around it..

I prefer to

[Dentafrice]

Why? It's pointless and stupid.

[Excellent2]

Why? It's pointless and stupid.

Because I prefer to.

[Dentafrice]

Because I prefer to.

Well, your preference is pointless.

You're basically telling PHP that you're going to output a string "", but you're actually outputting a variable inside that string, so you're making your program inefficient because it is having to check that string "" for variables ($_SESSION), so you're just putting more work on your program.

Compared to

Simple telling PHP you are going to output a variable, you save the time and hassle of having it look through the string for it.

[Iszak]

I agree with Dentafrice about the fact that it is rather pointless but it should be noted that the added extra execution time is so negligible it shouldn't matter. But still it's a bad habit and should be avoided.