Size: a a a

2020 March 12

EL

Evgeny Lukoyanov in JS for testing
источник

OP

Oleksandr Pelykh in JS for testing
😬
источник

ab

artem belikov in JS for testing
Ошибка в коде:
для ветки asc все норм.
нет конвертации к числу (Number) при сравнении
источник

OP

Oleksandr Pelykh in JS for testing
artem belikov
Ошибка в коде:
для ветки asc все норм.
нет конвертации к числу (Number) при сравнении
я использую только asc
источник

EL

Evgeny Lukoyanov in JS for testing
может если перевернуть функцию в else
с b[paramName] < a[paramName] ? -1: 1
на a[paramName] > b[paramName] ? 1: -1 то заработает?

и в одном случае вы приводите к Number, а в другом - нет
крайне сомнительно выглядит
источник

EL

Evgeny Lukoyanov in JS for testing
Более того у вас 0 не возвращается
хотя должен в случае равнества
источник

EL

Evgeny Lukoyanov in JS for testing
а вызов можно посмотреть?
какой paramName используется
источник

m

mkots in JS for testing
зачем вообще использовать > или < ? напишите просто return asc ? a-b : b-a
источник

ab

artem belikov in JS for testing
mkots
зачем вообще использовать > или < ? напишите просто return asc ? a-b : b-a
Потому что читаемость важнее хаков, если конечно тут не спортивное программирование.
источник

DK

Dmitriy Kovalenko in JS for testing
Просто это специально было придумано чтобы юзать a - b, потому что у > или < нету состояния равно; Поэтому для строк например есть
new Intl.Collator().compare('a', 'c')
источник

DK

Dmitriy Kovalenko in JS for testing
Если писать на >|< то это будет както так
.sort((a,b) => {
   if(a < b) { return -1; }
   if(a > b) { return 1; }
   return 0;
}
источник

DK

Dmitriy Kovalenko in JS for testing
Кстати ты делаешь очень грязную вещь, array.sort мутирует массив а ты еще и дополнительно переприсваиваешь его, лучше будет сделать
return […array].sort(…)
источник

m

mkots in JS for testing
лучше использовать slice() чтобы не выхватить проблем с многомерными массивами
источник

m

mkots in JS for testing
mkots
лучше использовать slice() чтобы не выхватить проблем с многомерными массивами
А вот я и неправду сказал. [...arr] это [].concat(arr) , а вот почему я думал что у него проблеммы со вложенными массивами. я не помню
источник

OP

Oleksandr Pelykh in JS for testing
Dmitriy Kovalenko
Кстати ты делаешь очень грязную вещь, array.sort мутирует массив а ты еще и дополнительно переприсваиваешь его, лучше будет сделать
return […array].sort(…)
учту. спасибо
источник
2020 March 13

S

Star naoki in JS for testing
Привет, подскажите что за ошибка - https://prnt.sc/rfind0
источник

OK

Oleksandr Khotemskyi in JS for testing
Star naoki
Привет, подскажите что за ошибка - https://prnt.sc/rfind0
Не используй эти тройные слеши
источник

OK

Oleksandr Khotemskyi in JS for testing
Подключи просто @types/cypress в package.json
источник

S

Star naoki in JS for testing
Oleksandr Khotemskyi
Подключи просто @types/cypress в package.json
спасибо
источник

AD

Andrei Dzeichyk in JS for testing
Oleksandr Khotemskyi
Пожалуйста, я тоже на такое напарывался с версиями
Так а в чем проблема с нодами?)
Почему одна сортирует, а другая нет? Мб конкретные версии помнишь
источник