Skip to content

Commit 4ff1a04

Browse files
authored
use consistent let for non-consts (#314)
1 parent cea0f17 commit 4ff1a04

File tree

10 files changed

+137
-129
lines changed

10 files changed

+137
-129
lines changed

src/aggregate-collection.js

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,14 @@
77
* @returns {Number} mode - The `mode` value of `arr`
88
*/
99
function Mode(arr) {
10-
const frequencies = new Map()
10+
let frequencies = new Map()
1111
let maxOccurrences = -1
1212
let maxOccurenceCount = 0
1313
let sum = 0
1414

1515
for (let i = 0; i < arr.length; i++) {
16-
const element = arr[i]
17-
const updatedCount = (frequencies.get(element) || 0) + 1
16+
let element = arr[i]
17+
let updatedCount = (frequencies.get(element) || 0) + 1
1818
frequencies.set(element, updatedCount)
1919

2020
if (updatedCount > maxOccurrences) {
@@ -40,8 +40,8 @@ function Mode(arr) {
4040
* @returns {Number} - The array's Median
4141
*/
4242
function Median(arr) {
43-
const middle = arr.length / 2
44-
const lowerMiddleRank = Math.floor(middle)
43+
let middle = arr.length / 2
44+
let lowerMiddleRank = Math.floor(middle)
4545

4646
if (middle !== lowerMiddleRank) {
4747
return arr[lowerMiddleRank]
@@ -84,12 +84,12 @@ class AggregateCollection {
8484

8585
// TODO: can we avoid this sort()? It's slow
8686
/** @type Number[] */
87-
const sorted = this._items.slice().sort((a, b) => a - b)
88-
const min = sorted[0]
89-
const max = sorted[sorted.length - 1]
87+
let sorted = this._items.slice().sort((a, b) => a - b)
88+
let min = sorted[0]
89+
let max = sorted[sorted.length - 1]
9090

91-
const mode = Mode(sorted)
92-
const median = Median(sorted)
91+
let mode = Mode(sorted)
92+
let median = Median(sorted)
9393

9494
return {
9595
min,

src/context-collection.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ class ContextCollection {
3131
* uniquenessRatio: number
3232
* }>}
3333
*/
34-
const itemsPerContext = new Map()
34+
let itemsPerContext = new Map()
3535

3636
for (let [context, value] of this._contexts.entries()) {
3737
itemsPerContext.set(context, value.count())

0 commit comments

Comments
 (0)