2016-05-11 13 views
1

年の配列を作成しようとしています。今年と昨年。次のコードは動作していますが、私はこれを行うよりクリーンな方法があるかどうかを見たいと思っていましたか?後で変数currentYearを使用していない場合現在の年と前年の配列を作成する角度

var d = new Date(); 
var currentYear = (d.getFullYear()); 
var lastYear = (currentYear - 1); //Remove another call to d.getFullYear() 

$scope.years = [ 
    currentYear, 
    lastYear 
]; 

を複製

var d = new Date(); 
    var currentYear = (d.getFullYear()); 
    var lastYear = (d.getFullYear() - 1); 

    $scope.years = [ 
     currentYear, 
     lastYear 
    ]; 
+0

がきれいに見えるお試しください私にきれいに。 – tymeJV

+0

ありがとう、私はJSとプログラミング一般には非常に新しく、改善できるかどうかを見たいと思っていました。 –

+0

私が変えるかもしれない*唯一のものは、 'var lastYear =(d.getFullYear() - 1);' 'var lastYear = currentYear-1;'を使うでしょう。 – Lex

答えて

0

この

var d = new Date(); 
    var currentYear = d.getFullYear(); 
    var lastYear = currentYear - 1; 
    $scope.years = [currentYear,lastYear]; 
0

避け、あなたもそれを減らすことができる:

var d = new Date(); 
var currentYear = (d.getFullYear()); 

$scope.years = [ 
    currentYear, 
    currentYear - 1 
]; 
関連する問題