Функцию бы покороче

[Parallelogram]

Здравствуйте. Есть код, который работает, но уж больно похож на индусский. Помогите сделать его красивым))

https://jsbin.com/waxugom/edit?js,output

let arr = ['n', 'n', 0, 0, 0, 'n', 1, 1, 2, 2, 2, 2, 2, 'n', 'n', 3];

let html = '<table><tr>';
for (let i=0; i<arr.length; i++) { html += `<th>${i+1}</th>`; }
html += '</tr>';

html += '<tr>';
let mem = 0;
for (let i=0; i<arr.length; i++) {
	if (i >= arr.length || arr[i] != arr[i+1]) {
		if (arr[i] == 'n') {
			html += "<td></td>";
		} else {
			html += `<td colspan="${++mem}">${arr[i]+1}</td>`;
			mem = 0;
		}
	} else {
		if (arr[i] == 'n') {
			html += "<td></td>";
			mem = 0;
		} else {
			mem++;
		}
	}
}
html += '</tr></table>';

document.body.insertAdjacentHTML("beforeend", html);

Есть массив в котором могут быть значения:
'n' - пустые ячейки
1, 2, 3, и тд - разные отдельные блоки

Нужно создать таблицу, чтобы 'n' превратились в пустые ячейки, а блоки 1, 1, 1 в ячейку с colspan="3", 2, 2, 2, 2, 2 в ячейку с colspan="5", и тд

Мой код решает эту задачу, но он совсем некрасивый

[p51x]

У вас цикл до

i<arr.length

смысл в проверке

i >= arr.length

?

arr[i] != arr[i+1]

С чем будет последний элемент сравниваться?

[Parallelogram]

p51x,

i >= arr.length нужна для последнего элемента
arr[i] != arr[i+1] будет проверяться на всех итерациях кроме последнего, там сработает i >= arr.length иначе ошибку выдаст

[macomics]

Сообщение от Parallelogram

i >= arr.length нужна для последнего элемента

Тогда надо проверять значение i+1, а не i. Цикл for закончится при i == arr.length и это условие всегда будет false

[Parallelogram]

Сообщение от macomics

Тогда надо проверять значение i+1, а не i. Цикл for закончится при i == arr.length и это условие всегда будет true

Да нееее. arr[i] != arr[i+1] нужен для того, чтобы определить, закончился этот блок или ещё нет. Если следующий элемент тоже будет с тем же значением, то значит блок ещё не закончился. Мы прибавляем mem++ и начинаем заново.

[Parallelogram]

Алгоритм такой: беру первый элемент и сравниваю его со следующим, если он такой же, то mem++ и continue. Если другой, то записываю colspan=mem и continue начинаю новый блок.

[macomics]

Сообщение от Parallelogram

Да нееее. arr[i] != arr[i+1] нужен для того, чтобы определить, закончился этот блок или ещё нет. Если следующий элемент тоже будет с тем же значением, то значит блок ещё не закончился. Мы прибавляем mem++ и начинаем заново.

Я вам написал только про часть условия с i >= arr.length. У цикла for не будет итерации с таким значением счетчика. А вот при i = arr.length - 1 (последний элемент массива) вы как раз и вылезете за его пределы во второй части условия (arr[i] != arr[i + 1]). Тогда значение arr[i+1] окажется вне массива arr (i+1 = arr.length), а это уже элемент на один дальше конца массива.

[Parallelogram]

Сообщение от macomics

Я вам написал только про часть условия с i >= arr.length.

Можно заменить на i == arr.length. Это вообще не играет роли.

Или Вы не запускали скрипт или я совсем ничего не понимаю.
Мой код абсолютно рабочий. Мне просто не нравится дублирование кода в цикле. Скорей всего это можно как-то исправить/переписать. Вот об этом я прошу. Вы же предлагаете сломать условие if до нерабочего варианта.

[macomics]

Что в условии

if ((i+1 == arr.length) || (arr[i] != arr[i+1]))

не рабочее. А вот за пределы массива с этим условием вы не вылезете.

Если чтобы оценить программу вам надо ее запустить, тогда добавьте вывод в консоль 4-х значений из цикла:
i, i+1, arr[i] и arr[i+1]
Дальше оцените сами полученный вывод.

P.S. Вам два человека на форуме указывают на одно и тоже место в программе. Но вы продолжаете препираться, вместо того чтобы проверить.

add: я за вас сделал вывод, чтобы вы больше не препирались, а прислушались

[Parallelogram]

Вы правы. Я совсем о другом думал и не видел в упор что мне говорят.
Либо i+1 == arr.length, либо i == arr.length-1. Мне же так и говорили :facepalm:
Но всё тлен, это условие не нужно, хоть и правильное. Так как arr[i] != arr[i+1] в последнем элементе сравнивается с undefined и всё равно выполняется корректно.