Является ли $view в программе MVC php правильным использованием глобальных переменных?

В моей программе я создаю $view как пустой массив() в index.php и превращаю его в глобальный в контроллерах, которые его используют.

Почему? Если вашему контроллеру нужен $view, просто передайте его через конструктор или сеттер. Зависимости от глобальных переменных всегда можно устранить с помощью внедрения зависимостей.

// index.php
$view = array();
$controller = new Controller($view);
$controller->doAction();

Кроме того, подумайте, должно ли представление быть просто массивом. Представление должно быть отображено в какой-то момент. Рендеринг — это ответственность, которую я вижу в представлении. Массив не может отображать себя, поэтому я предполагаю, что вы делаете это в другом месте. Это было бы нарушением принципа единой ответственности. Я бы предпочел сделать что-то вроде (упрощенно):

// index.php
$view = new View;
$controller = new Controller($view);
$controller->doAction();

// controller.php
...
public function doAction()
{
    ...
    $this->view->setTemplate('/path/to/template');
    $this->view->setVar('foo', 'bar');
    $this->view->render();
}
...

Следует избегать глобальных переменных, true. Однако они полезны для хранения глобального состояния, что не приводит к взаимосвязи, как, например, в массиве/объекте $config.

Но в вашем случае я не уверен, что $view - хорошая идея. Во-первых, я считаю, что вы собираете выходные данные. «Представление» в MVC - это скорее выходной шаблон, каким, я полагаю, является ваш index.php? Если это так, то вы на самом деле собираете $model. Неопределенно.
Теоретически лучше явно передать эту переменную контроллерам. Однако, если на самом деле это всего лишь одна глобальная переменная, и вы обещаете не переусердствовать с ней, можно передавать ее как глобальную.

Но вы также можете следовать упомянутому одноэлементному шаблону. Однако я бы рекомендовал процедурный вариант, потому что он чище:

function view() {
    static $view;
    if (!isset($view)) { $view = new ArrayObject(); }
    return $view;
}

Таким образом, вы можете использовать $view=view(); для его локализации в своих контроллерах, продолжая использовать его как массив $view["xy"]= и как объект $view->xy= одновременно.
Или просто напишите print view()->title; в шаблоне.

Делать глобальную переменную не очень хорошая идея, если вы используете шаблоны MVC в проекте. Есть и другие решения, которые будут более разумно использовать ваш MVC.

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

class My_View {

   private $_instance;

   static public function getInstance() {
     if (null === self::$_instance) {
        self::$_instance = new self();
     }
     return self::$_instance;
   }

   private function __construct() { }

   public function __clone() {
      trigger_error("Cannot make copy of this object", E_USER_ERROR);
   }

   // add public methods and/or properties here...
}

$view = My_View::getInstance();

В начальной загрузке вы объявляете свою функцию автозагрузки (как определено в руководство по PHP):

// My_Application.php located in /path/to/lib/My/Application.php
class My_View {

   private $_instance;

   static public function getInstance() {
     if (null === self::$_instance) {
        self::$_instance = new self();
     }
     return self::$_instance;
   }

   private $_basePath;
   public $view;

   private function __construct() { }

   public function __clone() {
      trigger_error("Cannot make copy of this object", E_USER_ERROR);
   }

   public function initAutoloader($base) {
      $this->_basePath = $base;
      spl_autoload_register(array($this, 'autoload'));
   }

   public function autoload($name) {
      require $this->_basePath . str_replace('_', '/', $name) . '.php';
   }

   // get the application global view
   public function getView() {
      if (null === $this->view) {
         $this->view = new My_View();
      }
      return $this->view;
   }
}
// perhaps have the parameter being a constant. I.g. APP_BASE_PATH
My_Application::getInstance()->initAutoload('/path/to/lib/');

И просто включите файл '/path/to/lib/My/Application.php', и когда вы получите доступ к My_View, функция load будет вызвана с $name = 'My_View', и функция просто потребует от вас файл '/path/to/lib/My/View.php'. Это немного для одного файла, но если все классы являются такими пространствами имен, вам нужен только один include (Autoloader), а все остальное загружается автоматически.

Решение этой проблемы, которое я видел, заключается в расширении контроллеров от базового контроллера. Внутри этого базового контроллера создается экземпляр свойства $data; новые данные представления добавляются к этому свойству. Например:

class PageController extends BaseController {

    function IndexAction() {
        $this->data['title'] = "Page Title";
        $this->data['content'] = "<p>Page content.</p>";

        $this->render();
    }
}

Метод рендеринга (который может быть загружен автоматически после запуска метода, а не явно) затем собирает шаблон, используя данные, хранящиеся в массиве $data.

Надеюсь, это поможет, и вы сможете реализовать рабочее решение.