这是代码:(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并找出你想要使用的角色范围。
对于Nathan,如果你确切地知道你想要哪些角色,这是你可以做到的另一种方式......
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<int>(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<int>(0, numberOfChars)
.Select(i => total[rnd.Next(total.Length)])
.ToArray();
var result = new string(chars);