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

    −1

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if (userSettingsErrorType != null && userSettingsErrorType.getError() != null &&
				userSettingsErrorType.getError().getMessage() != null && userSettingsErrorType.getError().getMessage().getName() != null &&
				userSettingsErrorType.getError().getMessage().getName().length != 0) {
			try {
				getView().ifPresent(v -> v.setFieldErrorMessage(SettingsField.NAME,
				                                                userSettingsErrorType.getError().getMessage().getName()[0]));
			} catch (Exception e) {
				Crashlytics.logException(e);
			}
}

    наткнулся на сие чудо во время рефакторинга.

    Запостил: copatel, 28 Марта 2019

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

    • ava Steve_Brown 28.03.2019 18:08 # 0

      Тебе не нужен if
      advice_negro.jpg
      Если у тебя уже есть try
      Ответить

    • ava Ray_Mints 29.03.2019 22:04 # 0

      Жесть. Даже не уверен теперь, что хуже: то, что сейчас или если бы это запихнули в try-catch с проверкой на NullPointerException.
      Ответить

    • ava codershitter 29.09.2019 17:31 # 0

      https://www.youtube.com/watch?v=87hYatWSE1c
      Ответить

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