Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [Kohana]Kontrola jakości kodu
MateuszS
post
Post #1





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----


Witam. Skończyłem mały projekt w Kohanie (prosta strona z newsami, galerią itp.). Nie ma sensu pokazywać strony bo najważniejsza jest w tym właśnie Kohana i szlifowanie jej. Dopiero zacząłem się go uczyć i chciałbym przejść swego rodzaju kontrolę jakości. Wiadomo, prawdopodobnie "forumowy sanepid" zamknąłby moją fabrykę kodu ale głównie chodzi mi o uzyskanie rad, co robię źle i jak to poprawić żeby było dobrze (albo chociaż lepiej). Przymierzam się w niedalekiej przyszłości do stworzenia większego projektu i chciałbym żeby to było zrobione PRO.

Chciałbym abyście przyjrzeli się kontrolerowi i modelowi newsów (dla panelu administracyjnego). Proszę nie traktować tego jako "kodu do oceny". Ja wiem że to do oceny się jeszcze nie nadaje, z tym że ja już nie bardzo wiem co mógłbym poprawić. Stąd moja prośba o "kontrolę" kodu. Wybaczcie durnowate komentarze w kodzie, czasami się gubiłem.

Kontroler:
http://wklej.org/id/966403/

Model:
http://wklej.org/id/966404/

Pozdrawiam

Ten post edytował MateuszS 26.02.2013, 01:07:52
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
MateuszS
post
Post #2





Grupa: Zarejestrowani
Postów: 1 429
Pomógł: 195
Dołączył: 6.10.2008
Skąd: Kraków/Tomaszów Lubelski

Ostrzeżenie: (0%)
-----


  1. $this->errors = $valid->errors('error');


  1. $error = $this->model->error();


Po prostu przypisujesz tablicę do tablicy a potem metodą error() ją zwracasz tak?

@Spawnm, szybko znaleziony na Google więc możliwe. Ale układ jest ten sam co w MVC. Kontroler ma "dostęp" do widoku, model już nie.


PS. Czyli to czy umieszczę w kontrolerze walidację czy w modelu to kwestia obranej drogi tak? Nie jest błędem pisanie "grubego kontrolera"?

Ten post edytował MateuszS 26.02.2013, 23:26:42
Go to the top of the page
+Quote Post
skowron-line
post
Post #3





Grupa: Zarejestrowani
Postów: 4 340
Pomógł: 542
Dołączył: 15.01.2006
Skąd: Olsztyn/Warszawa

Ostrzeżenie: (0%)
-----


Cytat(MateuszS @ 26.02.2013, 23:25:22 ) *
Po prostu przypisujesz tablicę do tablicy a potem metodą error() ją zwracasz tak?

Tak
Cytat(MateuszS @ 26.02.2013, 23:25:22 ) *
PS. Czyli to czy umieszczę w kontrolerze walidację czy w modelu to kwestia obranej drogi tak? Nie jest błędem pisanie "grubego kontrolera"?

I tak i nie. Wiele osób napisze Ci ze za przetwarzanie danych odpowiedzialny jest model, a kontroler łączy model z widokiem.
  1. public function action_index()
  2. {
  3. $this->template->content = View::factory('area/index') //widok
  4. ->set('items', $this->model->get()); // model
  5. }

i to koniec, jeżeli chesz spawdzić i wyświetlić błąd związany z brakiem danych do wyświetlenia to możesz to zrobić w widoku, ale też w mojej opini model też by był dobrym miejscem na sprawdzenie, ale wszystko zależy od tego jak masz zaprojektowaną aplikację.
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 8.10.2025 - 13:53