Add unambiguous section in the PGN + some fixes + code formatting and fix typos

[vchess.git] / client / src / utils / alea.js

diff --git a/client/src/utils/alea.js b/client/src/utils/alea.js
index 76d9c38..1b112ef 100644 (file)
--- a/client/src/utils/alea.js
+++ b/client/src/utils/alea.js
@@ -1,36 +1,32 @@
 // Random (enough) string for socket and game IDs
-export function getRandString()
-{
-  return (Date.now().toString(36) + Math.random().toString(36).substr(2, 7))
-    .toUpperCase();
+export function getRandString() {
+  return (
+    Date.now().toString(36) +
+    Math.random()
+      .toString(36)
+      .substr(2, 7)
+  ).toUpperCase();
 }
 
-export function random (min, max)
-{
-  if (!max)
-  {
+export function randInt(min, max) {
+  if (!max) {
     max = min;
     min = 0;
   }
-  return Math.floor(Math.random() * (max - min) ) + min;
+  return Math.floor(Math.random() * (max - min)) + min;
 }
 
-// Inspired by https://github.com/jashkenas/underscore/blob/master/underscore.js
-export function sample (arr, n)
-{
+// Inspired by https://github.com/jashkenas/underscore
+export function sample(arr, n) {
   n = n || 1;
   let cpArr = arr.map(e => e);
-  for (let index = 0; index < n; index++)
-  {
-    const rand = random(index, n);
-    const temp = cpArr[index];
-    cpArr[index] = cpArr[rand];
-    cpArr[rand] = temp;
+  for (let index = 0; index < n; index++) {
+    const rand = randInt(index, arr.length);
+    [ cpArr[index], cpArr[rand] ] = [ cpArr[rand], cpArr[index] ];
   }
   return cpArr.slice(0, n);
 }
 
-export function shuffle(arr)
-{
+export function shuffle(arr) {
   return sample(arr, arr.length);
 }