Trying to display an array of numbers onto label

Question

private static void Display(params Object[] things)
{
    foreach (Object num in number)
       Label5.Text = "{0} ", + num;
}

I'm not really sure what to really do. I want to display all the input numbers onto label5, but I'm not really sure how to do it.

For example: I have a textBox and the user enters in 23, 50, 89, 73, 40, etc into the textBox and I want to display the numbers the user entered into label5. Thank you.

ΦXocę 웃 Пepeúpa ツ · Answer 1 · 2016-10-15T19:31:14.983

2

You are overwriting the last value by every new iteration in the for loop

do instead this: put all the element in the array in one variable (use a StringBuilder) and then when that is done, set the content of the label with that variable

Example:

var sb = new StringBuilder();

foreach (Object num in number){
    sb.append(num);
}
Label5.Content = sb.ToString;

Edit

adding comma to separate the values:

var sb = new StringBuilder();
var comma = ""; 
foreach (Object num in number){
    sb.append(comma).append(num);
    comma = ", ";
}
Label5.Content = sb.ToString;

edited Oct 15 '16 at 19:31

answered Oct 15 '16 at 19:15

ΦXocę 웃 Пepeúpa ツ

47,427
17
69
97

The output of this would be `2350897340` missing the commas, also you missed `()` on the last `ToString` – Bojan B Oct 15 '16 at 19:28
Great, I upvoted :) – Bojan B Oct 15 '16 at 19:38

Dmitry Bychenko · Answer 2 · 2016-10-15T19:29:54.990

2

To concat IEnumerable<T> items into a string, try using String.Join:

Label5.Text = String.Join(", ", numbers);

edited Oct 15 '16 at 19:29

answered Oct 15 '16 at 19:23

Dmitry Bychenko

180,369
20
160
215

this is a pretty cool way... – ΦXocę 웃 Пepeúpa ツ Oct 15 '16 at 19:35
Surprising that noone come up with [that](http://stackoverflow.com/questions/26434458/how-to-print-the-contents-of-an-array-to-a-label-in-c-sharp) before... – Alexei Levenkov Oct 15 '16 at 19:36

score 0 · Accepted Answer · answered Oct 15 '16 at 19:15

0

If you want to display the text a user has entered into a textbox, why not assign it directly

label5.Text = textbox.Text;

The way you want to do it, to set it from an array you can do it like this:

private static void Display(params Object[] things)
{
    foreach (Object num in number)
       Label5.Text = Label5.Text + String.Format("{0}, " + num);
}

I don't quite understand though why you pass an array ob objects into the method if you don't use them.

answered Oct 15 '16 at 19:15

Bojan B

2,091
4
18
26

for somereason it says label5 does not exist in the current context, but it is displayed on the form. Am i doing something wrong? – Prodigy Oct 15 '16 at 19:18
how / where do you call the `Display` method? does it have to be static? – Bojan B Oct 15 '16 at 19:20
Nvm, I get it now thank you for your help. I call the display method on main and it doesnt have to be static. – Prodigy Oct 15 '16 at 19:21
You should also re-think if the `num` has to be an `Object` the same goes for the input params – Bojan B Oct 15 '16 at 19:24
yeah my book gave me a bad explanation or a brief explanation on when to use them. Can you help me understand them a bit? or give me a link where i can go read more about it? – Prodigy Oct 15 '16 at 19:28
Hm... you can try this and go from there https://msdn.microsoft.com/en-us/library/0f66670z.aspx. – Bojan B Oct 15 '16 at 19:33
thank you for your help. – Prodigy Oct 15 '16 at 19:36
use StringBuilder if you have large array to gain performance. – Vivek Nuna Oct 15 '16 at 19:37

Trying to display an array of numbers onto label

3 Answers3

Example:

Edit