DevProTalk

Forumi IT profesionalaca
web development, web design, e-business, SEO


Idite nazad   DevProTalk > Web development i web aplikacije > Programiranje
Želite da se reklamirate ekskluzivno na ovoj poziciji? Javite se

Programiranje Java, Perl, VB, ASP, .NET, C, C++, Pascal, Delphi Sponzor: VIP izazov 3

Odgovori
 
Alati teme Način prikaza
Staro 12. 04. 2012.   #31
Djuki
profesionalac
Professional
 
Avatar Djuki
 
Datum učlanjenja: 09.06.2005
Lokacija: Pancevo
Poruke: 251
Hvala: 60
47 "Hvala" u 34 poruka
Djuki is on a distinguished road
Pošaljite ICQ poruku za Djuki Pošaljite poruku preko Skype™ za Djuki
Default

Ja obično radim validaciju u kontrolerima, a svakako izbegavam da ponavljama kod.

Mislim da je najbolje da sama definicija validacije bude u samim modelima, koji će sami sebe da validirati pre save() metode.
__________________
Blog | Facebook | Twitter | PHP Srbija
Djuki je offline   Odgovorite uz citat
Staro 12. 04. 2012.   #32
ivanhoe
Ivan Dilber
Sir Write-a-Lot
 
Avatar ivanhoe
 
Datum učlanjenja: 18.10.2005
Lokacija: Bgd
Poruke: 5.320
Hvala: 104
2.344 "Hvala" u 583 poruka
ivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svima
Pošaljite poruku preko Skype™ za ivanhoe
Default

Citat:
Originalno napisao Djuki Pogledajte poruku
Mislim da je najbolje da sama definicija validacije bude u samim modelima, koji će sami sebe da validirati pre save() metode.
Upravo tako. To je rutinska stvar koja ce se uvek uraditi na insert() i update() i ne vidim zasto bi se to izdvajalo u kontroler kad ce ta jedna te ista provera postojati u svakom kontroleru koji nesto snima?

Naravno, sam proces validacije radi zasebna klasa, model samo pozove odgovarajucu metodu. Sad kad su dodati mix-ins u php to ce biti jos lepse za pravljenje.
__________________
Leadership is the art of getting people to want to do what you know must be done.
ivanhoe je offline   Odgovorite uz citat
Staro 12. 04. 2012.   #33
dee
Domagoj Horvat
Expert
 
Avatar dee
 
Datum učlanjenja: 24.07.2006
Lokacija: Zagreb
Poruke: 502
Hvala: 22
10 "Hvala" u 8 poruka
dee is on a distinguished road
Pošaljite ICQ poruku za dee
Default

Citat:
Upravo tako. To je rutinska stvar koja ce se uvek uraditi na insert() i update() i ne vidim zasto bi se to izdvajalo u kontroler kad ce ta jedna te ista provera postojati u svakom kontroleru koji nesto snima?
ako je uvijek ista provjera, onda da. ali, kako na taj nacin rijesiti slucaj da implementacija kod jednog korisnika trazi npr password od minimalno 6 znakova, a kod drugog 8, bar jedno veliko slovo i bar jedan broj? ili da pocinje/zavrsava s odredjenim stringom, a zelimo da koristi isti model? (recimo, jedan od trenutnih zahtjeva jednog korisnika je da mu sifra proizvoda ima tocno 7 znakova. drugi hoce 9 i to da mu pocinje sa 'xy')

ovo, recimo, pricam iz konteksta .NET gdje je zgodno jednom napraviti dll s modelom i ne pipati ga vise (za razlicite implementacije) iako ni u phpu nije nista sustinski drugacije. slozis model za npr cms i kasnije samo vozis front-end implementacije dio kojih je i neki validation rules mehanizam. model ne pipas.

naravno, ovo sve nema smisla ako reuse nije ozbiljan zahtjev jer je lakse promijeniti model na 2 implementacije nego ceprkati ovako. ali kad se dogadja slucaj od 50 instalacija, puno je ljepse imati jedinstven model i 50 front-endeva nego svakog po 50 (pa jos ne daj boze i verzija baza)
__________________
postoje ludosti bez kojih je nemoguce ljudsko dostojanstvo
dee je offline   Odgovorite uz citat
Staro 12. 04. 2012.   #34
ivanhoe
Ivan Dilber
Sir Write-a-Lot
 
Avatar ivanhoe
 
Datum učlanjenja: 18.10.2005
Lokacija: Bgd
Poruke: 5.320
Hvala: 104
2.344 "Hvala" u 583 poruka
ivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svimaivanhoe je ime poznato svima
Pošaljite poruku preko Skype™ za ivanhoe
Default

To su dodatne provere koje se onda urade u kontroleru isto kao npr. ono da li je kliknuto I agree. Ali model treba da brine o konzistenciji podataka koji idu u bazu, a to se ne menja ako se ne menjaju tabele (ili ako se drugacije cuvaju podaci onda fajlovi ili sta vec).

Ne kazem ja naravno da mora ovako ili onako, ali meni je zgodnije ovako, jer tako imas manje koda o kom moras da brines...
__________________
Leadership is the art of getting people to want to do what you know must be done.

Poslednja izmena od ivanhoe : 12. 04. 2012. u 15:26.
ivanhoe je offline   Odgovorite uz citat
Staro 12. 04. 2012.   #35
bluesman
Goran Pilipović
Sir Write-a-Lot
 
Avatar bluesman
 
Datum učlanjenja: 18.05.2005
Lokacija: Beograd
Poruke: 5.450
Hvala: 288
1.247 "Hvala" u 446 poruka
bluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušatibluesman je osoba koju treba slušati
Pošaljite ICQ poruku za bluesman
Default

Citat:
Originalno napisao Marko Medojevic Pogledajte poruku
Mislim da je upotreba $_POST u modelu loša sa aspekta ponovne upotrebljuvosti koda.

Uzmimo recimo primer gde se post koristi u modelu:
Kôd:
public function addUser() {
$this->db->insert($_POST['username'], $_POST['password'],$_POST['email']);
}
i primer gde se prosleđuju parametri:
Kôd:
public function addUser($username, $password, $email) {
$this->db->insert($username, $password, $email);
}
Prvi metod, koji koristi post, lako možeš upotrebiti iz kontrolera koji prima POST od forme sa očekivanim podatcima.
Međutim, da li možeš prvu metodu koristiti da upišeš podatke koji ne dolaze iz POST forme, recimo iz neke migracije koja uzima podatke iz stare baze i upisuje u novu?
Ili recimo, možda ćeš želeti da metodu addUser() testiraš kroz unit testove, gde ćeš sam prosleđivati podatke u kodu.

Mislim da je ponovna upotrebljivost druge metide daleko veća, za razliku od prve koja je zakucana za POST kontekst.
Nešto se nismo razumelli, ja nisam rekao da $_POST ide direktno u model nego sam rekao da se u modelu vrši validacija ulaznih podataka, bez obzira na poreklo

Po meni treba ovako:

Kôd:
public function addUser($data) {
// $data validacija
$this->db->insert(... whatever here .. );
}
A $data može da bude $_POST iz controllera ili bilo koji drugi array iz nekog drugog objekta

a ovo sa 17 parametara funkcije ne bih sada komentarisao osim da je to loša praksa
__________________
Goran Pilipović a.k.a. Ugly Fingers Bradley f.k.a. bluesman
I don't always know what I'm talking about but I know I'm right!
bluesman je offline   Odgovorite uz citat
"Hvala" bluesman za poruku:
Odgovori



Pravila pisanja
Možete ne započinjati nove teme
Možete ne slati odgovore
Možete ne slati priloge
Možete ne izmeniti svoje poruke
vB kôd je Uključen
Smajliji su Uključen
[IMG] kod je Uključen
HTML kôd je Isključen
Pogledajte forum


Vreme je GMT +2. Trenutno vreme je 01:02.


Powered by vBulletin® Verzija 3.6.8
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
Copyright © DevProTalk. All Rights Reserved.
Mišljenja, saveti, izjave, ponude ili druge informacije ili sadržaji nastali na Sajtu su vlasništvo onoga ko ih je kreirao, a ne DevProTalk.com, tako da ne morate da se oslanjate na njih.
Autori poruka su jedini odgovorni za ovakve sadržaje. DevProTalk.com ne garantuje tačnost, kompletnost ili upotrebnu vrednost informacija, stavova, saveta ili datih izjava. Ne postoje uslovi pod kojima bi mi bili odgovorni za štetu ili gubitak koji je posledica bilo čijeg oslanjanja na nepouzdane informacije, ili bilo kakve informacije nastale kroz komunikaciju između registrovanih članova.
Web sajt može sadržavati linkove na druge web sajtove na Internetu ili neke druge sadržaje. Ne kontrolišemo niti podržavamo te druge web sajtove, niti smo pregledali bilo kakve sadržaje na takvim sajtovima. Mi nećemo biti odgovorni za legalnost, tačnost ili prikladnost bilo kog sadržaja, oglasa, proizvoda, usluga ili informacije lociranim na ili distribuiranih kroz druge web sajtove, niti za bilo kakvu štetu nastalu kao posledica takvih informacija. DevProTalk.com drži i čuva druga prava vlasništva na web sajtu. Web sajt sadrže materijale zaštićene copyright-om, zaštitne znakove i druge informacije o pravu vlasništva ili softver. Članovi mogu poslatu informacije zaštićene pravima vlasništva njihovih nosilaca i ona ostaju zaštićena bez obzira da li su oni koji prenose te informacije to naveli ili ne. Osim informacija koje su u javnom vlasništvu ili za koje dobijete dozvolu, nemate pravo da kopirate, modifikujete ili na bilo koji način menjate, objavljujete, prenosite, distribuirate, izvršavate, prikazujete ili prodajte bilo koju informaciju zaštićenu pravima vlasništva. Slanjem informacija ili sadržaja na bilo koji deo DevProTalk.com, Vi automatski dozvoljavate i predstavljate garanciju da imate pravo da dozvolite DevProTalk.com ili članovima DevProTalk.com bespovratnu, kontinualnu, neograničenu, globalnu dozvolu da koriste, kopiraju, izvršavaju, prikazuju i distribuiraju takve informacije i sadržaje i da iz takvih sadžaja koriste bilo koji deo u bilo koje svrhe, kao i pravo i dozvolu da koriste gore navedene sadržaje. Svi zaštitni znakovi (trademarks), logotipi, oznake usluga, firme ili imena proizvoda koji se pominju na ovom web sajtu su vlasništvo kojim raspolažu njihovi vlasnici.