特定のラジオボタンが選択されたらテキストフィールドを入力可

質問

下のような6つのラジオボタンと1つのテキストフィールドのあるフォームがあります。
「あ・い・う・え・お」のラジオボタンが選択されたらテキストフィールドをdisabledに、「か」なら入力可能にしたくて、JavaScriptで次のように記述しました。
一応動作しているように見えるのですが、見よう見まねで書いた初めてJavaScriptなので大丈夫なのか不安です。これで良いのでしょうか。

function txtClose () {
if (document.f_frm.motive[5].checked) {
document.f_frm.motive_add.disabled = false;
} else {
document.f_frm.motive_add.disabled = true;
}
}

<form name="f_frm">
<input type="radio" name="motive" value="あ" onclick="txtClose();" />あ<br />
<input type="radio" name="motive" value="い" onclick="txtClose();" />い<br />
<input type="radio" name="motive" value="う" onclick="txtClose();" />う<br />
<input type="radio" name="motive" value="え" onclick="txtClose();" />え<br />
<input type="radio" name="motive" value="お" onclick="txtClose();" />お<br />
<input type="radio" name="motive" value="か" onclick="txtClose();" />か<br />
<input type="text" name="motive_add" size="20" maxlength="40" onclick="txtClose();" />
</form>

回答No.1

kazumero · ベストアンサー

正常動作するので問題ありません。
ただ、テキストフィールドのonclick属性は不必要です。
ラジオボタンが選択された時点で、テキストフィールドの入力可不可が決まりますので。

細かいことを言いますと、
<input type="radio" name="motive" value="あ" onclick="txtClose();" />あ<br />
でなく、
<p><input type="radio" name="motive" value="あ" onclick="txtClose();" />あ</p>
のように、<form>内ではbr要素は使用せずにp要素などのブロック要素で囲うのがベストです。

お礼

>テキストフィールドのonclick属性は不必要
よく考えると、確かにその通りです。
気付いていませんでした！ありがとうございます。

フォーム内、実際はテーブルの<td>～</td>で囲まれている部分だったのですが、質問の際にその旨を記述していませんでした。すみません。

素早い回答にとても助かりました。ありがとうございます！！