那里出现了问题,为什么死循环？

Question

<SCRIPT LANGUAGE="JavaScript1.2">
<!--//
if (navigator.appName == 'Netscape')
var language = navigator.language;
else
var language = navigator.browserLanguage;
if (language.indexOf('en') > -1) document.location.href = 'english.htm';
else if (language.indexOf('nl') > -1) document.location.href = 'dutch.htm';
else if (language.indexOf('fr') > -1) document.location.href = 'french.htm';
else if (language.indexOf('de') > -1) document.location.href = 'german.htm';
else if (language.indexOf('ja') > -1) document.location.href = 'japanese.htm';
else if (language.indexOf('it') > -1) document.location.href = 'italian.htm';
else if (language.indexOf('pt') > -1) document.location.href = 'portuguese.htm';
else if (language.indexOf('es') > -1) document.location.href = 'Spanish.htm';
else if (language.indexOf('sv') > -1) document.location.href = 'swedish.htm';
else if (language.indexOf('zh') > -1) document.location.href = 'chinese.htm';
else
document.location.href = 'german.htm';
// End -->
</script>

Answer 1

重复定义，浏览器无法识别到底是哪一个
var language = “”；
if (navigator.appName == 'Netscape') 
language  = navigator.language; 
else 
language = navigator.browserLanguage;

Answer 2

再次，为了规范，在条件里面+上{}，
if (language.indexOf('en') > -1) {document.location.href = 'english.htm'; }
else if (language.indexOf('nl') > -1) {document.location.href = 'dutch.htm'; }
else if (language.indexOf('fr') > -1) {document.location.href = 'french.htm'; }
else if (language.indexOf('de') > -1) {document.location.href = 'german.htm'; }

Answer 3

语言判断，language.indexOf('en') > -1，如果不存在，其值=-1,应该采用>=0