if else commands

The logic for playing the tones looks wrong.

Even if you turn the tone on, it will immediately be turned off by the next if statement, won't it ?