Как из одной функции передать значение в другую?

Question

[Дмитрий]

Код:

<?php
class Page_Portfolio extends Page_Basic
{
    function actionIndex() {
        $cms_class = cms_class::getInstance();
        $sql =  "SELECT * FROM site_portfoliotheme WHERE portfoliotheme_active='Y'";
        $menu = $cms_class->getRecords($sql, 3);
        
        $sql = "SELECT * FROM site_portfolio WHERE portfolio_active='Y' ORDER BY portfolio_id DESC";
        $viewportf = $cms_class->getRecords($sql, 150, "portfolio_image");
        
        return array("menu"=>$menu, "viewportf"=>$viewportf);
    }
    function actionHtml()
    {
        $cms_class = cms_class::getInstance();
        $sql = "SELECT * FROM site_portfolio WHERE portfolio_active='Y' AND portfolio_theme='".Router::$pageParams["html"]."'";    
	$records = $cms_class->getRecords($sql, 150, "portfolio_image");
        return array("records"=>$records, "menu"=>$menu);
        }
    }

Нужно передать из первой функции значение переменной $menu во вторую функцию, дабы там ее вернуть в массиве. Как это провернуть? Или же есть иной путь?

Answer 1

[Александр Литвинов]

можно создать свойство $menu и обращаться через $this

<?php
class Page_Portfolio extends Page_Basic{

    private $menu;

    public function actionIndex() {
        $this->menu;
    }

    public function actionHtml(){
         $this->menu;
    }

}

Или, быть может, нужно что-то подобное

<?php
class Page_Portfolio extends Page_Basic{

    public function actionIndex() {

    }

    public function actionHtml(){
         $this->actionIndex();
    }

}

Comments

[Дмитрий]

Спасибо

Answer 2

[1001001 111]

Можно использовать сессию, переменную как написал @Sander_Li , или просто вызвать ф-ию

$menu = $this->actionIndex();
$menu = $menu['menu'];

Comments

[Дмитрий]

Да, вызвать функцию. Об этом не подумал. Спасибо.

[Александр Литвинов]

@another_dream Сессию не нужно использовать в вашей задаче, как и глобальные переменные.

[Дмитрий]

@Sander_Li Да, тоже так подумал. Поэтому вызвал функцию как показал выше @IgorO2

Answer 3

[Станислав Маркин]

Если ближе к ООП, то вот так, пожалуй:

<?php
<?php
class Page_Portfolio extends Page_Basic
{
    function actionIndex() {
        $sql = "SELECT * FROM site_portfolio WHERE portfolio_active='Y' ORDER BY portfolio_id DESC";
        $viewportf = cms_class::getInstance()->getRecords($sql, 150, "portfolio_image");
        return array("menu"=>$this->getMenu(), "viewportf"=>$viewportf);
    }
	
	function getMenu() {
		$cms_class = cms_class::getInstance();
		return $cms_class->getRecords($sql, 3);
	}
	
    function actionHtml()    {
        $cms_class = cms_class::getInstance();
        $sql = "SELECT * FROM site_portfolio WHERE portfolio_active='Y' AND portfolio_theme='".Router::$pageParams["html"]."'";    
		$records = $cms_class->getRecords($sql, 150, "portfolio_image");
        return array("records"=>$records, "menu"=>$this->getMenu());
        }
    }

А уж если надо оптимизировать, то можно "одним движением руки" кэшировать getMenu()