PHP5 - Valid Class?

Does this look like a valid class? I mean if it were to be a part of a program, does it look ok?

class pSelector {

Public $pType;
$error = 'Please input a price.';

if(this->price > 2000) {

$pType = 'House';
echo(this->$pType);

}

else if(this->price < 2001) {

$pType = 'Apartment';
echo(this->$pType);
}

else {

echo($error);

}
}

No, you can't have code in the middle of a class, it needs to be inside a function.

Thanks, this better?

class pSelector {

Public $pType;
$error = 'Please input a price.';

function $pselect(){
if(this->price > 2000) {

$pType = 'House';
echo(this->$pType);

}

else if(this->price < 2001) {

$pType = 'Apartment';
echo(this->$pType);
}

else {

echo($error);

}
}

$pselect();
}

Closer but still wrong.

1. Function names do not use a $ in front, only variables.
2. this->price should have a $ in front - $this->price
3. $this->price actually needs to be declared. It may work depending on your error settings.
4. All internal class variables must be referenced by $this-> so you can't use $error as you have there, it should be $this->error
5. Again... You can't have code outside of functions, so your call to $pselect() is not valid (and remember to remove the $ in front of the function name).
6. Also, internal function names are referenced using $this-> as well, so it would be $this->pselect();

That's all that I can see for now.

Oh, and you should use PHP tags to wrap your code in when you paste in on here. That way it will keep the indenting and make it a lot more readable.

Thanks, ok - the class is is set to figure the type of rental it is likely to be based on price. $obj->price = '2000'; is set earlier on in the code. I appreciate the help.

Ok, what you've still missed:

1. You never actually create an object so you can't call any of it's methods.
2. You're still using $error when it should be $this->error
3. You're still using $this->$ptype when it should be $this->ptype
4. You never set up $price anywhere.
   e
5. elseif() should be a single word, no spaces. It still works as else if() but that's not the current way to do it.

Are you working on an editor with syntax highlighting and code "inteligence"? I don't believe so because if you were 99% of the problems that you're going through now would be shown straight away. I'd suggest looking at an editor that can do this for you. My editor of choice is Komodo Edit (the free version) and it does all of this for you.

As a valid example of what you are looking for:

Hi DAR1,

I think it might be a good idea to take some time and actually learn the syntax and logic of object orientation in PHP.

I mean, sure, you can probably put together some classes through trial and error and help from others. But you'll have much better results and be much faster if you actually know what you're doing.

There's even an OOP tutorial at php.net.

Do I need to define $price still?

Trust me Jacquez, in 5 min at this forum I've learned more than if I were to read a long *** tutorial. I learn differently.

Fair enough, there are some type of people who learn better that way. But are you learning or are you following directions? Do you understand what not just what they're telling you but why things are the way they are?

Yes, I learned that the object identifier $this-> should be used to identify which object we are utilizing. I learned that classes can utilize functions and are not to be mistaken with functions. I learned that else if and elseif are used differently.

I am still curious about whether I need to define $price since is referenced as $obj->price and set. I do not want to change the price.

In the example that you gave above, $price is set in the Property object type, so this means that the pSelect object doesn't need $price to be set.

BUT... How does the pSelect object know what the price is when it doesn't have any reference to the Property object where the price is actually set? You need to pas in the Property object to the pSelect object somewhere so that it knows what it's supposed to be looking at. As a suggestion...

Then in the pSelect object...

Thanks, I appreciate the help. I understand now. I redid the class here, if you don't mind taking a look and letting me know what you think. I am curious to know what you think about the way I structured the class and constructor.