What is wrong with my code?
The fact that you haven't declared the necessary variables volatile?
The fact that you haven't removed one of the attachInterrupt calls?
You haven't told us what changes you've made. All you've done is defend stupid code.