Üdv, PHP programozással kapcsolatban lenne kérdésem!?

PHP-val szeretném megoldani a menü rendszeremet.

Így csináltam meg:

<?php

$oldalak['0'] = 'menu1.php';

$oldalak['1'] = 'menu2.php';

$oldalak['2'] = 'menu3.php';

$oldalak['3'] = 'menu4.php';

$oldalak['4'] = 'menu5.php';

$oldalak['5'] = 'menu6.php';

$oldalak['6'] = 'menu7.php';

$oldalak['7'] = 'menu8.php';

?>

<ul>

<li><a href="index.php?oldal=0" class="active">Főoldal</a></li>

<li><a href="index.php?oldal=1">Eredmények</a></li>

<li><a href="index.php?oldal=2">Csapat</a></li>

<li><a href="index.php?oldal=3">Események</a></li>

<li><a href="index.php?oldal=4">Galéria</a></li>

<li><a href="index.php?oldal=5">Hírek</a></li>

<li><a href="index.php?oldal=6">Kapcsolat</a></li>

<li><a href="index.php?oldal=7">Impresszum</a></li>

</ul>

<div class="content">

<?php

$oldal = $_GET['oldal'];

if(isset($oldal) && isset($oldalak[$oldal])) {

include($oldalak[$oldal]);

} else {

include('menu1.php');

}

?>

</div> <!-- content end -->

Ez egy diven belül jeleníti meg a behívott oldalakat.

A kérdésem, hogy ez így "biztonságos" vagy kellene egyéb óvintézkedést tenni a kódba? Illetve, ha van jobb megoldás szívesen fogadom!