選択済みのラジオボタンを押すと選択が解除される

Question

###選択済みのラジオボタンを押すと選択が解除される
上記の通りの実装を考えております。下記のソースコードのようにラジオボタンが1セットのみでしたら、下記の通りのソースコードでも問題がないのですが、ラジオボタンのセットが30組近くある機能の中で、選択済みのラジオボタンを押すと選択が解除されるよう改修していくことになり、下記のようなソースコードだと、他のラジオボタンのセット(他のname)がある場合バグが生じてしまいます。例えば変数nowSelectedに1を保存し、他のセットのラジオボタンのvalue="1"のを選択したときなどです。これの対応策としてこちらのサイトの対応策も考慮しましたが、ラジオボタンのセットが30組近く考えると処理が重くなることが懸念され、できれば選択されているラジオボタンに値があっているかどうかだけで判断してラジオボタンをリセットしたいと思っているのですが、どう実装するべきか行き詰っています。もし他の解決策があった場合にも教えて頂けると嬉しいです。宜しくお願い致します。

###該当のソースコード

<label for="radio1">
    <input type="radio" name="hogehoge" id="radio1" value="1">
    <span>ラジオボタン1</span>
</label>
<label for="radio2">
    <input type="radio" name="hogehoge" id="radio2" value="2">
    <span>ラジオボタン2</span>
</label>
<label for="radio1">
    <input type="radio" name="hogehoge" id="radio3" value="3">
    <span>ラジオボタン3</span>
</label>
<label for="radio4">
    <input type="radio" name="hogehoge" id="radio4" value="4">
    <span>ラジオボタン4</span>
</label>
<script type="text/javascript">
var nowSelected = '';
$('input[type="radio"]').on('click', function(){
    if (nowSelected === $(this).val()) {
        nowSelected = '';
        $(this).prop('checked', false);
    } else {
        nowSelected = $(this).val();
    }
});

Answer 1

複数ラジオボタングループ(name)に対応するために、nowSelectedをオブジェクトにすれば良いのではないでしょうか？名前はcheckedRadioValueにしていますが。

<label for="hogehogeRadio1">
    <input type="radio" name="hogehoge" id="hogehogeRadio1" value="1">
    <span>ラジオボタン1</span>
</label>
<label for="hogehogeRadio2">
    <input type="radio" name="hogehoge" id="hogehogeRadio2" value="2">
    <span>ラジオボタン2</span>
</label>
<label for="hogehogeRadio3">
    <input type="radio" name="hogehoge" id="hogehogeRadio3" value="3">
    <span>ラジオボタン3</span>
</label>
<label for="hogehogeRadio4">
    <input type="radio" name="hogehoge" id="hogehogeRadio4" value="4">
    <span>ラジオボタン4</span>
</label>

<br>

<label for="fooRadio1">
    <input type="radio" name="foo" id="fooRadio1" value="1">
    <span>ラジオボタン1</span>
</label>
<label for="fooRadio2">
    <input type="radio" name="foo" id="fooRadio2" value="2">
    <span>ラジオボタン2</span>
</label>
<label for="fooRadio3">
    <input type="radio" name="foo" id="fooRadio3" value="3">
    <span>ラジオボタン3</span>
</label>

<script type="text/javascript">
	// ラジオボタングループ(name)ごとにプロパティを定義、初期化しておく
	var checkedRadioValue = { 'hogehoge': '', 'foo': '' };

	$('input[type="radio"]').on('click', function() {
		var name = $(this).attr('name');
		var value = $(this).val();

		if (checkedRadioValue.name === value) {
			checkedRadioValue.name = '';
			$(this).prop('checked', false);
		} else {
			checkedRadioValue.name = value;
		}
	});
</script>

せっかくベストアンサーに選んで頂きましたが、バグがありました。
お詫びして訂正します。

<label for="hogehogeRadio1">
    <input type="radio" name="hogehoge" id="hogehogeRadio1" value="1">
    <span>ラジオボタン1</span>
</label>
<label for="hogehogeRadio2">
    <input type="radio" name="hogehoge" id="hogehogeRadio2" value="2">
    <span>ラジオボタン2</span>
</label>
<label for="hogehogeRadio3">
    <input type="radio" name="hogehoge" id="hogehogeRadio3" value="3">
    <span>ラジオボタン3</span>
</label>
<label for="hogehogeRadio4">
    <input type="radio" name="hogehoge" id="hogehogeRadio4" value="4">
    <span>ラジオボタン4</span>
</label>

<br>

<label for="fooRadio1">
    <input type="radio" name="foo" id="fooRadio1" value="1">
    <span>ラジオボタン1</span>
</label>
<label for="fooRadio2">
    <input type="radio" name="foo" id="fooRadio2" value="2">
    <span>ラジオボタン2</span>
</label>
<label for="fooRadio3">
    <input type="radio" name="foo" id="fooRadio3" value="3">
    <span>ラジオボタン3</span>
</label>

<script type="text/javascript">
	// ラジオボタングループ(name)ごとにプロパティを定義、初期化しておく
	var checkedRadioValue = { 'hogehoge': '', 'foo': '' };

	$('input[type="radio"]').on('click', function() {
		var name = $(this).attr('name');
		var value = $(this).val();

		if (checkedRadioValue[name] === value) {
			checkedRadioValue[name] = '';
			$(this).prop('checked', false);
		} else {
			checkedRadioValue[name] = value;
		}
	});
</script>
</body>

checkedRadioValue.name　→　checkedRadioValue[name]

Answer 2

やりたいことは

• 選択済みのラジオボタンをクリックすると、そのグループでの選択を解除する（どれも選択されていない状態にする）
• ただし他のグループには影響を与えてはいけない

ですよね。

ということは、click のイベントハンドラで、
「自分が既に選択状態だったら、自分の選択状態を解除する」
すなわち

  if ($(this).prop('checked') == true) {
    $(this).prop('checked', false);
  }

じゃいけないんでしょうか。

Answer 3

直前のチェック状態を保持しておいて、ラジオボタンのクリック時には「直前のチェック状態がチェック済み」であればチェック解除をを行うようにしてはどうでしょうか？

<html>
<head>
<script src="http://code.jquery.com/jquery-1.10.1.min.js"></script>
<script>

function savePreChecked(name) {
	var selector = "input:radio";
	if (name) { selector += "[name='" + name + "']"; }
	$(selector).each(function(){ this.preChecked = this.checked; });
}

$(function() {
	savePreChecked();
	$("input:radio").on("click", function() {
		if (this.preChecked == true) {
			this.checked = false;
		}
		savePreChecked(this.name);
	});
});

</script>
</head>
<body>

<div id="group1">
	<input type="radio" name="rdoGroup1">
	<input type="radio" name="rdoGroup1">
	<input type="radio" name="rdoGroup1">
</div>

<div id="group2">
	<input type="radio" name="rdoGroup2">
	<input type="radio" name="rdoGroup2">
	<input type="radio" name="rdoGroup2">
</div>

<div id="group3">
	<input type="radio" name="rdoGroup3">
	<input type="radio" name="rdoGroup3">
	<input type="radio" name="rdoGroup3">
</div>

</body>
</html>

Comments

[sha]

質問で処理が重くなることを懸念されているので、clickイベント内のsavePreCheckedは別関数としてループを対象nameの要素のみにしてはどうでしょうか

[ku__ra__ge]

重くなるのでは？という懸念があるようなのでクリック時の直前状態保持は、同じ名前のラジオボタンに限るようにしました。

Answer 4

if ($(this).prop('checked') == true) {
    $(this).prop('checked', false);
  }

ではクリックして選択された状態になることがないので
参考にされたサイトのように各nameごとの選択中の値をどこかに保持するのが良いと思います。
30×4程度ならそこまで重くならない(mac + Chromeで検証しました)と思います。

Comments

[tacsheaven]

そういえばそうですね。click に飛んできた時点で元がどちらであっても checked == true になってるから、必ず false になってしまうか。

[sha]

そうですね。onmouseupであらかじめ変更前の値をとる方法もあるかもしれませんがバインドするイベントが増えますし、参考サイト通りで問題ないと思いました。

Answer 5

参考リンクの通りにやったらうまく機能しませんでした。
7行目を "input.prop('checked', false);" とすることでうまく機能しました。