My theme is sporadically slow and hangs. WordPress

Dbranes comments:

I have seen similar problems with sites that are fetching data from remote sites on each reload

(fx. using file_get_contents("http://example.com") in the theme )

You can also manually profile your code using [[LINK href="http://php.net/manual/en/function.microtime.php"]]microtime()[[/LINK]] function in your theme files.

Some prefer to use more advanced profiling, like [[LINK href="http://www.xdebug.org/docs/profiler"]]xdebug[[/LINK]] or [[LINK href="http://newrelic.com/"]]new relic[[/LINK]].

Dbranes comments:

ps: here is also a nice Wordpress timer function:

http://codex.wordpress.org/Function_Reference/timer_stop

Josh Cranwell comments:

You may have possible solved my issue from using the blackbox.

Possibly - I am going to hold open the question until the expiry time, while I continue to test.

I noticed in the SQL querie times, all my querys wer mili secs (1ms to 2ms) and then it became apparent that one in the list was taking 22 secs...

It was this... [[LINK href="https://gist.github.com/3341243"]]https://gist.github.com/3341243[[/LINK]]

Does look rough to you?

Thanks

Dbranes comments:

ok, that's great.

Next step is to try to comment out the $exhibitorMeta query and the corresponding if-loop

if ($exhibitorMeta->have_posts()))

to see if the page will run fast again, if so, then the next step is to investigate the $exhibitorMeta query or try to replace it, or skip it if possible.

Dbranes comments:

you can also reduce the number of get_post_meta calls,
you are calling it twice, you could reduce it by using fx:

$facebook_page=get_post_meta($post->ID, 'Facebook Page', true);
<?php if (strlen($facebook_page)>0) echo '
<li><a class="exhibitor-meta-icon-facebook" href="' . $facebook_page . '" target="_blank">View our Facebook Page</a></li>
'; ?>


instead of

<?php if ( get_post_meta($post->ID, 'Facebook Page', true) ) echo '
<li><a class="exhibitor-meta-icon-facebook" href="' . get_post_meta($post->ID, 'Facebook Page', true) . '" target="_blank">View our Facebook Page</a></li>
'; ?>

Dbranes comments:

you can also reduce the number of get_post_meta calls,
you are calling it twice, you could reduce it by using fx:

$facebook_page=get_post_meta($post->ID, 'Facebook Page', true);
<?php if (strlen($facebook_page)>0) echo '
<li><a class="exhibitor-meta-icon-facebook" href="' . $facebook_page . '" target="_blank">View our Facebook Page</a></li>
'; ?>


instead of

<?php if ( get_post_meta($post->ID, 'Facebook Page', true) ) echo '
<li><a class="exhibitor-meta-icon-facebook" href="' . get_post_meta($post->ID, 'Facebook Page', true) . '" target="_blank">View our Facebook Page</a></li>
'; ?>

Dbranes comments:

aha, or this is probably better, since these are boolean:

$bFacebookPage=get_post_meta($post->ID, 'Facebook Page', true);
$bTwitterUse=get_post_meta($post->ID, 'Twitter User', true);
...


<?php if ( $bFacebookPage || $bTwitterUser', true) || ...){?>


and so on.

Dbranes comments:

replace the above with:

<?php if ( $bFacebookPage || $bTwitterUser || ...){?>

Dbranes comments:

here is a pastebin rewrite to reduce the get_post_meta calls

[[LINK href="http://pastebin.com/347FmmVk"]]http://pastebin.com/347FmmVk[[/LINK]]

ps: sorry for the code mess above ;-)

Josh Cranwell comments:

If I'm honest I don't actually need it - runs a lot healthier with out it so I'm happy for now.

Site loads instantaneous compared to previous - but as it was so sporadic I will keep the question open till last min incase slowness returns.

Thanks

Dbranes comments:

ok let's see what happens ;-)

cheers