9 replies to this topic

[joe]

hai, now i learn PHP but with OOP style. And i have a problem :
source code :

<?
class Person
{
private $name;

function _construct($name)
{
$this->name=$name;
}

function getName()
{
return $this->name;
}
}

$judy = new Person("Judy") . "<br>";
$joe = new Person("Joe") ."<br>";

echo $judy->getName();
echo $joe->getName();
?>

error :
Fatal error: Call to a member function getName() on a non-object in
... on line 18

i use php version 5.1.0.

Please, correct me.

Thanx a lot...

[Mr. Matt]

well first off what error are you getting?

[Matthew.]

Read deadly, lol.

Fatal error: Call to a member function getName() on a non-object in
... on line 18

Use it like this to work:

<?
class Person
{
	private $name;

	function _construct($name)
	{
		$this->name = $name;
	}

	function getName()
	{
		return $this -> name;
	}
}

$cl = new Person();

$cl -> _construct("Judy");

echo $cl -> getName();
?>

btw. Doing...

$cl -> _construct("Name");

...again will not add a new name, it will simply overwrite the current variable value.

Edited by .Matt, 09 July 2006 - 11:48 AM.

[joe]

matt, i wanna ask u. Better i use OOP style for coding or use general style ( not use class and new....)

thanx...

[Matthew.]

OOP and functions etc kind of "constrain" coding ,makes it faster,neater and a lot less stressful on the coder lol.

I say use class based as the advantages are great. E.g. using a mysql class allows you to have constant error handling etc etc.

There are very few reasons why not to, thats what it comes down to really.

Edited by .Matt, 09 July 2006 - 12:09 PM.

[joe]

.Matt, on Jul 10 2006, 12:07 AM, said:

OOP and functions etc kind of "constrain" coding ,makes it faster,neater and a lot less stressful on the coder lol.

I say use class based as the advantages are great. E.g. using a mysql class allows you to have constant error handling etc etc.

There are very few reasons why not to, thats what it comes down to really.

yeah, we must learn alot of Java to know OOP style.
But, i agree with u.
Nice code, Thanx.

[rc69]

Three quick things i want to point out.

1. The reason for the error is that you set $judy and $joe as strings (since you added ."<br>" after declaring the object).

2. Matt is right in having to call the _construct function, but you could simplify things if you added another underscore before it.
http://php.net/manua....oop5.decon.php

class Person{
	private $name;

	function __construct($name){
		$this->name = $name;
	}

	function getName(){
		return $this -> name;
	}
}

$judy = new Person("Judy");
$joe = new Person("Joe");

echo $judy->getName()."<br>";
echo $joe->getName();

3. Calling the construct function of the same object will simply overwrite the name on that object. But, making a new object will make a new object, no harm, no foul. But, it would be wise to give your variables more general names (like how matt called the variable $cl, not $judy), that way, when you go to change something, you don't have to change every variable in order to still follow it.

[joe]

OK. i've clear about that. thanx RC69, Matt, and Deadly.

i've just learn OOP style...

[Mr. Matt]

lets just be honest I didn't help, i just didn't read.

[joe]

deadly, on Jul 10 2006, 03:22 PM, said:

lets just be honest I didn't help, i just didn't read.

it's ok !!