php link probleem

Pagina: 1
Acties:
  • 91 views sinds 30-01-2008

  • bloodrush
  • Registratie: Augustus 2004
  • Laatst online: 14-08 09:15
hoi ik heb een vraagje ik ben een website aan het maken.

Nu heb ik een leuke layout gemaakt met tables en wil nu in de body mijn php docs gaan includen.

nu ben ik op de volgende code gekomen:

<?
if(!$go) {
$go = "home"; }
include "" . $go . ".php";
?>


en de links naar de php docs zijn

<a href="index.php?go=home.php" style="color:#000000">Home</a><br>

<a href="index.php?go=news.php" style="color:#000000">Nieuws</a><br>


maar toch krijg ik dit niet werkende

:/ iemand een ideee?? :?

  • UltimateB
  • Registratie: April 2003
  • Niet online

UltimateB

Pomdiedom

PHP:
1
2
3
4
5
6
7
8
switch ( $go ) {
 case "home";
   include ( "temps/home.php" );
 case "whatever";
   include ( "temps/whatever.php" );
 default;
   include ( "temps/frontpage.php" );
}


Of zoeits. Ik raad het af om een variabele te gebruiken in een include dit is namelijk een aardig veiligheids lek in je code.

Zowiso klopt je syntax voor include niet helemaal. :p

[ Voor 18% gewijzigd door UltimateB op 04-10-2004 09:55 ]

"True skill is when luck becomes a habit"
SWIS


Verwijderd

Je include zet er altijd een .php achter en in je links zoals hieronder gebruik je ook de go met .php en die is naar mijn idee overbodig.

Je hebt dit

<a href="index.php?go=home.php" style="color:#000000">Home</a><br>

en naar mijn idee moet het dit worden

<a href="index.php?go=home" style="color:#000000">Home</a><br>

  • bloodrush
  • Registratie: Augustus 2004
  • Laatst online: 14-08 09:15
thnx

foutje van mij ... blij dat jullie er zijn

  • Skaah
  • Registratie: Juni 2001
  • Niet online
$go moet zijn: $_GET['go']

  • Kippenijzer
  • Registratie: Juni 2001
  • Laatst online: 16-08 09:43

Kippenijzer

McFallafel, nu met paardevlees

Daarnaast werkt het niet op php versies hoger dan 3 die register_globals uit hebben staan. Dan moet je $go door $_GET['go'] vervangen.

  • Thijsmans
  • Registratie: Juli 2001
  • Laatst online: 17-08 23:29

Thijsmans

⭐⭐⭐⭐⭐ (5/5)

hangt van zijn PHP-versie af :)

Overigens, deze manier is een major veiligheidslek. Gebruik liever een array en gebruik de key uit de URL:

PHP:
1
2
3
4
5
// (index.php)?news

$pages = array('news' => './php/actions/news.php', 'guestbook' => './php/actions/gb.php');

include( $pages[ $_SERVER['QUERY_STRING'] ] );


hth :)

sught, n00b :P Hope To Help

[ Voor 16% gewijzigd door Thijsmans op 04-10-2004 10:08 ]

Privacy-adepten vinden op AVGtekst.nl de Nederlandse AVG-tekst voorzien van uitspraken en besluiten.


  • UltimateB
  • Registratie: April 2003
  • Niet online

UltimateB

Pomdiedom

"True skill is when luck becomes a habit"
SWIS


  • Pelle
  • Registratie: Januari 2001
  • Laatst online: 17-08 22:37

Pelle

🚴‍♂️

Maar PHP is niets voor W&G, echter in dit niveau ook niet voor P&W :)

  • drm
  • Registratie: Februari 2001
  • Laatst online: 09-06-2025

drm

f0pc0dert

Als je de faq van Programming & Webscripting even doorleest, kom je er snel genoeg achter wat er hier niet goed gaat.

offtopic:
HTH betekent: "hope this helps" :)

Music is the pleasure the human mind experiences from counting without being aware that it is counting
~ Gottfried Leibniz

Pagina: 1

Dit topic is gesloten.