PHP 扩展包实战教程 - 作品分享

很棒哦！刚才看了一下代码，设计还挺不错，不过 “A book SDK.” 有点太简单了，尽量加上关键字以便于更多需要的人能找到它。

另外，关于开源可以参考我之前的演讲内容：博客：手把手教你如何构建一个优秀的开源项目

加油！:+1:

@overtrue 好的 谢谢

$params = \json_decode($params, true); 为什么要加\，感觉没必要

class BookFactory
{
    /**
     * make a book entity.
     *
     * @param $params
     *
     * @return Book|null
     */
    public static function make($params): ? Book
    {
        $params = \json_decode($params, true);
        if (!isset($params['title'])) {
            return null;
        }
        $book = new Book();

        return $book->setIsbn($params['isbn13'])
            ->setTitle($params['title'])
            ->setSubtitle($params['subtitle'])
            ->setAuthor($params['author'])
            ->setAuthorIntro($params['author_intro'])
            ->setPrice($params['price'])
            ->setCatalog($params['catalog'])
            ->setPublicationDate($params['pubdate'])
            ->setPublisher($params['publisher'])
            ->setSummary($params['summary'])
            ->setCover($params['images']['large'])
            ->setTags(array_column($params['tags'], 'name'));
    }
}

@lovecn 这里有相关的讨论，建议加 问答：\in_array (\strtolower ($format), ['xml', 'JSON']) 函数前面为什...

如果您想在方法中返回 null，您可以考虑抛出一个异常或返回一个特例（Special Case）对象，返回 null 会对调用者增加额外的工作量，看了一下您调用 make() 的地方并没有去判断返回 null 的情况。。。

@不会飞的猪 谢谢建议