Advent of Code - adventofcode.com

You are reading a single comment by @cyclotron3k and its replies. Click here to read the full conversation.

•

zooeyzooey in reply to @Arducius

The short answer is no, it still works.

The flow is it takes the string, pulls the number (word) or number (int) occurrences out into an object, then organises them into an array in the order they occurred in the original string, and then maps them to the relevant number. Then takes the first and last element of that array and concatenates them together. So 1 will result in an array of ['1'], so concatenating the first and last element of that array together results in 11.

But ty for the thought.

•

cyclotron3k in reply to @zooeyzooey
I think your problem is here:
```
Object.keys(strNumMap).reduce((res, key) => {
  const index = str.indexOf(key);
  if (index > -1) {
    res[key] = index;
  }
  return res;
}
```
It's not handling strings with multiple of the same number correctly. E.g. 121 is getting mapped to 12 instead of 11.

•

zooeyzooey in reply to @cyclotron3k

Nice one, that's definitely it.
•

zooeyzooey in reply to @cyclotron3k

Yep, that was it. Refactored to get arr of all indexes rather than just the first and got the right answer. Thank you!

Would have been nice to have an example in the test data with that structure, but oh well!

About

Avatar for cyclotron3k @cyclotron3k started