Witam,

ucze się oop...
i nasunęło mi się takie pytanie:

Dlaczego powinno się (chyba) pisać tak:

[PHP] pobierz, plaintext 
<?php
$model = new Nazwa_Modelu;
$model -> text = $text;
$model -> nazwa = $nazwa;
$model -> dodaj_strone();
?>
[PHP] pobierz, plaintext 

czy nie wystarczy po prostu

[PHP] pobierz, plaintext 
<?php
$model = new Nazwa_Modelu;
$modle = dodaj_strone($text, $nazwa);
?>
[PHP] pobierz, plaintext 

czy może chodzi o to że powinno się wszystko rozbijać na jak najmniejsze części (medy, classy) i każda mała część jest tylko za jedną rzecz odpowiedzialna?

Możesz i w jeden i w drugi sposób. Zależy ściśle od tego co chcesz zrobić.

Jaka jest zaleta drugiego przykładu - w pierwszym nie możesz kontrolować co zostaje wpisane do właściwości klasy przy jej wpisaniu, może się tam znaleźć największa bzdura, a przekazując dane do metody spokojnie możesz je już na wstępie walidować.

Jak już to tak

Ale to wszystko zalezy co to za klasa i co ona ma robic. Jezeli liczba parametrow jest cholernie dluga wiec warto robic 1 metoda.

a tak:

[PHP] pobierz, plaintext 
<?php
$model = new Nazwa_Modelu;
$model -> text($text);
$model -> nazwa($nazwa);
$model -> dodaj();
?>
[PHP] pobierz, plaintext 

i np teraz w metodzie text i nazwa moge sobie sprawdzac czy wszystko ok

Możesz i jak najbardziej jest to jedna z używanych metod. Oczywiście w tym momencie warto właściwości klasy opisać jako chronione lub prywatne, żeby umożliwić ustawianie tylko poprzez odpowiednie metody.

Możesz w metodzie __set daj sprawdzanie poprawności Wszystkie możliwości są prawidłowe, a to jak Ci będzie wygodniej pisać to musisz już sam sprawić Ja np. wole dac

niż rozbijać na oddzielne metody. A __set nie lubię jakoś We własnej metodzie set mogę sobie tablice itp. ustawiać, więc mam większą kontrolę i większe możliwości

Pisze sie tak aby uniknąć wywołania metody z parametrami bo czasem sa cholernie długie, po drugie masz w jeden obiekcie wywołanie funkcji z tego samego obiektu to po co do każdej przekazywać usera w parametrze skoro można to zrobić tak:

[PHP] pobierz, plaintext 
<?php
$user = new users;
$users->setUser(system::getLogined());
//a nie tak
$user->getUser(system::getLogined(), $model->getJakisModel($idjakies));
$user->getInfo(system::getLogined(), 'type');
//Bo takie długie molochy wychodzą
?>
[PHP] pobierz, plaintext 

Przepraszam bardzo za mały OT, ale nie poraz pierwszy widze takie coś:
co oznacza wyraz Logined? Chodzi o Logged In?

Ważne ze wiadomo o co chodzi

Nie chciałbym tu nikomu ubliżać, ale naukę programowania polecałbym od nauki podstaw języka angielskiego.

I nie ważne, że wiadomo o co chodzi - nie czepiam się literówki czy jakiegoś tam małego błędu gramatycznego, ale tworzenie nowego języka jest trochę dziwne.

Jak najbardziej można, ale moim zdaniem jest to troszeczkę bez sensu. Rozbijanie wszystkiego na siłę na oddzielne funkcje składowe (metody) jest co najmniej dziwne - ani to szybkie (więcej kodu), ani funkcjonalne, użyteczne.



Daj spokój

Rozbicie ma sens, jeśli można wywołać jedną metodę, a drugą już nie, jeśli obie są wymagane na raz, to lepiej jest aby przyjmowały parę argumentów (ale też nie za dużo, jak więcej to znaczy, że inaczej trzeba to rozwiązać, np. przez obiekt z danymi). Więc to czy będzie dwie metody wywołane, czy jedna, zależy od tego jak dokładnie działają.

Hmm.. może Ciebie źle zrozumiałem, ale: pakowanie wszystkiego do jednej metody na siłę jest bez sensu, wtedy dużo lepiej dać to do kilku osobnych metod. To jest prawda, i jak rozumiem o tym właśnie mówisz. Natomiast przesyłanie argumentów przez osobne metody, jak tu:

[PHP] pobierz, plaintext 
<?php
$model = new Nazwa_Modelu;
$model -> text($text);
$model -> nazwa($nazwa);
$model -> dodaj();
?>
[PHP] pobierz, plaintext 

to IMO kompletny nonsens. Zgodzisz się ?

Obrazowo: jak chcesz zrobić pranie, to wkładasz brudną odzież do pralki i wsypujesz do niej jakiegoś persila, czy czegośtam, czy może dzwonisz do jednego specjalisty, żeby wrzucił pranie, do drugiego żeby wsypał proszek, a na końcu dopiero włączasz pralkę :] ? Takie moje zdanie

Nie jest! Masz klasę np. comments a w niej funkcje: add i get. Kazda potrzebuje do działania id newsa dla którego ma operować wystarczy:

[PHP] pobierz, plaintext 
<?php
$comments->setIdnews($this->param3);
?>
[PHP] pobierz, plaintext 

niz za kazdym razem:

[PHP] pobierz, plaintext 
<?php
$comments->get($this->param3);
$comments->add($this->param3, $content);
?>
[PHP] pobierz, plaintext 

Dlaczego nonsens? A jeśli chce zmienić tylko jedno pole?

Też prawda. Nie pomyślałem o tym w taki sposób

A czy jak idziesz do swojego pokoju otwierasz wszystkie drzwi aby do niego wejść, czy tylko te co prowadzą do niego? Bo chyba nie ma sensu otwierać wszystkich jak chcesz tylko przez jedne przejść. To jest przykład kiedy mamy wiele metod, ale każda jest niezależna (w użyciu) od drugiej.

[PHP] pobierz, plaintext 
<?php
$model = new Nazwa_Modelu;
$model -> text($text);
$model -> nazwa($nazwa);
$model -> dodaj();
?>
[PHP] pobierz, plaintext 

Taki zapis jest wygodny gdy:
1) Możemy dodać wiele tekstów czy wiele nazw
2) Kod jest trochę rozrzucony

[PHP] pobierz, plaintext 
<?php
$model = new Nazwa_Modelu;
//jakis kod, ktory nam cos tam pobiera
$model->text($text);
//znowu jakis kod, ktory robic cos zupelnie innego
//i tu znowu cos tam
$model->nazwa($nazwa);
//cala reszta
//by na koncu dac
$model->dodaj();
?>
[PHP] pobierz, plaintext 

Jeżeli ma to być w "zwartym szyku" to można metodzie dodaj() dodać dwa parametry.

W praktyce, najczęściej łączy się oba sposoby. Np. w Propelu oprócz setterów i getterów jest też metoda fromArray().

No właśnie jest mowa o tym że nie zawsze chcesz podawać dwa parametry i wtedy to wychodzi dziwnie

[PHP] pobierz, plaintext 
<?php
$model->dodaj( $var1, null );
?>
[PHP] pobierz, plaintext 

Chodzi o to że np. jak edytujesz jakiś wpisz (AR) to przecież nie podajesz wszystkiego na raz, tylko pojedynczo, bo jak byś chciał zmienić status to nie ma sensu abyś musiał pisać resztę pól, tylko to potrzebne.
Co innego jak to ma być np. połączenie z Webservice, wtedy musisz podać adres, port, login, hasło, bo to jest konieczne do zainicjowania połączenia i zawsze muszę być wszystkie, i dlatego podaje się je jako argumenty jednej metody, czy też w konstruktorze.

Nie bez powodu napisałem to można metodzie dodaj() dodać dwa parametry.
Teoretycznie można dodać opcionalny argument, ale używanie metod typu

[PHP] pobierz, plaintext 
<?php
public function do Sth($param1 = null, $param2 = null, $param3 = null, $param4 = null, $param5 = null, $param6 = null){
  //...
}
?>
[PHP] pobierz, plaintext 

To lekki masochizm.

Można też połączyć obie metody

[PHP] pobierz, plaintext 
<?php
class someClass{
  public function text($x){
	//..
  }
 
  public function tytul($x){
	//..
  }
 
  public function dodaj($x = null, $y = null){
	if($x !== null) $this->text($x);
	if($y !== null) $this->tytul($y);
 
	//...
  }
}
?>
[PHP] pobierz, plaintext 

Naprawdę wszystko zależy od konkretnego przypadku - i najlepiej gdyby autor podał konkret.