複数設置してあるSNSの遷移先をPCとSPで別々のURLに遷移させたい

Question

forEachでclickイベント時にlocation.hrefで遷移させたかったのですがうまくいかず。
レクチャーください

  <ul class="sns_sub">
  <li><a class="js-changeLink" href="hoge1.html" target="_blank"><img src="img/icon.png" alt=""></a></li>
  <li>text</li>
  </ul>

      <div class="sns">
        <p>SNS</p>
        <ul>
 <li><a class="js-changeLink" href="hoge1.html" target="_blank"><img src="img/icon.png" alt=""></a></li>
  <li>text</li>
  <li>text</li>
        </ul>
      </div>

let link = document.querySelectorAll('.js-changeLink');
link.forEach(function(el, i) {
  el.addEventListener('click', function(){
if(navigator.userAgent.match(/(iPhone|iPod|Android.*Mobile)/i)){
  alert(i);
  location.href= "hoge1.html";
  
}else{
  alert(i);
  location.href= "xxx";
}
});
});

Comments

[miyabi_pudding]

何がどううまくいかなかったのでしょうか？
具体的に記載お願いします。
（コードを見る限りでは振り分けられているように見える）

[nanashissss]

特にコンソールエラーメッセージも表示されていなく、確認用alertもでてない状態でした。
ですが、DOMContentLoadedで処理を囲ったら振り分けれました。
コメントありがとうございます。

Answer 1

読み込み問題でした。以下で囲ってきちんと処理できました

document.addEventListener('DOMContentLoaded', () =>  {})

Answer 2

userAgentは信頼性が薄いしそれであればむしろクライアント側にPCかSPか最初に選ばせてクッキーか何かに保持しておけばよいのでは？

Comments

[nanashissss]

コメントありがとうございます。
そちらの内容もクライアント様にお伝えします。

[m.ts10806]

この場合の「クライアント」は「顧客」的な意味ではないですよ。