2019-05-03 12:29
浏览 281


my query won't show a timestamp if i call the function getDate_created();. how can i fix this? The function getAll() does work.


class Post
    public $date_created;

    public function getDate_created()
        return $this->date_created;

    public function setDate_created($date_created)
        $this->date_created = $date_created;

        return $this;

    public static function getAll()
        $conn = Db::getInstance();
        $result = $conn->query('SELECT * FROM images');

        return $result->fetchAll(PDO::FETCH_CLASS, __CLASS__);

$posts = Post::getAll();

$timepost = new Post();
$t = $timepost->getDate_created();

the table images:

CREATE TABLE `images` (
   `id` int(11) NOT NULL,
   `user_id` int(11) NOT NULL,
   `img_description` varchar(255) NOT NULL,
   `date_created` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
   `file_path` varchar(255) NOT NULL

Var_dump($t) shows NULL as result.

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 邀请回答

2条回答 默认 最新

  • douxun1934 2019-05-03 12:50

    You are creating a new instance of Post when you do $timepost = new Post();

    If you want to read one of those posts from database you should read one (or more) from those that came from getAll() method.


    $posts = Post::getAll();
    foreach($posts as $post) {
    点赞 打赏 评论
  • duankui3838 2019-05-03 13:03

    There is no point of setting timestamp value to the attribute in this case because you are getting ALL records so it will get value of the last one (it will override every previous one). It will make sense to set value to $this->date_created attribute only when you are fetching ONE record from DB, if you want to do it that way. In general you just need to make sth like:

    $received_data = $result->fetch(PDO::FETCH_ASSOC);
    $this->date_created = $received_data['date_created'];

    This solution will work ONLY FOR SINGLE record in assoc array. If you really want to make it for all records, just use simple foreach() loop iterating over received data from query.

    If you need all of records from DB, you do not want to (trust me) set it to attribute (unless you want to have array of objects but you would need to rebuild quite a lot of your current code and it is not efficient).

    In your case I would recommend to just use returned array without setting values to attributes.

    Also, I don't recommend using getters and setters in this case, you can achieve the same effect by simply using $timepost->created_date (of course you need to set value to that attribute before).

    点赞 打赏 评论

相关推荐 更多相似问题