13

I've done lots and lots of code in PHP that is object-oriented, but up until now, all of my classes have been, "singular", I suppose you can call it. I am in the process of changing several classes (that have 5 or so identical methods) to extend one class (to rid myself of duplicate code). I am running into a few issues.

I am trying to access a method in a parent class, but you can see the issue.

Parent class:

 class DatabaseObject { 

     public static function find_all() {
        return self::find_by_sql("SELECT * FROM " . self::$table_name);
    }
}

Child Class:

class Topics extends DatabaseObject {

    protected static $table_name = "master_cat";
    protected static $db_fields = array('cat_id', 'category');
    public $cat_id;
    public $category;

  }

Code trying to access all info from this table from php/html file:

$topics=Topics::find_all();

foreach($topics as $topic):
    echo $topic->category;
endforeach; 

As you can see, Most of the code has not been merged to the new way of doing things. I need to change the self::$table_name which no longer works in the new way I am doing things. I will have about 5 Classes extending this object, so what is the best way of coding this so I can access different tables with one method (rather than including this exact find_all() method in 5 different classes.

9
  • I recommend that you read more about OOP and especially the way it's implemented in PHP. Giving you a working code won't solve the real problem.
    – Adi
    Jun 25, 2012 at 17:49
  • 25
    On the contrary, I learn by other people's code. Perhaps I will wait for someone to actually answer the question now. Jun 25, 2012 at 17:52
  • Ive read "I learn from other peoples code" too often to know, that it doesn't work well. You should consider reading a book.
    – KingCrunch
    Jun 25, 2012 at 17:58
  • 1
    Linkspam: one, two, three, four
    – jedwards
    Jun 25, 2012 at 17:59
  • 4
    I think there was nothing wrong with the question, just not no one had an answer. But thanks to @Matthijs we are enlightened
    – Ibu
    Jul 12, 2012 at 18:15

2 Answers 2

32

You could try late static binding as mentioned below, or a singleton solution should work as well:

<?php
abstract class DatabaseObject {
  private $table;
  private $fields;

  protected function __construct($table, $fields) {
    $this->table = $table;
    $this->fields = $fields;
  }

  public function find_all() {
    return $this->find_by_sql('SELECT * FROM ' . $this->table);
  }
}

class Topics extends DatabaseObject {
  private static $instance;

  public static function get_instance() {
    if (!isset(self::$instance)) {
      self::$instance = new Topics('master_cat', array('cat_id', 'category'));
    }

    return self::$instance;
  }
}

Topics::get_instance()->find_all();
1
  • 17
    +1 for teaching us, and making every (go read the book) people shut up
    – Ibu
    Jul 12, 2012 at 18:13
5

You should look into the concept of Late Static Binding in PHP. This allows you to access a static constant or function from the class that was called.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.