Url design og kode dertil - optimering / forslag

Hej.
Er i gang med at lave et lille framework og er i gang med url routing delen har lige nu lavet det således at jeg laver url´en om til et array og derefter loader de forskellige ting så fx følgende url
http://www.example.com/categories.php/vis/users/
Bliver klassen 'vis' og funktionen users (som ligger i vis) kaldt.

Koden er som følger

    function url()
    {

        $request = preg_replace("|/*(.+?)/*$|", "\\1", $_SERVER['PATH_INFO']);
        $uri = explode('/', $request);

        $valid_modules = array('login', 'forum', 'bruger');
        if (empty($uri[0])) {
            $uri[0] = login;
            $uri[1] = view;
        }

        if (in_array($uri[0], $valid_modules)) {
            $controller = new $uri[0];
            return $controller->{$uri[1]}();
        } else
            return "Modulet $uri[0] ikke godkendt  (Vis 404 side her.))";
    }

Er dette den optimale måde at gøre det på ? eller er der sikkerhedsmæssige ting eller andre ting jeg skal tage højde for.

Ville i evt. gribe det an på en helt anden måde ?
alle indput og tanker er velkomne.