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

    +71

    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
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    @Override
    List search(Long ownerId, Long projectId, String docnumber, String ctr1, String ctr2, Long dateFrom, Long dateTo, String contract,
                Double amountFrom, Double amountTo, Double vatAmountFrom, Double vatAmountTo, Double withVatAmountFrom,
                Double withVatAmountTo, Boolean defect, DocumentDefect d, Long vatId, Integer limit, String sortField, String order, String docType) {

        // доступные проекты
        List projects = projectDAO.findAll(ownerId)
        
        if(!projects){
            logger.warn("Ошибка отображения списка документов: нет доступных проектов: ownerId:$ownerId")
            return []
        }
        
        Criteria criteria = currentSession.createCriteria(DocumentView)
            .createAlias("document", "d")//
            .createAlias("d.project", "p")//
            .add(Restrictions.in("p.id", projects.id))//

//            if(projectId)
//                criteria.add(Restrictions.eq("d.project.id", projectId))

        if (docnumber)
            criteria.add(Restrictions.ilike("docNumber", docnumber, MatchMode.ANYWHERE))
        if (ctr1)
            criteria.add(Restrictions.ilike("supplierName", ctr1, MatchMode.ANYWHERE))
        if (ctr2)
            criteria.add(Restrictions.ilike("customerName", ctr2, MatchMode.ANYWHERE))
        if (dateFrom) {
            Calendar date1 = Calendar.instance
            date1.setTime(new Date(dateFrom))
            criteria.add(Restrictions.ge("docDate", date1))
        }
        if (dateTo) {
            Calendar date1 = Calendar.instance
            date1.setTime(new Date(dateTo))
            date1.add(Calendar.DAY_OF_MONTH, 1)
            criteria.add(Restrictions.lt("docDate", date1))
        }
        if (contract)
            criteria.add(Restrictions.ilike("contract", contract, MatchMode.ANYWHERE))
        if (amountFrom)
            criteria.add(Restrictions.ge("amount", amountFrom))
        if (amountTo)
            criteria.add(Restrictions.le("amount", amountTo))
        if (vatAmountFrom)
            criteria.add(Restrictions.ge("vatAmount", vatAmountFrom))
        if (vatAmountTo)
            criteria.add(Restrictions.le("vatAmount", vatAmountTo))
        if (withVatAmountFrom)
            criteria.add(Restrictions.ge("totalAmount", withVatAmountFrom))
        if (withVatAmountTo)
            criteria.add(Restrictions.le("totalAmount", withVatAmountTo))
        if (defect) {
            criteria.add(Restrictions.isNotNull("defect"))
            if (d)
                criteria.add(Restrictions.eq("defect", d))
        }
//        if (vatId)
//            criteria.add(Restrictions.eq("d.", withVatAmountTo))

        if (docType) {
            def dt = DocumentType.values().find { it.link == docType || it.code == docType }
            if (dt)
                criteria.add(Restrictions.like("docLink", dt.code, MatchMode.START))
            else
                logger.error("Неправильное значение параметра фильтрации по виду документа [d_t:$docType]. Допустимые значения ${DocumentType.values().code}")
        }

        // получаем общее кол-во записей без ограничений по странице
        def rowCount = criteria.setProjection(Projections.rowCount()).uniqueResult() as Integer

        List result = criteria.setProjection(null)//
                .setResultTransformer(Criteria.ROOT_ENTITY)//
                .addOrder(Order."$order"(sortField))//
                .setMaxResults(limit)//
                .list()

        [rowCount, result]

    }

    поиск ?!

    Запостил: floppy, 20 Декабря 2013

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

    • ava taburetka 20.12.2013 18:51 # −1

      Глядя на эту "линейку" входных параметров, я тоже вспомнил как один парень из одной контры на заре карьеры, с упорством барана доказывал мне что в метод и по 100 параметров можно передавать. Главным аргументом было: "Но ведь работает же!". Там кстати тоже были vatAmountTo и withVatAmountTo.
      Ответить

      • ava floppy 20.12.2013 18:56 # +1

        я плакал кровью... аргумент кстати был тот же
        Ответить

    • ava 3.14159265 20.12.2013 19:19 # 0

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

      • ava WGH 21.12.2013 15:49 # −3

        Но в этом виноват по большей части язык. Были бы именованные параметры хотя бы, проблемы бы не было. А так имеем что имеем — либо жесть из поста, либо билдеры билдеров фабрик билдеров.
        Ответить

        • ava anonimb84a2f6fd141 21.12.2013 16:06 # −2

          Прочел комент в стоке и понял, что речь о яве.
          Ответить

        • ava 3.14159265 21.12.2013 16:59 # +2

          >А так имеем что имеем — либо жесть из поста
          Либо передача парметром структуры (POJO).
          Копировать столько данных через стек - однозначное западло, вне зависимости от именованных параметров и языка.
          Ответить

          • ava Stertor 21.12.2013 20:55 # +2

            >>передача парметром структуры
            +1
            Всегда так делаю в дельфях. А вообще, года 4 назад прочел у msrem-a, что в целях выйгрыша в скорости целесообразнее передавать в процедуру или функцию не более 3-4 параметров. Асм я так хорошо не знаю, но могу предположить, что в этом случае не будут задействованы дополнительные регистры.
            Ответить

            • ava kegdan 21.12.2013 21:05 # 0

              Тут не асм, тут jbc. В данном случае количество роли не играет
              Ответить

            • ava anonimb84a2f6fd141 21.12.2013 21:08 # +2

              В делфе структуры по значению, а в жавке по значению только примитивные типы.
              Ответить

              • ava kegdan 21.12.2013 21:12 # −1

                Проще - в жабе нет структур
                Ответить

                • ava anonimb84a2f6fd141 21.12.2013 22:05 # 0

                  В жабе есть почти структуры, если все типы в них - примитивные (численные/булевы)
                  Ответить

                • ava 3.14159265 21.12.2013 22:13 # +2

                  Уточнение в скобках (POJO) было специально для любителей придраться к букве.
                  Ответить

              • ava bormand 21.12.2013 21:45 # +2

                > В делфе структуры по значению
                Не всегда, можно поюзать var или const, чтобы передать по ссылке.
                Ответить

            • ava 3.14159265 21.12.2013 21:45 # +1

              >целесообразнее передавать в процедуру или функцию не более 3-4 параметров
              А Вы свой человек, я посмотрю.
              Ответить

      • ava kegdan 21.12.2013 20:30 # −1

        А нельзя было какой -нибудь структурой все оформить и передавать листом, по которому будут выполняться инструкции?

        Ну то есть передается набор команд, которые выполняются в зависимости от их контекста.
        Ответить

        • ava guest 21.12.2013 20:39 # −1

          конечно можно, и даже нужно... но поди объясни
          Ответить

    • ava guest 20.12.2013 19:32 # −1

      по мне 2, 3 параметра максимум, дальше тока каша получается.
      Этот метод тоже каша, причем полная.
      Ответить

    • ava anonimb84a2f6fd141 21.12.2013 16:08 # −1

      .addOrder(Order."$order"(sortField))//

      Это точно жава?
      Ответить

    • ava kegdan 21.12.2013 20:54 # −1

      сначала прочитал crаteria вместо criteria, подумал чувак в метроида переиграл)
      Ответить

    • ava Lure Of Chaos 22.12.2013 13:11 # +1

      groovy + hibernate здесь вижу я
      Ответить

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