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

    +70

    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
    Map<Integer, ArrayList<?>> // filterId : filterParams
{
	//
	// Фильтр занятости квартир (не вписывается в API, поэтому здесь заведены отдельные методы)
	//
	
    // хранит: showReserved, DateFrom, DateTo
    private ArrayList<?> reservedFilterParams

    public void onDateFilterChanged(Date dateFrom, Date dateTo, boolean showReserved)
    {
	this.put(QuickFilters.FILTER_RESERVED_ID, reservedFilterParams)
		
	reservedFilterParams = ArrayUtils.getOrCreateArrayListWithElements(
			this, QuickFilters.FILTER_RESERVED_ID, new Boolean(showReserved), dateFrom, dateTo )
	
	if (dateFrom == null && dateTo == null)
        {
            reservedFilterParams.set(0,new Boolean(true))
            this.remove(QuickFilters.FILTER_RESERVED_ID)
        }
    }

    public boolean doShowReserved()
    {
        return reservedFilterParams ? reservedFilterParams.get(0).booleanValue() : true
    }

    public Date getDateFrom()
    {
        return reservedFilterParams?.get(1)
    }

    public Date getDateTo()
    {
        return reservedFilterParams?.get(2)
    }

}

    Язык Groovy, стиль - моя переходная форма от Java. Это я сам написал. Вообще мне даже нравится. Есть фильтры для поиска. У каждого фильтра есть только ему ведомые параметры. Приведенный код - это мапка для хранения того, какие пользователь выбрал фильтры и что в них установил.

    Есть API который я делаю, состоящий из типовых фильтров. Этот API автоматически юзает мапку через get/put. Но бывает, придумывается какой-то нетиповой фильтр, и его параметры приходится хардкодить в этой мапке. Короче, мне понравилась идея с мапкой, ведь даже если параметры захардкодены, то они всё равно пихаются в мапку. И читаемость для таких случаев навысоте.

    Запостил: Bugdog, 20 Октября 2010

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

    • ava sectus 21.10.2010 04:19 # 0

      Надо чё? Сюда с какой целью постили?
      Ответить

      • ava Bugdog 21.10.2010 08:44 # 0

        Всё-таки можно посмеяться над reservedFilterParams и тем как с ним обращается метод - onDateFilterChanged (имею ввиду put/remove). Да, там тонкая идея, чтобы список не пересоздавался после того как его могут вынуть из мапки.
        Ответить

    • ava sectus 21.10.2010 08:04 # 0

      ЗЫ. А про "Стратегию" вы слышали?
      Ответить

      • ava Bugdog 21.10.2010 08:54 # 0

        Да, но как правило это использую когда сразу есть какой-то общий утилитарный алгоритм из предметной области.
        Ответить

    • ava Vermut 05.11.2010 18:13 # 0

      Поржать можно уже над самим форматированием и коментариями, автор ни разу не читал ни соглашения от sun ни исходников профессиональных либ, тем более смешно, что современные IDE и автоформатирование и javadoc поддерживают из коробки. ArrayList<?> равнозначно просто ArrayList, и ни чем не обоснована такое ранне связываниек, достаточно просто List
      Ответить

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