このJavaScriptのタイムスタンプビルダー関数の複雑さ/ネストされたifsを減らす方法

Question

こんにちは私は、私が呼んでいるAPIの行に部分的なISOタイムスタンプを構築するように設計された関数を持っています。私は本当に私はそれとしてであれば、ネストされた機能の必要性を取り除くことができるかわからない

function parsePartialDateTime(datetime = {}) { 
    if (datetime === null || datetime.year === undefined) { 
    return null; 
    } 

    let dateString = `${datetime.year}`; 

    if (datetime.month) { 
    dateString += `-${datetime.month}`; 
    if (datetime.day) { 
     dateString += `-${datetime.day}`; 
     if (datetime.hour) { 
     dateString += `T${datetime.hour}`; 
     if (datetime.minute) { 
      dateString += `:${datetime.minute}`; 
      if (datetime.second) { 
      dateString += `:${datetime.second}`; 
      } 
     } 
     } 
    } 
    } 

    return `${dateString}Z`; 
} 

：それは返す必要があり、タイムスタンプ：

2017Z 
2017-12Z 
2017-12-24Z 
2017-12-24T14Z 
2017-12-24T14:07Z 

私は次のようにこれは実行するために使用するコード前の項目が存在する場合は、タイムスタンプの各要素を作成する必要があります（ datetimeオブジェクトの文字列/数値として供給されます。

この上の任意の助けいただければ幸い:)

Answer 1

をあなたがそれらの上に配列し、ループ内の値を入れることができます：

// Just for brevity in this example 
var dt = datetime; 
// Put all the units you want checked in this array 
var units = [dt.month, dt.day, dt.hour, dt.minute, dt.second]; 

var dateString = dt.year 
// Then loop over the array... 
for (let unit of units) { 
    //... checking and adding like you were before 
    if (unit) { 
     dateString += `-${unit}`; 
    } else { 
     // So the rest of the object gets ignored 
     break; 
    } 
} 

かかわらず、コメントで述べたように、これはのために同じ区切り文字を使用していますそれぞれunit。これは、これを解決するための複雑さの少量を追加します。

// Each unit is paired with the delimiter that should be used on its left 
var units = [[dt.month, "-"], [dt.day, "-"], [dt.hour, "T"], 
      [dt.minute, ":"], [dt.second, ":"]]; 

var dateString = dt.year 
// Deconstruct out the unit and the delimiter 
for (let [unit, delim] of units) { 
    if (unit) { 
     dateString += delim + `${unit}`; 
    } else { 
     break; 
    } 
} 

あなたのコード内のすべての分岐がかなり簡単に抽出することができ、基本的if (x) y += xの形を取るので。ほぼ同じことを何度も何度もやり直している場合は、ループに到達してみてください。