这是代码:(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个字符长密码。
我的错误是什么?
答案 0 :(得分:14)
问题出在这里:
int[] numbers = { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' };
每次将数字附加到password
时,使用该声明,它将被视为ASCII编号,而不是实际值。所以你要添加48到57之间的整数,结果字符串会比预期更长。
e.g。当6
作为随机数生成时,您会在((int)'6').ToString()
变量中添加类似password
的内容,实际添加54
而不是6
。
将该数组声明为char[]
,它将正常工作。
答案 1 :(得分:5)
你可以试试这个小方法。
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并计算出您想要使用的字符范围。
对于Nathan,如果你确切地知道你想要的字符,这是你可以做到的另一种方式......
public static string Random(int length)
{
string allowed = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";
return new string(allowed
.OrderBy(o => Guid.NewGuid())
.Take(length)
.ToArray());
}
答案 2 :(得分:3)
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);
}
答案 3 :(得分:3)
这里是我的完整函数,用于生成具有所需长度的随机密码 (感谢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<int>(0, genlen)
.Select(i => total[rnd.Next(total.Length)])
.ToArray();
return new string(chars);
}
答案 4 :(得分:0)
您可以尝试以下代码:
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<int>(0, numberOfChars)
.Select(i => total[rnd.Next(total.Length)])
.ToArray();
var result = new string(chars);