使多个 PHP if 语句更高效 [英] Make multiple PHP if statements more efficient

问题描述

我从 PHP 开始用于 WordPress，并编写了一些代码来在页脚中放置一些社交网络图标.我完成它的方式是，我只是调用存储在数据库中的社交网络 URL 的内容，如果页脚中有图标/链接.但在我看来效率很低，这是代码，有没有人知道如何提高效率.

 <!-- 右侧社交网络图标栏的 div--><div class="八列"><div class="social"><ul><?php如果(！空($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="在 Twitter 上关注我们"></a></li><?php}?><?php如果(！空($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="在 Facebook 上关注我们"></a></li><?php}?><?php如果(！空($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="在 Linkedin 上关注我们"></a></li><?php}?>

解决方案

也许:

 <!-- 右侧社交网络图标栏的div--><div class="八列"><div class="social"><ul><?phpforeach (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="关注我们 '.ucfirst($option).'"></a></li>'));?>

I am starting out with PHP for a WordPress and have written some code to put some social network icons in the footer. The way I have done it works, I'm simply calling the content of social network URL stored in the DB and if there is something the icon/link in the footer. But looks very inefficient to me, here is the code, does anyone know how to make it more efficient.

      <?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>

解决方案

Maybe:

  <!-- 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>

这篇关于使多个 PHP if 语句更高效的文章就介绍到这了，希望我们推荐的答案对大家有所帮助，也希望大家多多支持IT屋！