この記事で紹介されていたソースなのだが。
これ、グローバル変数を汚染しているから、どうなんでしょう?var isMobile = {
via:スマホを判別するのに便利なJSスニペット
Android: function() {
return navigator.userAgent.match(/Android/i) ? true : false;
},
BlackBerry: function() {
return navigator.userAgent.match(/BlackBerry/i) ? true : false;
},
iOS: function() {
return navigator.userAgent.match(/iPhone|iPad|iPod/i) ? true : false;
},
Windows: function() {
return navigator.userAgent.match(/IEMobile/i) ? true : false;
},
any: function() {
return (isMobile.Android() || isMobile.BlackBerry() || isMobile.iOS() || isMobile.Windows());
}
};
一度、無名関数から値を返すようにしたほうがいいのではないのかと思うのだが。
0 コメント:
コメントを投稿