Как исправить Array indexOf () в JavaScript для браузеров Internet Explorer

295

Если вы когда-либо работали с JavaScript, вам известно, что Internet Explorer не реализует функцию ECMAScript для Array.prototype.indexOf () [включая Internet Explorer 8]. Это не большая проблема, потому что вы можете расширить функциональность на своей странице с помощью следующего кода.

Array.prototype.indexOf = function(obj, start) {
     for (var i = (start || 0), j = this.length; i < j; i++) {
         if (this[i] === obj) { return i; }
     }
     return -1;
}

Когда я должен реализовать это?

Должен ли я обернуть его на всех своих страницах с помощью следующей проверки, которая проверяет, существует ли функция прототипа, и если нет, продолжайте и расширять прототип Array?

if (!Array.prototype.indexOf) {

    // Implement function here

}

Или проверить браузер, и если это Internet Explorer, то просто реализовать его?

//Pseudo-code

if (browser == IE Style Browser) {

     // Implement function here

}
Бобби Борщич
источник
На самом деле Array.prototype.indexOfне является частью ECMA-262 / ECMAScript. См. Ecma-international.org/publications/files/ECMA-ST/ECMA-262.pdf Может быть, вы думаете String.prototype.indexOf...
Fresh Crescent
5
Это расширение, а не часть оригинального стандарта. Однако он должен быть реализован как часть Javascript 1.6 (что IE не может сделать) developer.mozilla.org/en/New_in_JavaScript_1.6
Джош Стодола
1
@Josh: просто имел в виду «IE не реализует функцию ECMAScript ...»
Crescent Fresh
4
Ваша реализация Array.indexOfне учитывает отрицательные стартовые индексы. Посмотрите на предложенную Mozilla
временную
3
Я обновил вопрос, чтобы использовать "===", потому что я волнуюсь, что люди скопируют его с "==", и это было бы неправильно - в остальном это нормально. Смотрите ответ Эли Грея.
Джошкомли

Ответы:

213

Делай это так...

if (!Array.prototype.indexOf) {

}

Как рекомендуется совместимость с MDC .

В общем, код обнаружения браузера - большой нет-нет.

Джош Стодола
источник
У меня недостаточно репутации, чтобы отредактировать вопрос, но не стесняйтесь удалить язык ECMAScript и заменить его соответствующей формулировкой.
Еще
12
Будьте осторожны, если вы используете этот вид обнаружения. Другая библиотека может реализовать эту функцию до того, как вы протестируете ее, и она может не соответствовать стандартам (прототип сделал это некоторое время назад). Если бы я работал во враждебной среде (многие другие кодеры использовали много разных библиотек), я бы не стал доверять ни одному из них ...
Пабло Кабрера
Столбец «Связано» ---> очень удобно! Мне нравится ответ здесь: stackoverflow.com/questions/1744310/…
Гордон
Должен ли он быть обернут в каждый файл JS?
rd22
Кто конкретно MDC?
Ferrybig
141

В качестве альтернативы вы можете использовать функцию jQuery 1.2 inArray , которая должна работать в разных браузерах:

jQuery.inArray( value, array [, fromIndex ] )
Моисей Ли
источник
'IndexOf' - это собственный код (справа), поэтому будет ли jQuery inArray () работать так же быстро, как, например, использовать native, когда он доступен, и poly-fill, если нет?
Jeach
10
Итак, чтобы ответить на мой собственный комментарий (выше), я только что реализовал его, и в Chrome это так же быстро, как когда я использовал 'indexOf ()', но в IE 8 это очень, очень медленно ... так что, по крайней мере, мы знаем что inArray () использует native, когда это возможно.
Jeach
78

Полный код будет таким:

if (!Array.prototype.indexOf) {
    Array.prototype.indexOf = function(obj, start) {
         for (var i = (start || 0), j = this.length; i < j; i++) {
             if (this[i] === obj) { return i; }
         }
         return -1;
    }
}

Для действительно подробного ответа и кода на этот вопрос, а также на другие функции массива, проверьте вопрос переполнения стека Исправление функций массива JavaScript в Internet Explorer (indexOf, forEach и т . Д.) .

Луис Перес
источник
2
спасибо за то, что у тебя есть все. Я часто захожу на эту страницу всякий раз, когда мне нужен кроссплатформенный indexOf в новом проекте, и ваш фрагмент является единственным с полным кодом. :) Эти несколько секунд действительно складываются, когда вы часто посещаете эту страницу.
dylnmc
16

Библиотека underscore.js имеет функцию indexOf, которую вы можете использовать вместо:

_.indexOf([1, 2, 3], 2)
scotta7exander
источник
4
Этот ответ позволяет избежать путаницы с прототипом массива , и он делегируется нативному indexOf, когда он доступен. Мне это нравится.
Брэд Кох
Кажется, самый простой способ , если вы можете включить или подчеркивание lodash
ChrisRich
10

Вы должны проверить, если это не определено с помощью if (!Array.prototype.indexOf).

Кроме того, ваша реализация indexOfне является правильной. Вы должны использовать ===вместо ==в своем if (this[i] == obj)заявлении, иначе [4,"5"].indexOf(5)будет 1 в соответствии с вашей реализацией, что неверно.

Я рекомендую вам использовать реализацию на MDC .

Эли Грей
источник
9

Существует официальное решение Mozilla: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf

(function() {
    /**Array*/
    // Production steps of ECMA-262, Edition 5, 15.4.4.14
    // Reference: http://es5.github.io/#x15.4.4.14
    if (!Array.prototype.indexOf) {
        Array.prototype.indexOf = function(searchElement, fromIndex) {
            var k;
            // 1. Let O be the result of calling ToObject passing
            //    the this value as the argument.
            if (null === this || undefined === this) {
                throw new TypeError('"this" is null or not defined');
            }
            var O = Object(this);
            // 2. Let lenValue be the result of calling the Get
            //    internal method of O with the argument "length".
            // 3. Let len be ToUint32(lenValue).
            var len = O.length >>> 0;
            // 4. If len is 0, return -1.
            if (len === 0) {
                return -1;
            }
            // 5. If argument fromIndex was passed let n be
            //    ToInteger(fromIndex); else let n be 0.
            var n = +fromIndex || 0;
            if (Math.abs(n) === Infinity) {
                n = 0;
            }
            // 6. If n >= len, return -1.
            if (n >= len) {
                return -1;
            }
            // 7. If n >= 0, then Let k be n.
            // 8. Else, n<0, Let k be len - abs(n).
            //    If k is less than 0, then let k be 0.
            k = Math.max(n >= 0 ? n : len - Math.abs(n), 0);
            // 9. Repeat, while k < len
            while (k < len) {
                // a. Let Pk be ToString(k).
                //   This is implicit for LHS operands of the in operator
                // b. Let kPresent be the result of calling the
                //    HasProperty internal method of O with argument Pk.
                //   This step can be combined with c
                // c. If kPresent is true, then
                //    i.  Let elementK be the result of calling the Get
                //        internal method of O with the argument ToString(k).
                //   ii.  Let same be the result of applying the
                //        Strict Equality Comparison Algorithm to
                //        searchElement and elementK.
                //  iii.  If same is true, return k.
                if (k in O && O[k] === searchElement) {
                    return k;
                }
                k++;
            }
            return -1;
        };
    }
})();
Будет V Король
источник
1
Просто быть педантичным, но MDN - это не просто Mozilla. Это проект, управляемый сообществом, в котором работают как сотрудники Mozilla, так и волонтеры, каждый может присоединиться и внести свой вклад.
ste2425
5

Я бы порекомендовал это всем, кто ищет недостающую функциональность:

http://code.google.com/p/ddr-ecma5/

Это приносит большую часть недостающей функциональности ecma5 для старых браузеров :)

Джош Мак
источник
** Хотя я отмечу, что у меня были проблемы в IE7 с этой библиотекой.
Джош Мак
2

Это была моя реализация. По сути, добавьте это перед любыми другими скриптами на странице. то есть в вашем мастере для глобального решения для Internet Explorer 8. Я также добавил функцию обрезки, которая, кажется, используется во многих фреймворках.

<!--[if lte IE 8]>
<script>
    if (!Array.prototype.indexOf) {
        Array.prototype.indexOf = function(obj, start) {
            for (var i = (start || 0), j = this.length; i < j; i++) {
                if (this[i] === obj) {
                    return i;
                }
            }
            return -1;
        };
    }

    if(typeof String.prototype.trim !== 'function') {
        String.prototype.trim = function() {
            return this.replace(/^\s+|\s+$/g, '');
        };
    };
</script>
<![endif]-->
Glennweb
источник
2

меня устраивает.

if (!Array.prototype.indexOf) {
  Array.prototype.indexOf = function(elt /*, from*/) {
    var len = this.length >>> 0;

    var from = Number(arguments[1]) || 0;
    from = (from < 0)? Math.ceil(from) : Math.floor(from);
    if (from < 0)
    from += len;

    for (; from < len; from++) {
      if (from in this && this[from] === elt)
        return from;
    }
    return -1;
  };
}
Аллен Вонг
источник
1

С Underscore.js

var arr=['a','a1','b'] _.filter(arr, function(a){ return a.indexOf('a') > -1; })

sri_bb
источник