Верно ли моё решение задачи по JavaScript?

Question

[Андрей]

Тест задачи:

Write a function which implement case-insensitive
lexicographical comparison of two strings
(also known as alphabetical compare)
Examples:
cmp('HELLO', 'hello') == 0// true
cmp('banaga', 'apple') > 0// true
cmp('banana', 'mango') < 0// true

Моё решение:

function cmp(a, b) {
    a = a.toLowerCase();
    b = b.toLowerCase();
    if (a === b) {
        return 0;
    }
    return a > b ? 1 : -1;
}

Как вы считаете, моё решение верно на данную конкретную задачу? Если нет, то почему?

Comments

[Александр]

var cmp = (a,b)=>a.toUpperCase()==b.toUpperCase()?0:(a.toUpperCase()<b.toUpperCase()?-1:1)

[Рональд Макдональд]

Вот, другое дело!

[Андрей]

Александр, в чём разница с моим решением? В том, что в одну строчку? toUpperCase() разве данном случае дает разницу?

[hack504]

Andchir, в данном случае нет никакой разницы в toUpperCase и toLowerCase , разница есть в === и ==

[Андрей]

hack504, и в чём она в данном конкретном случае?

[Robur]

Andchir, в данном конкретном случае в логике работы ни в чем, её нет.
Но есть в другом - лучше всегда писать === чем думать где надо === или == тоже подойдет, это упрощает и вашу работу и того кто будет этот код трогать после вас и уменьшает возможное количество будущих ошибок при изменении этого куска кода.

[Андрей]

Robur, согласен.

[RAX7]

function cmp(str1, str2) {
  return str1.toLowerCase().localeCompare(str2.toLowerCase());
}

[Андрей]

RAX7, этот ответ уже был, возможно вы подсмотрели.

[RAX7]

Andchir, если бы я подсмотрел, то сделал бы лучше https://developer.mozilla.org/ru/docs/Web/JavaScri...

[Андрей]

RAX7, но ваше решение тоже неверно) Догадались бы Вы что просит от вас человек по такому заданию? Оказывается нельзя использовать существующие в языке методы сравнения строк.

[RAX7]

Andchir, не догадался бы конечно, в задании сказано написать функцию, а не угадать что можно использовать, а что нельзя)

[Андрей]

RAX7, чел говорит что "имплементировать функцию сравнения строк" это и значит, что использовать существующие нельзя. Но мне кажется, что у слова "имплементировать" другое значение - воплотить в жизнь что-то или выполнить что-то, а в программировании - написать функцию - это и значит имплементировать. Не нашел точного определения.

[RAX7]

Andchir,

использовать существующие нельзя

Что именно нельзя? Методы строк? Операторы сравнения? По моему такое в тексте задачи должны указывать прямым текстом.
Есть такая интересная задачка на js: написать простенький калькулятор

calc(1, 2, "+") === 3 //should return true
calc(0, 0, "-") === 0 //should return true
calc(6, 7, "*") === 42 //should return true
calc(5, 4, "%") === 1 //should return true
isNaN(calc(9, 0, "/")) === true //should return true

если не смотреть на пункт Coding Limitation, то все просто, а если посмотреть...
https://www.codewars.com/kata/5a42497e55519ef5c5000015

[Robur]

Andchir, вы существующие и не использовали - написали свою.
А если для него "имплементировать" значит не использовать даже операторы сравнения - то что вообще остается.
Это классический пример вопроса на "угадай мой ответ" а не "реши задачу".

Answer 1

[Robur]

Как вы считаете, моё решение верно на данную конкретную задачу?

да.

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

Comments

[Андрей]

Это верно. Но в задаче даны конкретные примеры строк, с которыми должна работать функция. Если учесть, что строки могут быть на разных языках или содержать так же цифры, то зачем вообще эти примеры даны???

Вот какой был дан ответ проверяющего, который сказал, что моё решение не верно:

but you need to implement the comparison

because if you want to use existing methods, you can just do

const cmp = (a, b) => a.localeCompare(b, 'en', {sensitivity: 'base'}));

[Robur]

Andrew, там ничего не было про джунов :)

[Andrew]

Robur, поэтому удалил))
кажись, такое только джунам дают или, скорее даже, трейни, вообще без опыта, потому как, знания работы со строками, числами и DOM хватит

[Robur]

Andchir, Это просто неопытный проверяющий - он предполагает что вы должны знать те условия которые у его в голове. А так же вместо того чтобы оценивать ваше решение, он вам пишет про свое - зачем это вам? чем оно лучше вашего? Что оно делает такого что не делает ваше? в чем отличие кроме того что ваше вы написали, а его - он написал?
Он написал конкретную задачу. Вы ее решили. То что он ждал чего-то другого, или того что вы будете сидеть и писать свою логику без использования существующих методов (а по факту и без использования простых операторов сравнения) - его проблема, он должен был это указать в задании.

[Robur]

Ну понятно что если вы хотите пройти это интервью и получить эту работу то вам проще всего сыграть по этим правилам. Но учтите что потом вам с этим человеком работать :)

[Robur]

Andrew, на позицию сеноьр-архитектора тоже спрашивают порой как работает приведение типов и всякое такое. Проблема в том что просто делать хорошие интервью и реально оценивать человека - это очень крутой скилл, и мало кто умеет. Программисты вообще то не тем занимаются и подобные навыки не прокачивают.

[Андрей]

Andrew, показал свой "большой прибор"? Молодец.

[Andrew]

Andchir, вам плохо? комплексы?

[Андрей]

Andrew, посмотрел ваши ответы на другие вопросы и понял, что это только меня вы одарили своим флудом и демонстрацией "прибора" (все эти разговоры про джунов и т.п.). Поэтому просто замолкаю. Спасибо за ваши ответы на другие вопросы.

Answer 2

[Robert-critical-thoughts]

Напишите функцию, которая реализует регистро-независимую проверку

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

Comments

[Андрей]

регистро-независимую - значит результат не зависит от регистра строк. Что не так конкретно?

[Robur]

Для комментариев тоже надо вчитаться в задание. Все у него правильно.

[Robert-critical-thoughts]

Robur, К вечеру глаз замылен, х№№№ю написал, низкий поклон что поправили