Structure of PHP class, use static methods or not?

Question

I am writing a custom domain-management class which uses various external services, sort of a wrapper class to make them work together.

Where abouts would I put the connection logic in this class? I won't need all the services at once so it doesn't make sense to put it in the constructor, I'm actually thinking some of the methods will be better off as static methods as they don't really relate to each other, the only thing they have in common is the underlying connections.

I am going to have methods along the lines of:

registerDomain() (contacts Nominet)
updateDomain(),
domainAvailable(), (contacts Nominet)
registerDNS(), (contacts Amazon & Nominet)
updateDNS(),

Should I check for a connection property in each call (and create it if non-existent) or connect in the classes constructor?

Idea: The common denominator of your methods could be set up in the wrapper class construtor. For the other methods, you could extend the wrapper class or build separate classes and pass them only what they need to work. — hornetbzz
– hornetbzz, Commented Jul 20, 2011 at 9:58

NDM · Accepted Answer · 2011-07-20 10:01:19Z

2

I think it would make sense to create somthing along the lines of this:

class DomainManager {
    public function __construct($domainData) {}
    public function registerDomain() {
        //connect
        //do stuff
    }
    public function updateDomain() {
        //connect
        //do stuff
    }
    public function isAvailable() {
        //connect
        //do stuff
    }
    public function registerDns() {
        //connect
        //do stuff
    }
    public function updateDns() {
        //connect
        //do stuff
    }
    private function connectToNominet() {}
    private function connectToAmazon() {}
}

Then you have a nice object that encapsulates the logic available for a domain:

$domain1 = new DomainManager('example.com', $user, $foo);
$domain->registerDomain();
$domain->registerDns();

answered Jul 20, 2011 at 10:01

NDM

6,8383 gold badges42 silver badges54 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Dunhamzzz Over a year ago

Cool, would you store the connection in a property and something like this when it's needed: if(!$this->nominet) { $this->connectToNominet() } ? Or should connectToNominet return the connection?

NDM Over a year ago

Depends on the type of connection, if it is stateless like a regular ajax request it won't be needed. If it is a more permanent connection however, like you would make to a database server it would make sence to store the connection. The question you need to pose yourself is: can I do multiple request to the connection without having to reconnect?

hakre Over a year ago

Probably "DomainController" is a better name for the class than Manager.

genesis Over a year ago

@Nicky: This is wrong mate. You're defining $domain1 but using $domain ;)

Collectives™ on Stack Overflow

Structure of PHP class, use static methods or not?

1 Answer 1

4 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Related