Using millis for counting and cummulating a button push time

Thanks

I came up with this and it works

void loop() {

unsigned long NowMillis = millis();

if (NowMillis - BeforeMillis >= 1000 )

{
BeforeMillis = NowMillis;

if (digitalRead (BUTTON_PIN) == 0) // Pressed
{ CumulatedUpTime++;
}

else
{
CumulatedUpTime = CumulatedUpTime;
}

}

Serial.print("CumulatedUpTime = ");
Serial.print(CumulatedUpTime);

Serial.print("digitalRead ( BUTTON_PIN) = ");
Serial.println(digitalRead (BUTTON_PIN));
}