1. C# / Говнокод #6684

    +128

    1. 1
    long.Parse(Convert.ToString(Convert.ToSingle(Item.Value.ToString())))

    Запостил: wiz, 17 Мая 2011

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

    • ava abatishchev 17.05.2011 14:16 # 0

      убейте его кто-нибудь!
      Ответить

    • ava Lure Of Chaos 17.05.2011 14:27 # 0

      а какой тип Item.Value?
      вот весело, если там long
      Ответить

      • ava wiz 17.05.2011 14:30 # 0

        а внутри у него неонка object.
        Ответить

        • ava Lure Of Chaos 17.05.2011 14:33 # 0

          тогда я присоединяюсь к непонимающим, зачем нужна передергивающая конвертация object->string->single->string->long вместо object->string->long.
          Ответить

          • ava wiz 17.05.2011 14:37 # 0

            потому что без сингла кидает exception, что число в неправильном формате.
            Ответить

            • ava Lure Of Chaos 17.05.2011 14:40 # 0

              Convert.ToLong(), не кидающий исключение, существует?
              Ответить

              • ava wiz 17.05.2011 14:45 # 0

                Неа. Там число может быть дробным и разделитель точка/запятая зависит от локали на сервере, что само пос ебе прекрасно.
                Ответить

                • ava Lure Of Chaos 17.05.2011 14:52 # 0

                  ебе прекрасно!
                  мелкософт, как всегда, прекрасен, что бы конвертировать в строку в зависимости от локали.

                  на Java я бы сделал так:
                  long l=0;
                  try { l=Long.parseLong(String.valueOf(obj));} catch(NumberFormatException e) {Log.log(e);}
                  Ответить

                  • ava wiz 17.05.2011 14:53 # +1

                    Ну exception то поймать не проблема. Но надо извернуться чтобы работало на всей фигне кроме совсем нечисловых значений. Сцука энтерпрайз...
                    Ответить

                    • ava Lure Of Chaos 17.05.2011 14:57 # 0

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

                      • ava wiz 17.05.2011 14:58 # 0

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

                        • ava Lure Of Chaos 17.05.2011 15:38 # 0

                          спасибо мелкософту за костыли!
                          Ответить

                        • ava glilya 28.05.2011 21:43 # 0

                          значит юи недопилили - что там такое вводитися что так приходится парсить ?
                          Ответить

                          • ava wiz 28.05.2011 21:45 # 0

                            Это сервера между собой обмениваются. С учётом настроек системной локали блеять. Локали могут быть (и таки бывают) разные.
                            Ответить

                            • ava glilya 28.05.2011 22:06 # 0

                              да и хер бы сней если оно между флоат и лонгом то максимум один труексепшн и усе. Да и потом не сильно они отличабтся эти "локали блеать".
                              Ответить

                              • ava wiz 28.05.2011 22:08 # 0

                                Дык в том-то и дело, что на входе упакованый в object стринг, в который сериализовалось значение с учётом локали.
                                Ответить

                                • ava glilya 28.05.2011 22:10 # 0

                                  протокол предачи иксмл ? я для соапа просто класс накатал который все сам проверял и выдавал
                                  Ответить

                                  • ava wiz 28.05.2011 22:13 # 0

                                    Протокол передачи тупотекст... Впрочем, недолго ему жить осталось (надеюсь) и мы свалим наконец на няшненький питончик, где такая хуйня рядом не лежала.
                                    Ответить

                                    • ava glilya 28.05.2011 22:19 # 0

                                      Правильно, сам люблю питон потомучно нанем много поточные сервера и протоколы легко пишустя. И все таки какой протокол планируете использовать (иксмл, жисон ...)
                                      Ответить

                                      • ava wiz 28.05.2011 22:24 # 0

                                        Для умеренно-сложных данных мне больше всего симпатичен JSON.

                                        Через XML легко делать запросы, но принимать его и парсить - очень муторно, селекторы-фигекторы...

                                        А вообще надо конечно пользоваться минимально-достаточной штукой. Но и не забывать думать на пару шагов вперёд. Тоесть можно тупо делать urlencode, но если есть возможность не-плоских данных, то проще сразу забиться на JSON и не трахать в будущем мозги.
                                        Ответить

                    • ava wvxvw 17.05.2011 16:06 # 0

                      А потом кто-то номер телефона вместо суммы прописью впишет...
                      Ответить

                      • ava glilya 28.05.2011 21:44 # +1

                        херня, за доп деньги можно и так организовать
                        Ответить

              • ava glilya 28.05.2011 21:42 # 0

                тру парсе есть же
                Ответить

    • ava inser 17.05.2011 16:19 # 0

      Код говно. Вообще не понимаю, зачем всё это нужно?
      Раз уж на то пошло, и на CultureInfo насрать, то это всё делается так:
      long val = 0;
if(long.TryParse(Item.Value.ToString(), out val)) {
  //всё ок
}
else {
  //введи нормальное число, падла!
}


      А вообще, конечно же, надо использовать System.Globalization.NumberStyles и System.Globalization.CultureInfo.
      Ответить

      • ava Lure Of Chaos 17.05.2011 16:40 # 0

        глобализация?
        System.Globalization.GoldenBillion
        Ответить

        • ava inser 17.05.2011 16:44 # +1

          Думаю, те программисты, что пишут такой код, не входят в Golden Billion.
          Ответить

          • ava wiz 17.05.2011 16:47 # 0

            Входят. Иначе бы они не планктонили в своих офисах-аквариумах выдавая такой код.
            Ответить

            • ava Lure Of Chaos 17.05.2011 18:02 # 0

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

              • ava bugmenot 17.05.2011 18:25 # 0

                Каннибализм в среде разработчиков побежден! По крайней мере, мы держим проблему под контролем.
                Ответить

                • ava Lure Of Chaos 17.05.2011 19:21 # 0

                  офисный планктон - это те люди, которые не добиваются хороших результатов. Они отдают успех(и деньги) другим
                  Ответить

    • ava koodeer 17.05.2011 18:03 # +2

      В MSDN английским по белому написано, что обязательно следует использовать настройки глобализации при работе со строками! Анализатор кода в Студии выдаёт предупреждения, если таковых нет. Развели тут сыр-бор...
      Ответить

    • ava abatishchev 17.05.2011 19:49 # 0

      Если до упаковки в object, тип был long, то он им и остается: 
      Item.Value as long? ?? default(long);
      и только так :Р
      если это string, то 
      long.TryParse((string)Item.Value, out l);
      Ответить

    • ava guest8 08.04.2019 20:48 # −999

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

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