Notice: Undefined property: In creating new model using namespace

I am a newbie php coder. I have this main controller:

namespace AppCore;

class Controller

/** @var View View The view object */
public $View;
public $templates;
/**
* Construct the (base) controller. This happens when a real controller is constructed, like in
*/
public function __construct()



public function loadModel()

$this->model = new AppFrontModelIndexModel(); //error line





In IndexController I have:

namespace AppFrontController;

use AppFrontModelIndexModel;

class IndexController extends AppCoreController

public function index()
$this->loadModel->test();




In IndexModel I have:

namespace AppFrontModel;

class IndexModel


public function test()

echo 'test print';





In Action I get this error:

Notice: Undefined property: AppFrontControllerIndexController::$loadModel in /Applications/xampp/htdocs/cmstest/application/Front/Controller/IndexController.php on line 13

I load all classes using composer and PDR-4 method.

What is the problem and how do I fix it? Thanks

Controller::loadModel

$this->loadModel->test();

$this->loadModel()->test();

Fatal error: Uncaught Error: Call to a member function test() on null in /Applications/xammp/htdocs/cmstest/application/Front/Controller/IndexController.php:13

loadModel()

IndexModel

$this->model

public function loadModel() $this->model = new AppFrontModelIndexModel(); return $this->model

NULL

1 Answer
1

Notice that in your loadModel method you just assign the new model to this but you not returning anything -> so you cannot can function test() on null.

loadModel

this

test()

In order to fix it use:

class IndexController extends AppCoreController
public function index()
$this->loadModel();
$this->model->test();




If you insist on doing in index only one function you can modify your loadModel function to:

index

loadModel

public function loadModel()
if (!$this->model) // that way you load the Model only once. If you want to reload every time just remove the if
$this->model = new AppFrontModelIndexModel();
return $this->model;



And then you can do:

public function index()
$this->loadModel()->test();



IndexController

$this->loadModel(); $this->model->test();

Thanks for contributing an answer to Stack Overflow!

But avoid …

To learn more, see our tips on writing great answers.

Required, but never shown

Required, but never shown

By clicking "Post Your Answer", you acknowledge that you have read our updated terms of service, privacy policy and cookie policy, and that your continued use of the website is subject to these policies.