Mi corregete?

[dogobis]

Sto cercando di imparare l OOP, e probabilmente non ne so parecchio anche di php..

Posto una ciofeca che ho provato a scrivere e che ovviamente non funzioni (non restituisce mai notlogged..)

class auth {
    
  public function _constructor() 
  {
    if(!isset($_SESSION['userStatus'])) {
      return $_SESSION['userStatus'];
    } elseif (isset($_GET['username'])) {
      $this->userLogin();
    } elseif (isset($_GET['logout'])) {
      $this->userLogout();
    } else {
      $_SESSION['userStatus'] = 'notlogged';
      return $_SESSION['userStatus'];
    }
  }
  
  private function userLogin() 
  {
    $rs = $conn->Execute('SELECT * FROM admin_users WHERE username="$_GET[username]" AND password="$_GET[password]"');
    if ($rs->RecordCount > 0) {
      $_SESSION['userStatus'] = $rs->Fields['type'];
      $_SESSION['userName'] = $rs->Fields['name'];
      $_SESSION['userSurname'] = $rs->Fields['surname'];
      $_SESSION['userEmail'] = $rs->Fields['email'];
      $_SESSION['userLastLogin'] = $rs->Fields['lastlogin'];
      $conn->Execute('UPDATE admin_users SET lastlogin="'.time().'" WHERE username="$user"');
      return $_SESSION['userStatus'];
    } else {
      $_SESSION['userStatus'] = 'notlogged';
      return $_SESSION['userStatus'];
    }
  }
  
  private function userLogout() 
  {
    unset($_SESSION['userName']);
    unset($_SESSION['userSurname']);
    unset($_SESSION['userEmail']);
    unset($_SESSION['userLastLogin']);
    $_SESSION['userStatus'] = 'notlogged';
    return $_SESSION['userStatus'];
  }
  
} 

Se qualche buon anima potesse correggermi, e darmi qualche consiglio in generale sarebbe di grande aiuto.. Grazie

[buonste]

Qualche suggerimento:

hai inizializzato la sessione con

session_start()

prima dell'header?

Inoltre potresti sostiuire il codice della fx userLogout() con

private function userLogout()
  {
    unset($_SESSION);
    session_destroy();
  }

Ciao!
Stefano

[buonste]

Inoltre penso tu abbia un problema nel costruttore:

public function _constructor()
  {
    if(!isset($_SESSION['userStatus'])) {      //-- se la var userStatus non e' settata
      return $_SESSION['userStatus'];   //-- restituisce la var userStatus (che e' unset tanto quanto e' vero l'if di cui sopra)
    } elseif (isset($_GET['username'])) {
      $this->userLogin();
    } elseif (isset($_GET['logout'])) {
      $this->userLogout();
    } else {
      $_SESSION['userStatus'] = 'notlogged';
      return $_SESSION['userStatus'];
    }
  } 

[gm]

Il titolo della discussione non è conforme al regolamento