Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> problem z Switch
tdr
post
Post #1





Grupa: Zarejestrowani
Postów: 41
Pomógł: 0
Dołączył: 2.02.2013

Ostrzeżenie: (0%)
-----


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?

Ten post edytował tdr 6.05.2013, 14:45:50
Go to the top of the page
+Quote Post
motyl-pl
post
Post #2





Grupa: Zarejestrowani
Postów: 294
Pomógł: 31
Dołączył: 2.04.2010

Ostrzeżenie: (0%)
-----


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. }


--------------------
Go to the top of the page
+Quote Post
Crozin
post
Post #3





Grupa: Zarejestrowani
Postów: 6 476
Pomógł: 1306
Dołączył: 6.08.2006
Skąd: Kraków

Ostrzeżenie: (0%)
-----


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.
Go to the top of the page
+Quote Post
!*!
post
Post #4





Grupa: Zarejestrowani
Postów: 4 298
Pomógł: 447
Dołączył: 16.11.2006

Ostrzeżenie: (0%)
-----


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


--------------------
Nie udzielam pomocy poprzez PW i nie mam GG.
Niektóre języki programowania, na przykład C# są znane z niezwykłej przenośności (kompatybilność ze wszystkimi wersjami Visty jest wiele warta).
Go to the top of the page
+Quote Post
tdr
post
Post #5





Grupa: Zarejestrowani
Postów: 41
Pomógł: 0
Dołączył: 2.02.2013

Ostrzeżenie: (0%)
-----


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.

Go to the top of the page
+Quote Post
!*!
post
Post #6





Grupa: Zarejestrowani
Postów: 4 298
Pomógł: 447
Dołączył: 16.11.2006

Ostrzeżenie: (0%)
-----


Swoją drogą po co Ci te zmienne, skoro to w zasadzie powinno być w html, nie licząc include.


--------------------
Nie udzielam pomocy poprzez PW i nie mam GG.
Niektóre języki programowania, na przykład C# są znane z niezwykłej przenośności (kompatybilność ze wszystkimi wersjami Visty jest wiele warta).
Go to the top of the page
+Quote Post
PrinceOfPersia
post
Post #7





Grupa: Zarejestrowani
Postów: 717
Pomógł: 120
Dołączył: 18.04.2009

Ostrzeżenie: (0%)
-----


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. ?>


--------------------
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 20.08.2025 - 08:40