How to do this?

[Luke]

Right, I have this code to define a title on a div;



<?php

if(!$_GET['page']) {
echo "Homepage";
}
elseif($_GET['page']) {
if($_GET['subpage']) {
$page_name = str_replace("_", " ", $_GET['page']);
$subpage_name = str_replace("_", " ",$_GET['subpage']);
echo strtoupper($page_name." - ".$subpage_name);
}
elseif(!$_GET['subpage']) {
if($_GET['page'] == "ADMINISTRATIONCENTRE")
{
echo strtoupper("ADMINISTRATION CENTRE");
}
else
{
echo strtoupper($_GET['page']);
}
}
}
?>


Now, what I want another code to do is show an image defined by the page&subpage so it would be something like

<img src="images/$page_name;$subpage_name;.png"> if you get me..
Can anyone help?

[Dentafrice]

Try this...



$page = $_GET["page"];
$subpage = $_GET["subpage"];

if (!$page) {
$page_name = "Homepage";
echo $page_name;
echo "images/homepage.png";
} else {
if ($subpage) {
$page_name = str_replace("_", " ", $page);
$subpage_name = str_replace("_", " ", $subpage);
echo strtoupper($page_name . " - " . $subpage_name);

echo "images/{$page_name}-{$subpage_name}.png";

} else {
if ($page == "ADMINISTRATIONCENTRE") {
echo strtoupper("ADMINISTRATION CENTRE");
echo "images/{$page}.png";
} else {
echo strtoupper($page);
echo "images/{$page}.png";
}
}
}

[Luke]

Think you missed the <img> tags out :P That's the bit i'm struggling with xD sorry ;p

EDIT: Got it :')



<?php $page = $_GET["page"];
$subpage = $_GET["subpage"];

if (!$page) {
$page_name = "Homepage";
echo '<img src="images/homepage.png">';
} else {
if ($subpage) {
$page_name = str_replace("_", " ", $page);
$subpage_name = str_replace("_", " ", $subpage);
echo '<img src="images/' . $page_name . '' . $subpage_name . '.png">';
} else {
echo '<img src="images/' . $page_name . '.png">';
}
}
?>

[Dentafrice]

You don't have to keep breaking your echo '' for each variable... just use a ", and put {$page_name} and {$subpage_name}, makes it so much easier to read :P

Concatenating strings is so much easier like that. I mean it's a personal preference I guess, but I think it looks better.

[Luke]

You don't have to keep breaking your echo '' for each variable... just use a ", and put {$page_name} and {$subpage_name}, makes it so much easier to read :P

Concatenating strings is so much easier like that. I mean it's a personal preference I guess, but I think it looks better.

Fair enoughh, didn't know that :P
cheers xD +rep