要使用哪种集合类型?

我建议你把结果复制到新的 List<Question> 然后洗牌。

但是,我会使用 Fisher-Yates shuffle 而不是你在这里给的。在这个网站上有很多这样的例子。

例如,您可以执行以下操作:

// Don't create a new instance of Random each time. That's a detail
// for another question though.
Random rng = GetAppropriateRandomInstanceForThread();

List<Question> shuffled = new List<Question>(myQuestions);
for (int i = shuffled.Count - 1; i > 0; i--)
{
    // Swap element "i" with a random earlier element it (or itself)
    int swapIndex = rng.Next(i + 1);
    Question tmp = shuffled[i];
    shuffled[i] = shuffled[swapIndex];
    shuffled[swapIndex] = tmp;
}

您可以使用Linq和Order by a Random值:

List<string> items = new List<string>(); 
items.Add("Foo");
items.Add("Bar");
items.Add("Baz");

foreach (string item in items.OrderBy(c => Guid.NewGuid()))
{
    Console.WriteLine(item);
}

临时集合应与MyQuestions类型相同。

我还建议您更改代码:

for (int i = 0; i <= myQuestions.Count -1; i++)  

到

for (int i = 0; i < myQuestions.Count; i++)  

做同样的事情,但这是大多数编程人员的工作方式,这样可以使代码更容易阅读。