Comments on: JavaScript natural order string comparison http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/ Weblog of Kristof Coomans Tue, 05 Oct 2010 12:08:57 +0000 hourly 1 http://wordpress.org/?v=3.0.5 By: sasha http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/comment-page-1/#comment-11731 sasha Thu, 01 Mar 2007 17:51:10 +0000 http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/#comment-11731 You could add something like this: var cm = ca.localeCompare(cb) ; if (cm) return cm ; Rather then this if (ca cb) { return +1; } To compare strings based on system locale Additionally if you want case-insensitive comparisson you should not do it in the function itself, but rather send it lower-case arguments like this natcompare (arg1.toLowerCase() , arg2.toLowerCase()) or with locale support: natcompare (arg1.toLocaleLowerCase() , arg2.toLocaleLowerCase()) You could add something like this:
var cm = ca.localeCompare(cb) ;
if (cm) return cm ;
Rather then this

if (ca cb) {
return +1;
}
To compare strings based on system locale

Additionally if you want case-insensitive comparisson you should not do it in the function itself, but rather send it lower-case arguments like this

natcompare (arg1.toLowerCase() , arg2.toLowerCase())

or with locale support:

natcompare (arg1.toLocaleLowerCase() , arg2.toLocaleLowerCase())

]]> By: Braden http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/comment-page-1/#comment-3062 Braden Tue, 12 Dec 2006 19:24:10 +0000 http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/#comment-3062 Thank you. This script is great. I am adding a case-insensitive option for my own use. If you want to add it into the trunk, I just added this block, which may not be the most efficient way: <code> var ia = 0, ib = 0; var nza = 0, nzb = 0; var ca, cb; var result; + if (i) { + a = a.toLowerCase(); + b = b.toLowerCase(); + } + while (true) { </code> Thank you. This script is great. I am adding a case-insensitive option for my own use. If you want to add it into the trunk, I just added this block, which may not be the most efficient way:

var ia = 0, ib = 0;
var nza = 0, nzb = 0;
var ca, cb;
var result;

+ if (i) {
+ a = a.toLowerCase();
+ b = b.toLowerCase();
+ }
+
while (true)
{

]]> By: Kristof http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/comment-page-1/#comment-1118 Kristof Tue, 19 Sep 2006 06:08:12 +0000 http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/#comment-1118 Hello Ben I mailed Martin twice but he does not seem to be very responsive :-) Tried again with another address today, I hope the mail reaches his destination now. Hello Ben

I mailed Martin twice but he does not seem to be very responsive :-)

Tried again with another address today, I hope the mail reaches his destination now.

]]> By: Ben Reser http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/comment-page-1/#comment-1116 Ben Reser Tue, 19 Sep 2006 02:00:11 +0000 http://blog.coomanskristof.be/2006/08/01/javascript-natural-order-string-comparison/#comment-1116 You might want to get Martin Pool to update the script on the sourceforge site. I bumped into the but and debugged and fixed it myself. I was then looking for where I found it and bumped into your blog and found your already fixed version. But the one on the sourceforge page is still broken. You might want to get Martin Pool to update the script on the sourceforge site. I bumped into the but and debugged and fixed it myself. I was then looking for where I found it and bumped into your blog and found your already fixed version. But the one on the sourceforge page is still broken.

]]>