Правильно ли я использую класс? Парни, тренируюсь в написание простых классов. Недавно была задача написать генератор отчетов. Вот я и сделал. Посмотрите пожалуйста. Класс сильно сокращен, чтобы была понятна только суть.
1. "Круто" ли он написан и нет ли в нем ошибок?)
2. Что еще с ним можно сделать, чтоб дальше вникнуть в ООП?class Otchet
{
private $name;
private $boss;
private $directors = array();
public function __construct($name)
{
$this->name = $name;
}

public function setBoss($name)
{
$this->boss = $name;
}

public function replaceBoss($name)
{
if(!empty($boss))
{
$this->boss = $boss;
}
}
public function addDirector($name)
{
$this->directors[] = $name;

$this->directors = array_unique($this->directors);
}
public function delDirector($name)
{
if(($key = array_search($name, $this->directors)) !== false)
{
unset($this->directors[$key]);
}
}

private function generate_otchet_name()
{
return date('Y-m-d-H-i').'.txt';
}

public function save($filename = '')
{
if(empty($filename))
{
$filename = $this->generate_otchet_name();
}

$content = "Название отчета: ".$this->name."\n";
$content .= "Директор: ".$this->boss."\n";
if(count($this->directors) > 0)
{
$content .= "Зам директора: ".implode(', ', $this->directors);
}

if(file_put_contents($filename, $content))
{
return 'Отчет сохранен';
}
else
{
return 'Ошибка сохранения';
}
}
}
if(isset($_POST['name'], $_POST['boss'], $_POST['director']))
{
$f = new Otchet($_POST['name']);
$f->setBoss($_POST['boss']);

if(is_array($_POST['director']) && count($_POST['director']) > 0)
{
foreach($_POST['director'] as $v)
{
$f->addDirector($v);
}
}

echo $f->save();
}

Предыдущий
вопрос Следующий
вопрос