変数名や関数名にも命名規則を定めるように心がけないと、回答者は匙を投げることになりかねません。ご質問のタイトルフォームの表示非表示の切り替え「フォーム」という言葉だけでは不明瞭に感じます。利用したデータの取り扱いにも、2つが考えられます。要素の属性だけで行う。「データの扱い方」を定めないと、作用するイベントリスナは構築できません。CSSのセレクタで取得する方法があります。内で多用するからこそ、次のように短く書く例は役立つかもしれません。qsa("input[name=radio1]");直接的な回答には至りませんが、参考になればと思います。評価が高い回答ほどページの上位に表示されます。remove低評価0回答の評価を下げる下記のような回答は推奨されていません。評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。21:53ありがとうございます。可読性については全く意識していませんでした。マークアップから勉強し直します。ありがとうございます。可読性については全く意識していませんでした。マークアップから勉強し直します。strict'を使うなら宣言していない変数を使わない。評価が高い回答ほどページの上位に表示されます。remove低評価0回答の評価を下げる下記のような回答は推奨されていません。評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。03:35ありがとうございます。ご指摘いただいたか所は修正しました。ありがとうございます。ご指摘いただいたか所は修正しました。03:43radioに何個のラジオボタンが入っているかを調べてみましょう。radioに何個のラジオボタンが入っているかを調べてみましょう。