Arduino Timer to turn on LED

Projects Programming
1

I need to create a program that can turn on and off a series of LED's based on the current time in hours. I have tried to create a program that will do this but the LED's remain on when the current time parameters meet the requirement to shut off. Here is a copy of my code. Can anyone point out what I am doing wrong. Thanks!

#include <Time.h>
#include <TimeLib.h>

int ledPins[] = {2,3,4,5,6,7};
int pinCount = 6;

void setup() {
Serial.begin(9600);
for (int thisPin = 0; thisPin < pinCount; thisPin ++){
pinMode(ledPins[thisPin], OUTPUT);
}
}

void loop() {

time_t t = now();
if(hour(t) < 6 || hour(t) >= 20){
for (int thisPin = 0; thisPin < pinCount; thisPin ++){
digitalWrite(ledPins[thisPin], HIGH);
}
}
if(hour(t) >= 6 || hour(t) <8){
for (int thisPin = 0; thisPin < pinCount; thisPin ++){
digitalWrite(ledPins[thisPin], LOW);
}
}
}

2 
int pinCount = 6;

You want to be able to change the number of pins at run time? If not, this should be const.

You want to be able to set the number of pins to a negative number? If not, the type should be unsigned.

You want to be able to set this to more than 255? If not, the type should be byte.

  time_t t = now();

Without some help, the Arduino has NO idea what time it is. There is NOTHING in your code to give the Arduino that help, so the time is based STRICTLY on how long the Arduino has been running.

if(hour(t) >= 6 || hour(t) <8){
Is that happy hour? Buy me a drink?