Helper method to shuffle cards list for online gaming

Posted on


I have written this code to shuffle a deck of cards. I would like to hear your inputs.

public static class Helper

    public static Int32 GetRandomNo(this RNGCryptoServiceProvider rng, byte[] data)
        var randomNo = BitConverter.ToInt32(data, 0);
        return randomNo;

    public static void Shuffle<T>(this List<T> source)
        using (RNGCryptoServiceProvider rng = new RNGCryptoServiceProvider())
            // Buffer storage.
            byte[] data = new byte[4];

            source =
                source.Select(element => new {element, randomValue = rng.GetRandomNo(data)})
                    .OrderBy(entry => entry.randomValue)
                    .Select(entry => entry.element)


4 bytes is only 4,294,967,296
Consider there are 52! possible shuffles = 8 * 10^67

Use a proper shuffle like Fisher Yates. You get a proper shuffle with only 0-51 random. Also shuffle from the prior shuffle (not a sorted deck). If you look closely Fisher Yates it produces exactly 52! deals. It is perfect.

I am not sure about your algorithm but I think the problem is that it will produce too many shuffles and they will not have uniform distribution. I think it will produce 52^52 deals.

A perfect random with a less than perfect algorithm is a bigger vulnerability than a perfect algorithm and a less than perfect random.

You can use the regular Random and a proper algorithm and not be exploited unless they know the seed. But it does not hurt to use RNGCryptoServiceProvider.

Poker security

I wonder that the linked SO question does not mention the Sort method. With it you can randomize the list in place by creating a custom Comparison<T>. You should also return the new result so you can chain other extensions if necessary.

public static List<T> Shuffle<T>(this List<T> source)
    using (var rng = new RNGCryptoServiceProvider())
        var data = new byte[4];
        source.Sort(CompareRandomNumbers<T>(data, rng.GetRandomNo));            
        return source;

private static Comparison<T> CompareRandomNumbers<T>(byte[] data, Func<byte[], int> getRandomNumber)
    return (x, y) => getRandomNumber(data).CompareTo(getRandomNumber(data));


var randomizedList = new[] { "a", "b", "c", "d" }.ToList().Shuffle();

Leave a Reply

Your email address will not be published.