Jawa / Говнокод #5377 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 voice, (Updated )

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

  • ava gegMOPO4 # Ссылка на оригинал 0

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

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

    • ava voice # Ссылка на оригинал 0 ⇈

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

            • ava voice # Ссылка на оригинал 0 ⇈

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

                  • ava gegMOPO4 # Ссылка на оригинал 0 ⇈

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

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

                    • ava voice # Ссылка на оригинал 0 ⇈

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

                      • ava gegMOPO4 # Ссылка на оригинал 0 ⇈

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

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓