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

    • То есть бревна в глазу (определение элемента по его заголовку) вы не видите? :)
      Ответить
      • Вы что, действительно считаете, что это правильный подход? Эти тайтлы ведь из админки забиваются... Я что не могу название страницы сменить без ковыряния в коде? И что бы при этом страница не разваливалась...
        Ответить
    • > "switch - case" - Не не слышал!
      т.е. switch по строкам таки кошернее?

      > Стоимость темы - 45 уе...
      скорее за красивости, если они там есть, а не за код...
      Ответить
      • говно в том, что если title == "Portfolio", то все остальные if тоже будут проверяться. Либо switch для красивости, либо else if. Да и надо все эти выражения объединять все же.
        Ответить
        • это да, но switch по строкам в любом случае подванивает, а elseif и того хуже.
          Ответить
          • ну в контексте этого куска, если идти той же идеей то это будет вполне годным вариантом
            Ответить
    • Самое большое г... состоит в том, что все $title это названия страниц, которые задаются в админке. И изменив их полностью разваливается лейаут страницы. То есть я не могу использовать тему, не влазия в код... Вот такой хардкод...
      Ответить
    • еблан или шо? Нормально код писать религия не позволяла?
      Ответить

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