Создание глобальной переменной доступной для каждой функции внутри класса - PullRequest
13 голосов
/ 06 февраля 2009

У меня есть переменная в глобальной области видимости с именем ${SYSTEM}, где SYSTEM является определенной константой. У меня есть много классов с функциями, которым нужен доступ к этой переменной, и я нахожу раздражающим объявление global ${SYSTEM}; каждый раз.

Я попытался объявить переменную класса: public ${SYSTEM} = $GLOBALS[SYSTEM];, но это привело к синтаксической ошибке, что странно, потому что у меня есть другой класс, который объявляет переменные класса таким образом и, кажется, работает нормально. Единственное, о чем я могу думать, это то, что константа не распознается.

Мне удалось это осуществить с помощью конструктора, но я ищу более простое решение, прежде чем прибегнуть к этому.


EDIT Глобальная переменная $ {SYSTEM} - это массив с множеством других дочерних массивов. К сожалению, кажется, нет способа обойти использование конструктора ...

Ответы [ 7 ]

8 голосов
/ 06 февраля 2009

Хорошо, надеюсь, у меня есть суть того, чего вы пытаетесь достичь

<?php
    // the global array you want to access
    $GLOBALS['uname'] = array('kernel-name' => 'Linux', 'kernel-release' => '2.6.27-11-generic', 'machine' => 'i686');

    // the defined constant used to reference the global var
    define(_SYSTEM_, 'uname');

    class Foo {

        // a method where you'd liked to access the global var  
        public function bar() {
            print_r($this->{_SYSTEM_});
        }

        // the magic happens here using php5 overloading
        public function __get($d) {
            return $GLOBALS[$d];  
        }

    }

    $foo = new Foo;
    $foo->bar();

?>
8 голосов
/ 06 февраля 2009

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

class exampleGetInstance 
{

private static $instance;

public $value1;
public $value2;


private function initialize() 
{
    $this->value1 = 'test value';
    $this->value2 = 'test value2';

}

public function getInstance()
{
    if (!isset(self::$instance))
    {
        $class = __CLASS__;
        self::$instance = new $class();
        self::$instance->initialize();
    }
    return self::$instance;
}

}

$myInstance = exampleGetInstance::getInstance();

echo $myInstance->value1;

$myInstance теперь является ссылкой на экземпляр класса exampleGetInstance.

Исправлено форматирование

2 голосов
/ 06 февраля 2009

Вы можете использовать конструктор так:

class Myclass {
  public $classvar; 
  function Myclass() {
    $this->classvar = $GLOBALS[SYSTEM];
  }
}

РЕДАКТИРОВАТЬ: Спасибо за указание на опечатку, Питер!

Это работает и для массива. Если присвоение не требуется, взятие ссылки также работает:

$this->classvar =& $GLOBALS[SYSTEM];

EDIT2: следующий код был использован для тестирования этого метода, и он работал на моей системе:

<?php
define('MYCONST', 'varname');
$varname = array("This is varname", "and array?");

class Myclass {
  public $classvar;
  function Myclass() {
    $this->classvar =& $GLOBALS[MYCONST];
  }
  function printvar() {
    echo $this->classvar[0]; 
    echo $this->classvar[1];
  }
};

$myobj = new Myclass;
$myobj->printvar();
?>
1 голос
/ 06 февраля 2009

Вы пытаетесь сделать что-то действительно необычное здесь, так что вы можете ожидать, что это будет неловко. Работа с глобальными переменными никогда не бывает приятной, особенно если вы выбираете динамическое имя с помощью константы SYSTEM. Лично я бы порекомендовал вам вместо этого использовать $GLOBALS[SYSTEM] или ...

$sys = $GLOBALS[SYSTEM];

... если вы собираетесь использовать его много.

1 голос
/ 06 февраля 2009

Прямая спецификация переменных-членов не может содержать никаких ссылок на другие переменные (class {public $membervar = $outsidevar;} также недопустимо). Вместо этого используйте конструктор.

Однако, поскольку вы имеете дело с константой, почему бы вам не использовать константу php или константу класса средства?

0 голосов
/ 06 февраля 2009

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

<?php
class Singleton {

  // object instance
  private static $instance;

  // The protected construct prevents instantiating the class externally.  The construct can be
  // empty, or it can contain additional instructions...
  protected function __construct() {
    ...
  }

  // The clone and wakeup methods prevents external instantiation of copies of the Singleton class,
  // thus eliminating the possibility of duplicate objects.  The methods can be empty, or
  // can contain additional code (most probably generating error messages in response
  // to attempts to call).
  public function __clone() {
    trigger_error('Clone is not allowed.', E_USER_ERROR);
  }

  public function __wakeup() {
    trigger_error('Deserializing is not allowed.', E_USER_ERROR);
  }

  //This method must be static, and must return an instance of the object if the object
  //does not already exist.
  public static function getInstance() {
    if (!self::$instance instanceof self) { 
      self::$instance = new self;
    }
    return self::$instance;
  }

  //One or more public methods that grant access to the Singleton object, and its private
  //methods and properties via accessor methods.
  public function GetSystemVar() {
    ...
  }
}

//usage
Singleton::getInstance()->GetSystemVar();

?>

Этот пример немного изменен из википедии, но вы можете понять идею. Попробуйте прибегнуть к помощи шаблона синглтона для получения дополнительной информации

0 голосов
/ 06 февраля 2009

Я бы сказал, что первые две вещи, которые выделяются для меня:

  1. Вам не нужны скобки вокруг имени переменной, вы можете просто сделать public $ system или public $ SYSTEM.
  2. Хотя PHP может не всегда требовать этого, стандартной практикой является инкапсуляция индексов нечисловых массивов в одинарные или двойные кавычки на тот случай, если используемая вами строка в какой-то момент станет постоянной.

Это должно быть то, что вы ищете

class SomeClass {
  public $system = $GLOBALS['system'];
}

Вы также можете использовать константы класса, которые вместо этого будут

class SomeClass {
  const SYSTEM = $GLOBALS['system'];
}

На это можно ссылаться внутри класса с помощью «self :: SYSTEM» и извне с «SomeClass :: SYSTEM».

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