Ich bin das kein großer Fan von. Es hat den Nachteil, das man immer in zwei verschiedenen Dateien nache einenm Fehler suchen muss. Irgendwie lohnt es sich dann auch garnicht mehr, eine index.php zu schreiben
Es ggibt bei dir auch Funktionen, die egentlich nur zwei Zelen Code haben. Diese Funktionen sind imo nicht besonders sinnvoll. Du schreibst mehr Code und das ganze wird imo unübersichtlicher.
Was imo schon Sinn macht, sind Funktionen, die mehr Code haben und auch evtl. in mehreren anderen Dateien bzw. an unterschiedlichen Stellen im Code notwendig wären, wo also der Code unter Umständen mehrmals stehen müsste. Da spart man sich überflüssigen Code und erhäht durchaus die Übersichtlichkeit...

(BTW, der Code kann doch so garnicht funktionieren *verwirrt ist*
PHP-Code:
function sites_url($sitename, $inhalt)
{
if($_GET['site'] == "$sitename")
{
echo $inhalt;
} 
Die Bedingung wird bei dieser Funktion zB garnicht geschlossen.
Des weiteren ist diese Zeile nicht wirklich sinnvoll
PHP-Code:
while(sites("sitename", "login")) 
, weil die Funktion sites() keinen Rückgabewert hat...)