Having interrupts turned off between iterations of loop() is no good idea.
As I already have pointed out to you in your other thread
You should only disable interrupts for very short times, e.g. only for a couple of instructions to transfer highly volatile variables.
Instead of using noInterrupts() I'd rather use ATOMIC_BLOCK for such cases.