1. 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

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

    • ava Konardo 14.02.2014 15:37 # −16

      показать все, что скрытоЯ лабал твою мать на горбу твоего отца.
      Ответить

    • ava bormand 14.02.2014 15:39 # +2 

      return "Cache=" + (uint)key.GetHashCode();
      Как-то так?
      Ответить

    • ava anonimb84a2f6fd141 17.02.2014 17:50 # +1

      Напоминает баг с делением на -1
      Ответить

      • ava inkanus-gray 18.02.2014 16:54 # +2

        А что за баг? Я догадываюсь, что при делении на отрицательные числа разные платформы будут возвращать разный остаток (у кого-то он отрицательный, у кого-то — положительный), и что-то слышал про FDIV на первом Пентиуме, а про минус один ничего не помню.
        Ответить

        • ava anonimb84a2f6fd141 18.02.2014 17:37 # +2

          http://habrahabr.ru/post/179543/‎

          >Причем программа не требует никаких прав администратора и вызывает только одну функцию масштабирования окна.
          Слава богу, дрова видео в сперме вынесли в юзермод. Кто может отписаться, как сперма реагирует на падение видеодрайвера?

          Хотя это же не драйвер, это видеоподсистема.
          Ответить

          • ava inkanus-gray 19.02.2014 10:28 # 0

            Интересно.

            Кстати, http://habrahabr.ru/post/179543/‎/#comment_6233731
            Чем многозадачность OS/2 отличается от многозадачности Винды?
            Ответить

          • ava 3.14159265 19.02.2014 23:57 # 0

            Эти индусы совсем ничему не учатся.
            Иногда мне кажется что checked exceptions не такое и зло.
            Нормальным людям они конечно мешают.
            Но как иначе-то заставить говнокодеров читать спеку, доки по методам и проверять edge & corner case?
            Ответить

            • ava anonimb84a2f6fd141 20.02.2014 18:10 # −1

              Какое отношение checked exceptions имеют к runtime ошибкам? Насколько я помню, деление min_int на -1 относится к ним.
              Ответить

              • ava 3.14159265 20.02.2014 18:33 # 0

                Math.abs имел бы в сигнатуре исключение. Пусть даже runtime.
                Ответить

                • ava anonimb84a2f6fd141 20.02.2014 18:37 # −1

                  Да ну? Деление на ноль и OutOfMemoryError в жабе есть в исключениях?
                  Ответить

                  • ava 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.
                    Ответить

                    • ava anonimb84a2f6fd141 21.02.2014 03:36 # −1

                      >имел бы в сигнатуре исключение. Пусть даже runtime.
                      RuntimeException в сигнатуре не бывает, иначе во все места пришлось бы писать OutOfMemoryException & co.
                      Ответить

                      • ava bormand 21.02.2014 08:27 # 0

                        > OutOfMemoryException
                        А его вообще можно обработать?
                        Ответить

                        • ava roman-kashitsyn 21.02.2014 08:40 # 0

                          Нет такого исключения. Есть OutOfMemoryError. Отловить его можно (ловить можно всё, что Throwable), только не нужно. При непродуманной обработке исключений в многопоточном коде всё особенно грустно: воркер-треды могут дохнут от OOM, а основной тред может упорно продолжать заниматься ерундой, когда надо бы уже достойно умереть. У нас сейчас в проекте как раз такая проблема имеется.
                          Ответить

                      • ava 3.14159265 21.02.2014 13:04 # 0

                        >>RuntimeException в сигнатуре не бывает
                        ORLY?
                        Ответить

                        • ava anonimb84a2f6fd141 21.02.2014 16:35 # 0

                          А что, бывает? В любом случае, писать его абсолютно необязательно.
                          Ответить

                          • ava 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

                            Тем не менее их частенько пишут. Для документации - чтобы люди читали и корректно обрабатывали ошибки.
                            Ответить

                            • ava anonimb84a2f6fd141 21.02.2014 17:00 # 0

                              IllegalArgumentException это немножко другое, сечешь? Его бросает сам вызываемый код, а не среда.
                              Ответить

                              • ava 3.14159265 21.02.2014 17:01 # 0

                                >>RuntimeException в сигнатуре не бывает
                                >>IllegalArgumentException это немножко другое
                                class IllegalArgumentException extends RuntimeException
                                улавливаешь?
                                Ответить

                              • ava 3.14159265 21.02.2014 19:09 # 0

                                >>Его бросает сам вызываемый код, а не среда.
                                Какая среда? Сегодня пятница.

                                Чем т.н. среда (IDE?) отличается от вызываемого кода?
                                Ответить

            • ava anonimb84a2f6fd141 21.02.2014 16:37 # 0

              >Иногда мне кажется что checked exceptions не такое и зло.
              В коде, работающем с сокетами, в сигнатуре стоит IOError или как его там. Как определить, какой именно подкласс кидается, чтобы различать ошибки?
              Ответить

    • ava 3.14159265 18.02.2014 01:34 # +3

      Постили уже такое когда-то.
      Этот нюанс с Abs и в сырцах шарпа был проёбан.
      http://govnokod.ru/3117
      Если уж добавляете в свою жабу unsigned, то хоть используйте их. (Я о типе возврата Math.Abs)
      Ответить

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