Как правильно использовать методы и атрибуты классов PHP?

Question

[Karponter]

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

<?php
	class ClassName
	{
		var $myTestVar = 0;

		function readVar() {
			echo $this->$myTestVar;
		}
	}

	$a = new ClassName();
	$a->readVar();
?>

Валидатор говорит :

PHP Notice: Undefined variable: myTestVar in ~/.../test.php on line 13
PHP Fatal error: Cannot access empty property in ~/.../test.php on line 13

Проблема в коде или с валидатором?

update...

<?php
	class ClassName
	{
		public $myTestVar;

		public function __construct() 
		{
			$this->myTestVar  = 0;
		}

		function readVar() {
			echo $this->$myTestVar;
		}
	}

	$a = new ClassName();
	$a->readVar();
?>

Исход тот-же.

Answer 1

[vdem]

Лишняя $, вместо:
echo $this->$myTestVar;
надо
echo $this->myTestVar;

Comments

[Karponter]

Вот оно как xD
Спасибо большое.

[vdem]

@Karponter Проблема была в том, что обращаться к свойствам объекта нужно без $ перед именем свойства, т.к. $ уже стоит перед this. А в Вашем случае интерпретатор искал локальную переменную $myTestVar в методе readVar() для того, чтобы подставить ее значение в выражение $this->$myTestVar в качестве имени переменной. Т.е. такую конструкцию можно применять, чтобы обращаться к свойству, имя которого записано в переменной.

[Karponter]

@vdem Кстати мне этот маневр пригодится. Но не скажется ли такой вызов на безопасности, если я буду использовать параметры, переданные по GET?

[vdem]

@Karponter Обязательно скажется. Всё, что приходит в программу от пользователя - проверять от и до.

Answer 2

[Push Pull]

Начали изучать ООП модель, не пишите поля класса в старом стиле
var $myTestVar = 0;
пишите как надо
public|private|protected $myTestVar;
или же сделайте статическое поле, если надо.

вот это же - var $myTestVar = 0; уже делает ее статической, поэтому $this->myTestVar пустует
Если есть желание указывать значения по умолчанию, пишите в конструкторе

public function __construct() 
{
    $this->myTestVar  = 0;
}

Comments

[vdem]

"уже делает ее статической" - каким образом?

[Karponter]

Написал демо-код специально для темы в тостере.
public $myTestVar; и инициализация конструктором проблему с места не сдвигают. По сути проблему и обнаружил при таком раскладе.

[Push Pull]

@vdem магическим, каким же еще

[vdem]

@deadbyelpy Насколько мне известно, статические поля в PHP объявляются с использованием ключевого слова static. Вот и спросил, может я чего-то не знаю.

[Push Pull]

@vdem описание переменной в стиле var $asd - делает поле класса и обычным полем и статическим, т.к. php позволяет вызывать его в любом контексте, правда с ругательствами. тоже самое и про function func() внутри класса.

[vdem]

@deadbyelpy Создал класс A с полем $myVar, объявленным через var. Пытаюсь обратиться как к статическому: A::$myVar, получаю fatal error: "Fatal error: Access to undeclared static property: A::$myVar"

[vdem]

@deadbyelpy Fatal error - это не просто "ругательство", ошибка этого уровня прерывает работу скрипта.

[vdem]

@deadbyelpy Вероятно вы перепутали свойства/методы класса (статические) с приватными/публичными. В PHP да, свойство или метод, объявленные без модификатора доступа являются публичными по умолчанию (для совместимости с PHP4), в Java такие члены класса считаются приватными по умолчанию.

Answer 3

[Сергей Протько]

Потрудитесь для начала почитать документацию. И именно документацию а не статьи васи пупкина. Там даже на русском переведено (то что про ООП), и там очень даже неплохо раскрыты основы. Так же советую подключить википедию и почитать про "инкапсуляцию".