Witam. W PHP programuję od 6 lat, od kilku obiektowo. Napisałem własnego CMS'a, stworzyłem kilka portali, od 2 lat korzystam z frameworka Kohana. Potrafię zaprojektować sobie wszystko, czego potrzebuję, znam pojęcia, ale po wykonaniu 10% jakiejś aplikacji, mam wrażenie, że już panuje chaos.
Nigdzie nie znalazłem informacji jak projektować aplikacje, żeby były przejrzyste, elastyczne i zgodne z konwencją programowania obiektowego. Pokazując mój kod lepszym programistom, dostawałem zwroty, że kod jest chaotyczny, źle skonstruowany, że z obiektowością to on ma niewiele wspólnego, ale nie wiem co z tym zrobić.
Mam też dylematy typu jak zwracać kilka danych z jednej metody. Tablicą? Obiektem? Jeśli obiektem, to jak?
Może znacie jakieś książki lub materiały w internecie, z których mógłbym się tego nauczyć.

Wszystko zależy... Jakbyś tak dał np. cms'a moglibyśmy sobie kod pooglądać i dać Ci jakieś konkretniejsze uwagi.

Nigdy w życiu nie pisz takich metod! Za zwracanie kilku informacji z jednej metody powinno się zakuwać w dyby i chłostać na rynku miejskim.

Co do nauki. Polecam Clean Code: A Handbook of Agile Software Craftsmanship

http://helion.pl/ksiazki/czysty-kod-podrec...rtin,czykod.htm Tu po polsku.
Ale nie ma czegoś takiego jak "właściwie programowanie obiektowe". Dla każdego "właściwe " znaczy co innego.
Tutaj dałbym Ci link do DOBREJ książki na temat wzorców projektowych ale niestety takiej nie znam

Rozwalił mnie ten post . W takim razie jak inaczej byś napisał taką metodę?

[PHP] pobierz, plaintext 
/*
	 * Zwykłe zalogowanie
	 */
	public function login()
	{
		$callback = array(); // tablica z błędami lub komunikatem o powodzeniu
		if ( ! Auth::instance()->logged_in()) // nie jestem zalogowany w Auth
		{
			// walidacja
			$rules = Validation::factory($_POST)
				->rule('password', 'not_empty')
				->rule('password', 'min_length', array(':value', 6))
				->rule('password', 'max_length', array(':value', 32))
				->rule('username', 'not_empty')
				->rule('username', 'min_length', array(':value', 3))
				->rule('username', 'max_length', array(':value', 32))
				->rule('username', 'regex', array(':value', '/^[-\pL\pN_.]++$/uD'));
 
			if ( ! $rules->check()) // błąd walidacji
			{
				$callback['errors'] = $rules->errors('validation');
				return $callback;
			}
 
			$user = ORM::factory('user')->where('username', '=', $_POST['username'])->find();
 
			if ( ! $user->loaded())
				$callback['errors']['username'] = __('this user does not exist');
			else
			{
				if ($user->has('roles', ORM::factory('role', array('name' => 'login'))))
				{
					// logowanie
					$login = Auth::instance()->login(
						$_POST['username'],
						$_POST['password'],
						isset($_POST['remember']) ? (bool) $_POST['remember'] : FALSE
					);
 
					if ($login)
						$callback['success'] = __('you are logged in');
					else
						$callback['errors']['password'] = __('invalid password');
				}
				else
					$callback['error'] = __('this account has not been activated');
			}
			return $callback;
		}
		else
		{
			$callback['success'] = __('you are logged in');
			return $callback;
		}
	}
[PHP] pobierz, plaintext 

ja bym napisal ja zupelnie inaczej. Z ktorej wersji Kohany krozystasz? W 3.1 i 3.2 zalecanym sposobem na wykonywanie takich akcji jest przechwytywanie wyjatkow, w starszych wersjach jest od tego metoda check(). Metoda login powinna zwracac true/false a jezeli zwroci false to przechwytujesz wyjatki/bledy.

Edit:
juz widze, ze 3.1 lub 3.2. Nie dosc, ze nie przechwytujesz wyjatkow, to jeszcze przy uzyciu ORM robisz osobna walidacje. Jezeli uzywasz ORM cala walidacja powinna przebiegac w modelu.