1. C++ / Говнокод #17127

    +57

    1. 1
    2. 2
    float factor = ((float)sizeof(char) / sizeof(wchar_t));
fileSize = (long)((float)fileSize * factor);

    вот так выглядит деление на sizeof(wchar_t) :)

    Запостил: fsmoke, 18 Ноября 2014

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

    • ava guest 18.11.2014 11:30 # 0

      Кроссплатформенно, с расчетом - вдруг у нас будет платформа где char=16 бит, а wchar_t - 8.
      Ответить

      • ava guest 18.11.2014 11:31 # 0

        Правда там это все равно не сработает, ведь sizeof(char)==1. Но мы хотя бы попытались!
        Ответить

    • ava guest 18.11.2014 11:48 # 0

      много скобочек.
      long fileSize = ...;
      double factor = (double)sizeof(char) / sizeof(wchar_t);
      fileSize = fileSize * factor;

      и внезапно все не так уж и страшно
      Ответить

    • ava TarasB 18.11.2014 18:04 # 0

      нахуй тут флоат?!
      Ответить

      • ava guest 18.11.2014 18:11 # +1

        float работает быстрее, чем double
        Как не тебе, г-девщику это знать
        Ответить

        • ava TarasB 18.11.2014 18:12 # 0

          нахуй тут вообще плавучка?
          Ответить

          • ava Анонимус 19.11.2014 16:10 # 0

            >>плавучка
            Хотите сказать тут хватит целочисленки?

            Тоесть сам факт деления размера чара на размер широко чара с последующим умножением на размер файла кажется Вам вполне себе бест-практисом?
            Ответить

          • ava Psionic 19.11.2014 16:41 # 0

            Это необходимость - говно-ж не тонет.
            Ответить

      • ava guest 18.11.2014 18:15 # 0

        К.О. поясняет - он тут не нужен, в этом и говно. Можно было разделить fileSize на sizeof(wchar_t).
        Ответить

        • ava TarasB 18.11.2014 18:22 # 0

          А ещё при 1/3 можно получить неверный результат.
          Ответить

      • ava Анонимус 19.11.2014 02:57 # +1

        Вы так говорите, словно бы возмущаетесь: а почему не double??
        Ответить

        • ava kipar 19.11.2014 11:55 # +2

          Ты так говоришь, как будто double это что-то противоположное float. Extended - вот тип обеспечивающий приемлемую точность в данном случае.
          Ответить

          • ava bormand 19.11.2014 12:21 # +1

            Тут нужен rational (дроби).
            Ответить

            • ava Анонимус 19.11.2014 16:14 # 0

              BigDecimal
              Ответить

              • ava bormand 19.11.2014 16:18 # 0

                1/3 не схавает. Всё-таки надо настоящий Rational с числителем и знаменателем.
                Ответить

                • ava Анонимус 19.11.2014 16:21 # 0

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

                • ava 1024-- 19.11.2014 16:31 # 0

                  Откуда вы все берёте 1/3 и чем он плох? (только не надо говорить про то, что factor = 1/3 будет равен нулю, это не всегда правда и только отговорка)
                  В целых числах всё равно fileSize округляется. Если взять число не короче, чем sizeof(decltype(fileSize)) + log2(sizeof(char)), то всё должно нормально работать для любых fileSize. Или тут ещё подвох есть?
                  Ответить

                  • ava 1024-- 19.11.2014 16:38 # 0

                    Говоря более формально, чем плохо ниже написанное?
                    long long fileSize2 = fileSize * (long long) sizeof(char);
fileSize2 = fileSize / (long long) sizeof(wchar_t);
fileSize = fileSize2; // будет работать, даже если char не будет помещаться на CD
                    Ответить

                    • ava TarasB 19.11.2014 16:42 # 0

                      Не хватает операции MulDiv, которая способна работать с промежуточным результатом двойного размера. В Форте такая есть, кстати.
                      Ответить

                      • ava 1024-- 19.11.2014 16:46 # +1

                        И да, если учесть напоминание гостя, тут вообще всё хорошо. Ни плавающих питухов, ни mulDiv, ни 1/3
                        fileSize = fileSize / sizeof(wchar_t);
// будет работать до тех пор, пока конардо не пишет стандарт
                        Ответить

                        • ava guest 22.11.2014 00:29 # 0

                          и ведь будет работать.......
                          /*другой гость*/
                          Ответить

        • ava TarasB 19.11.2014 12:34 # 0

          лолшто?!
          Ответить

    • ava guest 18.11.2014 18:13 # 0

      *г-девщику,
      Ответить

    • ava Анонимус 19.11.2014 01:36 # +1

      И вышло у меня в ответе: Два землекопа и две трети
      Ответить

    • ava noganno 25.08.2021 08:35 # 0

      И вот результат! Так сказать, "на лице"... у этого пацана. Егорчик (усмехаюсь), новый дружбан. И мне совсем не стыдно. Мне интересно! Как там, show must go on. Думаю, ну, вчера всё было второпях-впопыхах, нормально даже не познакомились.
      Ответить

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