Added Linear Search Fix

This commit is contained in:
= 2020-10-17 20:30:01 +05:30
parent ceed279de8
commit 45762593b5
2 changed files with 16 additions and 17 deletions

View File

@ -4,17 +4,17 @@ describe('linearSearch', () => {
it('should search all numbers in array', () => {
const array = [1, 2, 4, 6, 2];
expect(linearSearch(array, 10)).toEqual([]);
expect(linearSearch(array, 1)).toEqual([0]);
expect(linearSearch(array, 2)).toEqual([1, 4]);
expect(linearSearch(array, 10)).toEqual(-1);
expect(linearSearch(array, 1)).toEqual(0);
expect(linearSearch(array, 2)).toEqual(1);
});
it('should search all strings in array', () => {
const array = ['a', 'b', 'a'];
expect(linearSearch(array, 'c')).toEqual([]);
expect(linearSearch(array, 'b')).toEqual([1]);
expect(linearSearch(array, 'a')).toEqual([0, 2]);
expect(linearSearch(array, 'c')).toEqual(-1);
expect(linearSearch(array, 'b')).toEqual(1);
expect(linearSearch(array, 'a')).toEqual(0);
});
it('should search through objects as well', () => {
@ -33,8 +33,8 @@ describe('linearSearch', () => {
{ key: 6 },
];
expect(linearSearch(array, { key: 10 }, comparatorCallback)).toEqual([]);
expect(linearSearch(array, { key: 5 }, comparatorCallback)).toEqual([0]);
expect(linearSearch(array, { key: 6 }, comparatorCallback)).toEqual([1, 3]);
expect(linearSearch(array, { key: 10 }, comparatorCallback)).toEqual(-1);
expect(linearSearch(array, { key: 5 }, comparatorCallback)).toEqual(0);
expect(linearSearch(array, { key: 6 }, comparatorCallback)).toEqual(1);
});
});

View File

@ -10,13 +10,12 @@ import Comparator from '../../../utils/comparator/Comparator';
*/
export default function linearSearch(array, seekElement, comparatorCallback) {
const comparator = new Comparator(comparatorCallback);
const foundIndices = [];
array.forEach((element, index) => {
if (comparator.equal(element, seekElement)) {
foundIndices.push(index);
for (let i = 0; i < array.length; i += 1) {
if (comparator.equal(array[i], seekElement)) {
return i;
}
}
});
return foundIndices;
return -1;
}