Как сделать логику приложения на PHP?

Question

[Twelfth Doctor]

Здравствуйте! У меня есть класс сайт и его методы:
1. Прикрепление алиасов (доменов) к сайту
2. Открепление доменов (алиасов) от сайта
3.Редактирование основной информации сайта
4. Удаление сайта
5.Получение информации о сайте
6.Создание нового сайта
Вопрос: в конструкторе класса должен быть метод получения информации о сайте, или метод создания нового сайта? И можно ли созданный объект класса сохранить в сессии?

Answer 1

[ThunderCat]

Конструктор для объектов может содержать неявный вызов инициализации. Например при передаче в конструктор id или имени сайта по нему загружается из базы все данные, однако обычно это делают через отдельный метод инициализации, который в конструкторе вызывают. если никаких параметров не передается - создается пустой(новй) объект.
По поводу сессии - если в объекте нет ссылок на внешние ресурсы(например на DB), то serialize() работает, храните как строку, потом unserialize() для чтения из сессии.

У вас в коде все же как-то все кривовато,
1) прям в объекте сайта создается подключение к бд и фигачатся запросы,
2) Названия классов пишутся с большой буквы
3) тут же вывод каких то обработок... Вообще как будто не ооп а попытка писать объекты процедурным стилем.

// конструктор объекта
function __construct($id = ""){
  if($id==="") {do some}// создаем пустой объект
  else{ $this->initById($id);}
}

// в коде где создаем новый сайт
$site = new Site();
$site->add($dataset); // передаем параметры, можно отдельно каждый, можно заранее созданным массивом.

Comments

[Twelfth Doctor]

При создании сайта в функцию(конструктор) передаётся 5 аргументов, при получении информации о сайте - только 1

[ThunderCat]

verdex: это не при создании, а при инициализации внешними данными. Как я и написал - создается пустой объект, вызывается ОТДЕЛЬНЫЙ метод класса для инициализации новыми данными, затем делается сохранение.

[Twelfth Doctor]

ThunderCat: Т.е. нужно сделать 2 отдельных метода - первый - инициализация внешними данными, второй - для случая с уже существующим сайтом. Эти функции есть, как теперь сделать так, чтобы конструктор различал, в каком случае происходит инициализация внешними данными?

[Twelfth Doctor]

Вот случай с уже существующим сайтом:

class site
{
public $id;
public $mainname;
public $aliases;
public $charset;
public $email;
public function getsiteinfo($id)
{
$mysqli = mysqli_connect($dbhost, $dbuser, $dbpass, $dbname);
$getinfo = $mysqli->query("SELECT * FROM vhosts WHERE unid = '$id'");
$info = $getinfo->fetch_assoc();
$getaliases = $mysqli->query("SELECT name FROM domains WHERE vhost = '$id'");
$aliases = $getaliases->fetch_assoc();

$this->mainname = $info['mainname'];
$this->charset = $info['charset'];
$this->email = $info['email'];
$this->aliases = $aliases;
return $this;
}
}

[Twelfth Doctor]

Сохранение:

public function add ($mainname,$charset,$email,$index,$login) {
        $mysqli = mysqli_connect($dbhost, $dbuser, $dbpass, $dbname);
        $check = $mysqli->query("SELECT unid FROM vhosts WHERE mainname = '$this->mainname'");
        if($check->num_rows>0) { echo "Произошла ошибка. Доменное имя, которое Вы ввели, уже привязано к другому сайту!";exit;}
        $mysqli->query("INSERT INTO vhosts (mainname,charset,email,index,userid) VALUES ('$this->mainname' , '$this->charset', '$this->email' , '$this->index' , '$this->login')");
        Include 'apacheconf.php';
        Include 'nginxconf.php';
        $vhostid = $mysqli->mysqli_insert_id;
        file_put_contents ("/etc/apache2/sites-available/$vhostid.conf", $apacheconf);
        file_put_contents ("/etc/nginx/sites-available/$vhostid", $nginxconf);
    }
}

[ThunderCat]

verdex: поправил ответ, в целом советую все же переписать данные... поделки на нормальный код ООП.

[Twelfth Doctor]

ThunderCat: Хорошо, буду переписывать, и смотреть, как схожий функционал реализован у других.