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

    +78

    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
    public void setDomainSearchFilter(String domainSearchFilter) {
	if (this.domainSearchFilter == null)
	{
		if (domainSearchFilter != null)
		{
			this.domainSearchFilter = domainSearchFilter;
		}

		return;
	}

	if (domainSearchFilter == null)
	{
		this.domainSearchFilter = null;
		return;
	}

	if (!this.domainSearchFilter.equals(domainSearchFilter))
	{
		this.domainSearchFilter = domainSearchFilter;
	}
}

    Запостил: voice, 24 Января 2011

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

    • ava gegMOPO4 24.01.2011 20:55 # 0

      Можно сократить до одного if-а
      if (this.domainSearchFilter == null ||
            domainSearchFilter == null ||
            !this.domainSearchFilter.equals(domainSearchFilter))
        {
                this.domainSearchFilter = domainSearchFilter;
        }

      --
      Ваш Капитан
      Ответить

      • ava voice 24.01.2011 21:15 # −1

        тогда уже так:
        if (domainSearchFilter != null || !domainSearchFilter.equals(this.domainSe archFilter)) {
        this.domainSearchFilter = domainSearchFilter;
        }
        Ответить

        • ava gegMOPO4 24.01.2011 21:22 # 0

          Так нельзя. Ошибка в !=.
          Ответить

          • ava voice 24.01.2011 21:26 # 0

            М.... почему же?
            Ответить

            • ava gegMOPO4 24.01.2011 21:38 # 0

              domainSearchFilter == null
              Ответить

              • ava voice 24.01.2011 21:41 # 0

                Что-то я Вас не понял. Чем следующий код не корректный?
                domainSearchFilter != null
                Ответить

                • ava gegMOPO4 24.01.2011 21:57 # 0

                  Подставьте domainSearchFilter равным null.
                  Ответить

                  • ava voice 24.01.2011 22:26 # 0

                    Ой, извиняюсь, я вместо || хотел && написать.
                    Ответить

                    • ava gegMOPO4 24.01.2011 22:40 # 0

                      Подставьте domainSearchFilter равным null. При this.domainSearchFilter не равному null.

                      Признайтесь, вы автор говнокода?
                      Ответить

                      • ava voice 24.01.2011 22:44 # 0

                        > Признайтесь, вы автор говнокода?
                        Нет. :) Я этот проект даже в глаза не видел. Сотрудник нашел в одном проекте. К сожалению, автор мне не известен.
                        Ответить

                        • ava gegMOPO4 24.01.2011 23:04 # 0

                          Ну, код-то рабочий, но вот с булевской алгеброй автор не знаком. Можно сократить, если нечего делать, можно так оставить.
                          Ответить

    • ava tir 25.01.2011 13:36 # 0

      аналог #5340
      Ответить

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