jquery - パラメータなしで関数を送信すると、未設定の変数エラーが発生します

Question

jQueryの基本的な理解についてお詫びします。ユーザーが値を設定するか、空白のままにして、自分自身を設定できる関数を作成しようとしています。次のようになります。

// Calendar
function eventCalendar(yearSet, monthSet) { 

    // Get current date
    var date = new Date();

    // Check to see if variables were set in function

    // Year
    if ((yearSet.length > 0)) {
            var year = yearSet;
        } else {
            var year  = date.getFullYear();
        }

    // Month
    if ((monthSet.length > 0)) {
            var month = monthSet;
        } else {
            var month = date.getMonth();
        }

    console.log(month + ', ' + year);   
}

ただし、関数が変数なしで呼び出されると、コンソールにエラーが表示されます。

'yearSet は定義されていません'

どうすればこれを回避できますか?

Answer 1

パラメータが明示的に定義されていないかどうかを確認するのではなく、パラメータの真偽を確認できます。

function eventCalendar(yearSet, monthSet) { 
  var date = new Date();
  var year = yearSet && yearSet.length ? yearSet : date.getFullYear();
  var month = monthSet && monthSet.length ? monthSet : date.getMonth();
  console.log(month + ', ' + year);   
}

参考までに、現在のエラーは、変数 ( )lengthのプロパティ ( )にアクセスしようとしたことが原因で発生しました。JavaScript IMO の真実性と偽りについて Google に問い合わせることをお勧めします。これは言語の非常に便利な機能です。undefinedyear

Answer 2

if(yearSet && yearSet.length) {
   //..
}

このようにメソッドを単純化できます。

function eventCalendar(yearSet, monthSet) { 
  var date = new Date();
  //if yearSet is present it will get that or else from new Date
  var year = yearSet || date.getFullYear();
  //if monthSet is present it will get that or else from new Date
  var month = monthSet || date.getMonth();

  console.log(month + ', ' + year);   
}

Answer 3

また、「 」は、フィールドに含める有効な値と見なされることも考慮してください。もう少しセクシーなことをしたい：

function isNumber(n) {
  return !isNaN(parseFloat(n)) && isFinite(n);
}

function eventCalendar (y,m) {
  var date = new Date(); 
  y=(isNumber(y) ? y : date.getFullYear(); 
  m=(isNumber(m) ? m : date.getMonth(); 
  console.log(m + " " + y);
}

テストされていませんが、必要に応じて実行する必要があります。

Answer 4

次のコードで変数が未定義かどうかを確認します。

if(typeof(variable) === 'undefined') {
    // var is undefined
}

.length使用すると、指定された変数が存在すると想定されるため、コードはエラーをスローします。

Answer 5

長さをチェックする代わりに、undefinedまたは比較しますnull