Witam,
chciałbym się dowiedzieć jak najlepiej napisać klasę edycji danych użytkownika. Chcę to napisać jak najoptymalniej i zgodnie z OOP ale nie do końca "ogarniam".
Chciałbym edytować takie dane jak, zmiana hasła, avatara, płci, informacji o sobie itp. Nazwę użytkownika mam zapisaną w sesji i po niej prawdopodobnie będę go identyfikował.
Jak dalej pisać ? Czy takie coś jak niżej jest ok ?

[PHP] pobierz, plaintext 
class User {
	private $username;	
 
	public function __construct(){
		$this->username = $_SESSION['username'];	
	}
 
	public function editPassword ($newpass, $confpass){
       }
[PHP] pobierz, plaintext 

Czy każdą opcje edycji (hasło, płeć etc.) robić jako oddzielna funkcja ? Czy może w ogóle, każda edycja to powinna być oddzielna klasa ? Powinienem też pewnie coś w konstruktorze przesyłać ale nie wiem za bardzo co ? :/
Czy tworzyć w tej klasie osobne funkcje odpowiadające za wyświetlenie formularzy ?
Chcę się nauczyć dobrze to wszystko pisać więc byłbym wdzięczny za wskazówki.
Pozdrawiam

Osobiście oddzieliłbym tą klasę od drugiej wyświetlającej formularze.

Tak, dla każdej opcji musisz mieć osobną funkcję, dla zamiany hasła,maila etc. :-)
Tak się robi i nikomu to jeszcze nie zaszkodziło.

A w konstruktorze przekaż username a nie, że pobierasz go sobie z session.
Dzięki temu będziesz mógł działać też na innych użytkownikach (nie tylko zalogowanych).

[PHP] pobierz, plaintext 
(...)
private $username;
public function __construct($username){
 $this->username = $username;
}
 
//i wtedy:
$user = new User($_SESSION['username']);//ale możesz też pracować na jakimś innym userze
[PHP] pobierz, plaintext 

ps. Nie wiem, czy nie lepiej byłoby przekazywać raczej id użytkownika w bazie zamiast username..

Jedna z najważniejszych zasad OOP: jeden obiekt, jedno zadanie. Klasa User ma reprezentować użytkownika (nie mylić z użytkownikiem w sensie internauty) czy zajmować się walidacją danych z formularza (wnioskuję po parametrach $newpass i $confpass)?

1. W konstruktorze przekazujemy tylko i wyłącznie to co jest niezbędne do zainicjalizowania obiektu. Czy nazwa/id użytkownika są niezbędne? W sytuacji, gdy tworzysz nowego użytkownika ID nawet z reguły nie znasz, a nazwę użytkownika po prostu wygodniej jest przekazać setterem (User::setUsername($username)).
2. Ta klasa User powinna mieć dwie metody od hasła: User::getPassword() i User::setPassword($password). Sama edycja hasła to już jakiś zewnętrzny proces, który jedynie wykorzystuje klasę User - całość mogła by wyglądać jakoś tak:

   [PHP] pobierz, plaintext 
   $user = /* pobierz aktualnie zalogowanego użytkownika */;
   $userServicel // Obiekt zamujący się zapisywaniem/usuwaniem użytkowników
   $validator; // Obiekt sprawdzający poprawność danych z formularza
    
   $data = $validator->getData(); // Powiedzmy, że to zwraca bezpieczne dane z formularza
    
   $user->setPassword($data['password']);
    
   $userService->update($user);
   [PHP] pobierz, plaintext 

Do tworzenia usera ( rejestracji i logowania ) mam 2 oddzielne klasy. Ta klasa "User" o której tutaj mówię ma za zadanie umożliwić już zalogowanemu użytkownikowi edycję własnych danych takich jak wymieniłem wyżej. Z tego co myślę, to wystarczy do tej klasy przekazać nazwę użytkownika aby po prostu widzieć, który rekord ma być edytowany.
Crozin - jakoś Twojego przykładu nie ogarniam Można prosić bardziej łopatologicznie albo może jest gdzieś jakaś prosta pełna klasa, która mogła by mi to jakoś uzmysłowić ?