Hallo
Ik heb mijn website bijna afgerond. Zelf gescript en elke pagina bevat zo'n 50 IF-functies met elk een include (waarvan er uiteindelijk maar 1 geladen zal worden, hier staat de tekst namelijk in) en nog eens een 5-tal andere includes (header, sidebar, ...).
Die IF-functies kan ik anders best vervangen door een SWITCH-functie heeft iemand mij aangeraden, zo zou de pagina toch lichter worden
Zal deze pagina nog snel geladen worden op de huidige servers? Ik heb namelijk geen idee wat zo'n shared hosting server aankan.
Alvast bedankt
Groetjes
Herman
Zo wordt gecheckt welke tekst moet worden included (via parameter in index.php?pagina=pagina1)
En ook in de header voor het ophalen van de metatags per pagina (die dus verschillend zijn wegens SEO-redenen):PHP Code:<?php
if($_GET['pagina'] == pagina1){
include('../pagina1.php');
}
else if($_GET['pagina'] == pagina2){
include('../pagina2.php');
}
else if($_GET['pagina'] == pagina3){
include('../pagina3.php');
}
.
. (zo gaat dit door tot de 20e pagina)
.
else{
include('../index.php');
}
?>
En daarnaast nog een 5-teal includes voor de header enz...PHP Code:<?php
if($_GET['pagina'] == pagina1){
include('../pagina1-meta.php');
}
else if($_GET['pagina'] == pagina2){
include('../pagina2-meta.php');
}
else if($_GET['pagina'] == pagina3){
include('../pagina3-meta.php');
}
.
. (zo gaat dit door tot de 20e pagina)
.
else{
include('../index-meta.php');
}
?>
De pagina's hebben overigens allemaal een andere naam, dus een kortere oplossing is volgens mij niet mogelijk?
-
31-03-2010, 19:34 #1
Particulier
- Berichten
- 92
- Lid sinds
- 15 Jaar
Pagina te zwaar?
-
In de schijnwerper
JPG omzetten naar vectorbestand ? Nu voor 15,95Freelance / WerkLaravel / Wordpress - Top kwaliteit voor een gunstig tariefFreelance / WerkSSD Webhosting al vanaf €0,85 ! 5GB SSD Opslag - Onbeperkt Dataverkeer - SitebuilderHostingFitness YouTube Kanaal met 52.000+ abonnees & 1.8M views afgelopen 30 dagenSocial media -
31-03-2010, 19:36 #2
Particulier
- Berichten
- 199
- Lid sinds
- 16 Jaar
Veel problemen in snelheid gaat dit niet geven vermoed ik
-
31-03-2010, 19:39 #3
Hostingpower
- Berichten
- 51
- Lid sinds
- 17 Jaar
Beste Herman,
Problemen qua performance zal een switch of een elseif niet heel veel schelen.
Echter qua overzichtelijkheid en professionaliteit zal een switch toch hoger scoren.
Problemen met performance van grote websites worden veelal veroorzaakt door verkeerde query's.
-
31-03-2010, 19:40 #4gast19806 Guest
Het is inderdaad veel beter en sneller als je een switch constructie gebruikt. Kleine aanpassing, groot effect.
-
31-03-2010, 19:48 #5
Jasper de Boer
- Berichten
- 359
- Lid sinds
- 17 Jaar
Volgens mij moet zoiets werken:
Ik ken geen php, maar als het goed is kan je alleen het getal uit de url lezen, en deze automatisch laten echo'en in je include. Correct me if I'm wrong.Code:<?php switch ($_GET['pagina']) { case "pagina1": $i = "1"; break; } ?> <?php include('../pagina<?=$i;?>.php'); include('../pagina<?=$i;?>-meta.php'); ?>Laatst aangepast door J. de Boer : 31-03-2010 om 19:53
-
31-03-2010, 19:56 #6
Particulier
- Berichten
- 92
- Lid sinds
- 15 Jaar
Alle pagina's hebben een andere naam, ik heb ze voor de eenvoud hier nu pagina1.php en pagina2.php genoemd. Staat in de openingspost :)
Origineel gepost door J. de Boer
Vanaf hoeveel SWITCH-condities zou een bezoeker al merken dat de pagina iets langer laadt?
Is dit zo rond de 50, 100 of spreken we dan van veel grotere getallen?
-
31-03-2010, 19:58 #7
Particulier
- Berichten
- 90
- Lid sinds
- 15 Jaar
Verder was de code goed de Boer ;)Code:<?php switch($_GET['pagina']){ case "pagina1": $i = "1"; break; } include('../pagina'.$i.'.php'); include('../pagina'.$i.'-meta.php'); ?>
Aanvullend bericht:
@Herman: Bij de 50 en 100 is zeker nog niet merkbaar. Dan gaat het om >1000 case's.
Denk maar aan een for loop in een switch statement, dit geeft snel ook veel keuzes, maar toch laad de pagina snel.Laatst aangepast door Michel Bardelmeijer : 31-03-2010 om 19:59 Reden: Automatisch samengevoegd.
-
31-03-2010, 20:06 #8
Particulier
- Berichten
- 92
- Lid sinds
- 15 Jaar
Oké bedankt voor de hulp :) Origineel gepost door Michel Bardelmeijer
Voor de metatags die worden opgehaald in de <head>, is het beter om die met een SWITCH te echo'en of om de keywords en description in een aparte pagina te zetten en dat dan te includen?
Met het echo'en zal de index pagina die sowieso elke keer wordt geladen de metatags van ALLE pagina's bevatten. Terwijl als ik ze laat includen wordne alleen de gewenste metatags voor die pagina opgehaald. Wat is de beste methode met het oog op de prestaties?
M.a.w. is het sneller om alle metatags al in de pagina te zetten en het juiste te laten echo'en, of gaat een include van de gewenste metatags sneller?
(de teksten zelf zal uiteraard beter met een include gedaan worden omdat deze veel te lang zijn voor een echo)Laatst aangepast door Herman Dewulf : 31-03-2010 om 20:13
-
31-03-2010, 20:40 #9
Particulier
- Berichten
- 11
- Lid sinds
- 15 Jaar
Case zowieso al is het puur estetisch...
-
01-04-2010, 20:28 #10
Particulier
- Berichten
- 155
- Lid sinds
- 18 Jaar
of je maakt gewoon een array aan zoals dit:
kun je hetzelfde doen voor je meta stuff, die ga ik alleen niet uittikken.PHP Code:<?php
$pages = array(
'pagina1' => '../pagina1.php',
'pagina2' => '../pagina2.php',
'pagina3' => '../blah.php');
if (in_array($_GET['pagina'],$pages)
{
include($pages[$_GET['pagina']]);
}
else
{
include('../index.php');
}
?>
daarnaast: al zou je 1000 if statements in je pagina plaatsen dan wordt je page nog niet traag, daar light het dus niet aan.
-edit: code tags ipv php tags -_-Laatst aangepast door Martijn H : 01-04-2010 om 23:28
-
01-04-2010, 21:02 #11
Particulier
- Berichten
- 64
- Lid sinds
- 18 Jaar
Wat ik er van kan opmaken, of hoe ik het interpreteer zou je er het volgende van kunnen maken:
Geen 1000 if statements nodig, toch?PHP Code:<?php
if( isset($_GET['pagina']) )
{
if( file_exists('../'.$_GET['pagina']'.php') )
{
include '../'.$_GET['pagina'].'.php';
include '../'.$_GET['pagina'].'-meta.php';
}
else
{
include '../default.php';
}
}
else
{
include '../default.php';
}
?>
