fokus-lop Mar 26 2019 at 13:07

5 типовых задач на собеседованиях по JavaScript: разбор и решения

8 min

91K

Skillbox corporate blogJavaScript*Programming*Algorithms*Studying in IT

+15

Comments 53

UFO just landed and posted this here

ZoNT Mar 26 2019 at 16:39

const convert = str => str.toLowerCase().match(/\w/g).sort().join('');
const anagram = (strA, strB) => convert(strA) === convert(strB);

Zavtramen Mar 26 2019 at 21:58

На пустой строке выдаст ошибку, match не отработает, равно как и на строке с русскими буквами или знаками препинания, в общем на любой где \w нет.
Мало того из строки вырежутся, к примеру, русские буквы, и строки 'abc' и 'abcабц' будут равны

ZoNT Mar 27 2019 at 09:18

Это проблемы постановки задачи, на это уже отвечали ниже: habr.com/ru/company/skillbox/blog/445360/?reply_to=19946684#comment_19944882

Zavtramen Mar 27 2019 at 11:10

Ваше решение падает на пустой строке, причем здесь постановка задачи?

ZoNT Mar 27 2019 at 12:33

const convert = str => str.toLowerCase().match(/\w/g).sort().join('');
const anagram = (strA, strB) => strA !== '' && strB !== '' && convert(strA) === convert(strB);

Так лучше?
Следует также учесть, что анаграмма это «Слово или словосочетание, образованное путём перестановки букв, составляющих другое слово (или словосочетание).». То есть пустая строка не может быть анаграммой.

drch May 2 2023 at 08:30

const str = s => [...s].sort().join('');
const anagram = (s1, s2) => s1 && str(s1) === str(s2) || false;

Так изящнее.

edogs Mar 26 2019 at 13:29

FizzBuzz — мы бы не делали лишних проверок.

тынц

Не так

const fizzBuzz = num => {
  for(let i = 1; i <= num; i++) {
    // check if the number is a multiple of 3 and 5
    if(i % 3 === 0 && i % 5 === 0) {
      console.log('fizzbuzz')
    } // check if the number is a multiple of 3
      else if(i % 3 === 0) {
      console.log('fizz')
    } // check if the number is a multiple of 5
      else if(i % 5 === 0) {
      console.log('buzz')
    } else {
      console.log(i)
    }
  }
}

А так

const fizzBuzz = num => {
  for(let i = 1; i <= num; i++) {
    if(i % 3 !== 0 || i % 5 !== 0) console.log(i);
   else {
    if(i % 3 === 0) console.log('fizz')
    if(i % 5 === 0) console.log('buzz')
  }
}

e_fail Mar 26 2019 at 14:09

У вас в выводе fizzbuzz будет размазан на две строки?

UFO just landed and posted this here

Stepanow Mar 26 2019 at 15:07

if(i % 3 !== 0 || i % 5 !== 0) console.log(i);

А почему ИЛИ? У вас так никогда не будет fizz или buzz, только fizzbuzz, да и то на разных строках.
Вот так правильно:

const fizzBuzz = num => {
  for(let i = 1; i <= num; i++) {
    if(i % 3 !== 0 && i % 5 !== 0){ 
      console.log(i);
    } else {
      console.log((i % 3 === 0 ? 'fizz' : '') + (i % 5 === 0 ? 'buzz' : ''));
    }
  }
};
fizzBuzz(100);

kahi4 Mar 26 2019 at 13:57

FizzBuzz

Оптимальным решением будет следующий код:

Ужасная конструкция с нулевой читаемостью. Я даже не буду придираться что совместную кратность можно было проверить как-то как i % (3 * 5), но вложенные if-else читать невозможно и стоит использовать continue.

Ну а вообще, в языках, откуда эта задачка, видимо, пришла — обычно есть printLn, и тогда первое условие нужно выкинуть вообще, потому что в случае совместной кратности будет fizz и buzz будут выведены на одну и ту же строку и без этого. Я так предполагаю, изначально это и проверялось — догадается ли кандидат до такого поведения, потому что задача, которая просит от тебя написать три условия — ну это не знаю, вступительная задача для пятого класса или типа того.

Вот код, который позволяет решить палиндром.

5 копий строки ради задачи, которая решается всего с одной дополнительной переменной. Браво. (Хорошо, к нижнему регистру приводить нужно все равно).

Второй шаг — реверс строки. Это сделать несложно: необходимо преобразовать ее в массив посредством метода .split() (библиотека String). Потом мы переворачиваем массив, используя .reverse() (библиотека Array). Последний этап — преобразование обратного массива в строку при помощи .join() (библиотека Array).

Первый раз слышу, чтобы в js кто-то Array и String называл библиотеками, а не встроенными типами, ну да ладно.

Если вместо этого вы используете рекурсию, это может понравиться интервьюеру и дать вам несколько дополнительных очков.

Всегда думал, что рекурсивное решение является "стандартным" и все всегда просят переписать на цикл (без использования массива. Я начинаю понимать почему весь веб тормозит), заодно спрашивают чем решение с рекурсией плохое.

Вообще либо я сильно переоцениваю уровень, для которого эти задачи сделаны, либо каждая задача содержит некие подводные камни, которые в статье не разобраны. Например, фибоначи часто реализуют через рекурсию, как самое очевидное решение. Но оказывается, что js (и много кто) не умеет в хвостовую рекурсию и на больших числах падает с переполнением стека.

На полиндром можно было бы просить проверить строку, заведомо большую, что даже вторая ее копия не влезет в память, а FizzBuzz — можно ли сократить количество условий.

ArVaganov Mar 26 2019 at 16:49

Но оказывается, что js (и много кто) не умеет в хвостовую рекурсию

В общем-то может, но для этого должны выполняться определенные условия.
Тут страница 62 и дальше.

Там как раз Николас рассматривает факториал

shaukote Jul 7 2019 at 13:00

1) Не увидел в книге по ссылке никаких дополнительных условий, кроме стандартного для tail call optimization требования, что рекурсивный вызов должен быть хвостовым (т. е. быть последней операцией в функции).

2) Рассуждения о преимуществах, которые даёт tail call optimization в JS, безусловно очень полезны и занимательны, но важно понимать, что в реальности они разбиваются о то, что в данный момент эта оптимизация поддерживается только в JSC (движок Safari). И, учитывая крайне принципиальную позицию разработчиков из Google, нет шансов, что это изменится в обозримом будущем.

ArVaganov Jul 7 2019 at 14:44

важно понимать, что в реальности они разбиваются о то, что в данный момент эта оптимизация поддерживается только в JSC (движок Safari). И, учитывая крайне принципиальную позицию разработчиков из Google, нет шансов, что это изменится в обозримом будущем

Это правда. Изучил вопрос подробнее. Действительно, на данный момент поддержка есть только в Safari. Видимо, имплементация данной оптимизации не тривиальная. =(

shaukote Jul 7 2019 at 19:17

Вы, видимо, изучили вопрос недостаточно подробно. :)
Дело в том, что в V8 (движок всех Chromium-based браузеров) tail call elimination давно реализована и… преднамеренно выключена его разработчиками. ¯\_(ツ)_/¯

Вот тут можно прочитать подробнее. Аргументация в целом очень похожа на таковую у Гвидо и у разработчиков Go (тут, увы, пруфов не будет), поэтому я упомянул об этом как о «принципиальной позиции разработчиков из Google».

ArVaganov Jul 7 2019 at 23:14

Интересные тексты.
И синтаксис от Mozilla и Microsoft интересный. ждем.jpg

shaukote Jul 9 2019 at 10:38

И синтаксис от Mozilla и Microsoft интересный. ждем.jpg

Я бы не особо надеялся. (:
Истории уже три с лишним года, а воз и ныне там. В в официальном списке proposal'ов в стандарт ES нет ни одного касающегося tail call, даже на stage 0.

Sirion Mar 26 2019 at 14:16

За такое рекурсивное нахождение чисел Фибоначчи лично я бы сразу развернул интервьюируемого на выход как злостного вредителя. И вообще статья жесть.

e_fail Mar 26 2019 at 14:18

Если вместо этого вы используете рекурсию, это может понравиться интервьюеру и дать вам несколько дополнительных очков.

const fibonacci = num => {
// if num is either 0 or 1 return num
if(num < 2) {
return num
}
// recursion here
return fibonacci(num — 1) + fibonacci(num — 2)
}

Аааа, ужас-ужас-ужас!!!

Если ваш интервьюер всерьёз считает, что приведённый рекурсивный код лучше тупого цикла — бегите оттуда бегом! Этот код работает за экспоненциальное время, в то время, как цикл — за линейное. В приличных же местах интервьюер должен знать, как эта задача решается за логарифмическое время.

Sirion Mar 26 2019 at 14:20

Ну, быстрое возведение в степень матриц — это всё-таки уже немного перебор для JS)

tuxi Mar 26 2019 at 14:24

Анаграммы в реальном проекте не используем, поэтому предлагаем вот такую задачку (одну из 3)
«написать на javascript (ECMAscript например не старше 5) код, который будет мониторить результаты любых ajax запросов расположенных на той же странице и в случае ошибок отправлять данные на сервер логов/статистики»
В ответах мы смотрим на оформление кода, на попытки подключения сторонних либ и самое главное, наличие списка уточняющих вопросов :)

Waxer Mar 26 2019 at 16:09

Мне вот интересно как вы fetch будете мониторить?

tuxi Mar 26 2019 at 17:36

а ограничение версии ecma не больше 5-й не напрягает?

Waxer Mar 27 2019 at 16:32

Меня напрягает что вы собственное условие задачи читать не умеете, где в ней сказано, что на странице, которую надо мониторить, нет fetch? В вашем условии задачи написано, используя es5 написать мониторинг всех ajax. Во-вторых как вам поможет, что кандидат знает или не знает как перекрыть метод send?

tuxi Mar 27 2019 at 17:59

Во-вторых как вам поможет, что кандидат знает или не знает как перекрыть метод send?

до сих пор помогало :)
Тестовое задание вполне себе покрывает часть наших нужд по теме фронтенда. Главное чтобы человек думать умел, остальному — научим/научится

Chamie Apr 19 2019 at 15:30

У вас написано «не старше», а не «не больше». 6й версии меньше лет, чем 5й => она моложе.

shaukote Mar 26 2019 at 14:25

По поводу проверки на палиндром.

Во-первых, странно, что вообще не упомянуто самое эффективное решение, оно не больно-то сложнее приведённого:

function palindrome(str) {
    for (let i = 0; i < Math.floor(str.length / 2); i++) {
        if (str[i] !== str[str.length - i - 1]) {
            return false;
        }
    } 

    return true;
}

А во-вторых задачки на проверку на палиндром и вообще на переворот строки простые только до тех пор, пока в дело не вступает Unicode. %)

Самый популярное решение работает корректно только для символов из BMP, для остальных разрушаются их суррогатные пары со всеми вытекающими последствиями:

const reverse = str => str.split('').reverse().join('');
// хабр не может в такие иероглифы ¯\_(ツ)_/¯
reverse('A \u{2f804} A');  // 'A �� A' 
reverse('A \u{2f804} A') === 'A \u{2f804} A'; // false

Это относительно несложно исправить, пользуясь тем, что при итератор по строке обходит code point'ы, а не code unit'ы:

const reverse = str => [...str].reverse().join('');
reverse('A \u{2f804} A');  // 'A � A' — тут всё ок, мамой клянусь
reverse('A \u{2f804} A') === 'A \u{2f804} A'; // true

Однако даже это не спасает, когда в дело вступают графемные кластеры:

const reverse = str => [...str].reverse().join('');
const str = 'А \u{0415}\u{0308} А';  // нормальная такая Ё
reverse(str);  // 'А ̈Е А'
reverse(str) === str; // true

Универсального решения, работающего во всех случаях, я не знаю, увы.

ArVaganov Mar 26 2019 at 16:32

Вот кстати, плюсую. Помимо неэффективного кол-ва операций, split+reverse+join занимает еще лишнюю память, в отличии от способа с двумя указателями, где памяти выделяется только на переменную в цикле.

Задача может усложняться разными регистрами и пробелами, вроде «А роза упала на лапу азора».

Тогда нужно отдельно двигать правый и левый указатель, (на пробелах предвигать). В целом в статье разбор типовых задач странный. Решения далеко не самые оптимальные и местами даже спорные.

UFO just landed and posted this here

shaukote May 5 2019 at 14:04

Действительно, я про него как-то не знал (упс). Ценная штука в некоторых случаях.

Проблема в том, что не любой графемный кластер можно нормализовать:

// devanagari ssi 
[...'षि'].length;  // => 2
[...'षि'.normalize()].length;  // => 2

// ok hand with dark skin tone
[...'\u{270c}\u{1f3ff}'].length;  // => 2
[...'\u{270c}\u{1f3ff}'.normalize()].length;  // => 2

WIBm May 2 2023 at 08:32

Проблему с графемными кластерами можно решить через метод normalize()

const reverse = str => [...str].reverse().join('');
const str = 'А \u{0415}\u{0308} А';
const strNormalized = str.normalize()
reverse(strNormalized );  // 'А Ё А'
reverse(strNormalized ) === strNormalized ; // true

shaukote Mar 26 2019 at 14:38

итератор по строке обходит code point'ы, а не code unit'ы

Это, кстати, приводит к некоторым любопытным последствиям:

const str = 'A \u{2f804} A';
[...str].length === str.length;  // false
[...str].map(char => char.length);  // [1, 1, 2, 1, 1]

Sirion Mar 26 2019 at 14:58

Каждый раз, когда я начинаю думать, что знаю JS досконально, находится что-нибудь в этом роде и меня разубеждает)

DeniSun Mar 26 2019 at 14:41

palindrome('race car') // false

Неплохо бы добавить

str = str.toLowerCase().replace(/ /g,'');

palindrome('А роза упала на лапу Азора') // true
// есть еще вариант от пенсионного фонда
palindrome('Оголи жопу пожилого'); // true

-1

Electrohedgehog Mar 26 2019 at 15:17

Мда, эксперты…
Код на проверку палиндрома не сработает, как писали выше, если будут пробелы и знаки препинания.
Fizzbuzz решён прямо в тупую, никакого понимания вообще.
Анаграммы… странноваты
Гласные вообще решаются в одну строку, два действия(если нет дополнительных условий).
Только Фибоначчи нормально.
Если эта дама людей собеседует, то ей стоит слегка поднять уровень.

keslo Mar 26 2019 at 15:25

Внесу и я свою лепту в fizzbuzz :)

function Fizzbuzz (num) {
  let response = '';
  if (!(num % 3)) response += 'fizz';
  if (!(num % 5)) response += 'buzz';
  return response || num;
}

Zavtramen Mar 26 2019 at 15:31

Написал точно такое же. Очень странно что оптимальным названо то что в статье, как впрочем и решения других задач.

function fizzbuzz(n) {
    var i, s; 
    for (i = 1; i <= n; i++) {
        s = '';
        if (i % 3 == 0) s = 'fizz';
        if (i % 5 == 0) s += 'buzz';
        console.log(s ? s : i);
    }
}

P.S. в задаче требуется написать функцию которая выводит от 1 до n в консоль. Ваша функция этого не делает, и еще со строчной обычно называют конструкторы, это если придираться ;)

keslo Mar 26 2019 at 16:28

Согласен с вами.

Chamie Apr 24 2019 at 16:45

А чем s ? s : i лучше, чем s || i для i ∈ ℕ?

Zavtramen Apr 24 2019 at 17:09

Ничем, просто я так написал, потому что мог.

Electrohedgehog Mar 26 2019 at 15:32

Каноничный вариант.

Cryvage Mar 26 2019 at 15:34

В анаграмме у вас две ошибки в одной строке:

for(let char of str.replace(/[^\w]/g).toLowerCase()) { ...

Во-первых, у метода replace должно быть 2 аргумента, иначе он заменит найденный символ на undefined. Вероятно это опечатка, но надо же проверять примеры, перед тем как публиковать статью.
Во-вторых, данное решение работает только с латинскими символами. В JS, паттерны, вроде \w не рассчитаны на юникод. ИМХО, это самое больное место регулярных выражений в JS. Правильная реализация будет выглядеть как-то так:

const notALetterRegEx = /[^A-Za-zªµºÀ-ÖØ-öø-ˁˆ-ˑˠ-ˤˬˮͰ-ʹͶ-ͷͺ-ͽΆΈ-ΊΌΎ-ΡΣ-ϵϷ-ҁҊ-ԣԱ-Ֆՙա-ևא-תװ-ײء-يٮ-ٯٱ-ۓەۥ-ۦۮ-ۯۺ-ۼۿܐܒ-ܯݍ-ޥޱߊ-ߪߴ-ߵߺऄ-हऽॐक़-ॡॱ-ॲॻ-ॿঅ-ঌএ-ঐও-নপ-রলশ-হঽৎড়-ঢ়য়-ৡৰ-ৱਅ-ਊਏ-ਐਓ-ਨਪ-ਰਲ-ਲ਼ਵ-ਸ਼ਸ-ਹਖ਼-ੜਫ਼ੲ-ੴઅ-ઍએ-ઑઓ-નપ-રલ-ળવ-હઽૐૠ-ૡଅ-ଌଏ-ଐଓ-ନପ-ରଲ-ଳଵ-ହଽଡ଼-ଢ଼ୟ-ୡୱஃஅ-ஊஎ-ஐஒ-கங-சஜஞ-டண-தந-பம-ஹௐఅ-ఌఎ-ఐఒ-నప-ళవ-హఽౘ-ౙౠ-ౡಅ-ಌಎ-ಐಒ-ನಪ-ಳವ-ಹಽೞೠ-ೡഅ-ഌഎ-ഐഒ-നപ-ഹഽൠ-ൡൺ-ൿඅ-ඖක-නඳ-රලව-ෆก-ะา-ำเ-ๆກ-ຂຄງ-ຈຊຍດ-ທນ-ຟມ-ຣລວສ-ຫອ-ະາ-ຳຽເ-ໄໆໜ-ໝༀཀ-ཇཉ-ཬྈ-ྋက-ဪဿၐ-ၕၚ-ၝၡၥ-ၦၮ-ၰၵ-ႁႎႠ-Ⴥა-ჺჼᄀ-ᅙᅟ-ᆢᆨ-ᇹሀ-ቈቊ-ቍቐ-ቖቘቚ-ቝበ-ኈኊ-ኍነ-ኰኲ-ኵኸ-ኾዀዂ-ዅወ-ዖዘ-ጐጒ-ጕጘ-ፚᎀ-ᎏᎠ-Ᏼᐁ-ᙬᙯ-ᙶᚁ-ᚚᚠ-ᛪᜀ-ᜌᜎ-ᜑᜠ-ᜱᝀ-ᝑᝠ-ᝬᝮ-ᝰក-ឳៗៜᠠ-ᡷᢀ-ᢨᢪᤀ-ᤜᥐ-ᥭᥰ-ᥴᦀ-ᦩᧁ-ᧇᨀ-ᨖᬅ-ᬳᭅ-ᭋᮃ-ᮠᮮ-ᮯᰀ-ᰣᱍ-ᱏᱚ-ᱽᴀ-ᶿḀ-ἕἘ-Ἕἠ-ὅὈ-Ὅὐ-ὗὙὛὝὟ-ώᾀ-ᾴᾶ-ᾼιῂ-ῄῆ-ῌῐ-ΐῖ-Ίῠ-Ῥῲ-ῴῶ-ῼⁱⁿₐ-ₔℂℇℊ-ℓℕℙ-ℝℤΩℨK-ℭℯ-ℼ-ℿⅅ-ⅉⅎↃ-ↄⰀ-Ⱞⰰ-ⱞⱠ-Ɐⱱ-ⱽⲀ-ⳤⴀ-ⴥⴰ-ⵥⵯⶀ-ⶖⶠ-ⶦⶨ-ⶮⶰ-ⶶⶸ-ⶾⷀ-ⷆⷈ-ⷎⷐ-ⷖⷘ-ⷞⸯ々-〆〱-〵〻-〼ぁ-ゖゝ-ゟァ-ヺー-ヿㄅ-ㄭㄱ-ㆎㆠ-ㆷㇰ-ㇿ㐀-䶵一-鿃ꀀ-ꒌꔀ-ꘌꘐ-ꘟꘪ-ꘫꙀ-ꙟꙢ-ꙮꙿ-ꚗꜗ-ꜟꜢ-ꞈꞋ-ꞌꟻ-ꠁꠃ-ꠅꠇ-ꠊꠌ-ꠢꡀ-ꡳꢂ-ꢳꤊ-ꤥꤰ-ꥆꨀ-ꨨꩀ-ꩂꩄ-ꩋ가-힣豈-鶴侮-頻並-龎ﬀ-ﬆﬓ-ﬗיִײַ-ﬨשׁ-זּטּ-לּמּנּ-סּףּ-פּצּ-ﮱﯓ-ﴽﵐ-ﶏﶒ-ﷇﷰ-ﷻﹰ-ﹴﹶ-ﻼＡ-Ｚａ-ｚｦ-ﾾￂ-ￇￊ-ￏￒ-ￗￚ-ￜ]|[\ud840-\ud868][\udc00-\udfff]|\ud800[\udc00-\udc0b\udc0d-\udc26\udc28-\udc3a\udc3c-\udc3d\udc3f-\udc4d\udc50-\udc5d\udc80-\udcfa\ude80-\ude9c\udea0-\uded0\udf00-\udf1e\udf30-\udf40\udf42-\udf49\udf80-\udf9d\udfa0-\udfc3\udfc8-\udfcf]|\ud801[\udc00-\udc9d]|\ud802[\udc00-\udc05\udc08\udc0a-\udc35\udc37-\udc38\udc3c\udc3f\udd00-\udd15\udd20-\udd39\ude00\ude10-\ude13\ude15-\ude17\ude19-\ude33]|\ud808[\udc00-\udf6e]|\ud835[\udc00-\udc54\udc56-\udc9c\udc9e-\udc9f\udca2\udca5-\udca6\udca9-\udcac\udcae-\udcb9\udcbb\udcbd-\udcc3\udcc5-\udd05\udd07-\udd0a\udd0d-\udd14\udd16-\udd1c\udd1e-\udd39\udd3b-\udd3e\udd40-\udd44\udd46\udd4a-\udd50\udd52-\udea5\udea8-\udec0\udec2-\udeda\udedc-\udefa\udefc-\udf14\udf16-\udf34\udf36-\udf4e\udf50-\udf6e\udf70-\udf88\udf8a-\udfa8\udfaa-\udfc2\udfc4-\udfcb]|\ud869[\udc00-\uded6]|\ud87e[\udc00-\ude1d]/ug;
for(let char of str.replace(notALetterRegEx, "").toLowerCase()) { ...

Выглядит дико, зато работает. Естественно, на собеседовании такое никто по памяти не напишет. Но, как мне кажется, кандидат должен отметить, что решение работает только с латинским алфавитом, а для поддержки других языков понадобится либо отдельная библиотека, такая как XRegExp, позволяющая использовать паттерны вроде p{L}, либо длиннющая регулярка, как правило сгенерированная каким-то сервисом. Например, данную простыню я сгенерировал здесь.

Aries_ua Mar 26 2019 at 22:13

Была еще задача на комплементарность ДНК. Решение ниже

const inp = 'ACCGGGTTTT';
const out = 'AAAACCCGGT';

// A -> T
// C -> G

// set mirror
// replace A to T
// replace C to G
// replace T to A
// replace G to T

function dnaComplement(str) {
  let newStr = '';

  for (let i = str.length - 1; i >= 0; i--) {
    let s = str[i];

    if (s === 'A') {
      newStr = newStr + 'T';
    } else if (s === 'T') {
      newStr = newStr + 'A';
    } else if (s === 'C') {
      newStr = newStr + 'G';
    } else if (s === 'G') {
      newStr = newStr + 'C';
    }
  }

  return newStr;
}

let resp = dnaComplement(inp);

console.log('resp ->', resp);
console.log(out === resp ? 'Done' : 'Wrong');

Немного оффтопа
Однажды пригласили меня на собеседование в одну фирму на позицию Senior JS developer (дело было в Кракове, Польша). В начале поговорили о жизни и ни о чем. Плавно перешли к менеджменту и микроменеджменту. И вот один из собеседователей говорит — ну что размялись, давайте теперь сделаем техническое интервью. Ну ок, не вопрос, конечно. И тут парень открывает гугл! А мне был виден его монитор немного, так как за спиной у него стеклянная стена и я видел отражение. И начинает читать вопросы для собеседования по JS с гугла Карл! Ты же сеньора собеседуешь! Ну как же так?!

Нда… теперь эта компания в моем личном черном списке. Нельзя же так не уважать людей, которые к вам приходят.

paratagas Mar 27 2019 at 12:10

Мне кажется, что анаграммы можно сделать намного проще:

 function anagram(srt1, str2) {
  const srt1Prepared = srt1.replace(/[^\w\s]|_/g, "").toLowerCase().split('').sort().join('');
  const srt2Prepared = str2.replace(/[^\w\s]|_/g, "").toLowerCase().split('').sort().join('');
  
  return srt1Prepared === srt2Prepared;
}

console.log(anagram('finder', 'Friend')); // true
console.log(anagram('hello', 'bye')); // false
console.log(anagram('', '')); // true

wheercool Mar 29 2019 at 07:08

К слову сказать Object.keys не сохраняет порядок ключей

shaukote May 5 2019 at 14:19

На самом деле, там всё немного сложнее — начиная с ES6 спецификация весьма строго описывает требуемый порядок ключей как для Object.keys(), так и для for (... in ...).
Сначала идут целочисленные свойства (в порядке возрастания), потом строковые (в порядке создания), потом symbols (тоже в порядке создания).

Проблема в том, что если попадётся браузер, не реализующий полностью ES6 (а как минимум в одном я почти уверен), то всё пойдёт не по плану, и вырулить на fallback-решение на полифиллах не очень-то получится.

RumataEstora Apr 15 2019 at 18:00

Мда. Всегда что-то можно оптимизировать. Даже фуз-буз задачку. Другие не стал читать — слишком много слов.

function getFizzBuzz(num) {
    var result = '';
    if ( num % 3 == 0 ) {
        result += 'fizz';
    }
    if ( num % 5 == 0 ) {
        result += 'buzz';
    }
    return result || num;
}

Sirion Apr 15 2019 at 19:58

const getFizzBuzz = num => `${num % 3 ? '' : 'fizz'}${num % 5 ? '' : 'buzz'}` || num;

Sirion Apr 15 2019 at 21:53

Или давайте ещё извратимся.

const getFizzBuzz = num => `${'fizz'.repeat(!(num % 3))}${ 'buzz'.repeat(!(num % 5))}` || num;

RumataEstora Apr 15 2019 at 22:10

Главный посыл не в количестве "крутых" фишек, а в оптимальности алгоритма. "Извращения" (как Вы сказали) не добавляют новизны, но читабельность кода ухудшают.

Sirion Apr 15 2019 at 22:22

Понятие оптимальности зависит от показателей, которые мы хотим оптимизировать. Если мы хотим оптимизировать показатель «продемонстрировать интервьюеру знакомство с фичами ES6 и умение писать write-only код»… =)

RumataEstora Apr 16 2019 at 07:45

Согласен. Но это уже из области "быстрее, выше, сильнее". В обычной жизни такие успехи не требуются, важнее чистота и лаконичность кода без потери ясности понимания написанного. Я не спорю с Вами, всего лишь дополняю Ваш ответ.