Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: problem z Switch
Forum PHP.pl > Forum > PHP
tdr
Oto mój kod:
  1. <?php
  2. if(isset($_GET['strona'])){
  3. switch($_GET['strona']){
  4. case 'kontakt':
  5. $url="includes/kontakt.php";
  6. $tytul="Kontakt";
  7. $opis="Zaczerpnij kontaktu o nas";
  8. break;
  9.  
  10. case 'onas':
  11. $url="includes/onas.php";
  12. $tytul="O NAs";
  13. $opis="Coś więcej o Nas";
  14. break;
  15.  
  16. case 'rejestruj':
  17. $url="includes/rejestruj.php";
  18. $tytul="Rejestracja";
  19. $opis="Zarejestruj się.";
  20. break;
  21.  
  22. default:
  23. $url="includes/glowna.php";
  24. $tytul="Strona glowna";
  25. $opis="Opis strony głównej.";
  26. }
  27. }else{
  28. $url = 'includes/glowna.php';
  29. $tytul="Strona glowna";
  30. $opis="Opis strony głównej.";
  31. }
  32. ?>
CHciałbym stworzyć skrypt który ma posłużyć za index.php, ale już sam nie wiem, czy dobrze robie... co dalej miałbym zrobić aby zaladować poszczególne strony?
motyl-pl
A gdzie includujesz strone ze switcha ?
Może:
  1.  
  2. }
  3.  
  4. @include_once($url);
  5.  
  6. }else{
  7. $url = 'includes/glowna.php';
  8. $tytul="Strona glowna";
  9. $opis="Opis strony głównej.";
  10. }
Crozin
1. Przede wszystkim: DRY - w bloku DEFAULT i ELSE masz dokładnie to samo nie wiadomo po co.
2.
  1. $strona = isset($_GET['strona']) ? $_GET['strona'] : '';
  2.  
  3. switch ($storna) {
  4. ...
  5. }
3. Kod wydaje się być OK, daj sobie po SWITCHu:
  1. var_dump($strona, $url, $tytul, $opis);
I zobacz co zwraca.
!*!
Cytat(tdr @ 6.05.2013, 15:40:59 ) *
CHciałbym stworzyć skrypt który ma posłużyć za index.php, ale już sam nie wiem, czy dobrze robie... co dalej miałbym zrobić aby zaladować poszczególne strony?

http://phpguru.pl/czytaj?id=5
tdr
Super smile.gif już mi mózg zlasował od tego PHPa, troszkę się w nim próbuje poruszać i chyba za długo dzisiaj wink.gif) a co do tego DRY? to jak by to rozwiązać, bo chyba średio gdybym usunął któraś z instrukcji.

Cytat(Crozin @ 6.05.2013, 15:48:34 ) *
1. Przede wszystkim: DRY - w bloku DEFAULT i ELSE masz dokładnie to samo nie wiadomo po co.
2.
  1. $strona = isset($_GET['strona']) ? $_GET['strona'] : '';
  2.  
  3. switch ($storna) {
  4. ...
  5. }
3. Kod wydaje się być OK, daj sobie po SWITCHu:
  1. var_dump($strona, $url, $tytul, $opis);
I zobacz co zwraca.

!*!
Swoją drogą po co Ci te zmienne, skoro to w zasadzie powinno być w html, nie licząc include.
PrinceOfPersia
ja bym to tak rozwiazał:
  1. <?php
  2.  
  3. $strona = $_GET['strona'];
  4.  
  5. $sitemap = array(
  6. 'kontakt' => array("includes/kontakt.php","Kontakt", "Zaczerpnij kontaktu o nas"),
  7. 'onas' => array("includes/onas.php", "O NAs", "Coś więcej o Nas"),
  8. 'rejestruj' => array("includes/rejestruj.php", "Rejestracja", "Zarejestruj się."),
  9. 'default' => array("includes/glowna.php", "Strona glowna", "Opis strony głównej.") );
  10.  
  11. $list = $sitemap[$strona];
  12. if (!$list) $list = $sitemap['default'];
  13. list($url, $tytul, $opis) = $list;
  14.  
  15.  
  16.  
  17. echo "$url, $tytul, $opis"; //do testow
  18. ?>
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.