パラメーター分岐のSplitエラー回避方法

Question

パラメータなしで閲覧した際に発生するsplitのエラーを回避させたいです。

http://irohacross.net/2016/04/jquery_tabpram.html
こちらのサイトを参考に、タブの切り替え＆指定タブへのダイレクトリンクを実装しました。

以下JSで問題なく動くのですが、
パラメータなしで閲覧した場合、Chromeのデベロッパーツールで見ると、
【Uncaught TypeError: Cannot read property 'split' of undefined】
が表示されています。
これをJSで回避させたいのですが、記述がわかりませんでした。
どなたかお教えいただけないでしょうか。

・hoge.htmlとtab.htmlを表示した際に、splitエラーが出る
・hoge.html?no=tab1など、?no=tabがついた場合はエラーが出ない
hoge.html

<ul class="tab_area">
 <li id="tab1" class="select">タブ1</a></li>
 <li id="tab2">タブ2</a></li>
</ul>

<div class="content_area">
 hoge.htmlのタブ1の中身
 <a href="tab.html?no=tab1">tab.htmlのタブ1へのリンク</a>
</div>
<div class="content_area hide">
 hoge.htmlのタブ2の中身
 <a href="tab.html?no=tab2">tab.htmlのタブ2へのリンク</a>
</div>

tab.html

<ul class="tab_area">
 <li id="tab1" class="select">タブ1</a></li>
 <li id="tab2">タブ2</a></li>
</ul>

<div class="content_area">
 tab.htmlのタブ1の中身
 <a href="hoge.html?no=tab1">hoge.htmlのタブ1へのリンク</a>
</div>
<div class="content_area hide">
 tab.htmlのタブ2の中身
 <a href="hoge.html?no=tab2">hoge.htmlのタブ2へのリンク</a>
</div>

Javascript

$(function() {
 $('ul.tab_area li').click(function() {
  var index = $('ul.tab_area li').index(this);
  $('.content_area').css('display','none');
  $('.content_area').eq(index).show();
  $('.tab_area li').removeClass('select');
  $(this).addClass('select');
 });

 var url = location.href;
 url = (url.match(/\?no=tab\d+$/) || [])[0];
 var params = url.split("?"); //ここがエラーになる
 var tab = params[1].split("=");
 if($(tab).length){
  var tabname = tab[1];
 } else {
  var tabname = "tab2";
 }

 $('.content_area').css('display','none');
 $('.tab_area li').removeClass('select');
 var tabno = $('ul.tab_area li#' + tabname).index();
 $('.content_area').eq(tabno).show();
 $('ul.tab_area li').eq(tabno).addClass('select');
});

Answer 1

location.href に ? が含まれなかったら、以下の解析処理をやらない
ってのじゃだめですかね。

Comments

[a2s]

「解析処理をやらない」で、リンクやタブはきちん動作するものなのでしょうか…？
するのであれば、その記述でまったく問題はないのですが…

[takasima20]

「解析処理」ってのは
url = (url.match(/\?no=tab\d+$/) || [])[0];
var params = url.split("?"); //ここがエラーになる
var tab = params[1].split("=");
このへんのことを意図しました。
tab にはなんか適当なのをつっこんどけばいいのかな。

[a2s]

なるほど。
? が含まれていなかったらその処理はやらなくて良いですね。
記述としてはどんなものがありますでしょうか？

[takasima20]

記述って…　具体的なコーディングですか?
match を使ってもいいのですが、単純な文字検索なので
indexOf あたりを使えばよいのではないでしょうか。

[a2s]

ありがとうございます！
indexOfは勉強不足なこともあり、よくわかっていないのですが…
var str = location.href;
if ( str.indexOf('?') != -1) {
var params = url.split("?");
var tab = params[1].split("=");
if($(tab).length){
var tabname = tab[1];
} else {
var tabname = "tab2";
}
}
でとりあえずエラーはなくなりました