Use spread syntax instead of for loop

Question

I'd like to reduce this code using the spread syntax to remove the for loop, any ideas?

function shiftChar() {
  let aCharArray = prompt("Enter a word").split("");
  for (let i = 0; i < aCharArray.length; i++) {
    aCharArray[i] = String.fromCharCode(aCharArray[i].charCodeAt(0) + 1);
  }
  alert(aCharArray);
}

This doesn't work

function shiftChar() {
  let aCharArray = prompt("Enter a word").split("");
  aCharArray = String.fromCharCode(...aCharArray.charCodeAt(0) + 1);
  alert(aCharArray);
}

how about using .map?

Roland Starke
– Roland Starke

2018-02-27 15:48:49 +00:00
Commented Feb 27, 2018 at 15:48 — Roland Starke
– Roland Starke, Commented Feb 27, 2018 at 15:48

Bergi · Accepted Answer · 2018-02-27 16:07:15Z

4

Spread syntax (it's not an operator!) is not a replacement for loops, it's a replacement for apply.

You can do

const string = prompt("Enter a word");
const charCodes = [];
for (let i = 0; i < aCharArray.length; i++) {
    aCharCodes[i] = aString.charCodeAt(i) + 1;
}

though and then use

String.fromCharCode(...charCodes)

instead of

String.fromCharCode.apply(String, charCodes)

answered Feb 27, 2018 at 16:07

Bergi

671k162 gold badges1k silver badges1.5k bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Andrew Bone · Accepted Answer · 2018-02-27 16:13:03Z

1

For every element in your array you are doing some manipulation, charCodeAt(0) + 1, so it is probably best you use map.

map calls a provided callback function once for each element in an array, in order, and constructs a new array from the results.

You can use the spread syntax in order to update the contents of your variable aCharArray from the array.

Spread syntax allows an iterable such as an array expression or string to be expanded in places where zero or more arguments

function shiftChar() {
  let aCharArray = prompt("Enter a word").split("").map(x => x.charCodeAt(0) + 1);
  aCharArray = String.fromCharCode(...aCharArray);
  alert(aCharArray);
}

edited Feb 27, 2018 at 16:13

Andrew Bone

7,3112 gold badges21 silver badges35 bronze badges

answered Feb 27, 2018 at 16:01

guijob

4,4883 gold badges23 silver badges41 bronze badges

Comments

stukilgore · Accepted Answer · 2018-02-27 16:20:21Z

1

A minified solution not using spread:

function shiftChar() {
    alert(
      prompt("Enter a word").split("")
      .map(letter => String.fromCharCode(letter.charCodeAt(0) + 1));
    );
}

A (strange) minified solution using spread:

function shiftChar() {
    alert(
      [...prompt("Enter a word")].map(letter => ( 
        String.fromCharCode(letter.charCodeAt(0) + 1)
      )
    );
}

edited Feb 27, 2018 at 16:20

answered Feb 27, 2018 at 16:01

stukilgore

1013 bronze badges

Collectives™ on Stack Overflow

Use spread syntax instead of for loop

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related