Ie7 javascript issue

Question

I have following code its working one IE8, FF and other browser but is not working for IE7 on checking i found formatIncludes[i] is not working, any idea whats wrong in this with IE7 and any work around

var formatIncludes = valueToSet.replace(/[^\D]/g, '');
    for(var i=formatIncludes.length-1; i >= 0 ; i--){
        if(valueToSet && valueToSet != null && valueToSet.endsWith(formatIncludes[i])){
            valueToSet = valueToSet.substr(0, valueToSet.length - 1);
        }else{
            break;
        }
    }

String.prototype.endsWith = function(suffix) {
    return (this.indexOf(suffix, this.length - suffix.length) != -1);
};

formatIncludes can contain any string value

error on IE7

Message: 'length' is null or not an object
Line: 352
Char: 2
Code: 0
URI: http://localhost:7001/HHSPortal/framework/skeletons/hhsa/js/util.js

What's `endWith`, I think we need the code for that to be able to tell. What's the error on console? — elclanrs, Oct 29 '12 at 05:46
@MuthuKumaran No, but you can still do indexing like that on a string. The better solution is to use `.charAt(i)` instead. — Ian, Oct 29 '12 at 05:48
@MuthuKumaran: Strings are _array-like_, the index look-up should work. — elclanrs, Oct 29 '12 at 05:48
@ianpgall - it worked can u put it as an answer so that i can accept it — Varun, Oct 29 '12 at 05:49

score 3 · Accepted Answer · edited May 23 '17 at 12:27

3

It seems that indexing a string with [i] does not work? And the more consistent/correct solution is to use .charAt(i)

The reasoning can be found in these:

string.charAt(x) or string[x]?

JavaScript cross-browser: Is it safe to treat a string as array?

edited May 23 '17 at 12:27

Community

1
1

answered Oct 29 '12 at 05:50

Ian

50,146
13
101
111

any idea why `[i]` doesnt work on IE7 though it works on other browsers – Varun Oct 29 '12 at 05:51
2

It might just be another one of those wonderful standards that IE decides to implement... :) – Ian Oct 29 '12 at 05:52
@ianpgall or neglect in this case – Phil Oct 29 '12 at 05:52
@Phil Haha I meant a fictional standard they decided to create that says `[]` is not allowed with strings. But true – Ian Oct 29 '12 at 05:53
@Varun Nevermind, found some sources online for this problem, updated my answer – Ian Oct 29 '12 at 06:03
@ianpgall - they just changed the question to look as an answer :D – Varun Oct 29 '12 at 06:05

Ie7 javascript issue

1 Answers1