1

所以我有一个支持多个单词的 Pig Latin Translator。但是每当我输入一些单词时（例如，我们将只使用“香蕉苹果剪巧克力西奥多火车”。）它会正确吐出翻译的单词，但它会重复！这是我的代码：

namespace Pig_Latin_Translator
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
    List<string> vowels = new List<string>();
    List<string> specials = new List<string>();

    private void TranslateButton_Click(object sender, EventArgs e)
    {
        String[] parts = TranslateBox.Text.Split();
        foreach (string s in specials)
        {
            if (TranslateBox.Text.Contains(s) || TranslateBox.Text == "\"")
            {
                TranslateOutput.Text = "";
                MessageBox.Show("No Special Characters!", "Warning!", MessageBoxButtons.OK, MessageBoxIcon.Warning);
                break;
            }
            else
            {
                 foreach (String part in parts)
                {
                    foreach (String v in vowels)
                    {
                        if (part.Substring(0, 1) == v)
                        {
                            TranslateOutput.Text = TranslateOutput.Text + " " + part + "ay";
                            break;
                        }
                        else
                        {
                            if (part.Substring(0, 2) == "sh" || part.Substring(0, 2) == "ch" || part.Substring(0, 2) == "th" || part.Substring(0, 2) == "tr")
                            {
                                string SwitchP = part.Substring(2) + part.Substring(0, 2);
                                TranslateOutput.Text = TranslateOutput.Text + " " + SwitchP + "ay";
                                break;
                            }
                            else
                            {
                                string Switch = part.Substring(1) + part.Substring(0, 1);
                                TranslateOutput.Text = TranslateOutput.Text + " " + Switch + "ay";
                                break;
                            }
                        }
                    }
                }
            }
        }
    }
    private void Form1_Load(object sender, EventArgs e)
    {
        vowels.Add("a");
        vowels.Add("e");
        vowels.Add("i");
        vowels.Add("o");
        vowels.Add("u");

        specials.Add("`");
        specials.Add("1");
        specials.Add("2");
        specials.Add("3");
        specials.Add("4");
        specials.Add("5");
        specials.Add("6");
        specials.Add("7");
        specials.Add("8");
        specials.Add("9");
        specials.Add("0");
        specials.Add("-");
        specials.Add("=");
        specials.Add("[");
        specials.Add("]");
        specials.Add(@"\");
        specials.Add(";");
        specials.Add("'");
        specials.Add(",");
        specials.Add(".");
        specials.Add("/");

        specials.Add("~");
        specials.Add("!");
        specials.Add("@");
        specials.Add("#");
        specials.Add("$");
        specials.Add("%");
        specials.Add("^");
        specials.Add("&");
        specials.Add("*");
        specials.Add("(");
        specials.Add(")");
        specials.Add("_");
        specials.Add("+");
        specials.Add("{");
        specials.Add("}");
        specials.Add("|");
        specials.Add(":");
        specials.Add("\"");
        specials.Add("<");
        specials.Add(">");
        specials.Add("?");
    }

    private void AboutButton_Click(object sender, EventArgs e)
    {
        MessageBox.Show("Pig Latin is a fake language. It works by taking the first letter (Or two if it's a pair like 'th' or 'ch') and bringing it to the end, unless the first letter is a vowel. Then add 'ay' to the end. So 'bus' becomes 'usbay', 'thank' becomes 'ankthay' and 'apple' becomes 'appleay'.", "About:", MessageBoxButtons.OK, MessageBoxIcon.Information);
    }
}

}

如果您输入“香蕉苹果剪巧克力西奥多火车”，则输出：

“ananabay appleay earshay ocolatechay heodoreTay aintray”重复了 10 次以上。

顺便说一句：对不起，如果你不能回答，因为我知道有很多代码。但这没关系，因为它仍然有用。只是它不应该发生并让我紧张。而且我知道还有很多故障和更多工作要做，但我想先解决这个问题。

4

3 回答 3

1

您正在为每个特殊字符迭代一次您的单词。您foreach要检查您的单词并进行翻译，foreach以检查文本框是否包含任何特殊字符。

换句话说，您将对每个特殊字符进行一次翻译。

你会想把foreach (String part in parts)你的foreach (string s in specials)

于 2015-12-20T02:27:52.613 回答
1

您的循环中有一些逻辑问题。

你的外循环：

foreach( string s in specials ) {

...正在遍历您的特殊字符列表中的所有 42 个字符。

你的内循环

foreach( String part in parts ) {

...然后执行 42 次。因此，对于您的六个单词示例，您实际上进行了 252 次猪拉丁语转换。

如果从外部提取内部循环，您的结果会更好。像这样：

foreach( string s in specials ) {
    if( TranslateBox.Text.Contains( s ) || TranslateBox.Text == "\"" ) {
        TranslateOutput.Text = "";
        MessageBox.Show( "No Special Characters!", "Warning!", MessageBoxButtons.OK, MessageBoxIcon.Warning );
        return;
    }
}

String[] parts = TranslateBox.Text.Split();
foreach( String part in parts ) {
    foreach( String v in vowels ) {
        if( part.Substring( 0, 1 ) == v ) {
            TranslateOutput.Text = TranslateOutput.Text + " " + part + "ay";
            break;
        }
        else {
            if( part.Substring( 0, 2 ) == "sh" || part.Substring( 0, 2 ) == "ch" || part.Substring( 0, 2 ) == "th" || part.Substring( 0, 2 ) == "tr" ) {
                string SwitchP = part.Substring( 2 ) + part.Substring( 0, 2 );
                TranslateOutput.Text = TranslateOutput.Text + " " + SwitchP + "ay";
                break;
            }
            else {
                string Switch = part.Substring( 1 ) + part.Substring( 0, 1 );
                TranslateOutput.Text = TranslateOutput.Text + " " + Switch + "ay";
                break;
            }
        }
    }
}

一个更简洁的实现是：

private void TranslateButton_Click( object sender, EventArgs e )
{
    char[] specials = "`1234567890-=[]\";',./~!@#$%^&*()_+{}|:\\<>?".ToArray();
    char[] vowels = "aeiou".ToArray();

    TranslateOutput.Text = String.Empty;

    if( TranslateBox.Text.IndexOfAny( specials ) > -1 ) {
        MessageBox.Show( "No Special Characters!", "Warning!", MessageBoxButtons.OK, MessageBoxIcon.Warning );
        return;
    }

    String[] parts = TranslateBox.Text.Split();
    foreach( String part in parts ) {
        int firstVowel = part.IndexOfAny( vowels );
        if( firstVowel > 0 ) {
            TranslateOutput.Text += part.Substring( firstVowel ) + part.Substring( 0, firstVowel ) + "ay ";
        }
        else {
            TranslateOutput.Text += part + "ay ";
        }
    }

    TranslateOutput.Text = TranslateOutput.Text.TrimEnd();
}

在此示例中，我为特殊字符和元音创建了两个字符数组。然后我可以利用框架的IndexOfAny方法来搜索数组中的任何字符并返回第一次出现的索引。这将在第一个循环中找到第一个特殊字符（如果有），在第二个循环中找到第一个元音。一旦我从单词中获得了字符索引，我就可以将单词解析成猪拉丁语。请注意，我正在检查零作为元音索引，因为在猪拉丁语中，前导元音保持在原位，并且“ay”只是附加到单词的末尾。

于 2015-12-20T03:58:52.237 回答
1

您将代码嵌套在不应嵌套的两个循环中

foreach (string s in specials)

和

foreach (String v in vowels)

你的break陈述让你摆脱了一个麻烦，但不是另一个。

如果您使用.Any(...)谓词，则可以完全避免这些循环。

您的代码可能如下所示：

private void TranslateButton_Click(object sender, EventArgs e)
{
    TranslateOutput.Text = "";
    if (specials.Any(s => TranslateBox.Text.Contains(s)))
    {
        MessageBox.Show("No Special Characters!", "Warning!", MessageBoxButtons.OK, MessageBoxIcon.Warning);
    }
    else
    {
        String[] parts = TranslateBox.Text.Split();
        foreach (var part in parts)
        {
            var index = 1;
            if (vowels.Any(v => part.Substring(0, 1).ToLower() == v))
            {
                index = 0;
            }
            else if (new [] { "sh", "ch", "th", "tr", }.Contains(part.Substring(0, 2).ToLower()))
            {
                index = 2;
            }
            TranslateOutput.Text += " " + part.Substring(index) + part.Substring(0, index);
        }
    }
    TranslateOutput.Text = TranslateOutput.Text.TrimEnd();
}

这将其归结为foreach您实际需要的一个循环。

您还可以让您的代码初始化vowels并specials执行以下操作：

vowels.AddRange("aeiou".Select(x => x.ToString()));
specials.AddRange(@"`1234567890-=[]\;',./~!@#$%^&*()_+{}|:""<>?".Select(x => x.ToString()));
于 2015-12-20T06:36:11.927 回答