Говнокод #14582 — C# — Говнокод.ру

C# / Говнокод #14582

+136
1. 1
2. 2
3. 3
4. 4
5. 5
```
public static string CreateBrowserCacheExtension(object key)
{
    //...Остальной код опущен для ясности
    return "Cache=" + Math.Abs(key.GetHashCode());
}
```
Legacy code из проекта над которым я работаю.
Косяк в том, что GetHashCode() иногда возвращает значение, равное System.Int32.MinValue.
А это в свою очередь приводит к OverflowException, в случае с Math.Abs(...);

Запостил: pikowatt, 14 Февраля 2014

Tweet
Комментарии (27) RSS
- Konardo 14.02.2014 15:37 # −16
  
  показать все, что скрытоЯ лабал твою мать на горбу твоего отца.
  Ответить
- bormand 14.02.2014 15:39 # +2
  return "Cache=" + (uint)key.GetHashCode();
  Как-то так?
  Ответить
  - Konardo 14.02.2014 15:40 # −15
    
    показать все, что скрытоПропаганда гомосексуализма.
    Ответить
  - anonimb84a2f6fd141 18.02.2014 19:05 # −1
    
    А что возвращает-то GetHashCode() и что Math.Abs()?
    Ответить
- anonimb84a2f6fd141 17.02.2014 17:50 # +1
  
  Напоминает баг с делением на -1
  Ответить
  - inkanus-gray 18.02.2014 16:54 # +2
    
    А что за баг? Я догадываюсь, что при делении на отрицательные числа разные платформы будут возвращать разный остаток (у кого-то он отрицательный, у кого-то — положительный), и что-то слышал про FDIV на первом Пентиуме, а про минус один ничего не помню.
    Ответить
    - anonimb84a2f6fd141 18.02.2014 17:37 # +2
      
      http://habrahabr.ru/post/179543/‎
      
      >Причем программа не требует никаких прав администратора и вызывает только одну функцию масштабирования окна.
      Слава богу, дрова видео в сперме вынесли в юзермод. Кто может отписаться, как сперма реагирует на падение видеодрайвера?
      
      Хотя это же не драйвер, это видеоподсистема.
      Ответить
      
      inkanus-gray 19.02.2014 10:28 # 0
      
      Интересно.
      
      Кстати, http://habrahabr.ru/post/179543/‎/#comment_6233731
      Чем многозадачность OS/2 отличается от многозадачности Винды?
      Ответить
      
      3.14159265 19.02.2014 23:57 # 0
      
      Эти индусы совсем ничему не учатся.
      Иногда мне кажется что checked exceptions не такое и зло.
      Нормальным людям они конечно мешают.
      Но как иначе-то заставить говнокодеров читать спеку, доки по методам и проверять edge & corner case?
      Ответить
      
      anonimb84a2f6fd141 20.02.2014 18:10 # −1
      
      Какое отношение checked exceptions имеют к runtime ошибкам? Насколько я помню, деление min_int на -1 относится к ним.
      Ответить
      
      3.14159265 20.02.2014 18:33 # 0
      
      Math.abs имел бы в сигнатуре исключение. Пусть даже runtime.
      Ответить
      
      anonimb84a2f6fd141 20.02.2014 18:37 # −1
      
      Да ну? Деление на ноль и OutOfMemoryError в жабе есть в исключениях?
      Ответить
      
      3.14159265 21.02.2014 02:07 # 0
      
      Как и в сишке в жабе вообще нет исключений по переполнению. Иногда это мешает. Лучше конечно было иметь возможность включать/отключать сиё.
      >Деление на ноль
      Thrown when an exceptional arithmetic condition has occurred. For example, an integer "divide by zero" throws an instance of this class.
      http://docs.oracle.com/javase/7/docs/api/java/lang/ArithmeticException.html
      Оно не checked.
      Ответить
      
      anonimb84a2f6fd141 21.02.2014 03:36 # −1
      
      >имел бы в сигнатуре исключение. Пусть даже runtime.
      RuntimeException в сигнатуре не бывает, иначе во все места пришлось бы писать OutOfMemoryException & co.
      Ответить
      
      bormand 21.02.2014 08:27 # 0
      
      > OutOfMemoryException
      А его вообще можно обработать?
      Ответить
      
      roman-kashitsyn 21.02.2014 08:40 # 0
      
      Нет такого исключения. Есть OutOfMemoryError. Отловить его можно (ловить можно всё, что Throwable), только не нужно. При непродуманной обработке исключений в многопоточном коде всё особенно грустно: воркер-треды могут дохнут от OOM, а основной тред может упорно продолжать заниматься ерундой, когда надо бы уже достойно умереть. У нас сейчас в проекте как раз такая проблема имеется.
      Ответить
      
      anonimb84a2f6fd141 21.02.2014 16:35 # 0
      
      >Есть OutOfMemoryError
      Ай-яй-яй, подловил :)
      Ответить
      
      3.14159265 21.02.2014 13:04 # 0
      
      >>RuntimeException в сигнатуре не бывает
      ORLY?
      Ответить
      
      anonimb84a2f6fd141 21.02.2014 16:35 # 0
      
      А что, бывает? В любом случае, писать его абсолютно необязательно.
      Ответить
      
      3.14159265 21.02.2014 16:51 # 0
      
      >> RuntimeException в сигнатуре не бывает
      >>писать его абсолютно необязательно.
      Это немного разные вещи. Я бы сказал даже противоречивые.
      http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/java/lang/Integer.java#Integer.valueOf%28java.lang.String%29
      
      Тем не менее их частенько пишут. Для документации - чтобы люди читали и корректно обрабатывали ошибки.
      
      Ответить
      
      anonimb84a2f6fd141 21.02.2014 17:00 # 0
      
      IllegalArgumentException это немножко другое, сечешь? Его бросает сам вызываемый код, а не среда.
      Ответить
      
      3.14159265 21.02.2014 17:01 # 0
      
      >>RuntimeException в сигнатуре не бывает
      >>IllegalArgumentException это немножко другое
      class IllegalArgumentException extends RuntimeException
      улавливаешь?
      Ответить
      
      anonimb84a2f6fd141 21.02.2014 18:28 # 0
      
      Это подкласс, но это немного другое.
      Ответить
      
      3.14159265 21.02.2014 19:09 # 0
      
      instanceof RuntimeException ==true
      Ответить
      
      3.14159265 21.02.2014 19:09 # 0
      
      >>Его бросает сам вызываемый код, а не среда.
      Какая среда? Сегодня пятница.
      
      Чем т.н. среда (IDE?) отличается от вызываемого кода?
      Ответить
      
      anonimb84a2f6fd141 21.02.2014 16:37 # 0
      
      >Иногда мне кажется что checked exceptions не такое и зло.
      В коде, работающем с сокетами, в сигнатуре стоит IOError или как его там. Как определить, какой именно подкласс кидается, чтобы различать ошибки?
      Ответить
- 3.14159265 18.02.2014 01:34 # +3
  
  Постили уже такое когда-то.
  Этот нюанс с Abs и в сырцах шарпа был проёбан.
  http://govnokod.ru/3117
  Если уж добавляете в свою жабу unsigned, то хоть используйте их. (Я о типе возврата Math.Abs)
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

C# / Говнокод #14582

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

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