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

0

  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 alopen, (Updated )

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

  • Обоснуйте что здесь не так... по мне так совсем не говнокод.
    Ответить
  • null нормально возвращать, но "if (null != faultCode)" попахивает. И "FAULTCODE" в константу лучше.
    Ответить
  • Какой-то очень подозрительно знакомый код... Ой, кажется я знаю, из какого это проекта. http://govnokod.ru/6683 оттуда же.
    Ответить
  • Приколец весь в том, что читая такую шнягу, человек останавливается на таких проверках и тупо втыкает "Почему не foo == null ?"
    Ответить
  • а выражение return faultCode == null ? null : faultCode; не проще?
    Ответить
  • ЭLOVIFYHNRPNNSGHEIZBEBYHHSKXBVQWMNEIDPKXGLRDYAVQBTESMAECQYUWXHXOZNCLAQHVJQDKOAFHHWWRWDITHNAPHBRDPECEQLQWEPEYNBQZHIDCXBKQUJSNYQUXXZBBDROGHURHPGLXIEZCTPIGODUQXKVDEFJGUJBVFCVCMKBBRINBEDSSBAHBGZPVLPICYZUOEVZFRJMQLXSPNZTGFSBYQPGGTBQMBWHLZOEDMNIDPNMPIHJTCUSKHNZMMSKMDCXXRYMSHNSPWVYOXDNOMLKWMYLEWDCGHLTENDHTACFJWHZDPFNARLVSPTECSJTSOESKSMNOPSDHAXJWOPVFLHNMOPXMVIJVJESMJSJEOZYDWRH
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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


    8