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

    +162

    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
    if ($page_name=='comments_content_sources.php' && $_REQUEST['batch_action']=="approve" && !in_array('comments|approve',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_content_sources.php' && $_REQUEST['batch_action']=="delete" && !in_array('comments|delete',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_content_sources.php' && $_REQUEST['batch_action']=="approve_and_delete" && (!in_array('comments|approve',$_SESSION['permissions']) || !in_array('comments|delete',$_SESSION['permissions']))) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_content_sources.php' && $_REQUEST['batch_action']=="delete_and_approve" && (!in_array('comments|approve',$_SESSION['permissions']) || !in_array('comments|delete',$_SESSION['permissions']))) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_models.php' && !in_array('comments|view',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_models.php' && $_REQUEST['action']=="change_complete" && !in_array('comments|edit_all',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_models.php' && $_REQUEST['batch_action']=="approve" && !in_array('comments|approve',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_models.php' && $_REQUEST['batch_action']=="delete" && !in_array('comments|delete',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_models.php' && $_REQUEST['batch_action']=="approve_and_delete" && (!in_array('comments|approve',$_SESSION['permissions']) || !in_array('comments|delete',$_SESSION['permissions']))) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='comments_models.php' && $_REQUEST['batch_action']=="delete_and_approve" && (!in_array('comments|approve',$_SESSION['permissions']) || !in_array('comments|delete',$_SESSION['permissions']))) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='categories.php' && !in_array('categories|view',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='categories.php' && in_array($_REQUEST['action'],array("add_new","add_new_complete")) && !in_array('categories|add',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='categories.php' && $_REQUEST['action']=="change_complete" && !in_array('categories|edit_all',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}
    if ($page_name=='categories.php' && $_REQUEST['batch_action']=="delete" && !in_array('categories|delete',$_SESSION['permissions'])) {header("Location: error.php?error=permission_denied");die;}

    Недавно перешел на новую фирму, и там меня вежливо попросили добавить новый функционал в готовый сайт. Как только я зашел в папку с сайтом, сразу же почувствовал аромат качественного кода, и это были только ромашки. Маршрутизация в 600 строк ифов(из за ограничения все не могу выложить), та что вы Александр над проектом работали профессионалы. Жесткие ссылки сайта по всему коду, один класс и моЩная архитектура в процедурном стиле, ну это только начало. Видели бы базу и запросы к ней, человек писавший все это явно Оптимус-Оптимизатор 80 уровня. В общем, ребят, не занимайтесь веб панорамированием, говна там ну очень много.

    Запостил: AslanV, 16 Июля 2014

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

    • ava gost 17.07.2014 09:39 # +1

      Тарасокод детектед.
      Ответить

    • ava kegdan 17.07.2014 11:47 # 0

      пыф.... расстрелять
      Ответить

    • ava Fike 17.07.2014 11:56 # +1

      интересно, сколько `die;` забыто?
      Ответить

    • ava Vasiliy 17.07.2014 15:51 # 0

      Да не страшный кодярник то в принципе, магии многовато. Но я бы подумал о том что с этой фирмой не надо связываться на долго. Ну только если ЗП там не космос.
      Ответить

    • ava Oliver_air 23.07.2014 11:25 # 0

      чпу уровень бог-индус.
      хотя код написан дальше грамотно, сама реализация хромает на голову.
      но с другой стороны код задефлен от всего чего только можно.
      этож сколько человекочасов потрачено на девелопинг?
      Ответить

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