Почему зависает приложение? Почему не работает функция сортировки?

Question

[aLeXxX_03]

let button = document.createElement('button');
button.textContent = 'Добавить товар';
button.style.background = '#228B22';
button.style.width = '130px';

let container = document.getElementById('myDiv');
container.append(button);

let arr = ['Арбуз', 'Книга', 'Кофе', 'Макароны', 'Молоко', 'Сахар', 'Яблоки'];
let listEL;

function renderList(arr, bookSearch) {
  if (listEL) listEL.remove();
  listEL = document.createElement('ul');
  for (i = 0; i < arr.length; i++) {
    const liEl = document.createElement('li');
    liEl.textContent = `${i + 1}) ${arr[i]}`;
    if (arr[i] === bookSearch) {
      liEl.style.width = '17%';
      liEl.style.background = 'yellow';
      liEl.style.margin = '0 0 0 12px ';
      liEl.style.padding = '7px';
    }
    listEL.append(liEl);
  }

  container.append(listEL);
}
renderList(arr);

function sort(arr) {
  for (let i = 0; i < arr.length; i++) {
    for (let j = 0; j < arr.length - 1; i++) {
      if (arr[j] > arr[j + 1]) {
        let temp = arr[j];
        arr[j] = arr[j + 1];
        arr[j + 1] = temp;
      }
    }
  }
}

button.onclick = function book() {
  let book = prompt('Введите название книги');
  if (book === '') {
    alert('Название книги не введено!');
  }
  arr.push(book);
  sort(arr);
  renderList(arr);
};

Почему не работает функция сортировки sort(arr); ? Нужно, чтобы товары добавлялись в список и сортировались в алфавитном порядке.

Когда нажимаю кнопку "Добавить товар", вызывается функция Prompt, то при вводе имени товара и нажатии кнопки "Ок" или "Отмена" приложение намертво зависает.
Что я сделал не так?

Comments

[VoidVolker]

aLeXxX_03 и на каком этапе отладки у вас возникли сложности? Что именно не получается?

[aLeXxX_03]

VoidVolker, Когда нажимаю кнопку Добавить товар, вызывается функция Prompt, то при вводе имени товара и нажатии кнопки "Ок" или "Отмена" приложение намертво зависает.
Что я сделал не так?

[aLeXxX_03]

VoidVolker, подскажите?))

[Rsa97]

Зачем извращаться, если есть штатная Array.sort?

[VoidVolker]

Rsa97, в образовательных целях же.

[Rsa97]

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

[VoidVolker]

Rsa97, конечно — тут я полностью согласен и поддерживаю. Поэтому каждый раз и спрашиваю про отладку, а не даю готовое решение.

Answer 1

[Пума Тайланд]

for (let j = 0; j < arr.length - 1; i++) — вот баг: i++ вместо j++. j не инкрементируется, внутренний цикл вечный, отсюда зависание.

p.s. в renderList переменная i без let — глобальная, потенциально тоже стрельнёт

Comments

[aLeXxX_03]

спасибо, не увидел))) как всегда проблема в мелочах

[Adamos]

aLeXxX_03, в такие мелочи хорошо тычут носом IDE. Даже на JS.