0
\$\begingroup\$

So, I have a toggle button to change a bool value, in this script 't1Bool'. The toggle button is working because when I use a Debug.log I can see the value changing from true o false when I press the button. When using the Debug.log with the variable turbina1 the value is always zero. I already checked the if condition syntax and don't know why this value is not being updated.

using System;
using UnityEngine;
namespace Assets.Code.PowerPlants
{
    public class HydroElectric
    {
        public bool t1Bool;
        public bool t2Bool;
        public bool t3Bool;

        int turbina1;
        int turbina2;
        int turbina3;

        public float prod;

        public HydroElectric ()
        {   
         t1Bool = true;
         t2Bool = true;
         t3Bool = false;
         prod = 0f; 
        }

        public float HydroControlPanel ()
        {
            turbina1 = t1Bool ? 2 : 0;
            turbina2 = t2Bool ? 3 : 0;
            turbina3 = t3Bool ? 1 : 0;

            prod = turbina1 + turbina2 + turbina3;
            Debug.Log (turbina1);
            return prod;
        }
    }
}

This is the code for the button:

public void ShowIt()
{
            ec.t1Bool = GUI.Toggle (new Rect (25, 55, 100, 50), ec.t1Bool, "Turbina 2 MW");

            ec.t2Bool = GUI.Toggle (new Rect (25, 95, 100, 50), ec.t2Bool, "Turbina 3 MW");

            ec.t3Bool = GUI.Toggle (new Rect (25, 135, 100, 50), ec.t3Bool, "Turbina 1 MW");

            GUI.Box (new Rect (Screen.width - 100, 60, 80, 25), ec.prod.ToString ());      // PRODUCED ENERGY

        }

This is where the object instance is created (it is in the same script as the button):

public class PlayState : IStateBase
    {
        private StateManager manager;
        private HydroElectric ec;
        public PlayState (StateManager managerRef)
        {
            manager = managerRef;
            Debug.Log ("Constructing PlayState");
            ec = new HydroElectric();
        }

Any hint?

\$\endgroup\$
5
  • \$\begingroup\$ turbina1 is marked as static, are you sure you meant for it to be static? \$\endgroup\$ Commented Dec 28, 2015 at 20:07
  • \$\begingroup\$ it seems not, I've already removed it but the problem persists. \$\endgroup\$
    – ClaudioA
    Commented Dec 28, 2015 at 20:11
  • \$\begingroup\$ The problem is with only turbina1 or with all three? \$\endgroup\$ Commented Dec 28, 2015 at 20:14
  • \$\begingroup\$ Can we see the code for the button? \$\endgroup\$ Commented Dec 28, 2015 at 20:14
  • \$\begingroup\$ The problem is with all three. I've just added the code for the button. \$\endgroup\$
    – ClaudioA
    Commented Dec 28, 2015 at 20:15

1 Answer 1

1
\$\begingroup\$

In last line, it should be ec.HydroControlPanel().ToString() instead of ec.prod.ToString() in ShowIt method

\$\endgroup\$
8
  • 1
    \$\begingroup\$ To explain: the button updated the internal on/off state, but HydroControlPanel() which calculates the total output based on that internal on/off state is never called, thus prod is never updated. \$\endgroup\$ Commented Dec 28, 2015 at 20:26
  • \$\begingroup\$ @Draco18s thanx man, m on cell phone. Thanx for explaining. And u r welcome ClaudioA :) \$\endgroup\$ Commented Dec 28, 2015 at 20:36
  • \$\begingroup\$ @HamzaHasan No problem. I know the joys (read "troubles") of trying to type anything longer than four words on a mobile device (mine has a quirk dating back a few months where the learned dictionary words got completely wiped, along with a number of real words, like "do." I cannot type "do" on my tablet except through painstaking manuality, if I forget I get "so" "Co" "dip" "drop" or something else). \$\endgroup\$ Commented Dec 28, 2015 at 20:40
  • \$\begingroup\$ @Draco18s A big LOL :D \$\endgroup\$ Commented Dec 28, 2015 at 20:42
  • \$\begingroup\$ @HamzaHasan Its also conveniently stopped learning new words 9.9 Right pain in the arse. Used to take it about six to ten usages of a word or proper noun for me to be able to use it in swipe-typing ("continuous input") but since the "update" I haven't seen it learn a single new word. \$\endgroup\$ Commented Dec 28, 2015 at 20:46

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .