[PHP]Pomoc przy optymalizacji klasy, czy da się to jeszcze bardziej uprościć? Opcje

[Czapla]

Napisałem klasę, która automatycznie generuje nazwy i linki dla danych stron znajdujących się w katalogu.

[PHP] pobierz, plaintext 
<?php
class tworzWidok 
 {
  function pokazMenu()
   {
   $np_nazwa_pliku = array();
    foreach (glob("./strona/*.php") as $np)
    {
     $np_nazwa_pliku[] = strtoupper(substr($np, 9, -4));
    }
   $ls_liczba_stron = count($np_nazwa_pliku);
    for ($i = 0; $i < $ls_liczba_stron; $i++)
     {
      print '<a href=index.php?strona='.$i.'> '.$np_nazwa_pliku[$i].' </a>';
     }
    }
 
  function pokazStrona($s_strona)
   {
   $np_nazwa_pliku = array();
    foreach (glob("./strona/*.php") as $np)
    {
     $np_nazwa_pliku[] = $np;
    }
   $ls_liczba_stron = count($np_nazwa_pliku);
    for ($i = 0; $i < $ls_liczba_stron; $i++)
     {
      switch ($s_strona) {
       case $i: 
        include ($np_nazwa_pliku[$i]);
        break;
       }
     }
   }
 }
?>
[PHP] pobierz, plaintext 

Przykład wykorzystania

[PHP] pobierz, plaintext 
<div id="menu"> //include menu.php;
  <?php
  $widok = new tworzWidok;
  $widok->pokazMenu();
  ?>
  </div>
 <div id="tresc"> //include tresc.php;
<?php
  $strona = (int)$_REQUEST['strona'];
  $widok->pokazStrona($strona);
?>
 </div>
[PHP] pobierz, plaintext 

Wiem że koncepcji rozwiązania problemu jest tyle co programistów (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ale być może dało by się bardziej zoptymalizować tą klasę?

Pozdrawiam

[Lars]

Hmm...mnie się na przykład nie podobają linijki 7-16. Po co robisz tablicę, skoro możesz od razu wyświetlić linki?
U ciebie

[PHP] pobierz, plaintext 
<?php
$np_nazwa_pliku = array();
   foreach (glob("./strona/*.php") as $np)
   {
    $np_nazwa_pliku[] = strtoupper(substr($np, 9, -4));
   }
  $ls_liczba_stron = count($np_nazwa_pliku);
   for ($i = 0; $i < $ls_liczba_stron; $i++)
    {
     print '<a href=index.php?strona='.$i.'> '.$np_nazwa_pliku[$i].' </a>';
    }
?>
[PHP] pobierz, plaintext 

Moje

[PHP] pobierz, plaintext 
<?php
$i=0;
foreach (glob("./strona/*.php") as $np)
   {
    $np= strtoupper(substr($np, 9, -4));
     print '<a href=index.php?strona='.$i.'> '.$np.' </a>';
     $i++;
    }
?>
[PHP] pobierz, plaintext 

Pozdrawiam