From 8c6059098d19e25d8b415180848af40e4c648bdb Mon Sep 17 00:00:00 2001
From: Darren Scerri <darren@scerri.me>
Date: Wed, 13 Dec 2017 22:56:36 +0100
Subject: [PATCH] Remove duplicate snippets. Simplify implementation

---
 README.md                            | 24 +++++-------------------
 snippets/randomize-order-of-array.md |  8 --------
 snippets/shuffle-array-values.md     | 12 ------------
 snippets/shuffle-array.md            |  8 ++++++++
 4 files changed, 13 insertions(+), 39 deletions(-)
 delete mode 100644 snippets/randomize-order-of-array.md
 delete mode 100644 snippets/shuffle-array-values.md
 create mode 100644 snippets/shuffle-array.md

diff --git a/README.md b/README.md
index b77013e78..7317a259d 100644
--- a/README.md
+++ b/README.md
@@ -50,13 +50,12 @@
 * [Promisify](#promisify)
 * [Random integer in range](#random-integer-in-range)
 * [Random number in range](#random-number-in-range)
-* [Randomize order of array](#randomize-order-of-array)
 * [Redirect to url](#redirect-to-url)
 * [Reverse a string](#reverse-a-string)
 * [RGB to hexadecimal](#rgb-to-hexadecimal)
 * [Run promises in series](#run-promises-in-series)
 * [Scroll to top](#scroll-to-top)
-* [Shuffle array values](#shuffle-array-values)
+* [Shuffle array](#shuffle-array)
 * [Similarity between arrays](#similarity-between-arrays)
 * [Sort characters in string (alphabetical)](#sort-characters-in-string-alphabetical)
 * [Sum of array of numbers](#sum-of-array-of-numbers)
@@ -510,15 +509,6 @@ const randomInRange = (min, max) => Math.random() * (max - min) + min;
 // randomInRange(2,10) -> 6.0211363285087005
 ```
 
-### Randomize order of array
-
-Use `Array.sort()` to reorder elements, utilizing `Math.random()` to randomize the sorting.
-
-```js
-const randomizeOrder = arr => arr.sort( (a,b) => Math.random() >= 0.5 ? -1 : 1);
-// randomizeOrder([1,2,3]) -> [1,3,2]
-```
-
 ### Redirect to URL
 
 Use `window.location.href` or `window.location.replace()` to redirect to `url`.
@@ -575,17 +565,13 @@ const scrollToTop = _ => {
 // scrollToTop()
 ```
 
-### Shuffle array values
+### Shuffle array
 
-Create an array of random values by using `Array.map()` and `Math.random()`.
-Use `Array.sort()` to sort the elements of the original array based on the random values.
+Use `Array.sort()` to reorder elements, using `Math.random()` in the comparator.
 
 ```js
-const shuffle = arr => {
-  let r = arr.map(Math.random);
-  return arr.sort((a,b) => r[a] - r[b]);
-}
-// shuffle([1,2,3]) -> [2, 1, 3]
+const shuffle = arr => arr.sort(() => Math.random() - 0.5);
+// shuffle([1,2,3]) -> [2,3,1]
 ```
 
 ### Similarity between arrays
diff --git a/snippets/randomize-order-of-array.md b/snippets/randomize-order-of-array.md
deleted file mode 100644
index d65aaf444..000000000
--- a/snippets/randomize-order-of-array.md
+++ /dev/null
@@ -1,8 +0,0 @@
-### Randomize order of array
-
-Use `Array.sort()` to reorder elements, utilizing `Math.random()` to randomize the sorting.
-
-```js
-const randomizeOrder = arr => arr.sort( (a,b) => Math.random() >= 0.5 ? -1 : 1);
-// randomizeOrder([1,2,3]) -> [1,3,2]
-```
diff --git a/snippets/shuffle-array-values.md b/snippets/shuffle-array-values.md
deleted file mode 100644
index a140bd647..000000000
--- a/snippets/shuffle-array-values.md
+++ /dev/null
@@ -1,12 +0,0 @@
-### Shuffle array values
-
-Create an array of random values by using `Array.map()` and `Math.random()`.
-Use `Array.sort()` to sort the elements of the original array based on the random values.
-
-```js
-const shuffle = arr => {
-  let r = arr.map(Math.random);
-  return arr.sort((a,b) => r[a] - r[b]);
-}
-// shuffle([1,2,3]) -> [2, 1, 3]
-```
diff --git a/snippets/shuffle-array.md b/snippets/shuffle-array.md
new file mode 100644
index 000000000..6266f9ecf
--- /dev/null
+++ b/snippets/shuffle-array.md
@@ -0,0 +1,8 @@
+### Shuffle array
+
+Use `Array.sort()` to reorder elements, using `Math.random()` in the comparator.
+
+```js
+const shuffle = arr => arr.sort(() => Math.random() - 0.5);
+// shuffle([1,2,3]) -> [2,3,1]
+```