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

    +24

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    int randomize(int Min, int Max)
{
qsrand(QTime::currentTime().msec());
if (Min > Max)
{
int Temp = Min;
Min = Max;
Max = Temp;
}
return ((qrand()%(Max-Min+1))+Min);
}

    Всё хорошо?

    Запостил: Hackeridze, 29 Мая 2013

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

    • ava neeedle 29.05.2013 13:17 # +1 

      if (Min > Max)
{
int Temp = Min;
Min = Max;
Max = Temp;
}


      Ыыы
      Ответить

    • ava bormand 29.05.2013 13:45 # +6

      1) Делать srand перед каждой генерацией числа - моветон. Если в одну миллисекунду пройдет несколько генераций - все вернут одинаковый ответ.
      2) Обмен min и max - защита от дурака, который не осилил доку к функции?
      3) Какой нахер randomize? Это ж генерация рандомной точки на отрезке... Если хочется назвать это рандомайзом - убирайте строки 4-10.

      В остальном сойдет.
      Ответить

      • ava defecate-plusplus 29.05.2013 13:53 # +6

        ошибка: в названии randomize не найдена буква Q
        Ответить

      • ava inkanus-gray 29.05.2013 15:32 # 0

        Обмен Min и Max вообще не нужен. Формула будет работать, даже если они перепутаны. Если, конечно, еретические отрицательные числа не запрещены.
        Ответить

        • ava inkanus-gray 29.05.2013 22:08 # +1

          P.S. Всё-таки обмен нужен. Разные компиляторы по-разному считают остаток от деления на отрицательное.
          Ответить

    • ava TarasB 29.05.2013 15:37 # 0

      Такое использование рандомайзера это сишкоблядство.
      В стиле Турбопаса это было бы так:

      int(((long long)(qrand()) * (long long)(max-min+1))>>32)
      (вменяемый компилятор это превратит в одно умножение и возврат edx)
      а остатки по модулю они для многих генераторов зацикливаются
      и ещё умножать быстрее чем делить
      Ответить

      • ava defecate-plusplus 29.05.2013 15:44 # +2

        предположим, что [min, max) это [10, 20)
        предположим, что qrand() вернул 30
        твоя формула вернет значение в пределах [10, 20) ?
        Ответить

        • ava TarasB 29.05.2013 16:33 # +1

          То, что к выражению следует дописать "+min", я оставил читателю в качестве упражнения.
          Я обсуждаю именно способ масштибирования числа, равномерно распределённого по очень большому отрезку, на число, равномерно распределённое по короткому отрезку.
          Ответить

          • ava LispGovno 29.05.2013 16:56 # +1

            1.0*(max-min+1)*rand()/RAND_MAX+min
            Ответить

            • ava bormand 29.05.2013 17:01 # +1

              Фу, сопроцессор.
              Ответить

              • ava LispGovno 29.05.2013 20:39 # 0

                Ты можешь с rand написать лучше?
                Ответить

                • ava bormand 29.05.2013 20:45 # +2

                  (((uint64_t)(max-min+1))*rand()) / RAND_MAX + min
                  Ответить

                  • ava LispGovno 29.05.2013 20:54 # +2

                    фу, медленное деление div
                    Ответить

                    • ava bormand 29.05.2013 20:57 # +1

                      Ну как уж генератор замутили. Если бы RAND_MAX не был кривым и произвольным - деление было бы ненужным.

                      P.S. Да и у тебя оно тоже есть ;)
                      Ответить

                      • ava LispGovno 29.05.2013 21:15 # 0

                        Ну у меня то fdiv, а не div.

                        > Если бы RAND_MAX
                        Мечтать не вредно. В qt что-то другое придумали? Если и придумали, то явно в сторону ухудшения генератора и гибкости от версии к версии.
                        Ответить

      • ava 3.14159265 29.05.2013 17:04 # +2

        >int(((long long)(qrand()) * (long long)(max-min+1))>>32)
        >(вменяемый компилятор это превратит в одно умножение и возврат edx)
        Какой вообще смысл в "высокоуровневой" сишке, если на асме оно и короче и понятней?
        Ответить

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