login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Probleempje

Offline bennieboy - 15/09/2005 16:00 (laatste wijziging 15/09/2005 16:01)
Avatar van bennieboyHTML beginner Ik gebruik in plaats van een iframe includes. De links leggen mbv includes, das geen probleem. Maar de pagina die automatisch geladen wordt krijg ik ni ingeprogrammeerd. Dit is wat ik heb:
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php 
  2.                     if (isset($_GET['pagina'])) 
  3.                     { 
  4.                         if($_GET['pagina']=='home') 
  5.                         { 
  6.                             include('inhoud.php'); 
  7.                         } 
  8.                         elseif($_GET['pagina']=='portfolio') 
  9.                         { 
  10.                             include('portfolio.php'); 
  11.                         } 
  12.                         elseif($_GET['pagina']=='webdesign') 
  13.                         { 
  14.                             include('webdesign.php'); 
  15.                         }
  16. 						elseif($_GET['pagina']=='tipsentricks') 
  17.                         { 
  18.                             include('tipsentricks.php'); 
  19.                         } 
  20. 						elseif($_GET['pagina']=='contact') 
  21.                         { 
  22.                             include('contact.php'); 
  23.                         }
  24. 						elseif($_GET['pagina']=='links') 
  25.                         { 
  26.                             include('links.php');
  27. 						}
  28. 						else
  29. 						{
  30. 							include('inhoud.php');
  31. 						}
  32. 					} 
  33.                 ?>

4 antwoorden

Gesponsorde links
Offline Rens - 15/09/2005 16:03 (laatste wijziging 15/09/2005 16:04)
Avatar van Rens Gouden medaille

Crew algemeen
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php 
  2.                     if (isset($_GET['pagina'])) 
  3.                     { 
  4.                         if($_GET['pagina']=='home') 
  5.                         { 
  6.                             include('inhoud.php'); 
  7.                         } 
  8.                         elseif($_GET['pagina']=='portfolio') 
  9.                         { 
  10.                             include('portfolio.php'); 
  11.                         } 
  12.                         elseif($_GET['pagina']=='webdesign') 
  13.                         { 
  14.                             include('webdesign.php'); 
  15.                         }
  16.                         elseif($_GET['pagina']=='tipsentricks') 
  17.                         { 
  18.                             include('tipsentricks.php'); 
  19.                         } 
  20.                         elseif($_GET['pagina']=='contact') 
  21.                         { 
  22.                             include('contact.php'); 
  23.                         }
  24.                         elseif($_GET['pagina']=='links') 
  25.                         { 
  26.                             include('links.php');
  27.                         }
  28.                         else
  29.                         {
  30.                             include('inhoud.php');
  31.                         }
  32.                     } else
  33.                     {
  34.                         include("default-pagina.php");
  35.                     }
  36. ?>


Simpel, gewoon een else eronder aan plakken.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?PHP
  2. if(IsSet($_GET['pagina']))
  3. {
  4.     // pagina wel geset
  5. } else
  6. {
  7.     // geen pagina in de url geset
  8.     // dus de standaard pagina
  9. }
  10. ?>
Offline Stijn - 15/09/2005 16:04 (laatste wijziging 15/09/2005 16:16)
Avatar van Stijn PHP expert
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?
  2. switch($_GET['pagina'])
  3. {
  4.   default:
  5.   include('inhoud.php');
  6.   break;
  7.   case "links":
  8.   include('links.php');
  9.   break;
  10.   /*enz voor iedere pagina. hiermee kan je ook die else verwijderen vanonderen, dit zal beter zijn, maar er zijn andere manieren:
  11. if(isset($_GET['pagina'])
  12. {
  13.   include($_GET['pagina'].".php");
  14. }
  15. //->werkt veiliger ;)
  16. */
  17. }
  18. ?>


mvg stijn

edit

of kijken of het gee http:// in de url voorkomt en of de file wel bestaat ... NOG Veiliger 
Offline Rens - 15/09/2005 16:09 (laatste wijziging 15/09/2005 16:18)
Avatar van Rens Gouden medaille

Crew algemeen		 Dan kun je beter een array aanmaken met alle toegestane pagina's erin.
En dan checken of de pagina in de array staat.
Weet je zeker dat alle pagina's toegestaan zijn.
Is nóg veiliger.

edit; Hoeft niet veiliger te zijn.
Het kan dat de file wel bestaat maar niet zichtbaar hoort te zijn.^^
Offline nemesiskoen - 15/09/2005 18:08
Avatar van nemesiskoen Gouden medaille

PHP expert
Citaat:
of kijken of het gee http:// in de url voorkomt en of de file wel bestaat ... NOG Veiliger

Dit is veilig, maar niet persé veiligER. Visa versa idem.
Ikzelf betwijfel ook of je default eerst mag zetten bij een switch structuur, vind het in ieder geval onlogisch omdat deze valt te vergelijken met ELSE. En je zet ook geen else voor een if.

Of je ramt het in een array, of je zet alles na elkaar via case en dan kan je nog een uitzondering maken.
Bv.

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2.  switch($_GET['pagina']){
  3.    case home: 
  4.        require 'inhoud.php';
  5.         break;
  6.    case portfolio: case webdesign: case tipsentricks: case contact: case links:
  7.        require $_GET['pagina'] .".php";
  8.         break;
  9.    default: require 'inhoud.php';
  10.         break
  11.  }
  12. ?>
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.23s