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

    • Тебе не нужен if
      advice_negro.jpg
      Если у тебя уже есть try
      Ответить
    • Жесть. Даже не уверен теперь, что хуже: то, что сейчас или если бы это запихнули в try-catch с проверкой на NullPointerException.
      Ответить
    • https://www.youtube.com/watch?v=87hYatWSE1c
      Ответить

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