PLEASE Help with simple mills timer

Projects Programming
1

I have been trying to get this for 4 days. I have read and looked at example after example. All the examples I can find are for blink, one shot,etc. I just want a simple timer to:

when torButton is pushed
turn on torRelay and torLed
hold them on for a set time and after times out
turn off torRelay and torLed and be ready for the next torButton push

I'm an old man and new at anything like this, but I'm trying to learn. Any help is appreciated.

int LDR = A1;     //analog pin to which LDR is connected
int LDRValue = 0;      //that's a variable to store LDR values
int light_sensitivity = 650;    //This is the approx value of light surrounding your LDR

unsigned long ThermoTimer;

const byte torLed = 13;//thermostst override on ledRED
const byte torRelay = 18; //short thermostat output
const int pin_torButton = 6; //button to short thermostat output RED
bool pushed_torButton = false;

void setup ()//(void)
{
  Serial.begin(9600);
  pinMode (torLed, OUTPUT);
  pinMode (torRelay, OUTPUT);
  pinMode (pin_torButton, INPUT_PULLUP);
}

void loop ()
{
  bool prev_torButton = pushed_torButton;
  pushed_torButton = digitalRead (pin_torButton) == LOW;

  LDRValue = analogRead(LDR);
  Serial.println (LDRValue);

  if (pushed_torButton) // thermostat override buttonpushed
  {
    ThermoTimer = millis();
    digitalWrite(torRelay, HIGH);//turn thermostat override relay on
    digitalWrite(torLed, HIGH);//turn thermostat override relay led on
  }
  if (millis() - ThermoTimer > 600UL)
      digitalWrite(torRelay, LOW);//turn thermostat override relay off
      digitalWrite(torLed, LOW);//turn thermostat override relay led off
}
}
2 
int LDR = A1;     //analog pin to which LDR is connected
int LDRValue = 0;      //that's a variable to store LDR values
int light_sensitivity = 650;    //This is the approx value of light surrounding your LDR

unsigned long ThermoTimer;

const byte torLed = 13;//thermostst override on ledRED
const byte torRelay = 18; //short thermostat output
const int pin_torButton = 6; //button to short thermostat output RED
bool pushed_torButton = false;

void setup ()//(void)
{
  Serial.begin(9600);
  pinMode (torLed, OUTPUT);
  pinMode (torRelay, OUTPUT);
  pinMode (pin_torButton, INPUT_PULLUP);
}

void loop ()
{
  bool prev_torButton = pushed_torButton;
  pushed_torButton = digitalRead (pin_torButton) == LOW;

  LDRValue = analogRead(LDR);
  Serial.println (LDRValue);

  if (pushed_torButton) // thermostat override buttonpushed
  {
    ThermoTimer = millis();
    digitalWrite(torRelay, HIGH);//turn thermostat override relay on
    digitalWrite(torLed, HIGH);//turn thermostat override relay led on
  } //millis(), not millis 
  if (millis() - ThermoTimer > 600UL) //This will only turn it on for 0.6 seconds (600 milliseconds) - is that right?
      digitalWrite(torRelay, LOW);//turn thermostat override relay off
      digitalWrite(torLed, LOW);//turn thermostat override relay led off
}
}
3

You had millis instead of millis()

millis is the function itself
millis() calls the function and evaluates to what the function returns.

(sorry, slipped up and put this in second post instead of after the fixed code.

4

Thanks Doc, but I still have something wrong. After that fix the LED and relay don't come on at all. What else did I @#$% up?