🦁
Size: a a a
2019 October 28
🦁
проблема в том, что если ты передашь 2 и 4 то у тебя начало массива будет 7 элемент, а конец - 2
а, 2 и 4
ПП
indexOf возвращает первое вхождение
ПП
если искомый элемент встретится дважды то вернет индекс первого
ПП
и если мы его будет использовть для определения конца массива то может возникнуть ситуация, когда нам нужно вырезать массив где начальный индекс больше конечного
ПП
и это не сработает
ПП
по этому мы отталкиваемся от литерала который был передан в функцию первым аргументом после масссива
ПП
опять же это исходя из кода который я видел, какая там изначално задача была хз)
ПП
ну тебе нужно правильно определить индекс второго элемента
ПП
и если он меньше индекса первого то тогда уже проигнорить и взять длину массива - 1
ПП
тогда будет работать
ПП
ладно, мой вариант решения такой:
let cream = (arr, a, b) => {
let startIndex = arr.indexOf(a);
startIndex = !~startIndex ? 0 : startIndex;
let endIndex = arr.indexOf(b, startIndex + 1);
endIndex = !~endIndex ? (arr.length - 1) : endIndex;
return arr.slice(startIndex, endIndex + 1);
};
let cream = (arr, a, b) => {
let startIndex = arr.indexOf(a);
startIndex = !~startIndex ? 0 : startIndex;
let endIndex = arr.indexOf(b, startIndex + 1);
endIndex = !~endIndex ? (arr.length - 1) : endIndex;
return arr.slice(startIndex, endIndex + 1);
};
ПП
я хз нужно ли поведение что бы если не найдено ни начало ни конец то возвращало весь массив, если нет то в принципе тут это не сложно поправить
ПП
воть?