Как правильно упростить код?

Question

[Ольга]

const obj = {
			inHistory: [1,6,8,10],
			inBook:[7,8,9,10,12]};

function _intersection(src) {
	let inHistory = []; //массив для колонки1
	let inBook = [];//массив для колонки2
	let inHistoryOrig = [...src.inHistory];
	let inBookOrig = [...src.inBook];
	const crossing = intersection(inHistoryOrig,inBookOrig); //массив пересекающихся объектов

	for(let i = 0; i < crossing.length; i++){
		let index1 = findIndex(inHistoryOrig, function(el){return el === crossing[i];}); //индекс элементов из пересеченного массива [2] , [3]
		let _take = take(inHistoryOrig, index1); //массив элементов до пересеченных [1,6] , [1,6,8]
		inHistoryOrig = drop(inHistoryOrig, index1); //выкидываем из начального массива элементы [1,6]; [1,6,8]
		inHistory.push(_take); //добавляет в массив inHistory  элементы, в inHistory [[1,6]]; [[ 1, 6], [1, 6, 8]]

		let index2 = findIndex(inBookOrig, function(el){return el === crossing[i];});
		let _take2 = take(inBookOrig, index2);
		inBookOrig = drop(inBookOrig,index2);
		inBook.push(_take2);
		}

	inHistory.push(inHistoryOrig);
	inBook.push(inBookOrig);
	return {
		inHistory: inHistory,
		inBook: inBook,
	};
}

console.log(_intersection(obj));
//{ inHistory: [ [ 1, 6 ], [ 8 ], [ 10 ] ],
 // inBook: [ [ 7 ], [ 8, 9 ], [ 10, 12 ] ] }

Comments

[Demian Smith]

Вот так можно упростить:

npm install -S lodash

_.intersection([1,6,8,10], [7,8,9,10,12])

Начните пользоваться lodash. Это невероятно полезная библиотека. Очень тяжело себе представить грамотный современный проект без неё.

[Ольга]

Demian Smith, тут я и использую lodash, суть кода не просто объединить два массива, а найти пересекающиеся элементы и разбить массив на подмассивы

[Demian Smith]

Ольга, ух, сорри, не сразу дошло про лодаш. Туповат.

Добавьте в код вашего вопроса коммент того что ожидается увидеть в консоли. Ато до сих пор непонятно что требуется.

[Ольга]

Demian Smith, добавила

[Demian Smith]

Ольга, можете словами описать что требуется? Ато всё равно непонятно. Скорее всего вам самой будет проще понять как упростить код, если вы опишите словами то, что требуется.

[Ольга]

Demian Smith, тут внутри цикла для обоих исходных массивов применяются одни и те же функции(findIndex, take, drop).Я думаю, нужно эти повторения запихнуть в функцию, где параметром и будут исходные массивы. Проблема в том, что у меня не получается эту функцию создать, т.е она не верно работает.
Собственно у меня получилось вот что

function columns(column) {
		let index = findIndex(column, function(el){return el === crossing[i];});
		let _take = take(column, index);
		column = drop(column, index);
		return _take;
}

Затем, при помощи push я добавляю результаты функции column в inHistory и inBook , но результат не тот что хотелось бы

[Demian Smith]

Ольга, а можете описать словами как должна работать функция _intersection? Ато по коду непонятно в чём цель всего этого.

[Demian Smith]

Ольга, описание можете добавить в текст вопроса. Будет больше вероятности что вам помогут.

[Ольга]

Demian Smith, _intersection выводить массив из элементов, которые одинаковы для начальных массивов. в моем случае для [1,6,8,10] и [7,8,9,10,12] _intersection выведет [8,10]

[Demian Smith]

Ольга, та нет. Опишите как работает ваша функция _intersection :)

Ато неясно по какому принципу

{
      inHistory: [1,6,8,10],
      inBook:[7,8,9,10,12]
}

превращается в

{
        inHistory: [ [ 1, 6 ], [ 8 ], [ 10 ] ],
        inBook: [ [ 7 ], [ 8, 9 ], [ 10, 12 ] ]
}

[Ольга]

Demian Smith,аааа)
1. создаются пустые массивы inHistory, inBook в них передаются результаты выполнения функции.
2. Создаем копии исходных массивов inHistoryOrig и inBookOrig, которые внутри obj, с ними и будет работать функция
3. в crossing храниться результат пересечения массивов inHistoryOrig и inBookOrig //[8,10]
4. Затем в цикле проходим по массиву пересечений (crossing)
4.1 при помощи findIndex находим индекс 8 из начального массива inHistoryOrig, при повторной итерации для 10 // индексы 2, 3
4.2 с помощью take берем из начального массива те элементы, которые стоят до пересечения ,т.е при первой итерации цикла это [1,6] , затем [1,6,8] и передаем эти значения в пустые массивы в начале (в inHistory)
4.3 drop удаляет из inHistoryOrig те элементы, которые мы тейком передали в inHistory
4.4 Первая интерация закончена, вторая аналогичная
5. передаем в пустой массив inHistory массив inHistoryOrig после всех итераций цикла
6. функция возвращает массив inHistory

[Demian Smith]

Ольга, если нашу переписку прочтут, то будет непонятно кто над кем издевается )))

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

Как-то так:

Функция _intersection принимает на вход два массива чисел. И возвращает два массива чисел, в которых числа сгруппированы в подмассивы по такому-то правилу.

На этом сайте обычно очень быстро отвечают и помогают, но сейчас у нас в ответе только один неработающий вариант с вечным циклом, потому что людям непонятно что должна делать функция. джаваскрипт-код-на-русском выше это тоже не проясняет. Вам помогут как только будет ясно как должна работать функция.

[Ольга]

Demian Smith, Могу Вас уверить, что над Вами я не издеваюсь)

Answer 1

[Павел Корнилов]

Тоже не понял всю глубину глубин, но отвечу на конкретный вопрос из комментов топика:

Demian Smith, _intersection выводить массив из элементов, которые одинаковы для начальных массивов. в моем случае для [1,6,8,10] и [7,8,9,10,12] _intersection выведет [8,10]

var a = [1,6,8,10], b = [7,8,9,10,12];
function _intersection (a,b) {
	return a.filter(function(obj) { return b.indexOf(obj) >= 0; });
	
}
console.log(_intersection (a,b));