Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit 9b429e4

Browse files
refactor(reversingAnArray): remove console
1 parent babb6a9 commit 9b429e4

File tree

1 file changed

+2
-16
lines changed

1 file changed

+2
-16
lines changed

‎chapters/chapter4/reversingAnArray.js

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
1-
const originalArray = [1, 2, 3, 4, 5]
2-
31
// Pure function
4-
const reverseArray = (numbers = []) => {
2+
exportconst reverseArray = (numbers = []) => {
53
const reversedArray = []
64

75
for (let i = numbers.length - 1; i >= 0; i -= 1) {
@@ -12,7 +10,7 @@ const reverseArray = (numbers = []) => {
1210
}
1311

1412
// With side-effects
15-
const reverseArrayInPlace = (numbers = []) => {
13+
exportconst reverseArrayInPlace = (numbers = []) => {
1614
const lastIndex = numbers.length - 1
1715
const stopIndex = Math.ceil(numbers.length / 2) - 1
1816

@@ -23,15 +21,3 @@ const reverseArrayInPlace = (numbers = []) => {
2321
numbers[lastIndex - i] = currentValue
2422
}
2523
}
26-
27-
// Returns a new array
28-
reverseArray(originalArray) // [5, 4, 3, 2, 1]
29-
30-
// originalArray is not modified
31-
console.log(originalArray) // => [1, 2, 3, 4, 5]
32-
33-
// Does not return anything
34-
reverseArrayInPlace(originalArray)
35-
36-
// originalArray is modified
37-
console.log(originalArray) // => [5, 4, 3, 2, 1]

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /