[php] Te simpele template-system

[frizzy]

Eigenlijk mag ik dit geen template parser noemen, maar ik vind hem wel geniaal. Gebruik altijd zoiets, is lekker snel en werkt perfect voor wat ik er van verwacht.

Code: Selecteer alles

<?php
   // Simpele Template-Dinges...
   $inpage   =   false;
   function viewer($name=false, $vars=array()){
      global $inpage;
      if($name !== false){
         if(@fopen('viewers/'.$name.'.tpl.php', 'r')){
            
            foreach($vars as $v=>$g){
               $$v   =   $g;
               define(strtoupper(str_replace(' ','_', $v)), $g);
            }
            
            if($inpage == false){
               include('viewers/header.tpl.php');
               $inpage   =   true;
            }
            include('viewers/'.$name.'.tpl.php');
            if($inpage == false){
               include('viewers/footer.tpl.php');
            }
            
            return true;
         }
         else{
            return false;
         }
      }
      else{
         return false;
      }
   }
?>

Even snel gerateld omdat ik me verveelde, kan dus foutje in zitten

[libia]

tip: pas op met global! als je niet oplet kunnen er onverwachte dingen gebeuren.. in princiepe heb je nooit global nodig
tip: gebruik nooit @ (@fopen()), gebruik gewoon een goede controle!

[Derk]

als je fopen in een if zet met een @ ervoor is je controle wat mij betreft genoeg hoor. Anders zou je moeten gaan kijken of het bestand bestaat voor je fopen draait. Nu zit die check gewoon bij fopen in.

Global is in de niet-OOP omgeving anders wel een verdraaid handige optie.

[frizzy]

[Derk]

[ img ]

Komt me vaag bekend voor...

[frizzy]

Geniaal gemaakt vriend

[libia]

als je fopen in een if zet met een @ ervoor is je controle wat mij betreft genoeg hoor. Anders zou je moeten gaan kijken of het bestand bestaat voor je fopen draait. Nu zit die check gewoon bij fopen in.

Global is in de niet-OOP omgeving anders wel een verdraaid handige optie.

@ is foten onderdrukken, maar fouten moet je voorkomen, niet mishandelen

En ja global kan handig zijn, maar - zeker bij een snippet - moet de gene die het script gebruikt goed oppassen en op blijven letten, daarom lijkt het me bij een snippet handig om dit zo min mogelijk te gebruiken

[frizzy]

Ik snap niet wat er zo moeilijk is aan een global, sorry. En als hij de pagina niet kan vinden geeft hij naast false terug ook een error, maar dit staat niet netjes. Daarom onderdruk ik hem

[Derk]

En als hij de pagina niet kan vinden geeft hij naast false terug ook een error, maar dit staat niet netjes. Daarom onderdruk ik hem

Wat je dus moet doen is eerst kijken of de file bestaat voordat je fopen gebruikt. Dan krijg je nooit een error die je moet onderdrukken. Maargoed, ik geef je groot gelijk dat je deze oplossing toepast. Doe ik ook altijd!

[Stefan]

Wat je dus moet doen is eerst kijken of de file bestaat voordat je fopen gebruikt.

Kan dat gewoon met file_exists()?

[libia]

Wat je dus moet doen is eerst kijken of de file bestaat voordat je fopen gebruikt.

Kan dat gewoon met file_exists()?

Is_file() geloof ik

[Remi]

Het doet ongeveer hetzelfde.
Maar het grote verschil is dat is_file alleen naar de bestanden kijkt, en file_exists ook de mappen mee neemt.

[libia]

Het doet ongeveer hetzelfde.
Maar het grote verschil is dat is_file alleen naar de bestanden kijkt, en file_exists ook de mappen mee neemt.

Dat lijkt me hier niet nodig of wel?