UKHeliBob:
if ((LED1 == HIGH) && (currentMillis - previousMillis >= OnTime))2 lines previously you read the state of LED1 (or at least try to) and throw away what is returned. Then in the line above you test the value of the LED1 pin not the state of the pin.
Have you tried compiling the program ?
Several errors are reported
This one is not with compiling issues
const int LED1 = 2;
const int LED2 = 4;
const int KNOP = 7;
unsigned long previousMillis = 0;
long OnTime = 500;
long OffTime = 750;
int ledState = LOW;
int Status ;
unsigned long vard = 0;
unsigned long var = 0;
void setup () {
pinMode (LED1, OUTPUT);
pinMode (LED2, OUTPUT);
pinMode (KNOP, INPUT);
}
void loop(){
Status = digitalRead(KNOP);
if(Status == HIGH){
LEDaan ();
LEDuit2();
}
else {
LEDuit ();
LEDaan2 ();
}
}
void LEDaan() {
while(vard < 400){
vard++;
digitalRead(LED2);
unsigned long currentMillis = millis();
if((LED2 == HIGH) && (currentMillis - previousMillis >= OnTime))
{
ledState = LOW;
previousMillis = currentMillis;
digitalWrite(LED2, LOW);
}
else if ((LED2 == LOW) && (currentMillis - previousMillis >= OffTime))
{
ledState = HIGH;
previousMillis = currentMillis;
digitalWrite(LED2, HIGH);
}
}
}
void LEDuit(){
digitalWrite(LED2, LOW);
}
void LEDaan2(){
while(var < 200){
var++;
digitalRead(LED1);
unsigned long currentMillis = millis();
if ((LED1 == HIGH) && (currentMillis - previousMillis >= OnTime))
{
ledState = LOW;
previousMillis = currentMillis;
digitalWrite(LED1, LOW);
}
else if ((LED1 == LOW) && (currentMillis - previousMillis >= OffTime))
{
ledState = HIGH;
previousMillis = currentMillis;
digitalWrite(LED1, HIGH);
}
}
}
void LEDuit2(){
digitalWrite (LED1, LOW);
}