[regex] cijfers EN letters

[Vinzzz243]

hmm, toch lastig te vinden, net de search gehad van got, maar nix...en ook nog een slotje, ook fijn...
wat is er mis?

Ik wil in javascript een controle uitvoeren op een password veld...en wel de volgende:
er MOET een letter EN een cijfer inzitten (en minimaal 5 karakters groot, maar das het probleem niet)
dus aaaaa mag niet
en 12345 ook niet
WEL een combinatie ervan, en natuurlijk geen andere karakters dan letters en cijfers

Zit vast, kan me iemand misschien helpen?

[ACM]

Je andere topic ging dicht met reden. Heb je geprobeerd die reden te begrijpen?

Dingen als: wat heb je zelf al geprobeerd, waar zit je vast etc...

[Vinzzz243]

ja beetje hulp is echt niet overbodig...

ben ik verplicht te zetten: heb alles gezocht, vanalles geprobeerd, kom er niet uit, kun je me aub helpen?

1 iemand met kleine tip, en ik ben geholpen...

maar goed...

[D2k]

een stukje geprobeerde code verwachten we van je
is wel veel hè?

[Grum]

Waarom moet dit met een regexp ??

[tomato]

Vinzzz: ben ik verplicht te zetten: heb alles gezocht, vanalles geprobeerd, kom er niet uit, kun je me aub helpen?

Hihi, nee, want dat zou ik toch niet geloven (zou je namelijk een eeuwigheid mee bezig geweest zijn, of je had het antwoord gevonden)

1 iemand met kleine tip, en ik ben geholpen...

Kijk naar character classes, quantifiers en alternation.

maar goed...

Euh... het is een programmeerforum...

[Vinzzz243]

niet dat je er veel aan hebt, maar daarom vroeg ik het ook hier

code:

if ( /^[0-9a-zA-Z]+$/.test( field.value )

[tomato]

Grum: Waarom moet dit met een regexp ??

Inderdaad, advies van mij geldt dus als je het met een regexp wilt doen.

Overigens is het makkelijker met 2 regexen en nog makkelijker zonder regexen (maar weer moeilijker als je veel meer leuke restricties gaat bedenken en ik denk dat je dat doet).

[Vinzzz243]

Op woensdag 24 april 2002 19:24 schreef Grum het volgende:
Waarom moet dit met een regexp ??

hoeft natuurlijk niet, wilde het al zo proberen, maar werd omslachtig en vroeg me af of het korter kon:

omslachtige methode = ieder karakter van wwveld controleren of dat binnen letters van alfabet valt of de 10 cijfers...

daarna, want alleen a's geeft ook goede result, kijken of een van 10 cijfers en 1 van de letters erin voorkomt

[Glimi]

Op woensdag 24 april 2002 19:42 schreef Vinzzz het volgende:

omslachtige methode = ieder karakter van wwveld controleren of dat binnen letters van alfabet valt of de 10 cijfers...

Mwoah, in een loopje moet dit kunnen

code:

psuedo:

boolean returnbool = false;
int nummers = 0;
int letters = 0;
if( lengte string < 5 ){
   returnbool = false;
}else{
   for( van 0 tot de lengte van de string ){
     if( char = een cijfer ){
         nummers++;
     }else if( char = een letter ){
         letters++;
     }
   }

   if( (nummers+letters) = lengte string && nummers != 0 && letters != 0){
     returnbool=true;
   }else{
     returnbool=false;
   }

   return returnbool;
}

Een regex zal ook gewoon alle tekens langs lopen vermoed ik.

[edit] idd de check op 0 vergeten.

[Vinzzz243]

edit:
ok, je oplossing is nu goed

[Soultaker]

Als regexp kan 'ie ook:

code:

^[[:alnum:]]*([[:alpha:]][[:alnum:]]*[[:digit:]]|[[:digit:]][[:alnum:]]*[[:alpha:]])[[:alnum:]]*$

De extra eis 'tenminste 5 letters' maakt 'm wel een heel stuk ingewikkelder.

[Vinzzz243]

Op woensdag 24 april 2002 20:44 schreef Soultaker het volgende:
Als regexp kan 'ie ook:

code:

^[[:alnum:]]*([[:alpha:]][[:alnum:]]*[[:digit:]]|[[:digit:]][[:alnum:]]*[[:alpha:]])[[:alnum:]]*$

De extra eis 'tenminste 5 letters' maakt 'm wel een heel stuk ingewikkelder.

die tenminste 5 voorwaarde, zou ik met javascript makkelijk kunnen doen...bedankt voor je reply, maar heb hem nu werkend onder de volgende code:

code:

function test2()
{
  var cijfers = 0;
  var letters = 0;
  var nummers = "0123456789";
  var alfabet = "abcdefghijklmnopqrstuvwxyz";
  for (var i=0;i<this.form1.ww.value.length;i++)
  {
    var chr=this.form1.ww.value.charAt(i)
    for (var j=0;j<alfabet.length;j++)
    {
    if (chr==alfabet.charAt(j)) letters++;
    }
    for (var j=0;j<nummers.length;j++)
    {
    if (chr==nummers.charAt(j)) cijfers++;
    }
  }
  if (!((letters+cijfers == this.form1.ww.value.length) && (cijfers != 0) && (letters != 0))) alert("fout");
}

[Glimi]

Heb je niet een methode isDigit() oid? Nu zit je in een dubbele for loop en doe je voor elk teken minstens 26 + 10 checks. Misschien zou je hem efficienter kunnen maken met een break als je een match heb (de match is immers exclusive)

Ook vind ik het nogal vies dat je geen gebruik maakt van variabelen die je meegeeft aan de functie maar alles global gaat aanroepen.

[Soultaker]

Ik kan het niet helemaal met Glimi oneens zijn. Zo belast je de processor van je websitebezoeker veel te veel!

Een wat efficientere methode (die ik voor de netheid heb opgesplitst in verschillende functies) zou deze zijn:

code:

function isNum(chr)
{
    return ((chr>='0')&&(chr<='9'));
}

function isAlpha(chr)
{
    return (((chr>='a')&&(char<='z'))||((chr>='A')&&(char<='Z')));
}

function isvalid(value)
{
    var alpha=false, num=false;
    for(var i=0;i<value.length;++i)
    {
        var chr=value.charAt(i);
        if(isNum(chr))
            if(alpha) return true; else num=true;
        else        
        if(isAlpha(chr))
            if(num) return true; else alpha=true;       
    }
    return false;
}

Overigens meer pseudo- dan javascript-code, want dat heb ik al jaren niet meer geschreven.

[Glimi]

Op woensdag 24 april 2002 22:32 schreef Soultaker het volgende:
Ik kan het niet helemaal met Glimi oneens zijn. Zo belast je de processor van je websitebezoeker veel te veel!

Maar je probeert het hopelijk wel

Een wat efficientere methode (die ik voor de netheid heb opgesplitst in verschillende functies) zou deze zijn:

code:

function isNum(chr)
{
    return ((chr>='0')&&(chr<='9'));
}

function isAlpha(chr)
{
    return (((chr>='a')&&(char<='z'))||((chr>='A')&&(char<='Z')));
}

function isvalid(value)
{
    var alpha=false, num=false;
    for(var i=0;i<value.length;++i)
    {
        var chr=value.charAt(i);
        if(isNum(chr))
            if(alpha) return true; else num=true;
        else        
        if(isAlpha(chr))
            if(num) return true; else alpha=true;       
    }
    return false;
}

Overigens meer pseudo- dan javascript-code, want dat heb ik al jaren niet meer geschreven.

Behalve dat je de >5 check vergeet (maar da's simpel te implementeren en daar zal niemand het over oneens zijn), ben ik het er niet mee eens dat je verschillende exitpoints in je code hebt. Naja, niet eens zijn is een groot woord, maar ik bedoel 'ut is niet netjes'.

Wel slim btw om te stoppen zodra je er één letter en één cijfer gevonden hebt, scheelt weer iig, maar daar check je niet of je reeks misschien uit 23a2~ of fg@21 bestaat ( dus niet letters of cijfers ). Het lijkt me dan ook de enige optie om alles langs te lopen, en te checken, maar niet op de manier die Vinzzz hierboven stelt iig

[Soultaker]

Op woensdag 24 april 2002 22:36 schreef Glimi het volgende:
Behalve dat je de >5 check vergeet

Oeps.

ben ik het er niet mee eens dat je verschillende exitpoints in je code hebt. Naja, niet eens zijn is een groot woord, maar ik bedoel 'ut is niet netjes'.

Ik ben van mening dat het de code overzichtelijk houdt, mits de functies waaruit gesprongen klein en overzichtelijk zijn. Pogingen een enkele exitpoint te forceren maken de code naar mijn ervaring vaak nodeloos complexer (en daardoor niet beter leesbaar).

Wel slim btw om te stoppen zodra je er één letter en één cijfer gevonden hebt, scheelt weer iig, maar daar check je niet of je reeks misschien uit 23a2~ of fg@21 bestaat ( dus niet letters of cijfers ).

Ik ben er niet helemaal bij vandaag.

Dan wil ik graag reviseren:

code:

function isvalid(value)
{
    if(value.length<5) return false;

    var alpha=false, num=false;
    for(var i=0;i<value.length;++i)
    {
        var chr=value.charAt(i);
        if(isAlpha(chr)) alpha=true;
        else if(isNum(chr) num=true;
        else return false;
    }
    return (alpha && num);
}

[Vinzzz243]

Op woensdag 24 april 2002 22:16 schreef Glimi het volgende:
Heb je niet een methode isDigit() oid? Nu zit je in een dubbele for loop en doe je voor elk teken minstens 26 + 10 checks. Misschien zou je hem efficienter kunnen maken met een break als je een match heb (de match is immers exclusive)

Ook vind ik het nogal vies dat je geen gebruik maakt van variabelen die je meegeeft aan de functie maar alles global gaat aanroepen.

hoezo een break, ik zal toch echt ieder karakter moeten controleren of t uberhaupt een letter of cijfer is? door op het einde te kijken of hij met variabele cijfers of letters NIETS gedaan heeft, weet ik toch dat er een van de 2 niet is ingevuld...

ben met je eens dat het sneller kan, ik ga verder programmeren

[Vinzzz243]

hmm net ff je code geprobeerd, zegt ie: verwacht object (bij de 3e if):

code:

function isvalid()
    {
      value = this.form1.ww.value
      if(value.length<5) return false;
      var alpha=false, num=false;
      for(var i=0;i<value.length;i++)
      {
        var chr=value.charAt(i);
        if (isAlpha(chr)) alpha=true;
        else 
        if (isNum(chr)) num=true;
        else return false;
      }
    return (alpha && num);
    }

[Guillome]

Waarom niet met een RegExp dan

code:

  function test(str)
  {
    if ( (str.match(/[A-Za-z]/i)) && (str.match(/[\d]/i)) && (str.length >= 5)) alert('s');
  }

[Vinzzz243]

Op donderdag 25 april 2002 10:54 schreef XLerator het volgende:
Waarom niet met een RegExp dan

code:

  function test(str)
  {
    if ( (str.match(/[A-Za-z]/i)) && (str.match(/[\d]/i)) && (str.length >= 5)) alert('s');
  }

is ook een mooie, alleen slikt ie ook special karakters

[Guillome]

oh moet dat ook?
Dan moet je "a-zA-Z" ff vervangen door "\D"
Zonder quotjes uiteraard
Dus dan heb je

code:

  function test(str)
  {
    if ( (str.match(/[\D]/i)) && (str.match(/[\d]/i)) && (str.length >= 5)) alert('s');
  }

Deze heb ik niet getest
Kijk hier eens

[Vinzzz243]

nee functioneel blijft ie hetzelfde, als ik @#abcde12 invul, slikt ie dat.
terwijl alleen letters en cijfers MOETEN

[Vinzzz243]

volgens mij is het deze, kan het? (na snel testen, klopt alles):

code:

function test()

 {
    str=this.form1.ww.value;
    if (!( ( /^[0-9a-zA-Z]+$/.test(str)) && (str.match(/[\D]/i)) && (str.match(/[\d]/i)) && (str.length >= 5))) alert('s');
  }

[Guillome]

euhm
Moeten die speciale dingen nou WEL of NIET mogen?
Zo ja, moet je de 2e hebben, zo niet, moet je de 1e hebben
Damn je hebt gelijk

[Vinzzz243]

Op donderdag 25 april 2002 11:14 schreef XLerator het volgende:
euhm
Moeten die speciale dingen nou WEL of NIET mogen?
Zo ja, moet je de 2e hebben, zo niet, moet je de 1e hebben

het mogen alleen letters en cijfers zijn...en er moet een combinatie zijn van letters en cijfers, maar kijk es naar mijn vorige oplossing, is dat hem?

[Guillome]

code:

function test(str)
  {
    if ( (!str.match(/\W/gi)) && (str.match(/[A-Za-z]/i)) && (str.match(/[\d]/i)) && (str.length >= 5)) alert('s');
  }

Die van jou kan ook wel, maar deze werkt ook

[drm]

meest flexibele oplossing is imo deze

Als je de regexen in verschillende if's zet, kun je gerichtere informatie geven over wat er mis is met het wachtwoord.

Processorload? Kom nou. We zitten toch niet meer met 1 MHz processors te werken, ofzo

[Vinzzz243]

Xlerator en drm allebei bedankt...

drm, jouw oplossing is het gaafst, thx!

[Guillome]

..
..
..

[jelmervos]

Op donderdag 25 april 2002 11:18 schreef drm het volgende:
meest flexibele oplossing is imo deze

[..]

http://gerard.yoursite.nl/got/Vinzzz/pass.php?pass=x

[Vinzzz243]

hmm idd, hoe los je zoiets op never mind, offtopic

[Glimi]

Op donderdag 25 april 2002 12:22 schreef Vinzzz het volgende:
hmm idd, hoe los je zoiets op never mind, offtopic

Dat doe je door je essentiele checks niet met JS te doen. Never trust the user

[drm]

jelmervos:
http://gerard.yoursite.nl/got/Vinzzz/pass.php?pass=x

da's natuurlijk weer een ander verhaal. Ik wil 'm in 't php-tje d'r ook wel even bijzetten, hoor. als je daar vrolijk van wordt ...

[Vinzzz243]

hehe neej hoeft niet..

kmoet zowiezo met asp gaan werken van school

[Vinzzz243]

ok toch nog klein vraagje...

werkt dit? (ik wil de postcode controleren)

code:

for (var i=0;i<4;i++)
  if (/[0-9]{1,1}/.test ( pc.value.charAt(i) )) alert('ok');
for (var i=4;i<6;i++)
  if (/[a-zA-Z]{1,1}/.test ( pc.value.charAt(i) )) alert('letterok');

edit:
heb van {0,0} {1,1} gemaakt, loopt nog niet helemaat vlekkeloos, maar wel beter

[edit2]BLOND, postcode for loop moest niet van 5 tot 7 , maar van 4 tot 6 lopen... [/edit2]

[drm]

Da's nou een typisch geval van een regex die je gewoon "als zodanig" * drm kunt doen:

code:

if ( /^[1-9][0-9]{3} ?[a-zA-Z]{2}$/.test ( postcode ) )
    |\___/\______/\/\_________/|
    |  |    |   |   |     |
    1  2    3   4   5     6

1. Begin string
2. Eerste getal moet tussen 1 en 9 zijn.
3. vervolgens 3 getallen tussen 0 en 9
4. Eventueel (is vraagteken) een spatie
5. vervolgens twee letters (kleine of hoofdletter mag beide)
6. Einde string

_edit:
* drm Ik bedoel dus zonder andere meuk als for enzo

[Vinzzz243]

eeje, hartstikke bedankt!!!

[Vinzzz243]

en die .test ?? (ben ik nog niet tegengekomen in enkele tutors...

[drm]

Vinzzz:
en die .test ?? (ben ik nog niet tegengekomen in enkele tutors...

de regex wordt "omgezet" in een RegExp object. test is een method van dat object.

In feite gebeurt er dus dit:

code:

tijdelijk = new RegExp ( /^[1-9][0-9]{3} ?[a-zA-Z]{2}$/ );

if ( tijdelijk.test ( postcode ) )

Dat bepaalde "waarden" omgezet worden in een default object voor dat type, wanneer er gepoogd wordt methodes van die waarde aan te roepen.

Hetzelfde idee is dit:

code:

alert ( "sjaak".lastIndexOf ( 'k' ) );

de string "sjaak" wordt omgezet in een String object, en daar wordt dan de methode lastIndexOf van aangeroepen.

dit verschijnsel heet autoboxing.

[Vinzzz243]

ok... en als ik nu de "." , "-" en "_" ook toesta voor een emailnaam (voor de @) wordt het dan:

code:

/^[a-zA-Z0-9]{1,}[.][-][_]$/.test ( value )

?

nee he? ff zoeken

[drm]

Vinzzz:
ok... en als ik nu de "." , "-" en "_" ook toesta voor een emailnaam (voor de @) wordt het dan:

code:

/^[a-zA-Z0-9]{1,}[.][-][_]$/.test ( vrijthof.value )

?

nee he? ff zoeken

Nee. 't wordt nu toch echt tijd dat je je een beetje in gaat lezen over hoe de regular expressions nou echt in elkaar zitten.
Zie de FAQ / Regex sites

[Vinzzz243]

hehe srry

code:

/^[a-zA-Z0-9\.\-\_]{1,}$/.test ( value )

[drm]

Vinzzz:
hehe srry

code:

/^[a-zA-Z0-9\.\-\_]{1,}$/.test ( value )

toch nog ff lezen
{1,} is hetzelfde als + in die context.
als je - aan het eind van je class zet hoef je 'm niet te escapen.
_ hoef je helemaal niet te escapen
. ook niet binnen een class.

wat is een class.

zoals ik al zei, toch nog ff lezen
ikzelf vond dit wel een aardige om mee te beginnen

[Vinzzz243]

kzie door de bomen het bos niet meer

code:

if (!( email.value.match(/\b(^(\S+@).+((\.nl)|(\.be)|(\.com)|(\.net)|(\.edu)|(\.mil)|(\.gov)|(\.org)|(\..{2,2}))$)\b/gi)))
  Fouten+="- E-mailadres onjuist ingevuld \n";

nu krijg ik het maar niet voor elkaar dat hij speciale tekens niet mag toestaan...probeerde al dit:

code:

if (!( email.value.match(/\b(^[a-zA-Z0-9.-_][\@].+((\.nl)|(\.be)|(\.com)|(\.net)|(\.edu)|(\.mil)|(\.gov)|(\.org)|(\..{2,2}))$)\b/gi)))
  Fouten+="- E-mailadres onjuist ingevuld \n";

ik heb nu zoveel veranderd, en ik meen dat ik 1 keertje zover had dat ik voor de "@" de restrictie goed had, maar na een poging om NA de "@" ditzelfde te bereiken, ging het fout...

ook niet na jouw , overigens duidelijke, url!

Lees je deze ook nog ff?

thx

[Vinzzz243]

ok opgelost :

code:

if (!( /^[A-Za-z0-9_-]+([.][A-Za-z0-9_-]+){0,4}[@][A-Za-z0-9_-]+([.][A-Za-z0-9_-]+){1,3}$/.test (email.value)))
 alert('onjuist');

[Grum]

-@-.- is geen valid email hoor
[e-mail] wel daarintegen

daarnaast is [@] ook nix .. gewoon @ gebruiken.

[edit]
http://gerard.yoursite.nl/got/Vinzzz/

werkt ook voor geen meter ..

12a34b56c78 zou een goede pass moeten zijn

[drm]

Grum:
[edit]
http://gerard.yoursite.nl/got/Vinzzz/

werkt ook voor geen meter ..

12a34b56c78 zou een goede pass moeten zijn

Als jij een beetje beter uit je doppen kijkt zie je dat je nu onzin blaat.

[Grum]

drm: Als jij een beetje beter uit je doppen kijkt zie je dat je nu onzin blaat.


Iek ik zie dat ik me letters en cijfers heb omgedraaid maar dan nog:

Code uit relevante link.

code:

     /^[a-zA-Z0-9]{8,}$/.test ( field.value )  // check minimaal 8 characters, alleen cijfers / letters
&&   /[a-zA-Z]{5,}/.test ( field.value )     // check minimaal 5 letters
&&   /[0-9]{2,}/.test ( field.value )       // check minimaal 2 cijfers

Eerste regex: klopt met wat erachter staat.
Tweede regex: 5 op een volgende letters ipv 5 letters.
Derde regex: 2 op een volgende cijfers ipv 2 cijfers

Volgens de regels zou ab1cd2ef3gh goed moeten zijn en dat is het dus niet.

[drm]

oopzie
excuus, je hebt gelijk...

stom

<sub>moet je maar niet van die ingewikkelde wachtwoorden willen

edit:</sub>
zo dan.