For loop skips values
243
January 12, 2018, at 05:03 AM
I have this program to recognize where an URL is within a message and leave in the string ONLY the URL, but it just skips values randomly and I din't know why. Here is my code and the debug console.
function DeleteNonURL(sep, URL)
{
var _Pos = new Number;
URL = URL.split(sep);
for (var i = 0; i < URL.length; i++)//deletes non-url substrings
{
Debug.Trace("Msg: "+URL + " i: "+i + " _Pos: "+_Pos + " Current Substring: "+URL[i]);
_Pos = URL[i].indexOf('.com');
if(_Pos == -1)
{
_Pos = URL[i].indexOf('.net');
if(_Pos == -1)
{
_Pos = URL[i].indexOf('.org');
if(_Pos == -1)
{
_Pos = URL[i].indexOf('.info');
if(_Pos == -1) //No URL in this substr
{
URL.splice(i, 1); //deleted.
}
}
}
}
}
Debug.Trace(URL);
}
Debug:
Msg: Hey,dude,look,at,this,amazon.com,what,is,it? i: 0 _Pos: 0 Current Substring: Hey
Msg: dude,look,at,this,amazon.com,what,is,it? i: 1 _Pos: -1 Current Substring: look
Msg: dude,at,this,amazon.com,what,is,it? i: 2 _Pos: -1 Current Substring: this
Msg: dude,at,amazon.com,what,is,it? i: 3 _Pos: -1 Current Substring: what
Msg: dude,at,amazon.com,is,it? i: 4 _Pos: -1 Current Substring: it?
dude,at,amazon.com,is
Answer 1
As noted in the comments, this is the very common problem of mutating the array that you're iterating when the mutation causes the array to be reindexed.
Ultimately, you seem to be wanting to check for substring that ends with one of the TLDs. This can be done much more simply.
const re = /\.(?:com|net|org|info)$/;
function test(sep, URL) {
return URL.split(sep).find(s => re.test(s));
}
const a = "http://example.com/foo/bar";
const b = "foo/bar/baz";
console.log("TLD part", test('/', a));
console.log("TLD part", test('/', b));
-
05:30
-
04:00
-
00:00
-
10:30
-
9:10
-
07:40
Firebase Cloud Functions: PubSub, "res.on is not a function"
-
04:00
TypeError: Cannot read properties of undefined (reading 'createMessageComponentCollector')
-
9:20
-
7:40
POPULAR ONLINE
- typescript: tsc is not recognized as an internal or external command, operable program or batch file 111624 visits
- search for elements in a list 111072 visits
- In Chrome 55, prevent showing Download button for HTML 5 video 98836 visits
- RxJS5 - error - TypeError: You provided an invalid object where a stream was expected 96341 visits
- Ionic 2 - how to make ion-button with icon and text on two lines? 92538 visits
- NetBeans IDE - ClassNotFoundException: net.ucanaccess.jdbc.UcanaccessDriver 62054 visits
- Adding methods to es6 child class 22381 visits
