Új hozzászólás Aktív témák

• #17686 dabadab titán bLaCkDoGoNe #17684

  Új Válasz 2022-09-02 17:15:03 #17686

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  dabadab

  titán

  válasz bLaCkDoGoNe #17684 üzenetére

  Egy kicsit beleszerelmesedtél a zárójelekbe

  Az elején a ^ az mehet a legelejére, mivel mindenképpen a string elejétől akarod checkelni.

  Az nem volt világos, hogy az elején az opcionális whitespace csak az egyes után lehet-e vagy mindenképpen: ha az előbbi (és szerintem erről van szó), akkor az a rész inkább úgy nézzen ki, hogy

  ^(1\s?)?...

  Ez így már megjavít egy csomó esetet.

  Ha már mindenhol máshol használod a \d-t, akkor itt is, szóval:

  ^(1\s?)?(\(\d{3}\)|\d{3})...

  A (\s|[-])? az feleslegesen barokkos, a sima [\s-]? pont ugyanazt csinálja

  Mivel mindenképpen tesztelni akarod a komplett stringet, a végére menjen egy $.

  És ezzel működőre is kalapáltuk:

  ^(1\s?)?(\(\d{3}\)|\d{3})[-\s]?\d{3}[-\s]?\d{4}$

• #17685 emvy félisten bLaCkDoGoNe #17684

  Új Válasz 2022-09-02 17:06:55 #17685

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  emvy

  félisten

  válasz bLaCkDoGoNe #17684 üzenetére

  /^1?\ ?(\(\d{3}\)|\d{3})[\ -]?\d{3}[\ -]?\d{4}$/

  1. feleslegesen hasznalsz capturing groupokat (zarojeleket)
  2. nem ellenorzod, hogy vege-e a sztringnek ($ a vegen)
  3. word boundary a vegen minek? (\b) - ezert nem matchelsz a 5555555555-re
  4. amikor az elso tripletre akarsz matchelni (a tiedben ez a resz: ((\([0-9]{3}\))*)|\d{3}, az 0 karakterre is matchel, ezert matchelsz a 555-5555-re, pedig nem kene (nezd meg az enyemet, az matchel a (123)-ra, vagy a 123-ra is, de a semmire nem matchel).

Új hozzászólás Aktív témák