2013-06-29 2 views
0

Я начинаю с PHP для WordPress и написал код для размещения значков социальных сетей в нижнем колонтитуле. То, как я это сделал, я просто вызываю контент URL социальной сети, хранящийся в БД, и если в нижнем колонтитуле есть что-то значок/ссылка. Но для меня это выглядит очень неэффективно, вот код, кто-нибудь знает, как сделать его более эффективным.Сделать несколько инструкций PHP, если более эффективны

 <?php 
     $social1 = of_get_option('fab_social_twitter_url'); 
     $social2 = of_get_option('fab_social_facebook_url'); 
     $social3 = of_get_option('fab_social_linkedin_url');    
     ?> 



     <!-- divs for right social network icons column --> 
     <div class="eight columns"> 
     <div class="social"> 
      <ul> 

      <?php 
       if(!empty($social1)) { 
      ?> 
       <li><a href="<?php echo of_get_option('fab_social_twitter_url'); ?>"><img src="<?php echo of_get_option('fab_social_twitter_icon'); ?>" alt="Follow us on Twitter"></a></li> 
      <?php 
      } 
      ?> 

      <?php 
       if(!empty($social2)) { 
      ?> 
       <li><a href="<?php echo of_get_option('fab_social_facebook_url'); ?>"><img src="<?php echo of_get_option('fab_social_facebook_icon'); ?>" alt="Follow us on Facebook"></a></li> 
      <?php 
      } 
      ?>    

      <?php 
       if(!empty($social3)) { 
      ?>     
       <li><a href="<?php echo of_get_option('fab_social_linkedin_url'); ?>"><img src="<?php echo of_get_option('fab_social_linkedin_icon'); ?>" alt="Follow us on Linkedin"></a></li> 
      <?php 
      } 
      ?>    

      </ul> 
     </div> 
     </div> 
+3

ли вам просто * думаю * это неэффективно, или у вас есть доказательства того, что он неэффективен? – delnan

ответ

2

Может быть:

<!-- divs for right social network icons column --> 
    <div class="eight columns"> 
    <div class="social"> 
     <ul> 
     <?php 
      foreach (array("twitter","facebook","linkedin") as $option) 
       ($tmp=of_get_option('fab_social_'.$option.'_url')) && (print('<li><a href="'.$tmp.'"><img src="'.of_get_option('fab_social_'.$option.'_icon').'" alt="Follow us on '.ucfirst($option).'"></a></li>')); 
     ?> 
     </ul> 
    </div> 
    </div> 
1

С точки зрения производительности, я не думаю, что есть что-нибудь, чтобы оптимизировать здесь три отдельных случаях проходят индивидуально

0
<?php 
    $social1 = of_get_option('fab_social_twitter_url'); 
    $social2 = of_get_option('fab_social_facebook_url'); 
    $social3 = of_get_option('fab_social_linkedin_url'); 

    $icon1 = of_get_option('fab_social_twitter_icon'); 
    $icon2 = of_get_option('fab_social_facebook_icon'); 
    $icon3 = of_get_option('fab_social_linkedin_icon');   
    ?> 



    <!-- divs for right social network icons column --> 
    <div class="eight columns"> 
    <div class="social"> 
     <ul> 

     <?php if(!empty($social1)) { ?> 
      <li> 
       <a href="<?php echo $social1; ?>"> 
        <img src="<?php echo $icon1; ?>" alt="Follow us on Twitter"> 
       </a> 
      </li> 
     <?php } ?> 

     <?php if(!empty($social2)) { ?> 
      <li> 
       <a href="<?php echo $social2; ?>"> 
        <img src="<?php echo $icon2; ?>" alt="Follow us on Facebook"> 
       </a> 
      </li> 
     <?php } ?> 

     <?php if(!empty($social3)) { ?> 
      <li> 
       <a href="<?php echo $social3; ?>"> 
        <img src="<?php echo $icon3; ?>" alt="Follow us on Linkedin"> 
       </a> 
      </li> 
     <?php } ?> 

     </ul> 
    </div> 
    </div> 
Смежные вопросы