HELP REWRITE - there must be a better way

mrbell

I have a long list of links on a page, but I want to "deactivate" the current page accordingly:

<?php $thispage = basename($_SERVER['PHP_SELF']) ; ?>

Contact Me</A>

There must be a better way to code this - in and out of HTML/PHP.
Thank for helping me clean it up if you can...😕

Weedpacket

Just echo the whole line; if you don't want it to be a link, leave out the <a> tags... depending on what exactly you mean by "deactivate".

Ceril

What you want to do is something like(assuming we are on contacts.php and the link list looks like the follwing:

Home[link]
About me[link]
PHP[link]
Contacts[not a link, diffrent color]

Usaly I just leave it be and let people click the currently active link.
Now for the neatness of it all ^_

links.php:
<?PHP
//what links do we got here?
$links=array("home.php", "aboutme.php","php.php","contacts.php");
$offcolor="#FFFFFF"; //Color for link currently displayed
$currentpage=basename($_SERVER['PHP_SELF']) ; //Snipped wildly from your code.

foreach($links as $key=>$v) //Go through each value in $links array and print it.
{
if($currentpage!=$v) //If its not currentpage, it should be a link
{
print "<a href=\"$v\">$v</a> \n";
}
elseif($currentpage==$v)
{
print "$v \n"; //its currentpage, not link, diffrent color.
}
}
?>
The above should work(didnt test it) and is easily modified to suit your layout(i hope).

Then do this where your linksection is to have them displayed:
Links:
<?PHP
include("links.php");
?>

And voila, you can add links as you please and dont have to rewrite every page to add a new link to a new page ^_

Good luck.

mrbell

Ceril,

Your code was executed brilliantly! I never would have thought to do that. Thank you so much.

If you're up for it (and I don't want to seem ungrateful), I'd like to take it one step further...

Is there a way to display the links (perhaps with another array) without their filenames and extensions.

e.g.
home.php = Home
about.php = About
contact.php = Contact Me

:bemused:

Ceril

Thank you cough cough Sorry for that one missed a little point ^_
$links=array("home.php"=>"Home", "aboutme.php"=>"About Me","php.php"=>"Php","contacts.php"=>"Contacts");

Thus we make the array key the document name and extenstion and let its value be the text displayed thus:

foreach($links as $key=>$v) //Go through each value in $links array and print it.
{
if($currentpage!=$key) //If its not currentpage, it should be a link
{
print "<a href=\"$key\">$v</a> \n";
}
elseif($currentpage==$key)
{
print "$v \n"; //its currentpage, not link, diffrent color.
}
}

Mmm, its kind of late but its either this way, or the other way around... good luck :-)

mrbell

Ceril,

You've got skills! I thought I'd be hacking at this for days. Thanks a million for saving me tons of time. 🆒