2017-10-11 16 views
0

j'ai mis le code suivant dans mon site pour vérifier que le navigateur je suis:travaux de détection de navigateur JavaScript mais montre « undefined » après

function myFunction() { 
    if ((navigator.userAgent.indexOf("Opera") || navigator.userAgent.indexOf('OPR')) != -1) { 
     document.write('Opera'); 
    } else if (navigator.userAgent.indexOf("Chrome") != -1) { 
     document.write('Google Chrome'); 
    } else if (navigator.userAgent.indexOf("Safari") != -1) { 
     document.write('Safari'); 
    } else if (navigator.userAgent.indexOf("Firefox") != -1) { 
     document.write('Firefox'); 
    } else if ((navigator.userAgent.indexOf("MSIE") != -1) || (!!document.documentMode == true)) { //IF IE > 10 
     document.write('IE (Internet Explorer)'); 
    } else { 
     document.write('Browser Undefined'); 
    } 
} 
document.write(myFunction()); 

Le code fonctionne très bien. Lorsque je suis dans Google Chrome, le nom est affiché, mais avec le mot "indéfini" à côté du nom du navigateur.

+0

retourner le nom du navigateur –

+0

de @SagarV que voulez-vous dire par là? –

+0

'myFunction' n'a pas de déclaration de retour ... donc, comme javascript est écrit de cette façon, c'est identique à' return undefined' –

Répondre

1

function myFunction() { 
 
    if ((navigator.userAgent.indexOf("Opera") || navigator.userAgent.indexOf('OPR')) != -1) { 
 
     return 'Opera' 
 
    } else if (navigator.userAgent.indexOf("Chrome") != -1) { 
 
     return 'chome' 
 
    } else if (navigator.userAgent.indexOf("Safari") != -1) { 
 
     return 'Safar' 
 
    } else if (navigator.userAgent.indexOf("Firefox") != -1) { 
 
     return 'Foxy' 
 
    } else if ((navigator.userAgent.indexOf("MSIE") != -1) || (!!document.documentMode == true)) { //IF IE > 10 
 
     return 'MIS' 
 
    } else { 
 
     return document.write('Browser Undefined'); 
 
    } 
 
} 
 
document.write(myFunction());

0

Vous pouvez écrire le code comme

function myFunction() { 
 
    var browser = ""; 
 
    if ((navigator.userAgent.indexOf("Opera") || navigator.userAgent.indexOf('OPR')) != -1) { 
 
    browser = 'Opera' 
 
    } else if (navigator.userAgent.indexOf("Chrome") != -1) { 
 
    browser = 'chrome' 
 
    } else if (navigator.userAgent.indexOf("Safari") != -1) { 
 
    browser = 'Safar' 
 
    } else if (navigator.userAgent.indexOf("Firefox") != -1) { 
 
    browser = 'Foxy' 
 
    } else if ((navigator.userAgent.indexOf("MSIE") != -1) || (!!document.documentMode == true)) { //IF IE > 10 
 
    browser = 'MIS' 
 
    } else { 
 
    browser = 'Browser Undefined'; 
 
    } 
 
    return browser; 
 
} 
 
document.write(myFunction());