search

datasektionen / signup-fu

1 stars 1 forks source link

dubbelnegationsvalideringar #10

Closed Frost closed 12 years ago

Frost commented 12 years ago

Valideringarna på User#name och Event#slug är inte helt uppenbara vad de innebär.

validates_format_of :slug, :without => /[^a-z0-9\-].*/

Det jag tror att det där ska betyda är i stort sett "validera att :slug inte innehåller något som inte är gemener, siffror eller bindestreck", vilket tog mig, Max och @linduxed ett par minuter att komma fram till när vi satt och tittade på det igår.

Jag tycker inte om när kod inte är uppenbar, och dubbelnegationer gör att jag måste tänka på vad som faktiskt står.

Den där valideringen skulle lika gärna kunna formuleras som "validera att :slug bara tillåt bara gemener, siffror och bindestreck", vilket kan skrivas såhär istället:

validates_format_of :slug, :with => /^[a-z0-9\-]+$/

Min personliga åsikt är att detta är mycket enklare att läsa.

Dessa valideringar finns här:

@linduxed fixar den ena i #9.

Koronen commented 12 years ago

...och jag fixade den andra i #12.