条件付き矢印関数オブジェクトの反復

Question

この質問は作業コードの改善についてです。私は値をとり、オブジェクトを反復し、その値をオブジェクトのキーと比較することによってどのラベルを返すかを定義する関数showLabelsを持っています。

showLabels = (value) => { 
    Object.keys(this.state.labels).forEach(key => { 
     if (value <= key) this.tooltip = this.state.labels[key]; 
    }); 
    return this.tooltip; 
    } 

質問：これをより簡潔にすることは可能ですか？特に関数の値をthis.tooltipに代入し、それを外部関数で返すことは不必要に冗長に思えます。ここでのコメントに基づいて

例

は一例です。このような

this.state.labels = {0: "Smallest amount", 50: "In the middle", 100: "Top"} 
value = 33 
//showLabels(value=33) should return "In the middle" 

Answer 1

あなたはところで、キーに対する反復の順序が保証されていないことを心に留めておくObject.entries

showLabels = value => Object.entries(this.state.labels) 
 
          // sort pairs by key 
 
          .sort(([keyA], [keyB]) => keyA > keyB) 
 
          .find(([key]) => key > value)[1] 
 

にArray.prototype.findを使用することができます。だから、値より大きい最初のラベルを見つけたいと思えば、ラベルをソートする必要があります。

Answer 2

showLabels = (value) => { 
 
    let tooltip, { labels } = this.state 
 
    Object.keys(labels).forEach(key => { 
 
    if (value <= key) tooltip = labels[key]; 
 
    }) 
 
    return tooltip; 
 
}

何かがより簡潔です。ここにObject.keys()を使用する理由はありますか？むしろ遅いです。

とにかく、始めに可変宣言と構造解除を追加しました。