1. Java / Говнокод #4178

    +69

    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
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    /**
     * Инициализация набора действий, определенных для контейнера.
     * @param context
     * @throws DfException
     */
    protected void initializeTransitions(EditorContext context) throws DfException{
        AbstractDocumentModel documentModel = getDocumentModel();
        AbstractState state = documentModel.getLifeCyclePolicy(context).getState(context);
        
        if (!documentModel.isNew())
        {
            addTransition(TransitionName.CREATE_DETACHED_COMMISSION, TransitionHandler.CREATE_DETACHED_COMMISSION);
            addTransition(TransitionName.PRINT, TransitionHandler.PRINT);
        }
        AbstractTaskModel taskModel = context.getTaskModel();
        if (taskModel  instanceof ReviewDocumentTaskModel && !(taskModel  instanceof ExpeditionaryProcessingDocumentTaskModel)){
//                 && !(taskModel instanceof AttachFileAndReviewDocumentTaskModel)){

                  
        if ((taskModel instanceof AttachFileToDocumentTaskModel || taskModel instanceof WaitForScanningTaskModel) && ((documentModel instanceof InternalDocumentModel) || (documentModel instanceof OutgoingDocumentModel) || (documentModel instanceof IncomingDocumentModel)))
            addTransition(TransitionName.PRINT_BARCODE, TransitionHandler.PRINT_BARCODE);

        if (taskModel instanceof AbstractDocumentTaskModel){
            if ( state instanceof AbstractDraftState ){
                if( state instanceof NewProtocolDraftState)
                    addTransition(ContainerAction.ACTION_NAME, TransitionName.SEND_DRAFT, TransitionName.REGISTER, TransitionHandler.SEND_DRAFT);
                else if (state instanceof NewDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionHandler.SEND_DRAFT);
                if (state instanceof AbstractApprovalDraftState)
                    addTransition(TransitionName.REWORK_DRAFT, TransitionHandler.REWORK_DRAFT);
                if (state instanceof AnalyseDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_ANALYSE, TransitionHandler.SEND_DRAFT);
                if (state instanceof ControlDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_CONTROL, TransitionHandler.SEND_DRAFT);
                if (state instanceof ApproveDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_APPROVAL, TransitionHandler.SEND_DRAFT);
                if (state instanceof SignDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_SIGN, TransitionHandler.SEND_DRAFT);
                if (state instanceof ExternalApproveDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_APPROVAL, TransitionHandler.SEND_DRAFT);
                if (state instanceof ControlApproveDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_CONTROLED, TransitionHandler.SEND_DRAFT);
                if (state instanceof ConfirmationDraftState)
                    addTransition(TransitionName.SEND_DRAFT, TransitionDescription.SEND_DRAFT_CONFIRM, TransitionHandler.SEND_DRAFT);
            }
          }
        }
        
.................. несколько страниц такого кода ...........................
      
        if (state instanceof AbstractApprovalDraftState ||
            state instanceof ReworkDraftState ||
            state instanceof NewDocumentState ||
            state instanceof RegisterDocumentState ||
            state instanceof ReviewDocumentState ||
            state instanceof ExecuteDocumentState){
        	addTransition(TransitionName.SAVE, TransitionHandler.SAVE);
        }

        super.initializeTransitions(context);
    }

    Автор решил захардкодить в одном месте логику переходов между всеми состояниями всех жизненных циклов всех документов, реализованных в системе документооборота. Получился говнокод-BPM-движок. А внешне код очень даже солидный: с константами и комментариями... и названиями классов вроде: ExpeditionaryProcessingDocumentTaskModel

    Запостил: dab84, 06 Сентября 2010

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

    • ava Анонимус 06.09.2010 13:52 # +4

      Автор не очень был в курсе полиморфизма
      Ответить

    • ava inkanus-gray 06.09.2010 21:22 # −1

      Ну вот, а все на 1С накидываются...
      Ответить

    • ava vov4a 07.09.2010 08:58 # −3

      А если имеется куча дочерних классов и лишь несколько будут реализовать "свою" "инициализацию транзакций"? Имхо целесообразнее использовать instanceof:)
      Ответить

      • ava absolut 07.09.2010 09:23 # −2

        Целесообразно переработать дерево классов.
        Ответить

      • ava Анонимус 07.09.2010 19:56 # −3

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

        инстансоф почти всегда говно, кроме случаев когда без него никак
        Ответить

        • ava vov4a 07.09.2010 21:42 # −1

          если уж на то пошло, то абстрактный.
          Ответить

          • ava Анонимус 07.09.2010 21:45 # −1

            не абстрактный, потому что "и лишь несколько будут реализовать"
            Ответить

            • ava vov4a 07.09.2010 21:49 # −1

              сорри, туплю к ночи:)
              Ответить

              • ava Анонимус 07.09.2010 21:52 # −1

                На самом деле.. если уж совсем круто... то конечно надо делать его абстрактным.
                А потомделать наследника "Adapter" с пустыми реализациями всех методов.

                В крупных проектах так часто делают
                Ответить

    • ava nikelin 08.10.2010 15:00 # +1

      а-а! отправь на [email protected] :-D
      Ответить

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