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

    +77

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    protected String getResponseFaultCode(Map<String, Object> response) {
		Object faultCode = response.get("FAULTCODE");
		if (null != faultCode){
			return faultCode.toString();
			}	
		return null;
	}

    Запостил: alopen, 25 Мая 2011

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

    • ava inser 25.05.2011 12:31 # 0

      Обоснуйте что здесь не так... по мне так совсем не говнокод.
      Ответить

    • ava Lure Of Chaos 25.05.2011 15:40 # −1

      и по мне тоже
      Ответить

    • ava absolut 25.05.2011 15:58 # 0

      лучше б пустую строку вернуть, чем null
      Ответить

      • ava guest 25.05.2011 16:00 # −2

        fucked she's hand?
        Ответить

      • ava Lure Of Chaos 25.05.2011 16:05 # 0

        и раз уж возвращаем код, неплохо бы попытаться извлечь число.
        ну да ладно, это вопрос архитектуры
        Ответить

    • ava vov4a 25.05.2011 15:59 # 0

      null нормально возвращать, но "if (null != faultCode)" попахивает. И "FAULTCODE" в константу лучше.
      Ответить

      • ava Lure Of Chaos 25.05.2011 16:06 # 0

        чем попахивает? как раз нормально, что бы не издеваться над null
        Ответить

    • ava Skipy 25.05.2011 18:29 # 0

      Какой-то очень подозрительно знакомый код... Ой, кажется я знаю, из какого это проекта. http://govnokod.ru/6683 оттуда же.
      Ответить

    • ava guest 25.05.2011 18:59 # +1

      Приколец весь в том, что читая такую шнягу, человек останавливается на таких проверках и тупо втыкает "Почему не foo == null ?"
      Ответить

      • ava Lure Of Chaos 25.05.2011 19:14 # 0

        ну, наверное, по привычке в бейсике не дописывать одно =
        Ответить

    • ava guest 26.05.2011 02:40 # +1

      а выражение return faultCode == null ? null : faultCode; не проще?
      Ответить

    • ava guest8 08.04.2019 20:50 # −999

      показать все, что скрытоvanished
      Ответить

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