测试超过了执行超时时间

本文关键字：超时 时间 执行 过了 测试 | 更新日期: 2023-09-27 18:00:19

我的main.cs代码：

 public string Generate(int length)
 {
     char[] chars = "$%#@!*abcdefghijklmnopqrstuvwxyz1234567890?;:ABCDEFGHIJKLMNOPQRSTUVWXYZ^&".ToCharArray();
     string password = string.Empty;
     Random random = new Random();
     for (int i = 0; i < length; i++)
     {
         int x = random.Next(1, chars.Length);
         if (!password.Contains(chars.GetValue(x).ToString()))
             password += chars.GetValue(x);
         else
             i--;
     }
     return password;
 }

我做了一个测试代码

[TestMethod]
[Timeout(1000)]
public void RenderingPasswordShouldHaveMaximumSize()
{
    var amountOfCharacters = Int32.MaxValue;
    var generator = new PasswordGenerator();
    var target = generator.Generate(amountOfCharacters);
    Assert.Fail("This method should throw an exception if you try to create a password with too many characters");
}

但它给了我以下错误：

消息：测试"RenderingPasswordShouldHaveMaximumSize"超过了的执行超时期

有人能帮我吗？我的密码需要最大大小为74。

测试超过了执行超时时间

我刚刚注意到您实际上希望您的测试花费很长时间，忽略关于Int32.MaxValue的部分。

你的循环有点奇怪，你正在减少迭代器，以便尝试使用所有的字母，但这可能会导致问题。这并不是它花费太长时间，而是它没有真正正确地生成密码。

我会像这样修复你的版本：

static string Generate(int length)
{
    // also, seed your random so you don't get the same password
    Random random = new Random((int)DateTime.Now.Ticks);
    char[] chars = "$%#@!*abcdefghijklmnopqrstuvwxyz1234567890?;:ABCDEFGHIJKLMNOPQRSTUVWXYZ^&".ToCharArray();
    string password = string.Empty;
    for (int i = 0; i < chars.Length; i++)
    {
        int x = random.Next(0, chars.Length);
        if (!password.Contains(chars.GetValue(x).ToString()))
            password += chars.GetValue(x);
        else
            i--;
    }
    if (length < password.Length) password = password.Substring(0, length);
    return password;
}

我已经测试过了，它有效：

测试代码：

Debug.WriteLine(Generate(5));
Thread.Sleep(50);
Debug.WriteLine(Generate(10));
Thread.Sleep(50);
Debug.WriteLine(Generate(20));
Thread.Sleep(50);
Debug.WriteLine(Generate(30));
Thread.Sleep(50);
Debug.WriteLine(Generate(40));
Thread.Sleep(50);
Debug.WriteLine(Generate(60));
Thread.Sleep(50);

结果：

SDxF0
i8ZLhm1gxn
@0Ldn7I&1:Kg2x3SYE;m
U?5uO%N4hkpq1*y;9SRVaer^Eij:bT
nvL;E3#D1MQgTicSdojHOwz:VFk2x&94a*PZ@X80
cSm39n:%1sL*lk2B?yVDTPZCA0vGb@udjeW5KoUw6qRNxY^pazH$JXiQ;tFg

此外，请记住，在向密码中添加字符的方式中，您要使其使用char[]中的所有字符，但没有重复。这意味着密码的最大长度等于char[]的长度。

我推荐这样的东西：

static string Generate(int length)
{
    Random random = new Random((int)DateTime.Now.Ticks);
    StringBuilder sb = new StringBuilder();
    for (int i = 0; i < length; i++)
    {
        int x = random.Next(33, 123);
        sb.Append((char)x);
    }
    return sb.ToString();
}

它给出了相同的结果，但也可以处理更长的长度，例如200:

t5[5l
WoEZG;8^9<
6(4Q*Y7k>`?ohte6F^pe
DVb'5l^JMKc`q&[#$U4Kq^'OW`JrRi
#iQSq0'WRoDe<]k36nOhNOb-#"Zt!cK8iaR.I>VF
1<%^"B)6bZhWEiazmfmO7Vv*Rw]TbKV+GRJUc%k23Lq/HC.I6Eha6!5V@v-&
jOnV_`(Cf$I'kFr/%%/loG"bx9lV1E?`[A"y1)pF5tA".x?**(E/>kzCN-XGnL2B`c!JEtxw0cXu'zTztcM*z0CkBYK%LIRcbz<Fxo`0*HzU4&=NTXM,z)CP@1)fJtKa2o3'_/#&#=zR1iW<<wLnwFdYXg&hgta0x:C?m6jEVH*])k@QTgKMIXdg,UJr*59)VkUrG8J&

添加以下内容怎么样：

if (length > chars.Length) {
    throw new ArgumentException("Password too long", length);
}

在测试中，您可以检查此Exception是否按预期抛出。

除非您传递一个小于74的数字，否则您的代码永远不会完成。

根据您的代码，如果随机选择的字符已经在密码中，则只需再次迭代（递减i）。

你有没有想过，一旦你使用了chars的所有字符，会发生什么？

相关文章：