mark下工作中JS的小问题

如图：

单选框是6天，文本框最多是1-15的正整数，而且单选框的值要和文本框对应，当文本框大于等于7时，单选框的check都为false;

而编辑器是根据天数来生成的，譬如选5天，就生成5个编辑器，代码如下：

//根据天数实例化UEditor
function getDaysDetail(num){
    for(var i=1;i<=num;i++)
    {
        var editor = new UE.ui.Editor();
        UE.getEditor('tuan_schedule_'+i, {
            maxImageSideLength:477 //宽度超过477px将自动等比缩至宽477px
             ,imagePath:"" 
        });
    }   
}

调用getDaysDetail()代码如下：

var daysRadio=$('#selectDays').find('input[name="schedule_day"]:radio');
var daysTxt=$('#selectDays').find('input[name="schedule_day_other"]');
var days=$('#selectDays').find('input[name="schedule_day"]:radio:checked').val()

//判断函数是否存在
if(typeof getDaysDetail==undefined) return;

daysTxt.val(days);
getDaysDetail(days);

daysRadio.click(function(){
    days=$('#selectDays').find('input[name="schedule_day"]:radio:checked').val();
    getDaysDetail(days);
    daysTxt.val(days);
});

daysTxt.mouseenter(function(){
    $(this).focus();
    $(this).select();
})

//判断是否为正整数 1~15
function isUnsignedInteger(a)
{
    var reg =/^[1-9][0-5]?$/;
    return reg.test(a);
}

daysTxt.keyup(function()
{
    days=parseInt($(this).val());
    if(isUnsignedInteger(days)){
        if(days>=1 && days<=15)
        {
            
            if(days>=1 && days<=6)
            {
                $('#selectDays').find('input[name="schedule_day"]:radio[value='+days+']').attr('checked',true);
            }
            else
            {
                daysRadio.each(function(i){
                    $(this).attr('checked', false);
                })
            }
            getDaysDetail(days);
        }
        else
        {
            daysTxt.val(1);
            $('#selectDays').find('input[name="schedule_day"]:radio[value=1]').attr('checked',true);
            return false
        }
    } 
    else
    {
        daysTxt.val(1);
        $('#selectDays').find('input[name="schedule_day"]:radio[value=1]').attr('checked',true);
        return false
    }
}); 

前段时间跑没什么问题，后来同事反映说当>=7时，无法调用getDaysDetail() 这个函数；用了几分钟check下代码，发现第3行代码出的问题：

var days=$('#selectDays').find('input[name="schedule_day"]:radio:checked').val()

days只算了radio的值；

而第9行getDaysDetail(days)  是只算的radio的值，没有传input=text的值，这样当然>=7时，就不能调用getDaysDetail() 函数了；

只需小改一下：

 

var days=$('#selectDays').find('input[name="schedule_day"]:radio:checked').val()||daysTxt.val();

 

就OK了。

看来工作中还是有些边缘问题没考虑到，mark于此，以后谨记于心。