[PHP/MySQL] Cached menu werkt niet

[BlackWhizz]

Oke, ik zit heir al wel 3 dagen mee te proberen. ik heb dus een treemenu gemaakt met ongelimiteerd subs maken. Alleen omdat dit per laadpoging ongelovelijk veel tijd kost om de hele tijd query's te maken wil ik hem cachen. Daarom heb ik het geheel omgebouwd in een class.

PHP:

<?php
class menu
{
    private $oDb,
            $sFormatted;
    protected $aMenuCache;
    
    function __construct()
    {
        global $oDb;
        $this->oDb          = $oDb;
        $this->sFormatted   = '';
        
        if ( !$this->cached() )
        {
            $this->write_cache();
        }
    }
    
    function build_menu( $aCached=null )
    {        
        if ( !is_array($aCached) )
        {
            $this->get_cache();
            $aCached = $this->aMenuCache;
        }
        
        $this->sFormatted .= '<ul>';
        
        foreach ( $aCached AS $aCache )
        {
            $this->sFormatted .= '<li><a href="'.$aCache['url'].'">'.htmlentities($aCache['title']).'</a></li>';
            
            if ( isset($aCache['_subs']) )
            {
                $this->sFormatted .= '<li>'.$this->build_menu( $aCache['_subs'] ).'</li>';
            }
        }
        $this->sFormatted .= '</ul>';
        
        return $this->sFormatted;
    }
    
    function cached ()
    {
        return file_exists( CACHE_PATH.'/cache/menu.txt' );
    }
    
    function get_cache()
    {
        if ( $sCache = @file_get_contents( CACHE_PATH.'/cache/menu.txt' ) )
        {
            $this->aMenuCache = unserialize( $sCache );
            
            return true;
        }
        else
        {
            trigger_error( 'Cannot load menu cache, please renew the cache in the admin panel.' );
            
            return false;
        }
    }
    
    function write_cache()
    {
        if ( $rHandle = @fopen( CACHE_PATH.'/cache/menu.txt', 'w' ) )
        {
            fwrite( $rHandle, serialize( $this->menu_cache() ) );
            fclose( $rHandle );
        }
        else
        {
            return trigger_error( 'Cannot create a cache file, please make shure the cache dir has chmod 777 and safe mode is turned off', E_USER_ERROR );
        }
    }

    function menu_cache( $iParent = 0, $sParent=ROOT_PATH )
    {
        if ( $this->oDb->count( 'categories', 'parent = ?', $iParent ) > 0 )
        {
            $aMenu = array();
            
            $aResult = $this->oDb->query( 'SELECT * FROM categories WHERE parent = ? ORDER BY niv ASC', $iParent );
            
            foreach ( $aResult AS $aData )
            {
                $aMenuCache = menu_cache( $aData['id'], $sParent.$aData['title_url'].'/' );
                
                if ( is_array($aMenuCache) )
                {
                    $aMenu[] = array( 'title' => $aData['title'], 'url' => $sParent.$aData['title_url'], '_subs' => $aMenuCache );
                    
                }
                else
                {
                    $aMenu[] = array( 'title' => $aData['title'], 'url' => $sParent.$aData['title_url'] );
                }
            }
            
            return $aMenu;
        }
        else
        {
            return false;
        }
    }
}
?>

In de array staat dit:

PHP:

Array
(
    [title] => Computers
    [url] => /computers
    [_subs] => Array
        (
            [0] => Array
                (
                    [title] => Hardware
                    [url] => /computers/hardware
                )

            [1] => Array
                (
                    [title] => Software
                    [url] => /computers/software
                    [_subs] => Array
                        (
                            [0] => Array
                                (
                                    [title] => Microsoft software
                                    [url] => /computers/software/microsoft-software
                                )

                        )

                )

        )

)

Echter, nu krijg ik dus doordat ik recursie gebruik de menu's 3x te zien.

Dat komt door dit:

code:

if ( !is_array($aCached) )
        {
            $this->get_cache();
            $aCached = $this->aMenuCache;
        }

Immers, die build_menu geeft een array door, zo niet dan maakt die een nieuwe array aan. Dit heb ik geprobeerd dooor het te onderdrukken om te kijken of $this->sFormatted leeg is, maar dat werkt dus nog steeds niet.

!is_array($aCached) && empty($this->sFormatted)

Kortom, ik weet niet wat ik fout doe, want als ik dat weghaal en het cache bestand verwijder dan werkt het nog steeds niet.

[Xcalibur]

is het niet gemakkelijker het hele menu in een sessievariabele te stoppen ofzo?
Per bezoeker wordt hij dan 1x gegenereerd, dat lijkt me niet zo'n probleem....

Overigens zijn er manieren om een tree wat efficienter in een database te stoppen, is dat niks?

[Johnny]

Ik zie de fout zo snel ook niet, maar de hele opbouw van de class lijkt me niet erg handig. Het idee van Object Georienteerd programmeren is dat je data binnen objecten afschermd. Als je $menu->get_menu() doet wordt het menu opgehaald, of het nu uit een cache-bestand komt of een database is buiten die class niet belangrijk. Functies zoals get_cache() hoef je dus niet te gebruiken.

Als je dit doet wordt het ontwerp ook meteen een stuk simpeler, zodra het menu wordt opgehaald wordt er gekeken of er een cache-bestand bestaat als dat zo is wordt dat direct ingelezen en terug gegeven. Als het niet zo is haal je de data uit de database, maak je het, schrijf het naar het cache en geef het terug. De eerste keer laden gaat dan wel iets langzamer, maar als je cache-bestand wordt verwijderd heb je geen last van fouten zoals "Cannot load menu cache, please renew the cache in the admin panel."

Verder kan je beter het hele menu in een enkele query ophalen uit de database en vervolgens recursief opbouwen, het is een beetje ingewikkelder maar gaat vele malen sneller. Als laatste tip kan je gewoon file_put_contents() gebruiken om je cache-bestand sneller en simpeler weg te schrijven.

[mocean]

Als je toch het hele menu cached, zou je die ook als HTML kunnen cachen. Dan hoef je bij iedere aanroep ook niet de array om te zetten in het menu.

Wat bepaald trouwens of het menu is gewijzigd? Je zou de functionaliteit ook kunnen opsplitsen: Je CMS (ik ga er van uit dat er iets van een CMS achter zit) kan weten als er iets in het menu wordt gewijzigd. Op dat moment kan je daar al een nieuwe cache file genereren.

Ter check in de front-end natuurlijk altijd kijken of de cache file wel bestaat, maar in principe heb je dan altijd een menu klaarstaan.

[BlackWhizz]

Oke, om het duidelijk te maken. Dit menu komt voor een CMS. Waarom ik hem opsla als serialized array? Omdat ik dan makkelijk kan of de bezoeker zich op die pagina bevindt. Echter, ik wilde eerst de basis functionaliteit maken.

Die renew cache error komt alleen als die cache niet geschreven kan worden (dus bijvoorbeeld de cache map bestaat niet of is niet beschrijfbaar).

Maar ik snap niet hoe ik dat probleem kan oplossen dat die het 3x herhaalt. Deze class is een class apart. Het wordt geen OOP, alleen maar OO.

Maar hoe kan ik dan onderdrukken dat die het 3x herhaalt. Of wat is die 2e manier?

[Voutloos]

BlackWhizz schreef op donderdag 26 juni 2008 @ 10:45:
Deze class is een class apart. Het wordt geen OOP, alleen maar OO.

Dus? Dan kan je deze class nog wel netjes doen.

[BlackWhizz]

Moderator, kan dit topic dicht?

[Janoz]

BlackWhizz schreef op donderdag 26 juni 2008 @ 11:17:
Moderator, kan dit topic dicht?

Waarom?

[BlackWhizz]

Ik heb de oplossing gevonden, dus het werkt.

[Creepy]

Misschien is het wel zo aardig even te vertellen wat het was zodat iemand die met een soortgelijk probleem zit daar ook nog wat aan heeft?

[BlackWhizz]

Oplossing was dit:

PHP:

<?php
class menu
{
    private $oDb;
    protected $aMenuCache,
              $sFormatted;
    
    function __construct()
    {
        global $oDb;
        $this->oDb          = $oDb;
        $this->sFormatted   = '';
        $this->aMenuCache   = array();
        
        if ( !$this->cached() )
        {
            $this->write_cache();
        }
        
        $this->get_cache();
    }
    
    function build_menu( $aCached=null )
    {        
        $sReqUri    = strtolower( trim( $_SERVER['REQUEST_URI'], '/' ) );
        
        $sFormatted = '';
        $sFormatted .= '<ul>'.PHP_EOL;
        
        foreach ( $aCached AS $aCacheData )
        {
            $sUrl = strtolower( trim( $aCacheData['url'], '/' ) );
            
            if ( $sUrl == substr( $sReqUri, 0, strlen($sUrl) ) )
            {
                $sFormatted .= '<li><a href="'.$aCacheData['url'].'" title="Ga naar de categorie '.htmlentities( $aCacheData['title'], ENT_QUOTES ).'" class="current">'.htmlentities( $aCacheData['title'], ENT_QUOTES ).'</li>'.PHP_EOL;
            }
            else
            {
                $sFormatted .= '<li><a href="'.$aCacheData['url'].'" title="Ga naar de categorie '.htmlentities( $aCacheData['title'], ENT_QUOTES ).'">'.htmlentities( $aCacheData['title'], ENT_QUOTES ).'</li>'.PHP_EOL;
            }
            
            if ( isset( $aCacheData['_subs'] ) && is_array( $aCacheData['_subs'] ) )
            {
                $sFormatted .= '<li>'.$this->build_menu( $aCacheData['_subs'] ).'</li>';
            }
        }
        
        $sFormatted .= '</ul>';
        
        $this->sFormatted = $sFormatted;
        
        return $sFormatted;
    }
    
    /**
    *   Menu::Get_Menu
    */
    
    function get_menu()
    {
        if ( empty($this->sFormatted) )
        {
            $this->build_menu( $this->aMenuCache );
        }
        
        return $this->sFormatted;
    }
    
    /**
    *   Menu::Cached
    *
    *   @desc   Checks if the cache file exists
    *   @access Public
    *   @return Bool    True/false
    */
    
    public function cached ()
    {
        return file_exists( CACHE_PATH.'menu.txt' );
    }
    
    /**
    *   Menu::Get_Cache
    *
    *   @desc   Gets the cache from a file and write it to a var
    *   @access Private
    *   @return Bool    True/false
    */
    private function get_cache()
    {
        if ( $sCache = @file_get_contents( CACHE_PATH.'menu.txt' ) )
        {
            $this->aMenuCache = unserialize( $sCache );
            
            return true;
        }
        else
        {
            trigger_error( 'Cannot load menu cache, please renew the cache in the admin panel.', E_USER_ERROR );
            
            return false;
        }
    }
    
    /**
    *   Menu::Write_Cache
    *
    *   @desc   Writes the cache to a file
    *   @return Bool    True/false
    *   @access Private
    */
    
    private function write_cache()
    {
        if ( $rHandle = @fopen( CACHE_PATH.'menu.txt', 'w' ) )
        {
            fwrite( $rHandle, serialize( $this->menu_cache() ) );
            fclose( $rHandle );
            
            return true;
        }
        else
        {
            return trigger_error( 'Cannot create a cache file, please make shure the cache dir has chmod 777 and safe mode is turned off', E_USER_ERROR );
        }
    }
    
    /**
    *   Menu::Menu_Cache
    *
    *   @desc   Creates an array which contains the menu cache
    *   @access Private
    *   @return Mixed   False on error, else an array which contains menu cache.
    *
    *   @param1 Int     Parent ( which contains the id of the parent to get from)
    *   @param2 String  Url from parent (to build seo-friendly url's)
    */
    
    private function menu_cache( $iParent = 0, $sParent=ROOT_PATH )
    {
        if ( $this->oDb->count( 'categories', 'parent = ?', $iParent ) > 0 )
        {
            $aMenu  = array();
            
            $aResult = $this->oDb->query( 'SELECT * FROM categories WHERE parent = ? ORDER BY niv ASC', $iParent );
            
            foreach ( $aResult AS $aData )
            {
                $aMenuCache = menu_cache( $aData['id'], $sParent.$aData['title_url'].'/' );
                
                if ( is_array($aMenuCache) )
                {
                    $aMenu[] = array( 'title' => $aData['title'], 'url' => $sParent.$aData['title_url'], '_subs' => $aMenuCache );
                }
                else
                {
                    $aMenu[] = array( 'title' => $aData['title'], 'url' => $sParent.$aData['title_url'] );
                }
            }
            
            return $aMenu;
        }
        else
        {
            return false;
        }
    }
    
    /**
    *   Menu::_Destruct
    *
    *   @desc   Menu class destructor, clears all filled variables
    *   @access Public
    *   @return None
    */
    
    public function __destruct ()
    {
        $this->sFormatted   = '';
        $this->oDb          = null;
        $this->aMenuCache   = null;
    }
}
?>