[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 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

[-gox-]

[PHP] pobierz, plaintext 
<?php
function pokazMenu() {
      $ls_liczba_stron = count($np_nazwa_pliku);
      for ($i = 0; $i < $ls_liczba_stron; $i++)  {
         print '<a href=index.php?strona='.$i.'> '.strtoupper(basename($np_nazwa_pliku[$i], '.php')).' </a>';
      }
   }
?>
[PHP] pobierz, plaintext 

[-gox-]

sorki, dalem ciala:

[PHP] pobierz, plaintext 
<?php
function pokazMenu() {
      $i=1;
      foreach (glob("./strona/*.php") as $np) {
           $np = strtoupper(basename($np, '.php'));
         print "<a href=index.php?strona=$i>$np</a>";
         $i++;
      }
   }
?>
[PHP] pobierz, plaintext 

[-gox-]

[PHP] pobierz, plaintext 
<?php
class tworzWidok {
    # ZOPTYMALIZOWANO! :)
    function pokazMenu() {
        foreach (glob("./strona/*.php") as $i=>$np) {
            $np = strtoupper(basename($np, '.php'));
            print "<a href=index.php?strona=$i>$np</a>";
        }
      }
    function pokazStrona($s_strona='') {
        $lista_plikow = glob("./strona/*.php");
        if ( ! in_array($s_strona, $lista_plikow))
            die('Hacking attempt!');
        include ($lista_plikow[$s_strona]);
    }
}
?>
[PHP] pobierz, plaintext 

[Czapla]

Gox takiej optymalizacji to ja się nie spodziewałem Ale jest błąd, ponieważ twoja klasa nie uwzględnia wartości zmiennej $s_strona większych niż liczba plików
Po wpisaniu wartości np. 187 [takiego pliku w indexie tablicy nie mam] pojawia się błąd

Czyli będzie tak Chyba tego się już nie da bardziej skrócić

[PHP] pobierz, plaintext 
<?php
class tworzWidok {
 
   function pokazMenu() {
 
       foreach (glob("./strona/*.php") as $i=>$np) {
           $np = strtoupper(basename($np, '.php'));
           print "<a href=index.php?strona=".$i."> ".$np." </a>";
       }
     }
     
   function pokazStrona($s_strona='') {
      $lista_plikow = glob("./strona/*.php");
      $lp_lista_plików = count($lista_plików);
       if ( ! array($s_strona, $lista_plikow)) 
          die('Hacking attempt!');
          include ($lista_plikow[$s_strona<$lp_lista_plików]);
    }
}    
?>
[PHP] pobierz, plaintext 

Pozdrawiam

Ten post edytował Czapla 23.10.2008, 17:57:16

[-gox-]

Da sie:

[PHP] pobierz, plaintext 
<?php
class tworzWidok {
 
  function pokazMenu() {
 
      foreach (glob("./strona/*.php") as $i=>$np) {
          $np = strtoupper(basename($np, '.php'));
          print "<a href=index.php?strona=".$i."> ".$np." </a>";
      }
    }
    
  function pokazStrona($s_strona='') {
     $lista_plikow = glob("./strona/*.php");
      if ( ! array_key_exists($s_strona, $lista_plikow)) 
         die('Hacking attempt!');
         include ($lista_plikow[$s_strona<$lp_lista_plików]);
   }
}    
?>
[PHP] pobierz, plaintext 

[-gox-]

a moze lepiej w ten sposob hmm?

[PHP] pobierz, plaintext 
<?php
class tworzWidok {
 
var $files_list = array();
 
function get_files_list() {
    if ( empty($this->files_list))
        $this->files_list = glob("./strona/*.php");
    return $this->files_list;
}
 
  function pokazMenu() {
      foreach ($this->get_files_list as $i=>$np) {
          $np = strtoupper(basename($np, '.php'));
          print "<a href=index.php?strona=".$i."> ".$np." </a>";
      }
    }
    
  function pokazStrona($s_strona='') {
      if ( ! array_key_exists($s_strona, $this->get_files_list)) 
         die('Hacking attempt!');
         include ($this->get_files_list[$s_strona<$lp_lista_plików]);
   }
}
?>
[PHP] pobierz, plaintext 

[-gox-]

czekaj bo znowu dalem ciala...

[PHP] pobierz, plaintext 
<?php
class tworzWidok {
 
var $files_list = array();
 
function tworzWidok() {
    if ( empty($this->files_list))
        $this->files_list = glob("./strona/*.php");
    return $this->files_list;
}
 
  function pokazMenu() {
      foreach ($this->files_list as $i=>$np) {
          $np = strtoupper(basename($np, '.php'));
          print "<a href=index.php?strona=".$i."> ".$np." </a>";
      }
    }
    
  function pokazStrona($s_strona='') {
      if ( ! array_key_exists($s_strona, $this->files_list)) 
         die('Hacking attempt!');
         include ($this->files_list[$s_strona<$lp_lista_plików]);
   }
}
?>
[PHP] pobierz, plaintext 

[Czapla]

Ta ostatnia jest imo najlepsza zwłaszcza z punku widzenia OOP i rozbijania modelu na mniejsze elementy

P.S Pomógł się należy, ale piszesz jako gość ...