به نظر من بعضی توابع رو بهتره استاتیک کنی
مثلا تابع userloign الان شما به این شکل استفاده میکنی
$user = new user();
$user->userloign($username , $passwod);
شما اینجا یه شی خالی میسازی و بعد چک میکنی وجود داره یا نه
که اگه وجود نداشت یعنی یوزر و پسورد غلط بود این شی شما عملا هدر میره و بی استفاده میشه
میتونی با استاتیک کردن تابع کاری کنی که اگه وجود داشت توی دیتبایس اونوقت تازه شی ساخته بشه
هم اینکه مقدار کد نویسی برای هر لاگین رو کمتر کنی
static function userloign($user, $pass)
{
if ($rows > 0) {
$user = $result->fetch(PDO::FETCH_OBJ);
$object = new user();
$object->user = $user->user;
$object->pass = $user->pass;
$object->name = $this->pass;
$object->id = $this->id;
setcookie("login", $this->user, time() + 86400);
echo 'Your Are SUscessfully LoGined In My SIte';
return $object ;
} else {
echo 'User Or Password Incorrect';
}
}
$user = user::userlogin($user, $pass);