vaj4088:
Complete code should be shown.
CTRL-T in the Arduino IDE should be used to auto-format the code.
Interrupt Service Routines (ISRs) should execute quickly. Thus, they should NEVER contain a call to delay(...).
Instead of using delay(...) in the loop function to waste ten seconds of time, a millis() based approach could do the same job and would not need an ISR.
Arduino programmers usually avoid pins 0 and 1 because they are shared with the serial port.
I have changed the delay() to millis() and I get "too many arguments to function 'long unsigned int millis()'". I don't know what to do about this, I'm brand new to Arduino...