Funciones innecesarias

[harvey]

Para gestionar los contenidos de una página tengo este código;

Código php:

$seccion = $_GET['seccion']; // la sección le llega por url; [url]www.dominio.com?seccion=inicio[/url]
        if(empty($seccion)){ //Si $seccion llega vacio presenta la portada.
            include("inicio.php");
        }else{// Si $seccion no esta vacia y existe un domuneto con el nombre que llega por $seccion...
            if(file_exists($seccion.".php")){
                include ($seccion.".php");
            }elseif (file_exists($seccion.".html")){
                include ($seccion.".html");
            }else{ // Si no existe ningún documento con ese nombre...
            echo 'La página solicitada no existe';
            }
        }
 

Alguien me dijo que lo metiera en una función pero esto no parece muy correcto así que... ¿como hago para llamarlo desde index.php sin meterlo en index.php ¿Hago otro include? ¿Hay alguna otra opción? ¿Hago lo mismo con el siguiente código?

Código php:

mysql_connect("localhost","usuario","pass"); //Conexion con la db
mysql_select_db("database") or die('Cannot select database'); //selección de la base de datos
 

Por otro lado en el código tengo repartido los típicos mensajes de "Login Incorrecto", "Se ha identificado correctamente", "Se ha registrado correctamente", etc. Y he pensado que quizás podría hacer una función en la que le pasara un número de mensaje y me devolviera el mensaje. Aunque la verdad es que estos mensajes son solamente utilizados una vez, no se repiten, por lo que la unica utilidad que tendría hacer esto seria tener todos los mensajes juntos, localizados en un mismo sitio, por si hubiera que hacer algún tipo de cambio. Tambien en dicha función se les aplicaría etiquetas css quedando algo así; '<p class="sysmes">'.$Mensaje($num).'</p>'
¿Que opinais? ¿Vale la pena o es complicarse demasiado?