Question

我有这个：

private void getAccount()
{
    string[] acct = File.ReadAllLines(Environment.GetFolderPath(Environment.SpecialFolder.Desktop) + @"\Accts.txt");
    for (int i = 0; i < acct[line].Length - 1; i++)
    {
        foreach (char c in acct[line])
        {
            if (c.ToString() == ":")
            {
                onPass = true;
                i += 1;
            }
            if (onPass == false) { user += acct[line][i]; }
            if (onPass == true) { pass += acct[line][i]; }
        }
    }
    MessageBox.Show("Username is " + user + ". \n\nPassword is " + pass + ".");
    onPass = false;
}

该文件包含：

minicl55:mypass

然而，这输出：

enter image description here

以下是以下问题：

字符重复很多
只有“mmmmmmm”被认为是用户名的一部分，直到冒号应该是用户名的一部分，通过后
：密码包含在内，应该完全忽略（除了告诉用户名停止在哪里，密码开始）

Answer 1

第一次通过for循环i == 0。然后，foreach循环会查看acct[line]中的每个字符，但i永远不会更改，因此对于:之前的所有字符，acct[line][i]部分会一直返回acct[line][0]或"m" 8次。这就是用户名显示为"mmmmmmmm"的原因。

然后遇到冒号，i增加1.现在onPass == true，pass最终得acct[line][1]，即"i"字符。这会对字符串的其余部分重复，因此pass似乎是"iiiiiii"（从冒号到结尾）。

现在我们回到for循环。除了i 内部循环之外1已经增加了（不好主意）所以现在for循环实际上在i == 2上。同样，开始部分执行8次（用户名中的每个字符一次），但始终引用acct[line][2]，因此用户名为"nnnnnnnn"。除onPass之外仍然如此，因此它会附加到密码变量中。然后在"i"增加后再增加7 i。

i变量在内部和for循环中再次增加，因此下次使用acct[line][4]时，"c"（8次），在i循环中foreach增加1，您获得acct[line][5] 7次，即"l"。

到目前为止，密码为"iiiiiiinnnnnnnniiiiiiicccccccclllllll"。希望你能看到模式。

你可以消除一些循环和复杂性，只需使用类似：（未经测试）

private void getAccount()
{
    var allAccounts = File.ReadAllLines(Environment.GetFolderPath(Environment.SpecialFolder.Desktop) + @"\Accts.txt");

    foreach (var account in allAccounts)
    {
        var pieces = account.Split(':');

        MessageBox.Show(string.Format("Username is {0}. \n\nPassword is {1}.", pieces[0], pieces[1]));
    }
}

Answer 2

您的外部循环正在迭代char中的每个acct[line]。然后你在你的内循环中做同样的事情，你只是表达一点不同。

Answer 3

请显示您的变量，但这是另一种方法：

       private void getAccount()
        {
            string user = "";
            string pass = "";
            string[] user_pass = new string[0];

            var accts = System.IO.File.ReadAllLines(Environment.GetFolderPath(Environment.SpecialFolder.Desktop) + @"\Accts.txt");


            foreach(var acct in accts)
            {
                user_pass = acct.Split(':');
            }


            //Add iteration for multiple lines
            if (user_pass.Length > 0)
            {
                MessageBox.Show("Username is " + user_pass[0] + ". \n\nPassword is " + user_pass[1] + ".");
            }
            else
            {
                MessageBox.Show("Chaos: Dogs and Cats Living Together!");
            }

        }
    }
}

Answer 4

好吧，我看到你的第一个循环得到一个特定线的长度，其位置根本没有变化。

for (int i = 0; i < acct[line].Length - 1; i++)

然后循环遍历该行的每个字符

foreach (char c in acct[line])

问题是如果你的acct [line]有X长度，你将循环通过acct [line] X次，这就是为什么重复的字符。你最终会读X次相同的角色。

Answer 5

正如其他人一样评论/回答，你的外圈和内圈几乎完全一样。我重写了for循环，所以外部循环遍历字符串数组的每一行，然后内部循环将遍历该行中的所有字符。

            for (int line = 0; line < acct.Length; line++)
            {
                int i = 0;
                foreach (char c in acct[line])
                {
                    if (c.ToString() == ":")
                    {
                        onPass = true;
                    }
                    else
                    {
                        if (!onPass)
                            user += acct[line][i];
                        else
                            pass += acct[line][i];
                    }
                    i++;
                }
            }

但我建议，为了您自己的利益，如果您需要循环遍历所有字符以将其用于内部循环：

            for (int i = 0; i < acct[line].Length; i++)
            {
                if (acct[line][i].ToString() == ":")
                {
                    onPass = true;
                }
                else
                {
                    if (!onPass)
                        user += acct[line][i];
                    else
                        pass += acct[line][i];
                }
            }

或者更好的是用更简单的东西替换所有东西，并且不太容易被小的变化打破：

        for (int line = 0; line < acct.Length; line++)
        {
            if (acct[line].Contains(":"))
            {
                string[] parts = acct[line].Split(':');
                user = parts[0];
                pass = parts[1];
                MessageBox.Show("Username is " + user + ". \n\nPassword is " + pass + ".");
            }
        }

对于循环读取，文件不起作用

5 个答案: