Question by TooLazyToBeOriginal · May 30, 2021 at 07:36 AM · arrays

Array displaying incorrect info when I try to show the array outside of Update loop

Console Log As you can see the values alternate between 0's and actual values :(. Below is shorterned code I use in my program. The class is just monobehaviour.

     int[] BallOrder = new int[5]; // me initializing the variable in the beginning of the program

     // int[] BallOrder = {4,6,3,2,7}; if I do this instead of showing 0's it 
     shows 4,6,3,2,7 when I click the object.
     void Start() {
     CreateBalls(5);
     }

     void CreateBalls(int HowMany) {
     for (int i = 0; i < HowMany; i++)  {
     BallClone = Instantiate(Ball);
     BallPic = BallClone.GetComponent<SpriteRenderer>();
     // give each ball a random image, each image is a number
     RandImg(); 
     // just using i for testing purposes.        
     BallOrder[i] = i;
     // array should just be 0,1,2,3,4,
 }
 }

   void ShowList() 
   {
 print(string.Join(" ", BallOrder));
 }

   void Update()
   {
 // here it shows the array correctly 0,1,2,3,4
 ShowList();
   }
    void OnMouseDown() {
    string BallNum = ConvertSprite(GetComponent<SpriteRenderer>().sprite);
    int test = int.Parse(BallNum);
    // here when I show list it just shows 0,0,0,0,0 or 4,6,3,2,7 depending on how I declared it
    ShowList(); 
    Destroy(gameObject);
     }

console.png (16.3 kB)

Add comment · Show 5

CodesCove · May 30, 2021 at 04:53 PM 0

Here are few things to check: Your console shows 6 values printed and your array has size of 5. Are you sure you have the right monobehaviour attached to GO (most probably not)? Also what are the other numbers than the zero (2 20 ..) in the console, how are these the actual ones as you say?? Also are you sure the mono is attached only to one GO and only once? You can add the object name (or instance id) to you print to check this. Also don't use print when debugging but use Debug.Log(..). You also destroy the gameobject this script is attached in onmousedown.

Well anyway I think the console print you are showing is not from the script you posted here and that is the problem (at least one of them)..

TooLazyToBeOriginal · May 30, 2021 at 06:07 PM 0

What do you mean by "right monobehaviour attached to GO"? I'm new to unity so not sure what that is.
I picked randoms number for when I ran the console log and choose 0,1,2,3,4,5 when I was testing out. so despite the picture showing random numbers, it was because I used a random number generator instead. My bad
I say actual ones because each ball has an image with a number on it and the image name is a number so I can easily check what's correct or not correct.
I checked and this script is only attached to one game object, I make duplicates and wonder if that is the issue.
how do I add object name (or instance id) to the Debug.Log?
I removed destroy and if I click again it still shows 0's

Show more comments

TooLazyToBeOriginal · May 30, 2021 at 07:35 PM 0

So I replaced the print with the line u specified and my instance id number is different. Is it bad that each ball has its own instance or is it fine (probably not but just to clarify). The parent gameobject is displaying the correct one but not any of the child ones. aka the clones. So the new values are only added to the parent gameobject (where I have the script attached to) and not to any of the clones. So I believe I have to make the clones first then add to the list.

Show more comments