Говнокод #14251 — Java — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

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

+71

@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

taburetka 20.12.2013 18:51 # −1

Глядя на эту "линейку" входных параметров, я тоже вспомнил как один парень из одной контры на заре карьеры, с упорством барана доказывал мне что в метод и по 100 параметров можно передавать. Главным аргументом было: "Но ведь работает же!". Там кстати тоже были vatAmountTo и withVatAmountTo.
Ответить
- floppy 20.12.2013 18:56 # +1
  
  я плакал кровью... аргумент кстати был тот же
  Ответить
3.14159265 20.12.2013 19:19 # 0

Сам код вроде ничего. Сигнатура метода безусловно говно.
Ответить
- WGH 21.12.2013 15:49 # −3
  
  Но в этом виноват по большей части язык. Были бы именованные параметры хотя бы, проблемы бы не было. А так имеем что имеем — либо жесть из поста, либо билдеры билдеров фабрик билдеров.
  Ответить
  - anonimb84a2f6fd141 21.12.2013 16:06 # −2
    
    Прочел комент в стоке и понял, что речь о яве.
    Ответить
  - 3.14159265 21.12.2013 16:59 # +2
    
    >А так имеем что имеем — либо жесть из поста
    Либо передача парметром структуры (POJO).
    Копировать столько данных через стек - однозначное западло, вне зависимости от именованных параметров и языка.
    Ответить
    - Stertor 21.12.2013 20:55 # +2
      
      >>передача парметром структуры
      +1
      Всегда так делаю в дельфях. А вообще, года 4 назад прочел у msrem-a, что в целях выйгрыша в скорости целесообразнее передавать в процедуру или функцию не более 3-4 параметров. Асм я так хорошо не знаю, но могу предположить, что в этом случае не будут задействованы дополнительные регистры.
      Ответить
      - kegdan 21.12.2013 21:05 # 0
        
        Тут не асм, тут jbc. В данном случае количество роли не играет
        Ответить
      - anonimb84a2f6fd141 21.12.2013 21:08 # +2
        
        В делфе структуры по значению, а в жавке по значению только примитивные типы.
        Ответить
        
        kegdan 21.12.2013 21:12 # −1
        
        Проще - в жабе нет структур
        Ответить
        
        anonimb84a2f6fd141 21.12.2013 22:05 # 0
        
        В жабе есть почти структуры, если все типы в них - примитивные (численные/булевы)
        Ответить
        
        3.14159265 21.12.2013 22:13 # +2
        
        Уточнение в скобках (POJO) было специально для любителей придраться к букве.
        Ответить
        
        bormand 21.12.2013 21:45 # +2
        
        > В делфе структуры по значению
        Не всегда, можно поюзать var или const, чтобы передать по ссылке.
        Ответить
      - 3.14159265 21.12.2013 21:45 # +1
        
        >целесообразнее передавать в процедуру или функцию не более 3-4 параметров
        А Вы свой человек, я посмотрю.
        Ответить
        
        Stertor 21.12.2013 22:31 # +1
        
        Спасибо.
        Ответить
- kegdan 21.12.2013 20:30 # −1
  
  А нельзя было какой -нибудь структурой все оформить и передавать листом, по которому будут выполняться инструкции?
  
  Ну то есть передается набор команд, которые выполняются в зависимости от их контекста.
  Ответить
  - guest 21.12.2013 20:39 # −1
    
    конечно можно, и даже нужно... но поди объясни
    Ответить
guest 20.12.2013 19:32 # −1

по мне 2, 3 параметра максимум, дальше тока каша получается.
Этот метод тоже каша, причем полная.
Ответить
anonimb84a2f6fd141 21.12.2013 16:08 # −1

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

Это точно жава?
Ответить
- guest 21.12.2013 17:19 # 0
  
  groovy
  Ответить
kegdan 21.12.2013 20:54 # −1

сначала прочитал crаteria вместо criteria, подумал чувак в метроида переиграл)
Ответить
Lure Of Chaos 22.12.2013 13:11 # +1

groovy + hibernate здесь вижу я
Ответить
- roman-kashitsyn 22.12.2013 15:25 # +1
  
  grails небось
  Ответить

Говнокод: по колено в коде.

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

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

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