Das oben hat sich erledigt hab ne komplett neue Datei gecodet!

Jetzt aber ne neue Frage ist das sinnvoll?

index.php
PHP-Code:
<?php
include("config.php");
include("header.php");
include("functions.php");

while(sites("sitename", "login"))
{
sites_url("login", sites_inhalt("login"))
}

admin_panel($_GET['adminname']);
?>
functions.php
PHP-Code:
function sites($orderby, $ordervalue)
{
$sql_sites = "SELECT * FROM `sites` ORDER BY '$orderby' = '$ordervalue'";
$query_sites = mysql_query($sql_sites);
while($row_sites = mysql_fetch_assoc($query_sites))
{
echo $row_sites['sitename'];
}
}

function sites_url($sitename, $inhalt)
{
if($_GET['site'] == "$sitename")
{
 echo $inhalt;
}

function sites_inhalt($site_name)
{
$sql_sites2 = "SELECT * FROM `sites` WHERE 'sitename' = '$site_name'";
$query_sites2 = mysql_query($sql_sites2);
while($row_sites2 = mysql_fetch_assoc($query_sites2))
{
echo $row_sites2['siteinhalt'];
}

function admin_panel($adminname)
{
$sql_admin = "SELECT * FROM `user` WHERE `adminrights` = '1' AND `username` = '$adminname'";
$query_admin = mysql_query($sql_admin);
$row_admin = mysql_fetch_assoc($query_admin);
if( $row_admin['adminrights'] == "1" ) {
echo "<a href=\"admin/\">Adminbereich</a>";
}
} 
Also ist das sinnvoll fast die ganze index.php in functions zu packen und die einfach in der index.php aufzurufen oder sowas sinnlos?