JavaScript - Regex-Code/Sonderzeichen/Zahlen usw.

Question

Antwort @Wiktor Stribiżew vorgeschlagen zu entfernen:

function myValidate(word) { 
    return (word.length === 1 || /[^A-Z]/i.test(word)) ? true : false; 
    } 

---

Hallo bei der Erstellung eines Arrays ich eine Funktion haben, die nicht Wörter mit bestimmten Zeichen erlaubt etc auf das Array

function myValidate(word) { 
     // No one letter words 
     if (word.length === 1) { 
      return true; 
     } 
     if (word.indexOf('^') > -1 || word.indexOf('$') > -1) { 
      return true; 
     } 
     return false; 
    } 

es mag nicht die richtige Art und Weise des Gehens über dieses und ive Blick in eine Regex scheint hinzugefügt werden, die damit umgehen würde, aber noch nicht erfolgreich gewesen Umsetzung versuchte zahlreiche Bemühungen wie:

if (word.match('/[^A-Za-z]+/g')) { 
      return true; 
     } 

kann jemand ein wenig Licht auf die richtige Art und Weise der Handhabung dieses?

Answer 1

Ich schlage vor, eine einfachere Lösung mit:

function myValidate(word) { 
 
    return (word.length === 1 || /[^A-Z]/i.test(word)) ? false : true; 
 
} 
 
var words = ["Fat", "Gnat", "x3-2741996", "1996", "user[50]", "definitions(edit)", "synopsis)"]; 
 
document.body.innerHTML = JSON.stringify(words.filter(x => myValidate(x)));

Wo:

• word.length === 1 Prüfungen für die Stringlänge
• /[^A-Z]/i.test(word) überprüft, ob ein Nicht-ASCII-Buchstaben-Symbol in der Zeichenfolge ist

Wenn eine der oben genannten Bedingung erfüllt ist, wird das Wort aus dem Array genommen . Der Rest bleibt.

Answer 2

EDIT: mit test statt match

Sie wollen test() verwenden, da es eine Bool zurückgibt Sie sagen, wenn Sie die Regex übereinstimmen oder nicht. Die match() gibt stattdessen immer die übereinstimmenden Elemente zurück. Diese können durch Zwang in true umgewandelt werden. Das ist nicht was du willst.

Um es allen zu Sie ihn nur Einzeiler verwenden können (keine if benötigt und keine entweder Zitate, kann jede einfacher nicht erhalten):

return word.test(/^[a-zA-Z][a-zA-Z]+$/); // two letter words 

---

Sie sollten Zeichen statt schwarze Listen weiße Liste. Das ist einer der Sicherheitsprinzipien. In Ihrem Fall nicht sagen, was falsch ist, aber sagen, was richtig ist:

if (word.test('/^[a-zA-Z]+$/')) { // two letter words 
    return false; 
} 

Diese für alle Wörter falsch zurück, die nur [a-zA-Z] Zeichen enthalten. Ich denke, das ist was du willst.

Ihre Regex hat stattdessen nach unzulässigen Zeichen gesucht, indem sie die Zeichengruppe mit der führenden ^ negiert.

---

Zwei Empfehlungen:

1. Gerade Regex in einer positiven Art und Weise verwenden (ohne Negation) und es wird viel einfacher zu verstehen sein.

2. Auch geben Validierungsfunktionen normalerweise true für gute Daten und false für schlechte Daten zurück.

Es ist besser lesbar auf diese Weise:

if (validate(data)) 
{ 
    // that's some good data we have here! 
}