我建议不要再重写这段代码了。我建议你使用Conditionizr库(http://conditionizr.com/)，它能够测试特定的IE版本以及其他浏览器、操作系统，甚至是否有视网膜显示。

只包含您需要的特定测试的代码，您还可以获得经过多次迭代的测试库的好处(并且很容易升级而不会破坏您的代码)。

它还可以很好地与Modernizr配合，Modernizr可以处理所有情况，在这些情况下，您最好测试特定的功能，而不是特定的浏览器。

var isIE9OrBelow = function()
{
   return /MSIE\s/.test(navigator.userAgent) && parseFloat(navigator.appVersion.split("MSIE")[1]) < 10;
}

// Detect ie <= 10
var ie = /MSIE ([0-9]+)/g.exec(window.navigator.userAgent)[1] || undefined;

console.log(ie);
// Return version ie or undefined if not ie or ie > 10

if (!document.addEventListener) {
    // ie8
} else if (!window.btoa) {
    // ie9
}
// others

function getIEVersion(){
     if (/MSIE |Trident\//.test( navigator.userAgent )=== false) return -1;
    /**[IE <=9]*/
    var isIE9L = typeof ( window.attachEvent ) === 'function' && !( Object.prototype.toString.call( window.opera ) == '[object Opera]' ) ? true : false;
    var re;
    if(isIE9L){
        re = new RegExp( "MSIE ([0-9]{1,}[\.0-9]{0,})" );
        if(re.exec( navigator.userAgent ) !== null)
            return parseFloat( RegExp.$1 );
        return -1;
    }
    /**[/IE <=9]*/
    /** [IE >= 10]*/
    if(navigator.userAgent.indexOf( 'Trident/' ) > -1){
        re = new RegExp( "rv:([0-9]{1,}[\.0-9]{0,})" );
        if(re.exec( navigator.userAgent ) !== null)
            return parseFloat( RegExp.$1 );
        return -1;
    }
    /**[/IE >= 10]*/
    return -1;
};

点击这里==>

var ieVersion = getIEVersion();

if(ieVersion < 0){
    //Not IE
}
//A version of IE

了解关于浏览器导航器的更多信息 https://developer.mozilla.org/en-US/docs/Web/API/Window/navigator

你的代码可以做检查，但正如你所认为的，如果有人试图使用IE v1或> v19访问你的页面将不会得到错误，所以可能更安全的做检查的Regex表达式如下代码:

var userAgent = navigator.userAgent.toLowerCase();
// Test if the browser is IE and check the version number is lower than 9
if (/msie/.test(userAgent) && 
    parseFloat((userAgent.match(/.*(?:rv|ie)[\/: ](.+?)([ \);]|$)/) || [])[1]) < 9) {
  // Navigate to error page
}