package javascript

[achraf.b]

new javascript package to validate data

# hasValidChars(string, characters, caseSensitive)
# isIP(ip)
# isAlpha(string)
# isLetter(string)
# isNotEmpty(string)
# isIntegerInRange(number,min,max)
# isNum(number)
# isEmail(string)
# isZipCode(number,country)
# isDate(date,format)
# isMD5(string)
# isURL(string)
# isGuid(string)
# isISBN(string)
# isSSN(number)
# isDecimal(number)

[achraf.b]

you can found it here :
http://www.mutationevent.com/projects/validate.js/

[Thomas 'PointedEars' Lahn]

ac******@gmail.com wrote:

new javascript package to validate data
- uses prototype.js nonsense (unnecessarily, of course)
# hasValidChars(string, characters, caseSensitive)
| [...]
| hasValidChars:function(string, characters, caseSensitive){
| if(!string || !string.length) return false;
| if(!characters || !characters.length) return false;

One could argue that the empty string always meets the requirement.

| if(!caseSensitive){
| string = string.toLowerCase();
| characters = characters.toLowerCase();
| }
|
| var cArr = new Array();
| cArr = string.split("");
| var len = cArr.length;
| for(var i=0; i<len; i++){
| if(characters.indexOf(cArr[i]) == -1){
| return false;
| }
| }
| return true;
| }

- Unnecessarily inefficient and error-prone, in contrast to

function hasValidChars(s, characters, caseSensitive)
{
function escapeSpecials(s)
{
// quick hack
return s.replace(new RegExp("([\\\\-])", "g"), "\\$1");
}

return new RegExp(
"^[" + escapeSpecials(characters) + "]+$",
(!caseSensitive ? "i" : "")).test(s);
}
# isIP(ip)
| isIP:function (ip){
| if(!this.hasValidChars(ip,"0123456789.")){
| return false;
| }
|
| var parts = ip.split(".");
| if(parts.length!=4) return false;
| if (Number(parts[0]) == 0) {
| return false;
| }
| var len = parts.length;
| var stri;
| var numi;
| for (var i=0; i<len; i++) {

for (var i = len; i--;) {

| stri = parts[i];
| numi = parseInt(stri);

The base is missing.

| if (numi > 255 || stri != numi.toString()) {
| return false;
| }
| }
| return true;
| }

- IP is the Internet Protocol, not the Internet Protocol address.
- It does not recognize or misrecognizes invalid IPv4 addresses;
"0.017.377.0" is read as "0.15.255.0" and not considered invalid
by this method.
- It does not recognize IPv6 addresses.
# isAlpha(string)
| isAlpha:function(string){
| return this.hasValidChars(string,"0123456789abcdefghijklm nopqrstuvwxyz");
| }

- It ignores that alphabets are not restricted to the Latin alphabet, i.e.
it does not support Unicode alphabetic characters (although they are
well-defined and supported). Since about one fifth of this planet's
population are born Chinese (and the population is growing and markets
are emerging especially there), that is bordering to suicide, or rather
seppuku in that case ;-)

Besides, you were looking for

function isAlphaLatin(string)
{
return /^[0-9a-z]+$/i.test(string);
}
# isLetter(string)
| isLetter:function(string){
| },

(sic!)

I see.
# isNotEmpty(string)
| isNotEmpty:function(str,allowWhite){
| if(!allowWhite){
| return parseInt(str + "1") != 1 || parseInt(str) == 1;
| }
| return (str.length > 0);
| },

You only /post/ to cljs, right? We have discussed this shortly ago.
The current approach

function isEmpty(s)
{
return !/\S/.test(s);
}

can be modified easily to

function isNotEmpty(s)
{
return /\S/.test(s);
}

and then to

function isNotEmpty(s, bAllowWhitespace)
{
// inner parentheses only for better legibility
return (bAllowWhiteSpace
? (/\S/.test(s))
: (!/^$/.test(s)));
}
# isIntegerInRange(number,min,max)
| isIntegerInRange:function(n,Nmin,Nmax){
| var num = Number(n);

Undeclared identifier, becoming a global property. Bad style.

| if(isNaN(num)){
| return false;
| }
| if(num != Math.round(num)){
| return false;
| }
| return (num >= Nmin && num <= Nmax);
| },

- It does not allow for open intervals.
# isNum(number)
| isNum:function(number){
| numRegExp = /^[0-9]+$/

- Unnecessary undeclared identifier, becoming a global property.
Bad style at the worst.

| return numRegExp.test(number);
| }

So you know about Regular Expressions, after all? However:

- It does not handle non-integer values.
- It does not handle non-decimal values.
- It does not handle strings that can be considered not a number
(e.g. "000200").
# isEmail(string)
| isEmail:function(string){

- Correct would have been `isEMailAddr'.

| if(!string) return;
| if (string.search(/^\w+((-\w+)|(\.\w+))*\@[A-Za-z0-9]+((\.|-)
| [A-Za-z0-9]+)*\.[A-Za-z0-9]+$/) != -1) return true;
| else return false;
| }

That's really a big laugh.

- Obviously you have never read STD11 or RFC2822.
- It does not handle IDNs.
# isZipCode(number,country)
| isZipCode:function(zipcode,country){
| if(!country) format = 'US';

- See above. (`country' or `format'?)

| switch(country){
| case'US': zpcRegExp = /^\d{5}$|^\d{5}-\d{4}$/; break;
| case'MA': zpcRegExp = /^\d{5}$/; break;
| case'CA': zpcRegExp = /^[A-Z]\d[A-Z] \d[A-Z]\d$/; break;
| case'DU': zpcRegExp = /^[1-9][0-9]{3}\s?[a-zA-Z]{2}$/; break;
| case'FR': zpcRegExp = /^\d{5}$/; break;
| case'Monaco':zpcRegExp = /^(MC-)\d{5}$/; break;
| }
| return zpcRegExp.test(zipcode);
| }

- It does not handle abbreviations of country names as prefixes but
for Monaco.
- Why you did not include support for ZIP codes of German-speaking
countries, the homelands of regular postal service in Europe,
remains a mystery.
# isDate(date,format)
| isDate:function(date,format){
| if(!format) format = 'FR';
| switch(format){
| case'FR': RegExpformat =
| /^(([0-2]\d|[3][0-1])\/([0]\d|[1][0-2])\/([2][0]|[1][9])\d{2})$/;
| break;
| case'US': RegExpformat =
| /^([2][0]|[1][9])\d{2}\-([0]\d|[1][0-2])\-([0-2]\d|[3][0-1])$/;
| break;
| case'SHORTFR': RegExpformat =
| /^([0-2]\d|[3][0-1])\/([0]\d|[1][0-2])\/\d{2}$/;
| break;
| case'SHORTUS': RegExpformat =
| /^\d{2}\-([0]\d|[1][0-2])\-([0-2]\d|[3][0-1])$/;
| break;
| case'dd MMM yyyy':RegExpformat =
| /^([0-2]\d|[3][0-1])\s(Jan(vier)?|Fév(rier)?|Mars|Avr(il)?|Mai|Juin
| |Juil(let)?|Aout|Septembre|Octobre|November|Decemb re)\s
| ([2][0]|[1][19])\d{2}$/; break;
| case'MMM dd, yyyy':RegExpformat =
| /^(J(anuary|u(ne|ly))|February|Ma(rch|y)|A(pril|ugu st)
| |(((Sept|Nov|Dec)em)|Octo)ber)\s([0-2]\d|[3][0-1])\,
| \s([2][0]|[1][9])\d{2}$/; break;
| }
| return RegExpformat.test(date);
| }

I can hardly wait for Dr Stockton's comment on this.
# isMD5(string)
| isMD5:function(string){
| if(!string) return false;
| md5RegExp = /^[a-f0-9]{32}$/;

- See above.

| return md5RegExp.test(string);
| }

I don't see the point, i.e. I doubt the usefulness of this method.
# isURL(string)
| isURL:function(string){
| string = string.toLowerCase();
| urlRegExp = /^http\:\/\/|https:\/\/[a-zA-Z0-9\-\.]+\.[a-zA-Z]{2,3
^^^^^^^^^ ^^^^^^^^^^^^^^^^^
- See above.

| (\/\S*)?$/
| return urlRegExp.test(string);
| }

- Obviously you have never read RFC3986.
- Because of the unparathesed alternation, it does not handle
correct "http:"-scheme URLs.
# isGuid(string)
| isGuid:function(guid){
| //guid format : xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx or
| xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
| GuidRegExp = /^[{|\(]?[0-9a-fA-F]{8}[-]?([0-9a-fA-F]{4}[-]?){3}
| [0-9a-fA-F]{12}[\)|}]?$/

- See above.

| return GuidRegExp.test(guid);
| },
# isISBN(string)
| isISBN:function(number){
| ISBNRegExp =
| /ISBN\x20(?=.{13}$)\d{1,5}([- ])\d{1,7}\1\d{1,6}\1(\d|X)$/

- See above.

| return ISBNRegExp.test(number);
| },
# isSSN(number)
| isSSN:function(number){
| //Social Security Number format : NNN-NN-NNNN
| ssnRegExp = /^\d{3}-\d{2}-\d{4}$/
| return ssnRegExp.test(number);
| }

Well, my valid SSN does not match.
# isDecimal(number)

| isDecimal:function(number){
| // positive or negative decimal
| decimalRegExp = /^-?\d*(\.\d+)?$/
| return decimalRegExp.test(number);
| }

- "-.0" really is definitely not a decimal number. Trust me.

And I really wonder why you did not use RegExp all the way, but insisted on
more inefficient approaches. This strikes me as just being copypasted from
others (and "forgotten" to mention the source) without having a minimum
clue yourself.

- Providing _only_ a RAR-compressed file on the Web is bordering to suicide.

That are enough minuses for me not only not to recommend it, but to
recommend against it.

Thanks for a long big laugh anyway.
PointedEars
--
#define QUESTION ((bb) || !(bb))
// William Shakespeare (if he would have been a hacker ;-))

[Dr John Stockton]

JRS: In article <11**********************@i39g2000cwa.googlegroups .com>
, dated Mon, 24 Apr 2006 06:49:54 remote, seen in
news:comp.lang.javascript, ac******@gmail.com posted :

new javascript package to validate data

# hasValidChars(string, characters, caseSensitive)
# isIP(ip)
# isAlpha(string)
# isLetter(string)
# isNotEmpty(string)
# isIntegerInRange(number,min,max)
# isNum(number)
# isEmail(string)
# isZipCode(number,country)
# isDate(date,format)
# isMD5(string)
# isURL(string)
# isGuid(string)
# isISBN(string)
# isSSN(number)
# isDecimal(number)

Post it here, in sections of reasonable size, and it will be publicly
evaluated. Or does that thought frighten you?

How about starting with isLetter?

--
© John Stockton, Surrey, UK. ?@merlyn.demon.co.uk Turnpike v4.00 IE 4 ©
<URL:http://www.jibbering.com/faq/> JL/RC: FAQ of news:comp.lang.javascript
<URL:http://www.merlyn.demon.co.uk/js-index.htm> jscr maths, dates, sources.
<URL:http://www.merlyn.demon.co.uk/> TP/BP/Delphi/jscr/&c, FAQ items, links.

[McKirahan]

"Thomas 'PointedEars' Lahn" <Po*********@web.de> wrote in message
news:15****************@PointedEars.de...

[snip]

- It ignores that alphabets are not restricted to the Latin alphabet, i.e.
it does not support Unicode alphabetic characters (although they are
well-defined and supported). Since about one fifth of this planet's
population are born Chinese (and the population is growing and markets
are emerging especially there), that is bordering to suicide, or rather
seppuku in that case ;-)

[snip]

Seppuku is ritual Japanese suicide, not Chinese. :)

[Thomas 'PointedEars' Lahn]

McKirahan wrote:

"Thomas 'PointedEars' Lahn" [...] wrote [...]

- It ignores that alphabets are not restricted to the Latin alphabet,
i.e. it does not support Unicode alphabetic characters (although they
are well-defined and supported). Since about one fifth of this
planet's population are born Chinese (and the population is growing
and markets are emerging especially there), that is bordering to
suicide, or rather seppuku in that case ;-)

[snip]

Seppuku is ritual Japanese suicide, not Chinese. :)

Yes, I knew, but the joke wouldn't have worked then :) (Hence "rather".)
F'up2 poster

PointedEars
--
Germany is the birth place of psychiatry. Psychs feel threatened by
Scientology as they are crazy. Many psychs become psychs to heal their own
mental problems and to control others. -- "The only real Barbara Schwarz",
dsw.scientology, <16**************************@posting.google.com >