随机大写字母不起作用

时间:2013-09-06 20:40:17

标签: c#

我想随机输入密码"生成不同字母长度的生成器。您输入长度,它使长度随机字符。我也想要一些随机字母大写。我有一个变量rndCap到1或2.如果rndCap == 2,if会把它变成大写字母,如果没有,它什么都不做。答案是所有角色的组合。当我执行时,数字都是小写的。一切都有效,除了它随机大写一个角色。

Random rnd = new Random();

string[] alphabet = new string[27];

alphabet[1] = "a";
alphabet[2] = "b";
alphabet[3] = "c";
alphabet[4] = "d";
alphabet[5] = "e";
alphabet[6] = "f";
alphabet[7] = "g";
alphabet[8] = "h";
alphabet[9] = "i";
alphabet[10] = "j";
alphabet[11] = "k";
alphabet[12] = "l";
alphabet[13] = "m";
alphabet[14] = "n";
alphabet[15] = "o";
alphabet[16] = "p";
alphabet[17] = "q";
alphabet[18] = "r";
alphabet[19] = "s";
alphabet[20] = "t";
alphabet[21] = "u";
alphabet[22] = "v";
alphabet[23] = "w";
alphabet[24] = "x";
alphabet[25] = "y";
alphabet[26] = "z";

string answer = "";
int length = Convert.ToInt32(lengthTextBox.Text);
int rndCap;
int rndLetter;

for (int i = 1; i <= length; i++)
{
    rndCap = rnd.Next(1, 3);
    rndLetter = rnd.Next(1, 27);
    string tempMem = alphabet[rndLetter];

    if (rndCap == 2)
    {
        tempMem.ToUpper();
    }
    answer = answer + tempMem;
}

passwordTextBox.Text = answer;

7 个答案:

答案 0 :(得分:10)

我很确定IF工作正常:)

String.ToUpper()但是不会将手头的字符串更改为大写,而是将其作为结果返回。您必须分配值。你想要的可能就是:

tempMem = tempMem.ToUpper();

另外,遵循以下规则:始终牢记,即 编码,文化和区域设置可能与您当前的不同。

即使在这种情况下,我们只使用“看起来无害”的ASCII字符,但要将其转换为使用大写字符串的实践:

正如Jeppe所指出的那样:在土耳其语中,我是一个与我不同的角色的上层 - 引入了不必要的后果。

答案 1 :(得分:3)

字符串为immutable,因此您需要再次分配

tempMem = tempMem.ToUpper();

答案 2 :(得分:1)

您调用ToUpper,但不指定值:

tempMem = tempMem.ToUpper()

也许这段代码通常更优雅:

    static void Main(string[] args)
    {
        Random cRandom = new Random(DateTime.Now.Millisecond);

        string answer = "";
        int length = Convert.ToInt32(6);
        int rndCap;
        int rndLetter;

        for (int i = 1; i <= length; i++)
        {

            rndCap = cRandom.Next(1, 3);

            rndLetter = cRandom.Next(0, 26);

            char tempMem = (char)('a' + rndLetter);
            string c2 = tempMem.ToString();
            if (rndCap == 2)
            {

               c2 = tempMem.ToString().ToUpper();

            }

            answer = answer + c2;

        }
        Console.WriteLine(answer);
        Console.ReadLine();
    }

答案 3 :(得分:0)

使用它:

tempMem = tempMem.ToUpper();

由于ToUpper()不会将大写字母设置为调用它的字符串,它会返回所有字母大写的字符串的副本。

答案 4 :(得分:0)

ToUpper()是一个返回大写字符串的方法,因此您需要将tempMem的值赋给该值:

tempMem = tempMem.ToUpper();

答案 5 :(得分:0)

我认为有更简单的方法可以实现您想要实现的目标。看看这段代码;

        var lowerchar = "abcdefghijklmnopqrstuvwxyz";
        var upperchar = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
        var numbers = "0123456789";
        var random = new Random();

        var result = new string(
            Enumerable.Repeat(lowerchar, 3)
                      .Select(s => s[random.Next(s.Length)])
                      .ToArray());

        result += new string(
            Enumerable.Repeat(upperchar, 3)
                      .Select(s => s[random.Next(s.Length)])
                      .ToArray());

         result += new string(
            Enumerable.Repeat(numbers, 2)
                      .Select(s => s[random.Next(s.Length)])
                      .ToArray());

        random = new Random();

        string password = new string(result.ToCharArray().OrderBy(s => random.Next().ToArray());

答案 6 :(得分:0)

所有字符串操作都返回新字符串而不修改原始字符串,因为字符串是不可变的。 像这样修改它:tempMem = tempMem.ToUpper();