Help with Arrays - Check if only 1 is toggled

Question

Hey guys,

I don't know if I'm going about t$$anonymous$$s the right way. Basically I need to be able to click a toggle, and when I click the toggle, I need to change the alpha of a few buttons. I wanted to do it in an Array because I'll be adding more buttons called Slots later

 public void ShowAllSlot()
     {
         
         GameObject[] buildOptionsFinder = GameObject.FindGameObjectsWithTag("BuildOptions");
 
         buildOptions = new Toggle[buildOptionsFinder.Length];
 
         for (int i = 0; i < buildOptionsFinder.Length; i++)
         {
             buildOptions[i] = buildOptionsFinder[i].GetComponent<Toggle>();
 
 
 
 
             //Debug.LogError(buildOptions[1]);
             if (buildOptions[i].isOn )
             {
                 GameObject[] slotFinder = new GameObject[transform.c$$anonymous$$ldCount];
                 
                 for (int ii = 0; ii < transform.c$$anonymous$$ldCount; ii++)
                 {
                     slotFinder[ii] = GameObject.Find("Level1Slots").transform.GetC$$anonymous$$ld(ii).gameObject;
 
                     Image button = slotFinder[ii].transform.GetC$$anonymous$$ld(0).transform.GetC$$anonymous$$ld(0).transform.GetComponent<Image>();
 
                     
 
                     Debug.LogError("IT SHOULD BE SHOWING NOW");
                     button.color = Alpha40;
                     
                 }
             }
 
   
 
 
         }
 
     }

I can't seem to figure a way to say :

if (1 out of all buildOptions[i].isOn == true) do t$$anonymous$$s else if (all buildOtions[i].isOn == false) do t$$anonymous$$s

Thanks!

Answer 1

Best Answer

Answer by flashframe · Sep 21, 2015 at 09:36 PM

You could use a bool and a foreach loop to check that at least one of the toggles is selected. Somet$$anonymous$$ng like t$$anonymous$$s (I haven't tested t$$anonymous$$s code sorry, just writing it from memory).

 bool isToggled = false; //declared at the start of your class
 
 foreach(Toggle t in buildOptions){
     if(t.isOn){
         isToggled = true;
     }
 }
 
 if(isToggled){
     //Do somet$$anonymous$$ng
 }else{
     //Do somet$$anonymous$$ng else
 }
 
 isToggled = false;

2 Show 1 · Share

Hathakas · Sep 21, 2015 at 11:50 PM 0

Share

Thank you so much! That totally worked!

Answer 2

Answer by Budoray · Sep 22, 2015 at 03:45 AM

I may be misunderstanding your question, but in the event that I'm not, here is an example that uses Linq to show another way to tackle your problem. Your question read to me as if you only wanted to do somet$$anonymous$$ng if only one true element was in an array and to do somet$$anonymous$$ng else otherwise. The above for loop runs the risk of changing the toggled flag back to false if the last element happens to be false.

In your case, the test would be more like if(buildOptions.Any(x => x.isOn) ... or if you're just looking for the count to be one and only one, if(buildOptions.Count(x => x) == 1).

 using System;
 using System.Linq;

 public static void Main()
 {
         var n1 = new bool[3] { true, true, true };
         var n2 = new bool[3] { true, false, false };
         var n3 = new bool[3] { false, false, false };
 
         if (n1.Any(x => x)) // short for (x => x == true)
         {
             Console.WriteLine("N1 has at least one that is true");
         }
         
         if (n2.Any(x => x))
         {
             Console.WriteLine("N2 has at least one that is true");
         }
         
         if (n3.All(x => !x)) //short for (x => x != true)
         {
             Console.WriteLine("N3 has none that are true");
         }
         
         // Count
         var n1TrueCount = n1.Count(x => x);
         var n2TrueCount = n2.Count(x => x);
         var n3TrueCount = n3.Count(x => x);
         
         Console.WriteLine("N1 True Count: " + n1TrueCount);
         Console.WriteLine("N2 True Count: " + n2TrueCount);
         Console.WriteLine("N3 True Count: " + n3TrueCount);
         
         
         //Testing Count
         if(n1.Count(x => x) == 3)
         {
             Console.WriteLine("Do T$$anonymous$$s");
         }
 }

1 Show 7 · Share

flashframe · Sep 22, 2015 at 12:55 PM 0

Share

Just to clarify, my solution doesn't run the risk of setting the flag to false if the last element happens to be false.

Budoray flashframe · Sep 22, 2015 at 01:16 PM 0

Share

No. It won't set it to false. I said it is risky. Using a for loop in the manner you showed is risky and inferred that care must be taken to manage that risk. My intent was not to challenge your programming ability or style. We all write code a bit differently. My apologies if I have offended you.

Hathakas · Sep 22, 2015 at 02:56 PM 0

Share

Thank you for the help! I'm still very new at programming so I'm happy to learn a different route. The first answer seems to work for what I need, but if you don't mind clarifying the risk factor so I can learn from it. Also, the script seems quite complicated lol but I wan't to know more about it. Can you refer me to were I can read up on it?

Budoray Hathakas · Sep 22, 2015 at 03:18 PM 0

Share

The risk is that the code uses a foreach loop to set a variable outside the loop with no break or way to exit the for loop until it loops through every item. Again, this isn't necessarily wrong. It's something that you have to be aware of and manage.

Take the code I shared and paste it into dotnetfiddle.net. Manipulate the code. Try other things with the code. The snippet of code I shared is not what you would use in your code. It only demonstrates how to use linq in a similar manner as a solution to your question. For testing and learning, replace n1, n2, n3 with buildOptions1, buildOptions2, buildOptions3. Once you get a grasp of what's going on with the snippet I shared, then you can go back to your code and change it if you wish.

if(buildOptions.Any(x => x.isOn) // same as if(buildOptions.Any(x => x.isOn == true) { // Do something }

If buildOptions has any toggle (x) such that (=>) toggle.isOn Do something

Does that help? Here's another primer for linq. http://www.dotnetperls.com/linq

Hathakas Budoray · Sep 22, 2015 at 10:08 PM 0

Share

Thank you for the information! I've been meaning to learn about Linq! I think I'm starting to get it. Really appreciate it.

Show more comments

flashframe · Sep 22, 2015 at 03:02 PM 0

Share

@Budoray No offence taken. I appreciate my comment was a bit curt, sorry. I too would like to know the risk if you could clarify.

Help with Arrays - Check if only 1 is toggled

2 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions