function return

[city_coder]

hi guys is it possible to do something like this??

class A {
   private $test;
   private $anotherTest;
   
   function getTest() {
      return $this->test;
   }

   function setTest($value = '') {
      $this->test = value;
   }

   function getAnotherTest() {
      return $this->anotherTest;
   }

   function setAnotherTest($value = '') {
      $this->anotherTest = value;
   }
}

class B {
    private $variable;

    function getVariable() {
       return $this->$variable;
    }

    function setVariable($value = '') {
       if($value != '') {
            $a = new A();
            $a->setAnotherTest($value);

            $return $a;
       } else {
            //do something else
       }
} 

$b = B::setVariable('text');
echo $b->getAnotherTest(); 

essentially i want to know whether i can pass a class in a return statement or you really just cant & it has to be as an array?

cheers in advance guys.

[djr33]

I'm not sure I understand exactly what you are trying to accomplish. Can you make a simpler example? There's a lot going on there that makes this confusing.
You should be able to return any variable regardless of its contents. Returning a variable just creates a copy of that variable in the other context... there should be no real problems.

The only thing that jumps out at me in your code is that you have a type: $return instead of return. "$return" is a variable named return, not a command.

[traq]

one other thing:
you are trying to call B::setVariable(), but it's not a static method. Choose one of two solutions:

<?php
$b = new B();
$b->setVariable();
?>

or

<?php
class B{
//  ...
static function setVariable(){
   //  ...
}
//  ...
}
?>

try that and see where it takes you. To answer your question ("can you return a class (actually, the object the class created)"), yes, but beyond being kinda interesting, I've rarely found it to be useful.

[djr33]

Also, since there are clearly some errors in this script, seeing those will help us. If you see no errors (blank screen?) then this means you need to turn on error reporting so you can diagnose the problems. There is information available by search for "php error reporting" and on php.net for the function error_reporting().

[djr33]

Also, since there are clearly some errors in this script, seeing those will help us. If you see no errors (blank screen?) then this means you need to turn on error reporting so you can diagnose the problems. There is information available by search for "php error reporting" and on php.net for the function error_reporting().

[city_coder]

Sorry for the confusing example guys....il make it more simple i promise :P

heres the scenario, i send a function in a class a parameter. The function then connects to the database (which as you probably noticed was my other thread about a database class, but thats besides the point) & returns a row from the table based on the data. So hopefully this is easier to understand....

(btw, this is not my code im using exactly, this is a cut down version)

class Customer {

   var $name;
   var $email;

function getEmail () {
   return $this->email;
}

function setEmail ($value = '') {
   $this->email = $value;
}

//etc etc for getting & setting

function readCustomer ($value = '') {
   //$value will be an email address
   $sql = "SELECT * FROM customer WHERE email = '$value'";
   $result = mysql_query();
   $customer = new Customer();
   if(//results greater than 0) {
      
      $customer->setEmail = //row from table
      $customer->setName = //row from table

   } else {
      $customer->setEmail = null;
      $customer->setName = null;
   }

   return $customer;
}

} 

then to access it....

$customer = new $customer::read('test@test.com');
echo $customer->getEmail();
echo $customer->getName(); 

The only reason i wanted to return a different type of object was cos i want to return the data in an object & a response message that i would then use to check whether any results came back & i could set error flags & messages etc.

I hope that explains things a little better.

Cheers guys

[traq]

again, yes, you can return an object. However, if you only need the data/messages/etc., it would be much simpler and more efficient to use an array. There's no reason to return an object unless you actually need to manipulate the object (e.g., you're going to use the object's functions) after it's returned.

also, your sample code above will still not work. If you want to use an object's methods outside the object (e.g., from your regular script), you need to declare them as static methods in the class definition. Otherwise, you won't be able to use them from the outside: you'll have to create a new object from the class and use that to access the methods.

(BTW, based on the way your code is written in your example, this:
$customer = new $customer::read('test@test.com');
should be something like this:
$customer = new customer();
$customer->readCustomer('test@test.com');
)

If this is one of your first attempts at using object-oriented programming, I'd suggest doing something much simpler to start. Find some tutorials you can understand and do lots of experimenting. It took me quite a while to figure out OOP; and it doesn't work quite the way you might expect it to.

Edit:

This is not correctly formed, either:

<?php
// ...
if(//results greater than 0) {
      
      $customer->setEmail = //row from table
      $customer->setName = //row from table

   } else {
      $customer->setEmail = null;
      $customer->setName = null;
   }
// ...
?>

$customer->setEmail (et.al.) are not variables; they're methods. You can't set them the way you set variables.
try something like:

<?php

if( /*  $results = results from DB*/ ){
   $customer->setEmail($results['email']);
   $customer->setName($results['name']);
}else{
   //  you don't have to "set" the values as null; they're already empty to start.
   //  setting another property might be more helpful:
   $customer->setFoundCustomer(FALSE);
   //  or even:
   //  return FALSE;
   //  instead of returning the customer object
}

?>

[city_coder]

OK, so its not my first attempt at using OOP. Iv been doing it in java for 3 months in SAP so I know how it works there. Iv just wanted to do this project in PHP because I know it better than moving to JSP & learning that more.

So you said it is possible to return an object but you didnt say how, which is what my question was.

Also you didnt give any examples of how that code was structured in the class & what was set.

If you can give me some examples please? That would be a great help!

Cheers.

[traq]

well, PHP is still different than Java. if you're already familiar with OOP, then you're ahead of the game; perhaps this would be helpful with sorting out php-specific syntax and constructs.

about examples: sorry - I thought you were just asking if what you had would work and how it would work. Aside from the typos and preceding syntax issues, the "return" part of your code was fine. An object is returned the same way anything else is:

<?php

function return_object(){
   $object = new myClass();
   return $object;
}

?>

To declare an object as static (so you can access it like $var = myClass::static_method(); ), just put the static keyword in front of it in the class definition:

<?php

myClass{

   static function static_method($var){
      //  do stuff
   }

}

?>

[city_coder]

Sorry its taken so long to reply...

Cheers for the response. So just to double check this now, if i do...

class Customer {
   private $fName;
   private $lName;
   private $errorOccurred;
   private $errorText;

   function setLName($value = '') {
      $this->lName = $value;
   }

   function 

   function readCustomer ($email = '') {
      $sql ="SELECT * FROM customer WHERE email = '$email'";
      $customer = new Customer();
      if result then {
         $customer->setLName($result['lName']);
         //etc
      } else {
           //set flag same as setting lName
      }

      return $customer;
   }
} 

then to read it

$customer = new Customer();
$customer->readCustomer('test@test.com');

if($customer->getErrorOccurred()) {
   //error couldnt get user details from db
} else {
   //great, get other customer details
   echo $customer->getLName();
} 

right? or wrong?

also could i do this?

$customer->Customer::readCustomer('test@test.com'); 

or does that mean im doing things differently?

cheers for the help guys, i know im being stupid.