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

    +122

    1. 1
    2. 2
    3. 3
    int valu=...;
...
string vals=""+valu;

    Запостил: Говногость, 28 Декабря 2010

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

    • ava Lure Of Chaos 28.12.2010 15:51 # 0

      самодокументированный код, однако
      Ответить

    • ava Dummy00001 28.12.2010 17:19 # +1

      Валс? Валю?? прибалты к нашим дивчонкам клеются!!!
      Ответить

      • ava Lure Of Chaos 28.12.2010 18:14 # 0

        а вы видели прибалтийских девушек? моментом все желание отклеивается
        вот и клеются на более сладкое )
        Ответить

    • ava burdakovd 28.12.2010 19:05 # −1

      В данном случае бред, но для преобразования int в float
      int x, y;

float z = x * 1.0f / y;
// короче и понятнее, чем
float z = static_cast<float>(x) / y;
      Ответить

      • ava bugmenot 28.12.2010 22:31 # 0

        Где здесь C#?
        Ответить

        • ava burdakovd 28.12.2010 22:37 # 0

          Специфики C# в исходном посте нет, конкатенировать строку с числом много где можно.
          Смысл моего сообщения в том, что иногда вместо явного преобразования типов (static_cast/SomeType.valueOf/...) проще и понятнее применять какую-то операцию, которая неявно приведёт аргумент к нужному типу. (например <<+ "">>, или <<* 1.0>>)

          Хотя именно преобразование в строку таким образом выглядит наиболее неприятно
          Ответить

          • ava bugmenot 28.12.2010 22:54 # +1

            неприятно, но придется терпеть?
            valu.toString() же!
            (в басиках не силен, возможно существует способ еще приличнее выразить то, чего хочется)
            Ответить

      • ava istem 28.12.2010 23:00 # +1

        Извините, но для преобразования int во float советую брять пример с http://govnokod.ru/5090
        Ответить

    • ava eigenein 28.12.2010 19:33 # 0

      Чем 
      "" + valu
      хуже, чем 
      String.Format("{0}", valu)
      ?
      Ответить

      • ava Lure Of Chaos 28.12.2010 19:41 # +3

        тоже зачетно
        Ответить

      • ava guest 28.12.2010 21:02 # 0

        Запостите это на говнокод. :D
        Ответить

      • ava guest 28.12.2010 21:05 # 0

        Нехочу постить, как верно. Буду ждать ещё говна. :)
        Ответить

        • ava eigenein 28.12.2010 21:22 # 0

          Вот так верно,
          string vals = String.Empty;
while (valu != 0)
{
    vals = (valu % 10)  + vals;
    value /= 10;
}

          ((:

          P.S. А на самом деле, короче, чем автор оригинала, вряд ли можно это сделать. При условии, что нельзя выкидывать переменные в результате неведомой аццкой оптимизации кода.
          Ответить

      • ava burdakovd 28.12.2010 22:33 # 0

        String.valueOf(valu) - в C# должен быть аналог такой функции
        Ответить

    • ava guest 28.12.2010 23:54 # 0

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

    • ava guest 28.12.2010 23:55 # 0

      String.TryParse(valu, out vals); :D
      Ответить

    • ava koodeer 29.12.2010 00:45 # 0

      Тут уже написали, что правильно будет valu.ToString(). Поясню, для недотнетчиков: в выражении ""+valu метод ToString() будет вызван автоматически. То есть будет вызов метода плюс конкатенация. Причём, если не ошибаюсь (лень уточнять - Новый год на носу, настроение не рабочее), ещё и сборщик мусора пнём, ибо строки иммутабельны.

      Хотя не исключено, что компилятор соптимизирует.
      Ответить

      • ava Nagg 29.12.2010 09:05 # +1

        в строке "" + vale произойдет вызов метода String.Concat(object, object), тобишь произойдет дополнительный boxing для valu. а там внутри уже ToString()
        Ответить

        • ava koodeer 29.12.2010 12:06 # 0

          Глянул в ILDASM. Действительно, боксинг и String.Concat(object, object). Печально... Я в тайне надеялся, что компилятор выкинет пустую строку вообще, и лишнего не будет.
          Ответить

          • ava guest 29.12.2010 12:12 # 0

            >Действительно, боксинг
            И так понятно. Зачем глядеть...
            Ответить

            • ava koodeer 30.12.2010 00:07 # 0

              > И так понятно.
              Ну не скажи! Я думал, сначала вызовется ToString(), а потом String.Concat(string, string)
              Ответить

          • ava guest 29.12.2010 12:13 # 0

            >компилятор выкинет пустую строку вообще
            Оптимизацию включали?
            Ответить

    • ava guest 29.12.2010 02:45 # −2

      .
      Ответить

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