[PHP] Include wordt meerdere keren uitgevoerd*

[Verwijderd]

Ik ben voor een vriend van mij een website aan het maken in php.

Ik heb header.php een footer.php

Daarnaast heb ik dan een main.php en nog een paar pagina's.

via

code:

<?
switch ($_GET['p']) {
   case 'index':
      include ('main.php');
   case 'music':
      include ('music.php'); 
      break;
   case 'links':
      include ('links.php');
      break;
   case 'informatie':
      include ('informatie.php');
      break;
   case 'about':
      include ('about.php'); 
      break;    
}
?>

roep ik dan elke pagina apart aan

het vreemde is nu dat de voorpagina gewoon goed is. precies zoals hij moet maar als je dan bv naar de pagina music gaat je de main pagina nog een keertje onderaan de site te zien krijgt.

de site is www.maydayproductions.nl/francois/index.php

[Gadgets]

Debuggen, je eerste main knalt er door heen, je hebt geen break staan!!

[BtM909]

1. lekkere titel, verzin voortaan een beter beschrijvende titel

2. Waar hoort mijn topic? -> Programming dus

3. [code=php][/]

4. Je snapt zelf dat we hierdoor eigenlijk niks weten, zitten er cross-references in de diverse PHP's, kan je het probleem isoleren, weet je het verschil tussen http://nl2.php.net/include-once en http://nl2.php.net/include?

5. Debuggen

[ Voor 10% gewijzigd door BtM909 op 19-03-2008 15:37 ]

[Verwijderd]

zag het maar ook met die break erbij heb ik het probleem

code:

switch ($_GET['p']) {
   case 'index':
      include ('main.php');
      break;
   case 'music':
      include ('music.php'); 
      break;
   case 'links':
      include ('links.php');
      break;
   case 'informatie':
      include ('informatie.php');
      break;
   case 'about':
      include ('about.php'); 
      break; 
    
}

[Sebazzz]

Klopt, en zet een 'default:' voor de index.

Check ook even je XHTML:

XML:

<a id="lala_gif" http://85.147.168.97:8010/listen.pls">

[Arethusa]

Op regel 3 omtrent $timeout$ gaat het ook mis volgens IE7.

[Verwijderd]

Arethusa schreef op woensdag 19 maart 2008 @ 15:42:
Op regel 3 omtrent $timeout$ gaat het ook mis volgens IE7.

dat mag volgens mij niets te doen hebben dat hij de pagina meerdere malen laat zien. Als ik

code:

   default:
      include ('main.php');
      break;
}

erbij zet dan gaat zelfs de index pagina in de stress

weird.

[Arethusa]

Dat weet ik Ik wilde je alleen even meedelen dat daar ook een foutje zit.

[ Voor 70% gewijzigd door Arethusa op 19-03-2008 15:48 ]

[Verwijderd]

tnx

[AaroN]

default is altijd de laatste optie he, dus je hoeft geen break te doen daar.

[Verwijderd]

AaroN schreef op woensdag 19 maart 2008 @ 15:49:
default is altijd de laatste optie he, dus je hoeft geen break te doen daar.

schiet mij maar lek

code:

<?
switch ($_GET['p']) {
   case 'index':
      include ('main.php');
      break;
   case 'music':
      include ('music.php'); 
      break;
   case 'links':
      include ('links.php');
      break;
   case 'informatie':
      include ('informatie.php');
      break;
   case 'about':
      include ('about.php'); 
      break;
   default:
      include ('main.php');    
    
}
?>

geeft dubbele pagina's nu, ook op de voorpagina. als ik default weghaal is de voorpagina weer goed.

[P.O. Box]

misschien moet je ietsiepietsie meer code posten, want dit is bijna hetzelfde als:
ik krijg bij de waarde van $a een waarde van 12, maar moet 13 zijn:

code:

if ($jaap == $joop) {
     $b = 45;
}

print "a=" . $a;
print "b=" . $b;

m.a.w. hier kunnen we weinig mee...

[ Voor 7% gewijzigd door P.O. Box op 19-03-2008 15:53 ]

[LuCarD]

Verwijderd schreef op woensdag 19 maart 2008 @ 15:36:
zag het maar ook met die break erbij heb ik het probleem

code:

switch ($_GET['p']) {
   case 'index':
      include ('main.php');
      break;
   case 'music':
      include ('music.php'); 
      break;
   case 'links':
      include ('links.php');
      break;
   case 'informatie':
      include ('informatie.php');
      break;
   case 'about':
      include ('about.php'); 
      break; 
    
}

Dan zit kennelijk het probleem niet in 'deze' switch case.

Voor de rest is het koffiedik kijken.

[Verwijderd]

zal eens de boel opnieuw gaan opzetten. zoek mij wezeloos zo.

[SinergyX]

Denk eerder dat je bij de footer moet gaan kijken, klopt het ook dat er diverse malen <body> </body> tags in de code zitten?

Bouw het anders eerst helemaal op zonder al die extra zut, koppelingen en advertenties. Dan klaar, dan pas content gaan invoegen.

[ Voor 34% gewijzigd door SinergyX op 19-03-2008 15:57 ]

[doeternietoe]

Dit lijkt me tussen haakjes iets mooier:

PHP:

switch ($_GET['p']) {
   case 'index':
      include ('main.php');
      break;
   case 'music':
      include ('music.php'); 
      break;
   case 'links':
      include ('links.php');
      break;
   case 'informatie':
         include ('informatie.php');
        break;
   case 'about':
      include ('about.php'); 
      break; 
   default:
      header( 'HTTP/1.0 404 Not Found' );
      include( '404.php' );
      break;
}

Anders als er een dood linkje ergens staat, wordt het èn door de zoekmachine geïndexeerd èn snapt die bezoeker maar niet waarom die telkens op de index komt.

En als je echt niet kan vinden waar het zit, kan je ook nog van alle includes include_once kunnen maken.

[LuCarD]

doeternietoe schreef op woensdag 19 maart 2008 @ 16:01:
Dit lijkt me tussen haakjes iets mooier:

PHP:

switch ($_GET['p']) {
   case 'index':
      include ('main.php');
      break;
   case 'music':
      include ('music.php'); 
      break;
   case 'links':
      include ('links.php');
      break;
   case 'informatie':
         include ('informatie.php');
        break;
   case 'about':
      include ('about.php'); 
      break; 
   default:
      header( 'HTTP/1.0 404 Not Found' );
      include( '404.php' );
      break;
}

Anders als er een dood linkje ergens staat, wordt het èn door de zoekmachine geïndexeerd èn snapt die bezoeker maar niet waarom die telkens op de index komt.

En als je echt niet kan vinden waar het zit, kan je ook nog van alle includes include_once kunnen maken.

Include_once gaat niet werken. Aangezien hij op de music pagina ook de main pagina include.

[doeternietoe]

LuCarD schreef op woensdag 19 maart 2008 @ 16:03:
[...]


Include_once gaat niet werken. Aangezien hij op de music pagina ook de main pagina include.

Tja, volgens mij kunnen we ook niet zo goed ruiken wat erop andere pagina's gebeurd.

Maar de TS zou ik het meest willen aanraden om z'n structuur eens wat te herzien. Je moet überhaubt niet tegen dit probleem aanlopen...

[MueR]

LuCarD schreef op woensdag 19 maart 2008 @ 16:03:
Include_once gaat niet werken. Aangezien hij op de music pagina ook de main pagina include.

Jawel hoor, want die include in de switch wordt niet uitgevoerd als de juiste case niet wordt bereikt.

[Verwijderd]

ik heb de fout al gevonden eikel die ik ben. in de index.php had ik ook de main.php ge-include

ik had dus niet

code:

<?php include("header.php"); ?>
<?php include("footer.php"); ?>

maar

code:

<?php include("header.php"); ?>
<?php include("main.php"); ?>
<?php include("footer.php"); ?>

gedaan. lekker snugger

[ Voor 54% gewijzigd door Verwijderd op 19-03-2008 17:45 ]

[DutchCommando]

Sebazzz schreef op woensdag 19 maart 2008 @ 15:37:
Klopt, en zet een 'default:' voor de index.

Als een waarde bekend is is het best practice om deze ook te specificeren. Anders verlies je een stukje documentatie. Immers, weet jij als je over een half jaar terug kijkt op die code nog dat default eigenlijk de case voor 'index' is? Ook raak je een mogelijkheid kwijt voor een stukje fout detectie (default kan in dit geval gebruikt worden voor fouten).

[mocean]

Als je nu eens de bestandnaam dezelfde waarde geeft (voor .php) als de waarde die je gebruikt voor _GET['p']

PHP:

$valid_options = ('index','music','links','informatie','main');
$inc = (in_array($_GET['p'],$valid_options)) ? $_GET['p'] : 'main' ;
include($inc.'php');

Dan ben je een stuk sneller klaar en makkelijk uitbreidbaar als steeds een case toevoegen.

[Verwijderd]

Sowieso een best practice:

Gebruik altijd require_once, tenzij iets ofwel mág misgaan, of iets meerdere keren mág worden geïnclude. Er is geen enkele reden om de functie include te gebruiken als bovenstaande niet zo is.

mocean schreef op woensdag 19 maart 2008 @ 20:39:
Als je nu eens de bestandnaam dezelfde waarde geeft (voor .php) als de waarde die je gebruikt voor _GET['p']

PHP:

$valid_options = ('index','music','links','informatie','main');
$inc = (in_array($_GET['p'],$valid_options)) ? $_GET['p'] : 'main' ;
include($inc.'php');

Dan ben je een stuk sneller klaar en makkelijk uitbreidbaar als steeds een case toevoegen.

Of bijvoorbeeld:

PHP:

if ( array_key_exists ( 'p', $_GET )
   && preg_match ( '~^[a-z]+([a-z-][a-z]+)*$~', $_GET['p'] )
   && is_file ( $file = './includes/' . $_GET['p'] . '.php' )
) {
   require_once ( $file );
}

Hoef je niet steeds je bestand aan te passen, maar kun je gewoon nieuwe scripts in die directory mikken. Hoe je het exact doet is niet boeiend. Dat je het veilig en gemakkelijk maakt wél.

[ Voor 65% gewijzigd door Verwijderd op 19-03-2008 20:47 ]

[doeternietoe]

Verwijderd schreef op woensdag 19 maart 2008 @ 20:41:
Sowieso een best practice:

Gebruik altijd require_once, tenzij iets ofwel mág misgaan, of iets meerdere keren mág worden geïnclude. Er is geen enkele reden om de functie include te gebruiken als bovenstaande niet zo is.

Om eerlijk te zijn lijkt me nog beter: Zorg dat include_once nooit echt nodig is. De meeste constructies waarbij (onbedoeld) hetzelfde bestand twee keer wordt geïnclude, zijn niet moeders mooiste.