Оптимизация кода вывода полей в любом месте

InstantCMS 2.X

Нужна помощь в PHP

#1 18 июля 2019 в 20:17
Добрый день уважаемые мастера PHP. Просьба помочь.
Создал четыре поля для соцсетей и сгруппировал их в "Соцсети" соответственно.

В произвольном месте шаблона я вывел так
  1.  
  2. <!--СОЦСЕТИ -->
  3. <?php if (!empty($item['vk'])||!empty($item['facebook'])||!empty($item['instagram'])){ ?>
  4. <h5><?php html($item['title']); ?> в соцсетях</h5>
  5. <div class="container-fluid" style="margin-bottom:20px">
  6. <div class="row">
  7. <?php if (!empty($item['facebook'])){ ?>
  8. <a href="<?php echo $item['facebook']; ?>" target="_blank" class="social col facebook"><i class="fa fa-facebook"></i></a>
  9. <?php } ?>
  10. <?php if (!empty($item['vk'])){ ?>
  11. <a href="<?php echo $item['vk']; ?>" target="_blank" class="social col vk"><i class="fa fa-vk"></i></a>
  12. <?php } ?>
  13. <?php if (!empty($item['instagram'])){ ?>
  14. <a href="<?php echo $item['instagram']; ?>" target="_blank" class="social col instagram"><i class="fa fa-instagram"></i></a>
  15. <?php } ?>
  16. <?php if (!empty($item['twitter'])){ ?>
  17. <a href="<?php echo $item['twitter']; ?>" target="_blank" class="social col twitter"><i class="fa fa-twitter"></i></a>
  18. <?php } ?>
  19. </div>
  20. </div>
  21. <?php } ?>
  22.  
Конечно же все работает, но я чувствую что это тупой грязный код. Огромная просьба подсказать как его оптимально и грамотно прописать, т.к я не силен в ПХП только использую примеры. Сама страница с выводом ТУТ. Скрин ниже
#2 18 июля 2019 в 20:22
Может как стоит циклом выводить и изменит тип поля?
#3 18 июля 2019 в 21:17


Может как стоит циклом выводить и изменит тип поля ?

Clear

А что тут грязного? Ничего грязного не вижу) Норм сделал)
#4 18 июля 2019 в 21:21
В конечно итоге можно заморочиться и сделать отдельное поле для соц сетей и не делать четыре штуки, а через одну и выводить в свой tpl. А так вполне норм сделал) Можно и так)
#5 18 июля 2019 в 21:38
Clear,
Насколько быстро выводится страница с этими полями?
#6 18 июля 2019 в 21:55

Насколько быстро выводится страница с этими полями?

Ris
ссылку дал замерь, пока не замерял
#7 18 июля 2019 в 22:13


Clear,
Насколько быстро выводится страница с этими полями?

Ris

Они будут быстрее выводиться так как не проходят через отдельный tpl. А сразу находятся в файле шаблона на сколько понимаю.
#8 18 июля 2019 в 23:14

В произвольном месте шаблона я вывел так

Clear
В первом условии проверки на пустоту в twitter не хватает

Насколько быстро выводится страница с этими полями?

Ris
Карта точно в разы больше повлияет. Или не факт?
#9 19 июля 2019 в 13:16
Ну если уже говорить об оптимизации, то там вобще все переделывать надо в Мапсе. Все жутко с шабом )))
Вы не можете отвечать в этой теме.
Войдите или зарегистрируйтесь, чтобы писать на форуме.
Используя этот сайт, вы соглашаетесь с тем, что мы используем файлы cookie.