Говнокод #16915 — Java — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Java / Говнокод #16915

+121

if (Ints.contains(new int[] { 4, 5 }, statusCode / 100)) {
	// error response
} else {
	// success response
}

Насколько я знаю, Apache HTTP Client не содержит "официального" метода для определения категории кода состояния. Приходится так.

Запостил:

someone, 23 Октября 2014

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

Lure Of Chaos 23.10.2014 12:26 # −32

показать все, что скрытоа классический (statusCode>=400 && statusCode<600) - это слишком просто?
Ответить
- someone 23.10.2014 12:29 # −38
  
  показать все, что скрытоКстати, мысль!
  Ответить
  - Lure Of Chaos 23.10.2014 12:44 # −37
    
    показать все, что скрытособственно, имеет смысл конкретизировать реакцию кодов, благо осмысленных (при адекватном сервере) совсем немного: 200 - успех, 403,404,500 - явный фейл.
    т.к. непонятно, код, например, 206 Partial Content - успех или не надо продолжать, и как реагировать на коды 3xx, по-хорошему.
    Ответить
    - Qwertiy 23.10.2014 14:17 # −39
      
      показать все, что скрыто206 - это как 200. Ты сказал серверу дай вот это с 123 байта по 412, он ответил 206 - держи.
      Ответить
- Qwertiy 23.10.2014 14:17 # −39
  
  показать все, что скрытоА зачем <600?
  Ответить
  - Lure Of Chaos 23.10.2014 14:21 # −38
    
    показать все, что скрытосогласно топик-коду.
    Ответить
- TarasB 24.10.2014 13:55 # −36
  
  показать все, что скрытоstatusCode/200==2
  Ответить
  - Lure Of Chaos 24.10.2014 14:41 # −36
    
    показать все, что скрытоумно, но неочевидно )
    это как использовать сдвиги вместо деления ))
    Ответить
    - TarasB 24.10.2014 18:33 # −36
      
      показать все, что скрытоСдвиги тут ни при чём, потому что такая запись нифига не ускоряет по сравнению с двумя сравнениями (а в выхлопе компилятора - одно), делить всяко дольше.
      Это просто нахуй не усравшаяся синтаксическая экономия, запощенная мной по приколу.
      Но кулхацкеры, сишники и прочие долбоёбы должны одобрить, они любят, когда буковок меньше на ту же конструкцию.
      Ответить
Lure Of Chaos 23.10.2014 12:29 # −33

показать все, что скрытоа вообще, уже не раз обсуждалось тут, какие коды - успех и какие - провал, и как их разделять. и сюда же http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
Ответить
myaut 23.10.2014 15:34 # −26
показать все, что скрыто
А по индусски это-бы можно было записать так:
```
String httpStatusCodes[] = {"one", "two", "three", "four", "five"};
String statusCodeString = httpStatusCodes[statusCode / 100];

if(statusCodeString.length() == 4) {
   // error
}
```
Ответить
- inkanus-gray 24.10.2014 01:00 # −25
  
  показать все, что скрытоАнгличане как в воду глядели, когда придумывали названия чисел.
  Ответить
anonimb84a2f6fd141 23.10.2014 18:53 # −22

показать все, что скрытоВъебал минус, чтоб неповадно было пиздой вилять тут.
Ответить

Говнокод: по колено в коде.

Java / Говнокод #16915

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

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