Forum PHP.pl

extract + dowolny sposób wczytania pliku z widokiem (require, include etc.)

Pierwsza klasa:
1. Niepotrzebnie jest statyczna, poza tym skoro używasz statycznych metod używaj static *typ* function zamiast *typ* function.
2. Metody takie jak Database, Router, Session itp. to dość dobry pomysł, ale lepiej jakbyś użył do tego jakiegoś kontenera (Poczytaj o DependencyInjection).

Druga klasa:
1. "class Controller extends Services" - Nie powinno być takiego dziedziczenia, ty nie rozszerzasz serwisów, tylko z nich korzystasz/robisz na nich operacje - Przekaż obiekt Services poprzez konstruktor/metodę do obiektu Controller.
2. Nie zauważyłem byś gdziekolwiek przekazywał jakąkolwiek konfigurację, a to poniekąd zły znak.

Trzecia klasa:
1. Ten sam błąd co w klasie wyżej, pkt. 1
2. Skoro korzystasz z jakiejś ścieżki kilkukrotnie to zapisz ją gdzieś do zmiennej i używaj w ten sposób. Poza tym lepiej byś tę ścieżkę przekazał jako argument / klucz konfiguracji, zamiast ustawiać na sztywno w klasie.
3. Do tworzenia obiektów przyda Ci się call_user_func_array lub ReflectionClass.
4. Porozdzielaj metode Run() na inne, wyspecjalizowane metody dla poszczególnych elementów ponieważ robi się bałagan.
5. Dane imho nt. modułu itp powinny zostać wcześniej sprawdzone (długość, czy to string itp) i dopiero później powinny być przekazane do Applications w celach sprawdzenia istnienia i utworzenia obiektu.

Przekaż obiekt poprzez argument metody sendNewPassword

Dlaczego metodą? Może być równie dobrze obiektem klasy Umiejętność. Dlaczego upierasz się przy swojej racji i tłumaczysz wszystkim dookoła, że to my mamy coś nie równo, zamiast dać coś komuś wyjaśnić i pomyśleć trochę LOGICZNIE.

Dokładnie tak, różnicą w tym przypadku może być wygoda pisania, zaoszczędzać parę linijek kodu

Po pierwsze twój kod nawet nie zadziała. Żeby zrobić "ciąg" wykonywanych metod, w każdej z nich musisz zwracać jakiś obiekt, w twoim przypadku tej samej klasy.

W klasie masz metody save oraz setData. W metodzie setData ustawiasz dane i w metodzie zwracasz obiekt, a dalej możesz wykonywać inne operacje.

[PHP] pobierz, plaintext 
class X 
{
 
	private $data = array();
 
	public function setData(array $data) 
	{
		$this->data = $data;
 
		return $this; // Zwracasz obiekt klasy X
	}
 
	public function save() 
	{
		// Zapis danych...
	}
 
}
 
$data = array('foo' => 'bar');
 
$x = new X();
$x->setData($data)->save();
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
class Lancuch 
{
 
	public function ala() 
	{
		return $this;
	}
 
	public function ma() 
	{
		return $this;
	}
 
	public function kota() 
	{
		return $this;
	}
 
}
 
$lancuch = new Lancuch();
$lancuch->ala()->ma()->kota();
 
[PHP] pobierz, plaintext 

Po prostu w metodzie zwracasz obiekt klasy.

Nie Zend_DB, a Zend_Db. Wielkość znaków ma znaczenie

Funkcja conn nie istnieje.

Zamień:

[PHP] pobierz, plaintext 
$result = conn($query);
[PHP] pobierz, plaintext 

Na:

[PHP] pobierz, plaintext 
$result = mysql_query($query);
[PHP] pobierz, plaintext 

W 6 linijce

[PHP] pobierz, plaintext 
public function Dodaj( $a, $b ){
    $this -> wynik = $a + $b;
    return $this; // zwracasz obiekt...
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php 
header('refresh: 5; url=/tutaj/link/');
?>
[PHP] pobierz, plaintext 

Gdzie 5 to czas w sekundach po jakim ma przekierować

Błędem jest to co podał mat-bi.
kadlub - Bardzo inteligentnie, teraz przy każdej wykonanej akcji będzie niszczyć sesję i przenosić do głównej strony. Chyba jasno widnieje instrukcja switch i stan wyrażenia "wyloguj".

Popraw linie 17 - ?action=wylogyj -> ?action=wyloguj

1. Definiujesz dane do bazy danych, które równie dobrze mógłbyś wpisać do klasy poprzez choćby tablicę z konfiguracją.
2. Klasa jest kompletnie nie tyle co źle, a bezsensownie napisana.
3. Pakujesz wszystko do jednego - Połączenie z bazą danych, operacje na niej, dane użytkownika, autoryzacja, wyświetlanie.
4. Łączysz się za każdym razem przy próbie utworzenia obiektu User.
5. Prawie w każdej metodzie wykonujesz zapytanie o_O.
6. Wspomniałem wyżej, że klasa jest napisana bezsensu, ponieważ niektóre "metody" zawarte w niej są na siłę wepchane (czyt. Twoja klasa to kontener na funkcję)
7. Podstawy OOP się kłaniają - Single Responsibility Principle - Jedna sprawa - Jedna klasa.

Przede wszystkim musisz oddzielić od siebie taki rzeczy jak baza danych (dostęp, komunikacja, obsługa), zarządzanie użytkownikiem, autoryzacja.

Stwórz klasę, która zajmie się połączeniem z bazą danych, zwróci Ci obiekt jakiegoś sterownika bazy danych (np. klasę dziedziczącą po PDO).

Zrób osobną klasę dla obsługi użytkownika (tworzenie, usuwanie, edycja / pobieranie danych) oraz osobną do autoryzacji (logowanie, wylogowywanie, sprawdzanie).

pozdrawiam

Zawsze można skorzystać z funkcji array_search, bez utrudniania sobie życia