0

我的程序中有某种逻辑错误。每当我输入一个带有 1 个字母的短语时,我都会收到 ArgumentOutOfRange 异常,并且每当我输入一个多字母单词时,文本框就会清除,显示“Apple”(我的数组中的第一个值)并且什么都不做。任何人都可以看到其中的逻辑错误吗?

string[] d = { "Apple", "Bass", "Cat", "Dog", "Ear", "Flamingo", "Gear", "Hat", "Infidel", "Jackrabbit", "Kangaroo", "Lathargic", "Monkey", "Nude", "Ozzymandis", "Python", "Queen", "Rat", "Sarcastic", "Tungston", "Urine", "Virginia", "Wool", "Xylophone", "Yo-yo", "Zebra", " " };
string var;
int len = 0;

private void button1_Click(object sender, EventArgs e)
{
    var = textBox2.Text;
    textBox1.Text = "";
    for (int y = 0; y < var.Length; y++)
    {
        for (int x = 0; x < d.Length; x++)
        {
            if (d[x].ToUpper().Substring(0, 0) == var.ToUpper().Substring(len, len))
            {
                len = len + 1;
                textBox1.Text = textBox1.Text + "\n" + d[x];
            }
        }
    }
}
4

3 回答 3

0

这一行解释了这种行为。

if (d[x].ToUpper().Substring(0, 0) == var.ToUpper().Substring(len, len))

substring 的第二个参数是字符串长度。所以在左边你总是有一个空字符串。在右侧,当 len==0 时,您也有空字符串(这就是您的代码总是选择 Apple 的原因)。

之后,您更改 len,并重复循环。然后右边的表达式是 var.Substring(1,1) 如果你的字符串是 1 个字符长,它会给你错误。因为这读取 - 1 以 1 开头的符号(这是字符串的第二个字符)

除此之外,代码的目的是完全神秘的,所以肯定还有其他错误。

于 2013-09-19T15:06:04.340 回答
0

Substring(0, 0)真的没有意义。这将始终是一个空字符串。

Substring(len, len)也是一个坏主意,因为它会返回一个len从 index 开始的长度字符串len。这是您获得例外的地方。

我假设,你真正想要的是第二个参数在这两种情况下都是 1。这可以进一步简化为通过索引访问:

d[x].ToUpper()[0] == var.ToUpper()[len]
于 2013-09-19T14:54:34.877 回答
0

您可以使用 LINQ 轻松完成相同的操作:

private void button1_Click(object sender, EventArgs e)
{
    var dict = d.ToDictionary(x => x.First(), x => x);
    textBox1.Text = string.Join(Environment.NewLine, textBox2.Text.Select(x => dict[char.ToUpper(x)]));
}

要在没有 LINQ 的情况下执行此操作,我建议如下:

for (int y = 0; y < input.Length; y++)
{
    for (int x = 0; x < d.Length; x++)
    {
        if (char.ToUpper(d[x][0]) == char.ToUpper(input[y]))
        {
            result = result + "\n" + d[x];
        }
    }
}

变化是:

  • 你不需要len变量。改为使用y
  • 你不需要全部string大写。请改用char.ToUpper静态方法。
  • 你不需要string.Substring方法。请改用索引器。
于 2013-09-19T14:55:43.670 回答