Validacija

[knee-cola]

Neki dan sam napravio Pull Request za tvoj generator bar koda.

Sada sam na temelju tvoje stranice izradio generator opće uplanice, koji koristi tvoj lib za generiranje bar koda, te pomoću njega generira kompletnu uplatnicu: https://github.com/knee-cola/generator-opce-uplatnice

Zašto sam to radio? Zato jer mi stalno stižu uplatnice bez bar koda, a kako sve plaćam na kioscima Tiska, prepravljmanje uplatnica je pain in the butt.

Zašto ti sve ovo pišem? Zato jer nisam uspio skužiti kako radi tvoja validacija. Naime validacija uredno prolazi ako su polja prazna. Naime validacijski blok izgleda ovako:

if (!StringNotDefinedOrEmpty(paymentParams.ImePlatitelja) && fieldLength == -1) {
	result |= BarcodePayment.ValidationResult.PayerNameInvalid;
}

Kada je polje prazno funkcija StringNotDefinedOrEmpty će vratiti true, što znači da neće biti izvršen if blok. Ako je polje pak puno funkcija će vratiti false, no fieldLength će biti veći od nule, što znači da if blok ponovo neće biti izvršen.

Ja sam u kopiji tvojeg source-a zakrpa (ako se to može tako zvat), pa blok sada glasi ovako:

if (StringNotDefinedOrEmpty(paymentParams.ImePlatitelja) || fieldLength == -1) {
	result |= BarcodePayment.ValidationResult.PayerNameInvalid;
}

No ne razumijem kako je stvar originalno zamišljena, pa mi je teško reći da li ovo što sam ja napravio ima nekog smisla. Možeš li objasniti u čemu je trik? Thanks!

[Bikonja]

Pozdrav! Tako nekako je izgledala i moja motivacija za taj kod :) Proslo je puno vremena pa se vise ne sjecam, ali gledanjem u kod mogu reci da: A) uzasno izgleda kod :) B) izgleda mi kao da je ideja da polje smije biti prazno, ali ako nije prazno onda provjera da odgovara uvjetima (nema nedozvoljenih znakova sto se provjerava sa fieldLength) i ostalo po polju (regex ili sto vec)... Na mobitelu gledam pa nisam 100% siguran, ali tako mi izgleda. Jel ima smisla? Lijepi pozdrav, Igor Loborec

…

On Fri, 28 Sep 2018, 21:16 knee-cola, ***@***.***> wrote: Neki dan sam napravio Pull Request za tvoj generator bar koda. Sada sam na temelju tvoje stranice izradio generator opće uplanice, koji koristi tvoj lib za generiranje bar koda, te pomoću njega generira kompletnu uplatnicu: https://github.com/knee-cola/generator-opce-uplatnice Zašto sam to radio? Zato jer mi stalno stižu uplatnice bez bar koda, a kako sve plaćam na kioscima Tiska, prepravljmanje uplatnica je pain in the butt. Zašto ti sve ovo pišem? Zato jer nisam uspio skužiti kako radi tvoja validacija. Naime validacija uredno prolazi ako su polja prazna. Naime validacijski blok izgleda ovako: if (!StringNotDefinedOrEmpty(paymentParams.ImePlatitelja) && fieldLength == -1) { result |= BarcodePayment.ValidationResult.PayerNameInvalid; } Kada je polje prazno funkcija StringNotDefinedOrEmpty će vratiti true, što znači da neće biti izvršen if blok. Ako je polje pak puno funkcija će vratiti false, no fieldLength će biti veći od nule, što znači da if blok ponovo neće biti izvršen. Ja sam u kopiji tvojeg source-a *zakrpa* (ako se to može tako zvat), pa blok sada glasi ovako: if (StringNotDefinedOrEmpty(paymentParams.ImePlatitelja) || fieldLength == -1) { result |= BarcodePayment.ValidationResult.PayerNameInvalid; } No ne razumijem kako je stvar originalno zamišljena, pa mi je teško reći da li ovo što sam ja napravio ima nekog smisla. Možeš li objasniti u čemu je trik? Thanks! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHZOGwkJk-tJ1vJX1DLjgZYg-WZxVTogks5ufoObgaJpZM4W_oht> .

[knee-cola]

Hm ... čini mi se da if uvijet možemo opisati ovako (pseudokod):

AKO polje nije prazano
I AKO polje ne sadrži nedopuštene znakove
> polje nije ispravno

Validacija se dakle ne bavi time je li neko polje popunjeno ili ne, već da li sadrži nedopuštene znakove.

Promatrajući funkciju koja određuje sadrži li polje nedopuštene znakove vidim da i ona prvo provjerava je li polje prazno, tako da je suvišno to još jednom provjeravati. Dovoljno je dakle provjeriti je li fieldLength == -1.

Unatoč tome što se ista stvar 2x provjerava, ne vidim da je to neka greška.

Jedino što mi je čudno jest da niti jedno polje nije obavezno.

[Bikonja]

Tocno tako. Nije striktno nuzno ponovno provjeravati da li je polje prazno, ali nije greska (i poanta je da je tamo to dio validacije, dok unutar funkcije za duljinu je to dio izracuna duljine tako da iako funkcionalno nije potrebno ponovno pozvati, semanticki ima smisla. Sto se tice toga da niti jedno polje nije obavezno, mislim da je to jednostavno ostalo kao TODO koji nikad nisam napravio, najvjerojatnije jer iako sam stavio kod na Github, svrha je bila da sebi olaksam zivot s tim kodom, a za to sam uvijek prepisao ista polja pa mi nije striktno trebala takva validacija. Mislim da je cijela ideja mog repozitorija bila vise orjentirana na generalno dobivanje koda iz input forme, a sve sto je dodatno tome je lijepo, ali nije striktno nuzno bilo mi napraviti. Naravno, da sam radio stranicu za "javnost" da ju upotrebljava, nebi smjela izaci bez takve validacije nikako.

…

On Fri, Sep 28, 2018 at 11:06 PM knee-cola ***@***.***> wrote: Hm ... čini mi se da if uvijet možemo opisati ovako (pseudokod): AKO polje nije prazano I AKO polje ne sadrži nedopuštene znakove > polje nije ispravno Validacija se dakle ne bavi time je li neko polje popunjeno ili ne, već da li sadrži nedopuštene znakove. Promatrajući funkciju koja određuje sadrži li polje nedopuštene znakove vidim da i ona prvo provjerava je li polje prazno, tako da je suvišno to još jednom provjeravati. Dovoljno je dakle provjeriti je li fieldLength == -1. Unatoč tome što se ista stvar 2x provjerava, ne vidim da je to neka greška. Jedino što mi je čudno jest da niti jedno polje nije obavezno. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHZOG3OClMG7jwXv94a02niX-lwGF-EGks5ufp15gaJpZM4W_oht> .