Comments on: JavaScript GUID Generator

By: Ron Fredericks

Ron Fredericks — Thu, 11 Feb 2010 22:55:24 +0000

I made a few more fixes…

Untitled Document

function S4() {
// add a seed to randome number generator, RDF
now = new Date();
seed = now.getSeconds();
// replace 0×10000 with this format: *parseInt(’10000′,16); RDF
// replace substring(1) with a 4 digit hex string like this: substring(1,5); RDF
return ((1+Math.random(seed))*parseInt(’10000′,16)).toString(16).substring(1,5);
}
function generateGuid(){
return (S4()+S4()+”-”+S4()+”-”+S4()+”-”+S4()+”-”+S4()+S4()+S4()).toUpperCase();
}

// test the GUID generator
document.writeln(generateGuid());

By: Ron Fredericks

Ron Fredericks — Thu, 11 Feb 2010 22:52:49 +0000

Oops, I have two more fixes to make the GUID a little more usable and a little more random…

Untitled Document

// test the GUID generator
document.writeln(generateGuid());

By: Ton Fredericks

Ton Fredericks — Thu, 11 Feb 2010 22:23:08 +0000

The term 0×10000 in function s4() seems to be the source of error in my use of the proposed javascript GUID creator. Here is my proposed solution:

Untitled Document

function S4() {
// replace 0×10000 with this format: *parseInt(’10000′,16); RDF
return ((1+Math.random())*parseInt(’10000′,16)).toString(16).substring(1);
}
function generateGuid(){
return (S4()+S4()+”-”+S4()+”-”+S4()+”-”+S4()+”-”+S4()+S4()+S4()).toUpperCase();
}

// test the GUID generator
alert(generateGuid());