1. PHP / Говнокод #12462

    +48

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    $title = get_the_title();
if ( $title == "Portfolio")  $data['sl_portfolio_style'] = "2 Columns Portfolio";
if ( $title == "3 Columns Portfolio")  $data['sl_portfolio_style'] = "3 Columns Portfolio";
if ( $title == "4 Columns Portfolio")  $data['sl_portfolio_style'] = "4 Columns Portfolio";
if ( $title == "6 Columns Portfolio")  $data['sl_portfolio_style'] = "6 Columns Portfolio";
if ( $title == "Portfolio")  query_posts( '&post_type=portfolio-type&posts_per_page=4&paged=' . $paged );
if ( $title == "4 Columns Portfolio")  query_posts( '&post_type=portfolio-type&posts_per_page=8&paged=' . $paged );
if ( $title == "6 Columns Portfolio")  query_posts( '&post_type=portfolio-type&posts_per_page=12&paged=' . $paged );
if ( $title == "3 Columns Portfolio")  query_posts( '&post_type=portfolio-type&posts_per_page=6&paged=' . $paged );
if ( $title == "Portfolio Right Sidebar")  $data['sl_portfolio_style'] = "Portfolio with Sidebar";
if ( $title == "Portfolio Right Sidebar")  $data['portfolio_sidebar_position'] = "Right Sidebar";
if ( $title == "Portfolio Left Sidebar")  $data['sl_portfolio_style'] = "Portfolio with Sidebar";
if ( $title == "Portfolio Left Sidebar")  $data['portfolio_sidebar_position'] = "Left Sidebar";

if ( $title == "Portfolio Left Sidebar")  query_posts( '&post_type=portfolio-type&posts_per_page=12&paged=' . $paged );
if ( $title == "Portfolio Right Sidebar")  query_posts( '&post_type=portfolio-type&posts_per_page=12&paged=' . $paged );

    "switch - case" - Не не слышал!
    И это в покупной теме для wordpress на тимфоресте.
    Стоимость темы - 45 уе...

    Запостил: tsybulskyserg, 22 Января 2013

    Комментарии (8) RSS

    • ava krypt 22.01.2013 23:35 # +2

      То есть бревна в глазу (определение элемента по его заголовку) вы не видите? :)
      Ответить

      • ava tsybulskyserg 25.01.2013 12:44 # 0

        Вы что, действительно считаете, что это правильный подход? Эти тайтлы ведь из админки забиваются... Я что не могу название страницы сменить без ковыряния в коде? И что бы при этом страница не разваливалась...
        Ответить

    • ava Lure Of Chaos 23.01.2013 00:02 # +1

      > "switch - case" - Не не слышал!
      т.е. switch по строкам таки кошернее?

      > Стоимость темы - 45 уе...
      скорее за красивости, если они там есть, а не за код...
      Ответить

      • ava guest 25.01.2013 12:00 # +1

        говно в том, что если title == "Portfolio", то все остальные if тоже будут проверяться. Либо switch для красивости, либо else if. Да и надо все эти выражения объединять все же.
        Ответить

        • ava Lure Of Chaos 25.01.2013 12:04 # −1

          это да, но switch по строкам в любом случае подванивает, а elseif и того хуже.
          Ответить

          • ava guest 25.01.2013 18:29 # +1

            ну в контексте этого куска, если идти той же идеей то это будет вполне годным вариантом
            Ответить

    • ava tsybulskyserg 25.01.2013 12:39 # +1

      Самое большое г... состоит в том, что все $title это названия страниц, которые задаются в админке. И изменив их полностью разваливается лейаут страницы. То есть я не могу использовать тему, не влазия в код... Вот такой хардкод...
      Ответить

    • ava ELked 26.01.2013 13:02 # −4

      еблан или шо? Нормально код писать религия не позволяла?
      Ответить

    Добавить комментарий