回答編集履歴

テキスト修正

2020/09/28 09:25

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -17,17 +17,14 @@
 上記で決めた `DISQUALIFIED_SCORE` を使って、冒頭に書いた `map`を挟むコードの一案は以下です。
 ```javascript
-data = data.map(a => {
+data = data.map(a => (
-  const score = parseFloat(a[key]);
-  return {
-    ...a,
-    score: Number.isNaN(score) ? DISQUALIFIED_SCORE : score }
+    { ...a, score: a[key] >= 0 ? +a[key] : DISQUALIFIED_SCORE }
-  })
+  ))
- .sort((a, b) => (ii === 'asc' ? 1 : -1) * (a.score - b.score));
+  .sort((a, b) => (ii === 'asc' ? 1 : -1) * (a.score - b.score));
 ```
-上記では、ソート用に追加するキーを `score`としています。以下で動作確認できます。
+上記では、ソート用に追加するキーを `score`としています。また、有効なスコアを0以上の数値としています。以下で動作確認できます。
-- **動作確認用Codepen: **[codepen.io/jun68ykt/pen/OJNGNqM](https://codepen.io/jun68ykt/pen/OJNGNqM?editors=0012)
+- **動作確認用Codepen: **[codepen.io/jun68ykt/pen/abNxJLZ](https://codepen.io/jun68ykt/pen/abNxJLZ?editors=0012)
 以上、参考になれば幸いです。

テキスト修正

2020/09/28 09:25

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -21,7 +21,7 @@
   const score = parseFloat(a[key]);
   return {
     ...a,
-    score: Number.isNaN(score) ? DISQUALIFIED_SCORE : score };
+    score: Number.isNaN(score) ? DISQUALIFIED_SCORE : score }
   })
  .sort((a, b) => (ii === 'asc' ? 1 : -1) * (a.score - b.score));
 ```

テキスト修正

2020/09/28 08:58

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -34,7 +34,7 @@
 ### 追記
-もうひとつ、別の方法を挙げておきます。与えられた`data`全体をソートするのではなく、ソート対象プロパティを数値に変換したときに`NaN`にならないオブジェクトだけを集めた配列を作り、ソートはこの配列に対して行って、ソート後に失格のオブジェクトのみを集めた配列と結合するという考え方です。
+もうひとつ、別の方法を挙げておきます。与えられた`data`全体をソートするのではなく、失格ではないデータ（＝ ソート対象プロパティを数値に変換したときに`NaN`にならないオブジェクト) だけを集めた配列を作り、ソートはこの配列に対して行って、ソート後に失格のオブジェクトのみを集めた配列と結合するという考え方です。
 以下はそのコード例です。（有効なスコアは０以上の数値であることを前提にしています。）

テキスト修正

2020/09/28 08:36

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -30,4 +30,28 @@
 - **動作確認用Codepen: **[codepen.io/jun68ykt/pen/OJNGNqM](https://codepen.io/jun68ykt/pen/OJNGNqM?editors=0012)
-以上、参考になれば幸いです。
+以上、参考になれば幸いです。
+### 追記
+もうひとつ、別の方法を挙げておきます。与えられた`data`全体をソートするのではなく、ソート対象プロパティを数値に変換したときに`NaN`にならないオブジェクトだけを集めた配列を作り、ソートはこの配列に対して行って、ソート後に失格のオブジェクトのみを集めた配列と結合するという考え方です。
+以下はそのコード例です。（有効なスコアは０以上の数値であることを前提にしています。）
+```javascript
+// 有効なスコアを持つオブジェクトのみの配列を作り、昇順にソートする。
+const qualifiedData = data.filter(a => a[key] >= 0).sort((a,b) => a[key] - b[key]);
+// 失格のオブジェクトのみを持つ配列を作る
+const disqualifiedData = data.filter(a => Number.isNaN(+a[key]));
+// data を、有効スコアの配列と失格スコアの配列を結合した配列に置き換える。
+data = [...qualifiedData, ...disqualifiedData];
+// 降順指定であれば、全体の順序を逆にする。
+if (ii == 'desc') {
+  data.reverse();
+}
+```
+- **動作確認用Codepen: **[codepen.io/jun68ykt/pen/poyBEEB](https://codepen.io/jun68ykt/pen/poyBEEB?editors=0012)

テキスト修正

2020/09/28 05:49

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -1,6 +1,6 @@
 こんにちは
-比較対象のどちらか（あるいは両方）の`parseFloat`が失敗して、`NaN`が返されてきた場合の条件分岐や対応を、`sort`に与える関数の中に入れてしまうと、やや煩雑になってしまいます。このようなときに自分がよくやる手は、`sort` の前に `map` を挟んで、配列要素のオブジェクトに 、`NaN`のような（全順序関係を崩す）値をとらない、ソートキーになる別のプロパティを追加して、`sort`に与える関数をできるだけ簡単にすることです。
+比較対象のどちらか（あるいは両方）の`parseFloat`が失敗して、`NaN`が返されてきた場合の条件分岐や対応を、`sort`に与える比較関数の中に入れてしまうと、やや煩雑になってしまいます。このようなときに自分がよくやる手は、`sort` の前に `map` を挟んで、配列要素のオブジェクトに 、ソートキーになる別のプロパティを追加することです。追加プロパティの値は、`NaN`のような（全順序関係を崩す）値をとらないように配慮された数値とすることで、`sort`に与える比較関数を簡単に書けるようにします。
 以下はそのサンプルです。

テキスト修正

2020/09/28 04:48

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -1,6 +1,6 @@
 こんにちは
-`sort`に与える関数の中に比較対象のどちらかが文字列だった場合の処理を入れてしまうと、やや煩雑になってしまうので、このようなときに自分がよくやる手は、`sort` の前に `map` を挟んで、配列要素のオブジェクトに 、`NaN`のような（全順序関係を崩す）値をとらない、ソートキーになるプロパティを追加することです。
+比較対象のどちらか（あるいは両方）の`parseFloat`が失敗して、`NaN`が返されてきた場合の条件分岐や対応を、`sort`に与える関数の中に入れてしまうと、やや煩雑になってしまいます。このようなときに自分がよくやる手は、`sort` の前に `map` を挟んで、配列要素のオブジェクトに 、`NaN`のような（全順序関係を崩す）値をとらない、ソートキーになる別のプロパティを追加して、`sort`に与える関数をできるだけ簡単にすることです。
 以下はそのサンプルです。

テキスト修正

2020/09/28 04:21

投稿

jun68ykt

スコア9058

answer CHANGED Viewed

@@ -23,7 +23,7 @@
     ...a,
     score: Number.isNaN(score) ? DISQUALIFIED_SCORE : score };
   })
- .sort((a, b) => a.score - b.score);
+ .sort((a, b) => (ii === 'asc' ? 1 : -1) * (a.score - b.score));
 ```
 上記では、ソート用に追加するキーを `score`としています。以下で動作確認できます。