いつ服用しますか?

JavaScript部分

function noticeIt() {
	var msg = document.getElementById("msg");
	msg.innerHTML = checkedValues();
	}
function checkedValues() { //チェックボックスを見つける
	var times = document.getElementById("times")
	var cinput = times.getElementsByTagName("input");
	var cvals= ""; //表示する文章を返す
	var cnum = 0; //チェックボックスの数を数える
	for (var i = 0; i < cinput.length; i ++) {
		if (cinput[i].checked == true) {
			cvals += cinput[i].value;
			cnum ++;
			}
		}
	if (cnum > 0) {
		cvals += "の" + cnum + "回、服用してください";
		}
	else {
		cvals = "1回は飲まなきゃだめですよ";
		}
	return cvals;
	}

body部分

<p>いつ服用しますか?</p>
<div id="times">
	<input type="checkbox" value="朝">朝
	<input type="checkbox" value="昼">昼
	<input type="checkbox" value="晩">晩
</div>
<p><input type="button" value="反映!" onclick="noticeIt()" /></p>
<div id="msg"></div>