aepnat / simhelpdesk

js/time.js

function startTime()

{ 	var today=new Date();

	var weekday=new Array(7);

The assignment to variable weekday seems to be never used. Consider removing it.

	var weekday=["Minggu","Senin","Selasa","Rabu","Kamis","Jumat","Sabtu"];

The variable weekday already seems to be declared on line 3. Consider using another variable name or omitting the var keyword.

	var monthname=new Array(12);

The assignment to variable monthname seems to be never used. Consider removing it.

	var monthname=["Jan","Feb","Mar","Apr","May","Jun","Jul","Aug","Sep","Oct","Nov","Dec"];

The variable monthname already seems to be declared on line 5. Consider using another variable name or omitting the var keyword.

	var dayname=weekday[today.getDay()];

	var day=today.getDate();

	var month=monthname[today.getMonth()]; 

	var year=today.getFullYear();

	var h=today.getHours();

	var m=today.getMinutes();

	var s=today.getSeconds();

	h=checkTime(h);

	m=checkTime(m);

	s=checkTime(s);

	document.getElementById('clocktime').innerHTML=dayname+", "+day+"-"+month+"-"+year+", "+h+":"+m+":"+s;

	t=setTimeout(function(){startTime()},500);

The variable t seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.t.

}

// function checkTime to add a zero in front of numbers<10

function checkTime(i)

{	if(i<10){i="0"+i;}

	return i;

}