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

    +110

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    public static int getRandomNumber(int from, int to)
{
       System.Threading.Thread.Sleep(10);
       Random r = new Random();
       int var = r.Next(from, to);
       return var;
}

    No comments

    Запостил: bazhenovc, 20 Августа 2012

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

    • ava 3.14159265 20.08.2012 11:52 # +5

      Счастливо поймать InterruptException.
      Ответить

    • ava Lure Of Chaos 20.08.2012 11:56 # +4

      создаем видимость трудной работы?
      Ответить

      • ava krypt 20.08.2012 12:40 # +5

        Нет. Здесь каждый раз переинициализируется класс, и если вызывать этот код в цикле - можно запросто словить 5-10 одинаковых значений.
        Но программист настолько суров, что вместо того, чтобы вынести класс за пределы фунции, ждёт пока сменится рандом сид, зависящий от таймера

        В общем это труЪговнокод.
        Ответить

        • ava Lure Of Chaos 20.08.2012 13:20 # 0

          > переинициализируется класс, и если вызывать этот код в цикле - можно запросто словить 5-10 одинаковых значений
          ну здесь же нет стартового сида, поэтому трудно сказать, будут последовательности одинаковы, или нет
          Ответить

          • ava phoenixx 20.08.2012 13:37 # +3

            По дефолту, при создании Random - старовый seed равен - Environment.TickCount. И поэтому при быстром создании Random обьектов, seed может совпадать. Автор не понял причины проблемы, и сделал фикс через одно место.
            Ответить

            • ava vistefan 20.08.2012 19:42 # 0

              Причём, как я понимаю, если не знать причины, то такой фикс можно сделать только наугад?
              Ответить

              • ava phoenixx 20.08.2012 19:46 # 0

                Причину можно легко узнать из MSDN-а http://msdn.microsoft.com/ru-ru/library/h343ddh9.aspx
                Либо с помощью Reflector и подобных посмотреть исходный код.

                Имхо, чтобы использовать рандомную генерацию, нужно хотя бы знать, что она зависит от начального seed, а не просто тупо скопипастить сэмпл из гугла. Что автор скорее всего и сделал.
                Ответить

                • ava krypt 20.08.2012 21:54 # 0 

                  public static void Main()
{
   Random rand1 = new Random();
   Random rand2 = new Random();
   Thread.Sleep(2000);
   Random rand3 = new Random();
   ShowRandomNumbers(rand1);
   ShowRandomNumbers(rand2);
   ShowRandomNumbers(rand3);
}


                  Так вот откуда ноги растут!
                  Ответить

        • ava Steve_Brown 20.08.2012 15:49 # 0

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

    • ava argos 20.08.2012 11:59 # +5

      По объекту на каждый вызов. Стресс-тест GC.
      Ответить

      • ava 3.14159265 20.08.2012 12:06 # −1

        Ну это кстати нормально.
        А Random в шарпах thread-safe?
        Ответить

        • ava krypt 20.08.2012 12:42 # 0

          Конкретно с потокобезопасностью тут всё в порядке, потому как для каждого вызова создастся свой объект.
          Ответить

          • ava 3.14159265 20.08.2012 12:49 # 0

            А. Вот граждане меня минусуют. Но я не вижу обоснованной критики.

            >Конкретно с потокобезопасностью тут всё в порядке
            Спасибо кеп, мой вопрос о другом.
            Ответить

            • ava absolut 20.08.2012 13:01 # +1

              Политика .net по потокобезопасности гласит:
              Any public static (Shared in Visual Basic) members of this type are thread safe. Any instance members are not guaranteed to be thread safe.
              Ответить

      • ava absolut 20.08.2012 13:17 # +1

        >По объекту на каждый вызов.
        первый признак конструктора
        Ответить

    • ava phoenixx 20.08.2012 13:39 # +1

      В конце поста автор этого кода опубликовал коммент - "Спасибо за критику, спасибо пользователю general, он спустил меня на землю. Исправлю функции, жестоко протестирую и опубликую топик вновь.".
      Теперь нужно подождать нового топика чтобы понять, извлек ли он из этого что-то полезное.
      Ответить

      • ava krypt 20.08.2012 13:43 # +2

        coll3ctor карма: -11,0
        Думаю - уже не дождёмся
        Ответить

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