草庐IT

php - 帮助使用 DRY 原则在服务类中创建灵活的基础 'find' 方法

coder 2024-04-06 原文

多年来,我一直在一遍又一遍地(随着进化)重新实现相同的代码,却没有找到一些干净、有效地将其抽象出来的方法。

该模式是我的服务层中的基本“find[Type]s”方法,它将选择查询创建抽象到服务中的单个点,但支持快速创建更易于使用的代理方法的能力(参见示例 PostServivce::getPostById () 方法方法如下)。

不幸的是,到目前为止,我一直无法满足这些目标:

  • 减少由不同的重新实现引入的错误的可能性
  • 向 IDE 公开有效/无效的参数选项以进行自动完成
  • 遵循 DRY 原则

  • 我最近的实现通常类似于以下示例。该方法接受一组条件和一组选项,并从中创建并执行一个 Doctrine_Query(我今天主要在这里重写了这个,所以可能会有一些拼写错误/语法错误,它不是直接剪切和粘贴)。

    class PostService
    {
        /* ... */
    
        /**
         * Return a set of Posts
         *
         * @param Array $conditions Optional. An array of conditions in the format
         *                          array('condition1' => 'value', ...)
         * @param Array $options    Optional. An array of options 
         * @return Array An array of post objects or false if no matches for conditions
         */
        public function getPosts($conditions = array(), $options = array()) {
            $defaultOptions =  = array(
                'orderBy' => array('date_created' => 'DESC'),
                'paginate' => true,
                'hydrate' => 'array',
                'includeAuthor' => false,
                'includeCategories' => false,
            );
    
            $q = Doctrine_Query::create()
                            ->select('p.*')
                            ->from('Posts p');
    
            foreach($conditions as $condition => $value) {
                $not = false;
                $in = is_array($value);
                $null = is_null($value);                
    
                $operator = '=';
                // This part is particularly nasty :(
                // allow for conditions operator specification like
                //   'slug LIKE' => 'foo%',
                //   'comment_count >=' => 1,
                //   'approved NOT' => null,
                //   'id NOT IN' => array(...),
                if(false !== ($spacePos = strpos($conditions, ' '))) {
                    $operator = substr($condition, $spacePost+1);
                    $conditionStr = substr($condition, 0, $spacePos);
    
                    /* ... snip validate matched condition, throw exception ... */
                    if(substr($operatorStr, 0, 4) == 'NOT ') {
                      $not = true;
                      $operatorStr = substr($operatorStr, 4);
                    }
                    if($operatorStr == 'IN') {
                        $in = true;
                    } elseif($operatorStr == 'NOT') {
                        $not = true;
                    } else {
                        /* ... snip validate matched condition, throw exception ... */
                        $operator = $operatorStr;
                    }
    
                }
    
                switch($condition) {
                    // Joined table conditions
                    case 'Author.role':
                    case 'Author.id':
                        // hard set the inclusion of the author table
                        $options['includeAuthor'] = true;
    
                        // break; intentionally omitted
                    /* ... snip other similar cases with omitted breaks ... */
                        // allow the condition to fall through to logic below
    
                    // Model specific condition fields
                    case 'id': 
                    case 'title':
                    case 'body':
                    /* ... snip various valid conditions ... */
                        if($in) {
                            if($not) {
                                $q->andWhereNotIn("p.{$condition}", $value);
                            } else {
                                $q->andWhereIn("p.{$condition}", $value);
                            }
                        } elseif ($null) {
                            $q->andWhere("p.{$condition} IS " 
                                         . ($not ? 'NOT ' : '') 
                                         . " NULL");
                        } else {
                            $q->andWhere(
                                "p.{condition} {$operator} ?" 
                                    . ($operator == 'BETWEEN' ? ' AND ?' : ''),
                                $value
                            );
                        }
                        break;
                    default:
                        throw new Exception("Unknown condition '$condition'");
                }
            }
    
            // Process options
    
            // init some later processing flags
            $includeAuthor = $includeCategories = $paginate = false;
            foreach(array_merge_recursivce($detaultOptions, $options) as $option => $value) {
                switch($option) {
                    case 'includeAuthor':
                    case 'includeCategories':
                    case 'paginate':
                    /* ... snip ... */
                        $$option = (bool)$value;
                        break;
                    case 'limit':
                    case 'offset':
                    case 'orderBy':
                        $q->$option($value);
                        break;
                    case 'hydrate':
                        /* ... set a doctrine hydration mode into $hydration */ 
                        break;
                    default:
                        throw new Exception("Invalid option '$option'");
                }
            }
    
            // Manage some flags...
            if($includeAuthor) {
                $q->leftJoin('p.Authors a')
                  ->addSelect('a.*');
            } 
    
            if($paginate) {
                /* ... wrap query in some custom Doctrine Zend_Paginator class ... */
                return $paginator;
            }
    
            return $q->execute(array(), $hydration);
        }
    
        /* ... snip ... */
    }
    



    这个基函数的好处是:
  • 它使我能够随着模式的发展快速支持新的条件和选项
  • 它使我能够快速实现查询的全局条件(例如,添加一个默认为 true 的“excludeDisabled”选项,并过滤所有 disabled = 0 模型,除非调用者明确表示不同)。
  • 它允许我快速创建新的、更易于使用的方法,这些方法代理回调 findPosts 方法。例如:

  • class PostService
    {
        /* ... snip ... */
    
        // A proxy to getPosts that limits results to 1 and returns just that element
        public function getPost($conditions = array(), $options()) {
            $conditions['id'] = $id;
            $options['limit'] = 1;
            $options['paginate'] = false;
            $results = $this->getPosts($conditions, $options);
            if(!empty($results) AND is_array($results)) {
                return array_shift($results);
            }
            return false;
        }
    
        /* ... docblock ...*/       
        public function getPostById(int $id, $conditions = array(), $options()) {
            $conditions['id'] = $id;
            return $this->getPost($conditions, $options);
        }
    
        /* ... docblock ...*/
        public function getPostsByAuthorId(int $id, $conditions = array(), $options()) {
            $conditions['Author.id'] = $id;
            return $this->getPosts($conditions, $options);
        }
    
        /* ... snip ... */
    }
    

    专业 这种方法的缺点是:
  • 在每个模型访问服务中都创建了相同的整体“find[Model]s”方法,主要只是条件切换构造和基表名称发生了变化。
  • 没有简单的方法来执行 AND/OR 条件操作。所有条件都明确地进行了 AND 运算。
  • 引入了许多拼写错误的机会
  • 在基于约定的 API 中引入了许多中断的机会(例如,稍后的服务可能需要实现不同的语法约定来指定 orderBy 选项,这对于向后移植到所有以前的服务变得乏味)。
  • 违反 DRY 原则。
  • IDE 自动完成解析器隐藏了有效的条件和选项,选项和条件参数需要冗长的文档块解释来跟踪允许的选项。

  • 在过去的几天里,我试图为这个问题开发一个更面向对象的解决方案,但我觉得我正在开发一个太复杂的解决方案,这个解决方案过于僵化和限制使用。

    我正在努力的想法类似于以下内容(当前项目将是 Doctrine2 fyi,因此略有变化)...

    namespace Foo\Service;
    
    use Foo\Service\PostService\FindConditions; // extends a common \Foo\FindConditions abstract
    use Foo\FindConditions\Mapper\Dql as DqlConditionsMapper;
    
    use Foo\Service\PostService\FindOptions; // extends a common \Foo\FindOptions abstract
    use Foo\FindOptions\Mapper\Dql as DqlOptionsMapper;
    
    use \Doctrine\ORM\QueryBuilder;
    
    class PostService
    {
        /* ... snip ... */
        public function findUsers(FindConditions $conditions = null, FindOptions $options = null) {
    
            /* ... snip instantiate $q as a Doctrine\ORM\QueryBuilder ... */
    
            // Verbose
            $mapper = new DqlConditionsMapper();
            $q = $mapper
                    ->setQuery($q)
                    ->setConditions($conditions)
                    ->map();
    
            // Concise
            $optionsMapper = new DqlOptionsMapper($q);        
            $q = $optionsMapper->map($options);
    
    
            if($conditionsMapper->hasUnmappedConditions()) {
                /* .. very specific condition handling ... */
            }
            if($optionsMapper->hasUnmappedConditions()) {
                /* .. very specific condition handling ... */
            }
    
            if($conditions->paginate) {
                return new Some_Doctrine2_Zend_Paginator_Adapter($q);
            } else {
                return $q->execute();
            }
        }
    
        /* ... snip ... */
    }
    

    最后,Foo\Service\PostService\FindConditions 类的示例:

    namespace Foo\Service\PostService;
    
    use Foo\Options\FindConditions as FindConditionsAbstract;
    
    class FindConditions extends FindConditionsAbstract {
    
        protected $_allowedOptions = array(
            'user_id',
            'status',
            'Credentials.credential',
        );
    
        /* ... snip explicit get/sets for allowed options to provide ide autocompletion help */
    }
    

    Foo\Options\FindConditions 和 Foo\Options\FindOptions 非常相似,所以,至少现在,它们都扩展了一个共同的 Foo\Options 父类。这个父类处理初始化允许的变量和默认值,访问设置的选项,限制只访问定义的选项,并为 DqlOptionsMapper 提供一个迭代器接口(interface)来循环选项。

    不幸的是,经过几天的黑客攻击后,我对这个系统的复杂性感到沮丧。照原样,仍然不支持条件组和 OR 条件,并且在指定 FindConditions 时创建 Foo\Options\FindConditions\Comparison 类环绕一个值的能力指定备用条件比较运算符已经是一个完整的泥潭值 ( $conditions->setCondition('Foo', new Comparison('NOT LIKE', 'bar')); )。

    如果存在,我更愿意使用其他人的解决方案,但我还没有遇到任何可以满足我的需求的解决方案。

    我想超越这个过程并回到实际构建我正在处理的项目中,但我什至看不到尽头。

    所以,堆栈溢出者:
    - 有没有更好的方法可以提供我已经确定的好处而不包括缺点?

    最佳答案

    我认为你把事情复杂化了。

    我使用 Doctrine 2 开发了一个项目,它有很多实体,它们的不同用途,各种服务,自定义存储库等,我发现这样的东西效果很好(无论如何对我来说)。

    1. 用于查询的存储库

    首先,我通常不会在服务中进行查询。 Doctrine 2 提供了 EntityRepository 以及为此确切目的为每个实体对其进行子类化的选项。

  • 只要有可能,我就会使用标准的 findOneBy... 和 findBy... 风格的魔术方法。这使我不必自己编写 DQL,并且开箱即用。
  • 如果我需要更复杂的查询逻辑,我通常会在存储库中创建特定于用例的查找器。这些是诸如 UserRepository.findByNameStartsWith 之类的东西以及诸如此类的事情。
  • 我通常不会创造一个 super 花哨的“我可以接受你给我的任何参数!”类型的魔法探测器。如果我需要一个特定的查询,我会添加一个特定的方法。虽然这似乎需要您编写更多代码,但我认为这是一种更简单、更容易理解的做事方式。 (我尝试查看您的查找程序代码,但在某些地方查找相当复杂)

  • 所以换句话说...
  • 尝试使用教义已经给你的东西(魔法查找方法)
  • 如果您需要自定义查询逻辑,请使用自定义存储库类
  • 为每个查询类型创建一个方法

  • 2. 结合非实体逻辑的服务

    使用服务将“事务”组合在一个简单的界面后面,您可以从 Controller 使用或通过单元测试轻松测试。

    例如,假设您的用户可以添加 friend 。每当用户与其他人成为 friend 时,就会向其他人发送一封电子邮件以进行通知。这是您在服务中会拥有的东西。

    您的服务将(例如)包含一个方法 addNewFriend这需要两个用户。然后,它可以使用存储库来查询一些数据,更新用户的 friend 数组,并调用一些其他类然后发送电子邮件。

    您可以在服务中使用 entitymanager 来获取存储库类或持久化实体。

    3. 实体特定逻辑的实体

    最后,您应该尝试将特定于实体的业务逻辑直接放入实体类中。

    这种情况的一个简单示例可能是,在上述场景中发送的电子邮件可能使用某种问候语......“你好安德森先生”或“你好安德森女士”。

    例如,您需要一些逻辑来确定适当的问候语。这是您可以在实体类中拥有的东西 - 例如,getGreeting或其他东西,然后可以考虑用户的性别和国籍,并基于此返回一些内容。 (假设性别和国籍将存储在数据库中,而不是问候语本身 - 问候语将由函数的逻辑计算)

    我可能还应该指出,实体通常不应该知道实体管理器或存储库。如果逻辑需要其中任何一个,它可能不属于实体类本身。

    这种方法的好处

    我发现我在这里详述的方法效果很好。它是可维护的,因为它通常对事物的作用非常“明显”,它不依赖于复杂的查询行为,并且由于事物被清楚地划分为不同的“区域”(存储库、服务、实体),因此单元测试非常简单好。

    关于php - 帮助使用 DRY 原则在服务类中创建灵活的基础 'find' 方法,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/6629432/

    有关php - 帮助使用 DRY 原则在服务类中创建灵活的基础 'find' 方法的更多相关文章

    1. ruby - 如何使用 Nokogiri 的 xpath 和 at_xpath 方法 - 2

      我正在学习如何使用Nokogiri,根据这段代码我遇到了一些问题:require'rubygems'require'mechanize'post_agent=WWW::Mechanize.newpost_page=post_agent.get('http://www.vbulletin.org/forum/showthread.php?t=230708')puts"\nabsolutepathwithtbodygivesnil"putspost_page.parser.xpath('/html/body/div/div/div/div/div/table/tbody/tr/td/div

    2. ruby - 使用 RubyZip 生成 ZIP 文件时设置压缩级别 - 2

      我有一个Ruby程序,它使用rubyzip压缩XML文件的目录树。gem。我的问题是文件开始变得很重,我想提高压缩级别,因为压缩时间不是问题。我在rubyzipdocumentation中找不到一种为创建的ZIP文件指定压缩级别的方法。有人知道如何更改此设置吗?是否有另一个允许指定压缩级别的Ruby库? 最佳答案 这是我通过查看ruby​​zip内部创建的代码。level=Zlib::BEST_COMPRESSIONZip::ZipOutputStream.open(zip_file)do|zip|Dir.glob("**/*")d

    3. ruby - 为什么我可以在 Ruby 中使用 Object#send 访问私有(private)/ protected 方法? - 2

      类classAprivatedeffooputs:fooendpublicdefbarputs:barendprivatedefzimputs:zimendprotecteddefdibputs:dibendendA的实例a=A.new测试a.foorescueputs:faila.barrescueputs:faila.zimrescueputs:faila.dibrescueputs:faila.gazrescueputs:fail测试输出failbarfailfailfail.发送测试[:foo,:bar,:zim,:dib,:gaz].each{|m|a.send(m)resc

    4. ruby-on-rails - 使用 Ruby on Rails 进行自动化测试 - 最佳实践 - 2

      很好奇,就使用ruby​​onrails自动化单元测试而言,你们正在做什么?您是否创建了一个脚本来在cron中运行rake作业并将结果邮寄给您?git中的预提交Hook?只是手动调用?我完全理解测试,但想知道在错误发生之前捕获错误的最佳实践是什么。让我们理所当然地认为测试本身是完美无缺的,并且可以正常工作。下一步是什么以确保他们在正确的时间将可能有害的结果传达给您? 最佳答案 不确定您到底想听什么,但是有几个级别的自动代码库控制:在处理某项功能时,您可以使用类似autotest的内容获得关于哪些有效,哪些无效的即时反馈。要确保您的提

    5. ruby-on-rails - rails : keeping DRY with ActiveRecord models that share similar complex attributes - 2

      这似乎应该有一个直截了当的答案,但在Google上花了很多时间,所以我找不到它。这可能是缺少正确关键字的情况。在我的RoR应用程序中,我有几个模型共享一种特定类型的字符串属性,该属性具有特殊验证和其他功能。我能想到的最接近的类似示例是表示URL的字符串。这会导致模型中出现大量重复(甚至单元测试中会出现更多重复),但我不确定如何让它更DRY。我能想到几个可能的方向...按照“validates_url_format_of”插件,但这只会让验证干给这个特殊的字符串它自己的模型,但这看起来很像重溶液为这个特殊的字符串创建一个ruby​​类,但是我如何得到ActiveRecord关联这个类模型

    6. ruby - 在 Ruby 中使用匿名模块 - 2

      假设我做了一个模块如下:m=Module.newdoclassCendend三个问题:除了对m的引用之外,还有什么方法可以访问C和m中的其他内容?我可以在创建匿名模块后为其命名吗(就像我输入“module...”一样)?如何在使用完匿名模块后将其删除,使其定义的常量不再存在? 最佳答案 三个答案:是的,使用ObjectSpace.此代码使c引用你的类(class)C不引用m:c=nilObjectSpace.each_object{|obj|c=objif(Class===objandobj.name=~/::C$/)}当然这取决于

    7. ruby - 使用 ruby​​ 和 savon 的 SOAP 服务 - 2

      我正在尝试使用ruby​​和Savon来使用网络服务。测试服务为http://www.webservicex.net/WS/WSDetails.aspx?WSID=9&CATID=2require'rubygems'require'savon'client=Savon::Client.new"http://www.webservicex.net/stockquote.asmx?WSDL"client.get_quotedo|soap|soap.body={:symbol=>"AAPL"}end返回SOAP异常。检查soap信封,在我看来soap请求没有正确的命名空间。任何人都可以建议我

    8. python - 如何使用 Ruby 或 Python 创建一系列高音调和低音调的蜂鸣声? - 2

      关闭。这个问题是opinion-based.它目前不接受答案。想要改进这个问题?更新问题,以便editingthispost可以用事实和引用来回答它.关闭4年前。Improvethisquestion我想在固定时间创建一系列低音和高音调的哔哔声。例如:在150毫秒时发出高音调的蜂鸣声在151毫秒时发出低音调的蜂鸣声200毫秒时发出低音调的蜂鸣声250毫秒的高音调蜂鸣声有没有办法在Ruby或Python中做到这一点?我真的不在乎输出编码是什么(.wav、.mp3、.ogg等等),但我确实想创建一个输出文件。

    9. ruby-on-rails - rails : "missing partial" when calling 'render' in RSpec test - 2

      我正在尝试测试是否存在表单。我是Rails新手。我的new.html.erb_spec.rb文件的内容是:require'spec_helper'describe"messages/new.html.erb"doit"shouldrendertheform"dorender'/messages/new.html.erb'reponse.shouldhave_form_putting_to(@message)with_submit_buttonendendView本身,new.html.erb,有代码:当我运行rspec时,它失败了:1)messages/new.html.erbshou

    10. ruby-on-rails - 'compass watch' 是如何工作的/它是如何与 rails 一起使用的 - 2

      我在我的项目目录中完成了compasscreate.和compassinitrails。几个问题:我已将我的.sass文件放在public/stylesheets中。这是放置它们的正确位置吗?当我运行compasswatch时,它不会自动编译这些.sass文件。我必须手动指定文件:compasswatchpublic/stylesheets/myfile.sass等。如何让它自动运行?文件ie.css、print.css和screen.css已放在stylesheets/compiled。如何在编译后不让它们重新出现的情况下删除它们?我自己编译的.sass文件编译成compiled/t

    随机推荐