I have a string being grabbed from a page in the format "4m 26s", how can I strip this into just seconds?
Many thanks,
+2
A:
Simple regex will work:
var s = '21m 06s';
var m = /(\d{1,2})m\s(\d{1,2})s/.exec(s);
var mins = parseInt(m[1], 10);
var secs = parseInt(m[2], 10);
Evan Trimboli
2010-06-28 10:31:35
-1 regexp is unnecessary
galambalazs
2010-06-29 11:49:18
This is a silly comment; you could technically say regexes are always unnecessary. It solves the problem as requested. I could downvote your answer saying that using split is unnecessary.
Evan Trimboli
2010-06-29 13:09:49
no, here regexp is slow, harder to read and harder to maintain, thus overcomplicates the problem. you should use regexp when you have a good reason for it. you could've edited your code but you're rather arguing with me.
galambalazs
2010-06-29 16:23:28
Slow? Care to back that up with some numbers? I think you'll find there's little to no difference in terms of performance.Harder to read and harder to maintain are subjective, especially since it's a small and easily readable regex. Just because you aren't comfortable with it, doesn't make it complicated.Again, stupid comment.
Evan Trimboli
2010-06-29 16:52:42
why is it so hard to admit there is a better way? happens to everyone... *"it's easy to make things bigger, it's hard to make things better..."*
galambalazs
2010-06-29 17:17:21
I like how you ignored everything I said. Nice one ;)
Evan Trimboli
2010-06-30 01:39:22
A:
var str = "4m 26s";
console.log(str.match(/\d+m\s+(\d+)s/)[1]);//26
Amarghosh
2010-06-28 10:32:33
+2
A:
A non-regex way:
Do a string.split(" ") on your string; then do string.slice(0, -1) on both arrays. Multiply the first entry by 60. Add them together.
Jan Kuboschek
2010-06-28 10:32:55
+2
A:
var str = "4m 26s";
var arr = str.split(" ");
var sec = parseInt(arr[0], 10)*60 + parseInt(arr[1], 10);
You don't need regex if you use parseInt...
galambalazs
2010-06-28 12:43:02