2017-10-16 133 views
-2

我需要一些帮助完成这个简单的名字“发电机”
我们刚开始在高中编程,所以我想要一个更简单的解决方案。这是我的一切,现在,但在文本框中输出System.String[]简单的名字生成器在C#

private void Btn_gen_Click(object sender, EventArgs e) 
{ 
    textbox.Text = textbox.Text + name((int) count.Value) + Environment.NewLine; 
} 

private string name(int count) 
{ 
    string heslo = ""; 
    string[] names = new string[] {"Abby", "Uther", "Thomas", "Michelle", "Abraham", "Bendy"}; 
    string[] surnames = {"The Mighty", "The Clumsy", "The Strong", "The Lightbringer", "The Pyromancer", "The Necromancer"}; 
    Random gnč = new Random(); 
    for (int i = 1; i <= count; i++) 
    { 
    int nč = gnč.Next(0, names.Length); 
    heslo = heslo + names; 
    } 
    return heslo; 
} 
+4

用 - > heslo = heslo + names [nc]替换循环中的最后一条语句; ,如果你想打动你的老师将“heslo”字符串改为StringBuilder并使用append。当“计数”超过3-4时,性能更好 –

+0

问题是什么?这个问题适用于Stackoverflow,如果它有一个bug并且你不知道如何解决它。如果你的代码可以工作,但你只是想要一个评论,请尝试发布[CodeReview StackExchange](https://codereview.stackexchange.com/)。 –

回答

3

的问题是,你是不是追加随机名称,而是类型名称System.String[],这是正规型你的names变量名称,你的字符串。

变化heslo = heslo + names

heslo = heslo + names[nč]; 

这将需要一个随机元素出来的名字排列的。