Здравствуйте. Для того что бы не передавать аргументы в каждый метод, я присваиваю значения полям в конструкторе. Потом эти переменные я использую внутри методов.

Правильно ли так делать? Или это неверный подход?

class Myclass{

private $a;
private $filepath;

function __construct()
{
     $this->a = $this->generate_a();
     $this->filepath = $this->gen_filapath();
}

function generate_a()
{
 // что то возвращаем
}

function gen_filepath()
{
 // что то возвращаем
}



function all()
{
 // используем $this->a и $this->filepath в методах этого класса
}

}

Всё зависит от задачи. Одно могу сказать, private - это зря. Лучше protected.

_____________
Если вам недостаточно собственных заблуждений, можно расширить их мнениями экспертов.

Нужно уважать мнение оппонета. Ведь заблуждаться - его святое право.

Настаивал, настаиваю и буду настаивать на своем. На кедровых орешках.

Если в методах тебе нужны будут уже инициализированные поля - да, все верно.

будет полезно, когда вычисляемое значение используется в нескольких местах, например, типичный геттер.
если одноразовое обращение, то смысла нет.

private $foo;

public function getFoo() {
  if(is_null($this->foo)) {
    $this->foo = ... // чет вычисляем, с базы читаем и т.д.
  }
   return $this->foo;
}

.....
  $bar = $obj->getFoo();
....
....
  $fooBar = $obj->getFoo(); // тут не будет повторного вычисления

_____________
[продано копирайтерам]

Как раз эти значения и используются во многих методах класса. Кажись я понемногу начинаю понимать все преимущества ООП)
Спасибо за ответы.

и High cohesion