Хочу обратить Ваше внимание на один нюанс, который называется temporal coupling. Многие (большинство? все?) методы Вашего класса зависят от результата работы метода f1()
. Вызов f2()
, должен предваряться вызовом f1()
для корректной работы приложения. Т.е. другим разработчикам, которые будут работать с Вашим классом, нужно постоянно держать в уме эту деталь реализации, которая, к слову, никак не следует из интерфейса. Т.е. разработчик придет к пониманию этого порядка либо прочитав документацию, либо путем проб и ошибок, либо покопавшись в исходном коде Вашего класса. Это все признаки плохого дизайна класса. Ведь получается, что экземпляр класса после своего создания не сконфигурирован для выполнения методов f2()
, f3()
и т.д. Трудно что-то сказать в отрыве от конкретной задачи, но я бы сначала ответил бы на ряд вопросов:
- Имеет ли смысл существование экземпляра класса без вызова
f1()
?
- Имеет ли смысл вызов
f2()
вне контекста состояния объекта (обрабатывая скажем скалярное значение на входе)?
- Являются ли методы
f2()
, f3()
etc. обязанностью этого класса? Или может метод f1()
должен возвращать должным образом сконфигурированный объект другого класса, который уже сможет обрабатывать f2()
и f3()
?
- Имеет ли для системы ценность результат работы
f1()
вне контекста состояния объекта?
Возможно будет разумно уже в конструкторе ожидать аргумент для f1()
, и записывать в поле объекта результат, гарантируя, что при вызове других методов объект будет сконфигурирован. Может все таки стоит провести декомпозицию класса, разграничивая обязанности, может это сделает систему гибче и более расширяемой. Скорее всего для вашей задачи это оверкил, но советую проводить подобного рода анализ всегда, когда сталкиваетесь с подобными проблемами - это типичный code smell, признак плохого дизайна, не придать этому значения при разработке важного системообразующего модуля - с высокой вероятностью увеличить грядущие издержки на поддержку системы.
$data
и результат первой функции складируете туда$this->data = 123765123651237;
; а во второй пользуетесь этой переменной – Алексей Шиманский 2 мая '16 в 14:22$data; $this->data = $this->f1();
– Ислам 2 мая '16 в 14:26private $data = null;
........ первая функцияfunction f1(){ $v; //some code $this->data = $v; }
.....втораяfunction f2(){ echo $this->data; }
.......а все остальное без изменений$obj = new example; $obj -> f1(); $obj -> f2();
.........ideone.com/JtG4jq ....... а если вам нужнодля каждой f2() нужен результат выполнение f1()
то тогда можно делать как cheops сказал. Это правильнее будет – Алексей Шиманский 2 мая '16 в 14:30