C#随机密码生成器

这是代码:(passwordLengthBox是NumericUpDown Box,r和k是随机数)

private void generateButton_Click(object sender, EventArgs e) { int r, k; int passwordLength = (Int32)passwordLengthBox.Value; string password = ""; char[] upperCase = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z' }; char[] lowerCase = { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z' }; int[] numbers = { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' }; Random rRandom = new Random(); for (int i = 0; i < passwordLength; i++) { r = rRandom.Next(3); if (r == 0) { k = rRandom.Next(0, 25); password += upperCase[k]; } else if (r == 1) { k = rRandom.Next(0, 25); password += lowerCase[k]; } else if (r == 2) { k = rRandom.Next(0, 9); password += numbers[k]; } } textBox.Text = password; } 

该程序的作用是创建一个随机密码,其中包含字母(大写和小写)以及我选择的长度的数字。 问题是该程序没有像我选择的那样设置密码长度。

例如:如果我在NumericUpDown Box(passwordLengthBox)中键入5设置密码长度,有时它会给我5个字符长的密码,有时6/7/8字符长密码。

我的错是什么?

问题在这里:

 int[] numbers = { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' }; 

使用该声明,每次将数字附加到password它将被视为ASCII编号,而不是实际值。 所以你要添加48到57之间的整数,结果字符串会比预期更长。

例如,当6作为随机数生成时,您将附加类似于: ((int)'6').ToString()到您的password变量中,实际上添加了54而不是6

将该数组声明为char[] ,它将正常工作。

你可以试试这个小方法。

 public static string Random(int length) { try { byte[] result = new byte[length]; for (int index = 0; index < length; index++) { result[index] = (byte)new Random().Next(33, 126); } return System.Text.Encoding.ASCII.GetString(result); } catch (Exception ex) { throw new Exception(ex.Message, ex); } } 

与此唯一的区别是它也会使用字母数字字符,例如它可能会生成字符串,如f6Dx3$5d£4hG7

看看www.asciitable.com并找出你想要使用的角色范围。

对于内森来说,如果你确切地知道你想要哪些角色,这是你可以做到的另一种方式......

 public static string Random(int length) { string allowed = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"; return new string(allowed .OrderBy(o => Guid.NewGuid()) .Take(length) .ToArray()); } 

这是来自series0ne的答案略有改进。 那个答案给了一个相同字符的密码。 (例如%%%%%)

  var random = new Random((int) DateTime.Now.Ticks); try { var result = new byte[length]; for (var index = 0; index < length; index++) { result[index] = (byte) random.Next(33, 126); } return System.Text.Encoding.ASCII.GetString(result); } catch (Exception ex) { throw new Exception(ex.Message, ex); } 

这里我的完整函数生成所需长度的随机密码(感谢Viacheslav Smityukh)

  private String GeneratePassword(int genlen = 21, bool usenumbers = true, bool uselowalphabets = true, bool usehighalphabets = true, bool usesymbols = true) { var upperCase = new char[] { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z' }; var lowerCase = new char[] { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z' }; var numerals = new char[] { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' }; var symbols = new char[] { '~', '`', '!', '@', '#', '$', '%', '^', '&', '*', '(', ')', '{', '[', '}', ']', '-', '_', '=', '+', ':', ';', '|', '/', '?', ',', '<', '.', '>' }; char[] total = (new char[0]) .Concat(usehighalphabets ? upperCase : new char[0]) .Concat(uselowalphabets ? lowerCase : new char[0]) .Concat(usenumbers ? numerals : new char[0]) .Concat(usesymbols ? symbols : new char[0]) .ToArray(); var rnd = new Random(); var chars = Enumerable .Repeat(0, genlen) .Select(i => total[rnd.Next(total.Length)]) .ToArray(); return new string(chars); } 

您可以尝试以下代码:

 var numberOfChars = 6; var upperCase = new char[] { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z' }; var lowerCase = new char[] { 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z' }; var numbers = new char[] { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' }; var rnd = new Random(); var total = upperCase .Concat(lowerCase) .Concat(numbers) .ToArray(); var chars = Enumerable .Repeat(0, numberOfChars) .Select(i => total[rnd.Next(total.Length)]) .ToArray(); var result = new string(chars);