Как занести число в соответствующую ячейку?

Question

[bigos]

Здравствуйте. Делаю задачу, в которой нужно каждое число из массива вывести в соотстветствующую ячейку в таблице. Всего ячеек 9, а чисел гораздо больше. Например, в массиве есть числа 2,1,4,3,6,8. При чем числа могут повторяться. Видим 2-ку - заносим в ячейку под номером 2.. 1-цу в ячейку 1 и так далее.
У меня есть массив и переменные с 1 до 9

var massiv = [];
var fkol_1 = 0;
var fkol_2 = 0;
---------
var fkol_9 = 0;

Эти переменные нужны для того, чтобы проверять числа из массива и выводить их в ячейку таблицы(td). Например, вот так:

if ((massiv[i] == 1)) {fkol_1++;}
if ((massiv[i] == 2)) {fkol_2++;}
//и так далее до 9

Также есть 2 переменные a=42, b=79. Их значение может быть разное, но они двухзначные. И мне нужно их разделить, чтобы они стали однозначным(4,2,7,9). Затем значения этих переменных заношу в массив massiv[] через функцию:

function splitNumber(number) {
	number.toString().split('').forEach(number => massiv.push('('+number+')'));
}
splitNumber(a);
splitNumber(b);

Теперь сама суть вопроса: как мне проверить последние 4-е цифры в массиве (в нашем примере эти четыре цифры и являются переменными a,b(4,2,7,9) ) и занести их в определенную ячейку таблицы в скобках? Например, у нас в ячейке есть уже 2-ка, и так как в последних 4-х числах массива встречается 2-ка, то заносим её в скобочках (2), чтобы получилось 2(2) ? Вывожу так переменные:

<td><p>'+fkol_1+'</p></td>
<td><p>'+fkol_2+'</p></td>
/*  и так до fkol_9' */

Для наглядности живой пример - https://jsfiddle.net/e1c81ujg/

Answer 1

[Алексей Самсонов]

Простите, но это нечитабельно.

if ((massiv[i] == 1)) {fkol_1++;}
if ((massiv[i] == 2)) {fkol_2++;}
//и так далее до 9

замените хотя бы на
fkol[massiv[i]]++;

и так далее, не нужно писать по 10 строк на операции, которые при правильном подходе можно написать одной строчкой.

И именование переменных, старайтесь именовать их понятно и по английски

Comments

[bigos]

Заменил на
if ((massiv[i] == i )) { fkol[massiv[i]]++ }
Если вы перейдете по "живому примеру", то увидите, что я вовзращаю таблицу с переменными fkol_1 и т.д. Как в данном случае мне возвращать значения?

[bigos]

if ((massiv[i] == i )) { fkol[massiv[i]]++ } - некорректно работает...

[Алексей Самсонов]

Иф избыточен просто оставьте тело ифа, и смысл не поменяется.

fkol_1 заменяете на fkol[1] и так далее.

[Алексей Самсонов]

bigos: некорректно потому что наверное не объявили: var fkol = [];

[bigos]

Алексей Самсонов: объявил. Оставил тело, вывожу fkol[1] = отображается в 1-й ячейке NaN.. Вы смотрели живой пример?

[bigos]

Алексей Самсонов: https://jsfiddle.net/e1c81ujg/1/ - вот я обновил

[Алексей Самсонов]

bigos: массив нужно инициализировать, либо использовать тренарный оператор или проверять на существование элемента.
В общем если я правильно понял ваш вопрос, то вот чуток обновленный фидл https://jsfiddle.net/e1c81ujg/3/
в целом ваш стиль кода сохранен, так что вопросов со стороны возникнуть не должно.

Но в целом: уделите больше времени на изучение культуры кода и реализации логики ваших программ

[bigos]

Алексей Самсонов: огромнейшее спасибо. Последний вопрос:
Вот ваш обновленный фидл - https://jsfiddle.net/e1c81ujg/5/ Я изменил условие в if. Там у вас 9, но я поставил 16, так как єто условие считает количество повторяющихся чисел и выводит в таблицу. Если взять онлайн пример www.astrocentr.ru/index.php?przd=pif, то там выводит такой же вывод как и ваш, но исправленный мною.
Теперь касательно вопроса: https://jsfiddle.net/e1c81ujg/4/ в моем примере (чуть измененном) в четвертой ячейке складываются числа между собой и выводится их сумма: Допустим, 11111 2 3 будет равняться 10, так как 1+1+1+1+1+2+3. Поэтому 10-ку записываем в четверную ячейку. И так для каждой строки(TR). У вас 4-я ячейка(td) в вашем примере (https://jsfiddle.net/e1c81ujg/5/) как-то странно считает и выводит 0. Как сделать, чтобы числа НЕ в скобках считались только между собой, а числа в скобках - только между собой.
В конце javascript'a показано как считается сумма 4-й ячейки. Подскажите

[Алексей Самсонов]

bigos: я уже отписал что код не читабелен) я не могу понять что куда выводиться.
Я даже в целом не понимаю где у вас складываются одни числа с другими.
Числа в скобках хранятся в 16,17,18,19 индексах массива, сумму по строкам можно посчитать как-то так:

var str1 = 0;
var str2 = 0;
var str3 = 0;

for (var i = 16; i < 20; i++) {
if (massiv[i] < 4) {
str1 += massiv[i];
} else if (massiv[i] < 7) {
str2 += massiv[i];
} else {
str3 += massiv[i];
}
}

так в str1 будет хранится сумма в ячейках 1,2,3
в str2 будет хранится сумма в ячейках 4,5,6
в str3 будет хранится сумма в ячейках 7,8,9

[Алексей Самсонов]

просто сумму считайте не в редьюсе, а прямо в вашей функции, и спокойно выводите что хотите.

Мне кажется на вопрос отвечено довольно подробно, и я заслуживаю галочки ;)
Иначе надо формулировать вопрос, аля: "решите за меня лабораторную работу"

[bigos]

Алексей Самсонов: спасибо, вы очень помогли) Поставил галочку. Буду своим методом делать. Направили меня