public function setBoss($name)
{
$this->boss = $name;
}
В данном случае, от этого метода - сеттера нет никакой практической пользы, он не проверяет ничего, не возвращает объект, чтобы можно было производить вызовы методов цепочкой $obj->a()->b(); В данном случае, целесообразнее использовать свойство напрямую.
Не нужно писать конструкции вот так:
if (...)
{
//
}
Пишем так:
if (...) {
//
}
Это не относится к классам, методам и функциям (в них скобку нужно писать на отдельной строке). Про это все, кстати, написано в PSR. Выберите стиль именования методов, чтобы он был один на протяжении всего кода, а то у вас в одном месте camelCase, в другом - under_scope.
if(file_put_contents($filename, $content))
{
return 'Отчет сохранен';
}
else
{
return 'Ошибка сохранения';
}
Не надо возвращать строку! Возвратите true или false. И запомните: если что-то не так, бросайте исключение!
