Timer fading problem

[B-Navigator]

Trying to put together a simple javascript code to make a series of text snips fade into and out of visibility in random intervals. It isn't working now, but it's not producing any specific erors to focus debugging on either. Any thoughts on what I'm doing wrong?:

var CommentList = ['Com1','Com2','Com3','Com4','Com5','Com6','Com7','Com8','Com9','Com10','Com11','Com12','Com13','Com14','Com15','Com16'];
var CommentOn = [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0]

function SetOpacity(ComID,Value){
document.getElementById(ComID).style.opacity = Value;
}
function resetOn(OnID){
CommentOn[OnID] = 0;
}

function CommentTwinkle(comment){
document.getElementById(CommentList[comment]).style.visibility=visible;
CommentOn[comment] = 1;
for (i = 0; i <= 1; i = i + 0.05) {
    setTimeout("setOpacity(" +CommentList[comment]+","+ i + ")", i * 3000);
    setTimeout("setOpacity(" +CommentList[comment]+","+ (1-i) + ")", 8000+(i * 3000));
    setTimeout(resetOn(comment),12000)
}
Rand = 0;
CueIt=0;
function CueComment(){
while (CueIt == 1){
Rand = math.random()*(CommentList.count);
CueIt = CommentOn[Rand];
}
CommentTwinkle(Rand);
setTimeout('CueComment()',1000)
}
document.body.onload=CueComment();

[Logic Ali]

A few things that stand-out immediately are:

document.body.onload=CueComment();

should be document.body.onload=CueComment; although it's more usual to attach the handler to window.

function CueComment(){
while (CueIt == 1){
Rand = math.random()*(CommentList.count);
CueIt = CommentOn[Rand];
}
CommentTwinkle(Rand);
setTimeout('CueComment()',1000)
}

Nothing ever sets the value of CueIt to 1.

math.random()

should be Math.random()

[B-Navigator]

Thanks for the assistance, the errors pointed out, plus a missing } has definately helped, so now the trouble looks to be down to a one line consideration, though I recall having trouble with it before.

var CommentList = new Array('Com1','Com2','Com3','Com4','Com5','Com6','Com7','Com8','Com9','Com10','Com11','Com12','Com13','Com14','Com15','Com16');

document.getElementById(CommentList[comment]).style.visibility="visible";

getElementById keeps reading null when I try using an array to simplify defining what the ID is, anyone know what problem I'm missing?

[Logic Ali]

Can't be certain without seeing it in context, but either comment is out of range, or the elements don't exist at the point of execution.