Skip to content

Не могу найти ошибку в коде. По преобразованию кода в http://jslint.com/ вопросов нет.#9

Open
NBronina wants to merge 1 commit intocripi-javascript:masterfrom
NBronina:master
Open

Не могу найти ошибку в коде. По преобразованию кода в http://jslint.com/ вопросов нет.#9
NBronina wants to merge 1 commit intocripi-javascript:masterfrom
NBronina:master

Conversation

@NBronina
Copy link

Программа работает не совсем корректно. Неправильно считываются
данные из формы. Например, массив [1, 33, 65, 3, 5, 6] будет
отсортирован не правильно, тогда как массив [1, 7, 8, 3, 5, 6]
будет отсортирован верно. Подскажите, пожалуйста, в чем ошибка?

Выполнение первого задания по преобразованию кода для совтировки
массива. Программа работает не совсем корректно. Неправильно считываются
данные их формы. Например, массив [1, 33, 65, 3, 5, 6] будет
отсортирован не правильно, тогда как массив [1, 7, 8, 3, 5, 6]
отсортируется верно.
@azproduction
Copy link
Member

value.split(" "); возвращает массив из строк, строки сравниваются поэлементно "4" > "33" === true необходимо их всех привести к числу, а потом сравнивать.

Проще всего это сделать так

value.split(" ").map(function (item) {
    return +item;
});

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments