How do I Add value to a list every time you click a button. Then display list in text box?

Code (CSharp):

1. public void onClick()
2.     {
3.         List<Studio> beatStorage = new List<Studio>();
4.         beatmade++;
5.         beatStorage.Add(new Studio("Beat" + beatmade, ga.beatScore));
6.         //beatStorage.Sort();
7.  
8.         foreach(Studio value in beatStorage)
9.         {
10.             beatText.transform.SetParent(parentPanel.transform);
11.             beatText.text = value.beatNames + "\n";
12.             beatText.color = Color.black;
13.             beatText.fontSize = 14;
14.         }
15.  
16.     }

That's the function that's called when you click a button. But it only shows one value. What do I do?

 

You are instantiating a new list each OnClick, declare and instantiate the list outside your onClick method.

Also you'll want to set the text color, position and fontSize outside of the function. Then t oadd the text to the textbox, use:

Code (CSharp):

1. beatText.text += value.beatNames + "\n";

And for next time, when posting code always use code tags. That makes is a lot better readable for everyone.

 

Sorry fixed the code. Also If I take the new list and beatStorage.add lines outside the code I wouldn't be able to access it in the foreach statement correct?

 

As long as they are in the same class you can access them as they are in the same scope.

 

Looks like it's working. Should I be doing all the text stuff in void start. Then the beatStorage.add in another functions that's called everytime you click the button?

 

Something like that. I was just re-reading the code and noticed I missed some things (see how good code tags are? ).

I quickly made this:

Code (CSharp):

1. List<Studio> beatStorage = new List<Studio>();
2.  
3. // Call this for example in Start()
4. public void SetText()
5. {
6.         beatText.transform.SetParent(parentPanel.transform);
7.         beatText.color = Color.black;
8.         beatText.fontSize = 14;
9. }
10.  
11. public void onClick()
12. {
13.     beatmade++;
14.     string beatName = "Beat" + beatmade;
15.     beatStorage.Add(new Studio(beatName, ga.beatScore));
16.  
17.     // No need to rewrite the entire textbox each time you click a button,
18.     // you should be able to just add the new beat to the textbox.
19.     beatText.text += beatName + "\n";
20. }

 

Thanks it's working that's great. The other problem I have is I want to randomly generate a number and save it to each beat. The variable ga.beatscore gives me an ArgumentException error. I tried moving the randomgenerator if function into the onClick function but that doesn't work.

Code (CSharp):

1. if (pm.level == 1)
2.          {
3.         float randomnumber = Random.Range(1, 100);
4.         if (randomnumber <= 3)
5.         {
6.             beatRating = 5;
7.         }
8.         else if (randomnumber > 3 && randomnumber < 7)
9.         {
10.             beatRating = 4;
11.         }
12.         else if (randomnumber > 7 && randomnumber < 20)
13.         {
14.             beatRating = 3;
15.         }
16.         else if (randomnumber > 20 && randomnumber < 50)
17.         {
18.             beatRating = 2;
19.         }
20.         else if (randomnumber > 50 && randomnumber <= 100)
21.         {
22.             beatRating = 1;
23.         }
24.          }