Hi Safdar,
Jscript is case sensitive so get<b>m</b>onth() and get<b>M</b>onth() are different, getmonth() is not a method of object Date, getMonth() instead is.
The same applies to getYear() instead of getyear();
Said that, a few notes:
1) in jscript you do not need to initialize your variables so you can simply write:
var this_month = today.getMonth();
2) if you need a 4 digit year, eg: 2006 use method getFullYear() instead of getYear()
3) getMonth() returns a string, if you need an int, as for the if block you can use parseInt function;
4) as far as i remember (don't have a reference manual in my hands) you can't use switch with multiple conditions (0,1,2) you need an if for this
5) as jscript is a scripting language next time you have problem i suggest you to have a look at
, maybe you'll get a faster response.
This is your code reviewed:
function get_quarter(){
var today = new Date();
var this_month = today.getMonth();
var this_year = today.getYear();
var quarter;
var intMonth = parseInt(this_month);
if(intMonth >= 0 && intMonth < 3){
quarter = this_year+'1';
}else if(intMonth >= 3 && intMonth < 6){
quarter = this_year+'2';
}else if(intMonth >= 6 && intMonth < 9){
quarter = this_year+'3';
}else{
quarter = this_year+'4';
}
return quarter;
}
Hope it helps!
Kind Regards,
Sergio
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.