OnChange is for one change. If the radio button is unchecked and then checked, that is one change.
Not only have you not closed your input tags, you have declared two inputs with the same name. This already causes unexpected behavior.