liten fråga kring regular expressions

Ursprungligen inskrivet av matt76
Har en kontroll som ser över att epost inte innehåller några otillåtna tecken, dvs att enbar bokstäver, siffror, punkt och @ ska finnas med , men med följande

if ($this->content == "email" && preg_match('/[^a-zA-Z0-9.@]/', trim($this->value)))
{
$this->errorMessage = $this->name." contains illegal characters.";
}

så händer ingenting, dvs den påpekar att ex eposten test@test.se innehåller illegala tecken. Så vad är det för miss jag gjort? (testade även att skriva \. och \@ men blir samma sak.)

Är troligen något enkelt, men i denna tidiga morgon/sena natt så ser jag tredubbelt och missar det uppenbara

EDIT: Glöm mitt inlägg, var inget fel på uttrycket, enbart min klass

Ursprungligen inskrivet av KurreKula
Det finns mer officiella regex för email:

[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+(?:[A-Z]{2}|com|org|net|gov|mil|biz|info|mobi|name|aero|jobs|museum)\b

Ursprungligen inskrivet av muppens
Som dock inte är bra för då fungerar inte .se adresser.

Ursprungligen inskrivet av muppens
Som dock inte är bra för då fungerar inte .se adresser.

Ursprungligen inskrivet av pscs3
Utan att kika allt för noga så är den uppdelat så att parentesen innan '\b' söker topdomäner, delen [A-Z]{2} tordes vara för att representera alla två-bokstavs-domäner, ex. .se, .eu, .cz etc.

Ursprungligen inskrivet av badboll
Vilket också borde innebära att ogiltiga domäner som .xx, .zf, .vw och liknande passerar, vilket gör uttrycket värdelöst.

Korrekt validering av epostadresser är ett (mycket!) svårt problem, och på det hela taget meningslöst. Det finns ju ingenting som hindrar en användare att ange en korrekt formaterad men icke-existerande epostadress... och vad har valideringen då tjänat till?

Det finns bättre tekniker för att säkerställa en giltig epostadress, om en sådan nu verkligen krävs.

Ursprungligen inskrivet av matt76
Jo har en mindre avancerad sådan också, men istället för

[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)

före @ borde det väl räcka med att köra [^a-z0-9._], dvs de grundläggande tecknen som finns med i en mejladress

Felet i min if-sats var ju annars att de inte funkade att köra $this->content == "email" ihop med preg_match i samma, men var väl kanske tur i oturen då klassen blev bättre optimerad nu med mindre skräpkod.

Ursprungligen inskrivet av badboll
[...]

Ursprungligen inskrivet av KurreKula
Det blir ju inte värdelöst. Det är bättre att fånga upp så mycket som möjligt så tidigt som möjligt. Som du säger kan man ju ange en giltig men icke existerande epostadress. Men det gör inte uttrycket meningslöst. Nånting blir ju inte meningslöst för att det går att gå runt, då är den mesta säkerheten meingslös. Även om denna är rätt så lätt att gå runt är den ju verkligen inte svår att implementera

Ursprungligen inskrivet av badboll
Jo, det är värdelöst.

1) Syftet med den föreslagna valideringen är att fånga upp ogiltiga epostadresser och underkänna dessa.

2) Valideringen misslyckas med ovanstående.

3) Istället är risken överhängande att uttrycket underkänner epostadresser som faktiskt är giltiga. Det blir inte drabbade användare glada av!

1 + 2 + 3 = valideringen fyller inte sitt syfte. Släng lösningen i papperskorgen och hitta en annan, eller släpp valideringen helt om korrekta epostadresser inte är så viktigt trots allt.

Ursprungligen inskrivet av Leedow
Vad har du för regexp som säger att bajs@bajs.com inte är en valid epostadress?

Om en människa skriver "skit" i fälten så kan du aldrig validera om det är korrekt eller inte.