このシナリオでjsゲッターを実装するためのエレガントな方法は何ですか？

Question

私は、次のグローバルJSモジュールを持っている：

var globals = (function(){ 
    var selectedReportPeriodId = 2; 
    return{ 
     SelectedReportPeriodId : selectedReportPeriodId 
    }; 
})(); 

私のコード・セットは、ユーザーが選択したレポート期間のオプションに基づいてSelectedReportPeriodId。

if(globals.IsCurrentReportPeriod) 
{ 
    //do this 
} 

エレガントな方法だろう何を：私はこのようなゲッターを呼び出すために、下流のコードを希望

function IsCurrentReportPeriod() 
{ 
    return selectedReportPeriodId === 1 
} 

：私は、機能的には、このように動作します。このモジュールにjsのゲッターを追加します上記のjsモジュールでゲッターとしてこれを実装するには？

UPDATE＃1

私はこのように実装してみました：

var globals = (function(){ 

    var selectedReportPeriodId; 

    return{ 
     SelectedReportPeriodId : selectedReportPeriodId, 
     get IsCurrentReportPeriod() { 
     return selectedReportPeriodId == 1; 
     } 
    }; 
})(); 

私は前のコードでglobals.SelectedReportPeriodId = 1を設定しています。また、メソッドにブレークポイントを設定し、globals.SelectedReportPeriodIdのウォッチを1に設定します。ただし、globals.IsCurrentReportPeriodはこのコードの下の行にfalseを返し、定義されたゲッターが期待通りに動作しないように見えます。助言がありますか？

UPDATE＃2

selectedReportPeriodIdは、2の値が、私は、これはゲッターが更新された値であっても後selectedReportPeriodId = 2ではなく1見ている理由であり得ることを推測していると初期化されますセット。これには良い解決策はありますか？私はselectedReportPeriodIdのゲッターを作るべきですか？ハードコードされた値で初期化するのではなく、いくつかの条件付きロジックを行うのですか？

Answer 1

return{ 
     SelectedReportPeriodId : selectedReportPeriodId, 
     IsCurrentReportPeriod : function() { 
      return selectedReportPeriodId === 1 
     } 
}; 

しかし、あなたの呼び出しは次のようになりする必要があります：

if (globals.IsCurrentReportPeriod()) { 
} 

Answer 2

最も簡単でシンプルなソリューションがあなたのコードではなくselectedReportPeriodId変数のglobals.IsCurrentReportPeriodプロパティを設定できるようになります。しかし、あなたは絶対にゲッターを使用する必要がある場合は、あなたがそうすることができます。

var globals = (function(){ 
    var selectedReportPeriodId; 
    return { 
     get IsCurrentReportPeriod() { 
      return selectedReportPeriodId === 1; 
     } 
    }; 
})(); 

Answer 3

あなたが作成したモジュールのこの方法を試すことができます。

var Globals = Globals || {}; 
 

 
(function(exports) { 
 
    var selectedReportPeriodId = 1, 
 
     otherPrivateVariable = 'private !'; 
 
    
 
    exports.getIsCurrentReportPeriod = function() { 
 
    return (selectedReportPeriodId === 1); 
 
    }; 
 

 
    function thisIsAPrivateMethod() { 
 
    } 
 

 
})(Globals); 
 

 
alert(Globals.getIsCurrentReportPeriod())