猪拉丁语翻译吐出多行?C#

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

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 earsshay ocolatechay heodoreDay aintray"重复10多次。

BTW：很抱歉你不能回答，因为我知道有很多代码。但这并不重要，因为这个东西仍然有用。只是这不应该发生，让我很紧张。我知道还有很多小故障，还有很多事情要做，但我想先解决这个问题。

您将代码嵌套在两个循环中，而这两个循环不应该嵌套在中

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()));

对于每个特殊字符，您将对单词进行一次迭代。您的foreach用于浏览您的单词并进行翻译，它位于foreach内部，用于检查文本框是否包含任何特殊字符。

换句话说，你要为每个特殊的字符做一次翻译。

您需要将foreach (String part in parts)从foreach (string s in specials) 中移出

循环中存在一些逻辑问题。

你的外环：

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"只是附加在单词的末尾。

相关内容

最新更新

热门标签：