jjzjj

java - 如何避免上课自用

coder 2024-03-08 原文

我有以下类(class):

public class MyClass
{

    public void deleteOrganization(Organization organization)
    {
        /*Delete organization*/

        /*Delete related users*/
        for (User user : organization.getUsers()) {
            deleteUser(user);
        }
    }

    public void deleteUser(User user)
    {
        /*Delete user logic*/
    }
}

此类表示一种自用,因为其公共(public)方法deleteOrganization使用其其他公共(public)方法deleteUser。在我的情况下,该类是旧代码,我开始在上面添加单元测试。因此,我首先针对第一种方法deleteOrganization添加了一个单元测试,最后确定该测试已扩展为也可以测试deleteUser方法。

问题

问题在于该测试不再孤立(它应该仅测试deleteOrganization方法)。为了通过它,我不得不处理与deleteUser方法相关的不同条件,以便通过测试,这极大地增加了测试的复杂性。

解决方案

解决方案是监视被测类和 stub deleteUser方法:
@Test
public void shouldDeleteOrganization()
{
    MyClass spy = spy(new MyClass());

    // avoid invoking the method
    doNothing().when(spy).deleteUser(any(User.class));

    // invoke method under test
    spy.deleteOrganization(new Organization());
}

新问题

尽管先前的解决方案解决了该问题,但不建议使用该方法,因为spy方法的javadoc指出:

As usual you are going to read the partial mock warning: Object oriented programming is more less tackling complexity by dividing the complexity into separate, specific, SRPy objects. How does partial mock fit into this paradigm? Well, it just doesn't... Partial mock usually means that the complexity has been moved to a different method on the same object. In most cases, this is not the way you want to design your application.


deleteOrganization方法的复杂性已移至deleteUser方法,这是由类的自用引起的。除了In most cases, this is not the way you want to design your application语句之外,不建议使用此解决方案的事实表明存在代码异味,确实需要重构来改进此代码。

如何消除这种自用?是否有可以应用的设计模式或重构技术?

最佳答案

类的自用使用并不一定是问题:我尚未确信“除其他个人或团队风格外,它仅应测试deleteOrganization方法”。尽管将deleteUserdeleteOrganization保持在独立的隔离单元中很有帮助,但这并不总是可行或实际的-特别是如果方法彼此调用或依赖于一个公共(public)状态。测试的重点是测试“最小的可测试单元”,它不需要独立的方法或可以独立测试的方法。

您可以根据自己的需要以及代码库的发展方式进行选择。其中两个来自您的问题,但我在下面重新介绍它们的优点。

  • 测试黑盒。

    如果将MyClass视为不透明的接口(interface),则可能不会期望或要求deleteOrganization重复调用deleteUser,并且可以想象实现会发生变化,而这样做不会。 (例如,将来的升级可能会使数据库触发器负责级联删除,或者单个文件删除或API调用可能会负责组织删除。)

    如果您将对deleteOrganizationdeleteUser调用视为私有(private)方法调用,那么您将只测试MyClass的契约(Contract)而不是其实现:创建具有某些用户的组织,调用该方法,并检查该组织是否消失以及用户太。它可能很冗长,但却是最正确,最灵活的测试。

    如果您希望MyClass发生巨大变化或获得全新的替代实现,那么这可能是一个有吸引力的选择。
  • 将类水平拆分为OrganizationDeleter和UserDeleter。

    正如Mockito文档所暗示的那样,为了使类更“SRPy”(即更好地符合Single Responsibility Principle),您会看到deleteUserdeleteOrganization是独立的。通过将它们分成两个不同的类,可以使OrganizationDeleter接受模拟UserDeleter,从而消除了对部分模拟的需求。

    如果您希望用户删除业务逻辑发生变化,或者希望编写另一个UserDeleter实现,那么这可能是一个有吸引力的选择。
  • 将类垂直拆分为MyClass和MyClassService/MyClassHelper。

    如果底层基础结构和高层调用之间有足够的差异,则可能需要将它们分开。 MyClass将保留deleteUserdeleteOrganization,执行所需的任何准备或验证步骤,然后对MyClassService中的原语进行一些调用。 deleteUser可能是一个简单的委托(delegate),而deleteOrganization可以在不调用其邻居的情况下调用该服务。

    如果您有足够的低级调用来保证这种额外的分离,或者MyClass处理高级和低级问题,这可能是一个有吸引力的选择,尤其是在您之前曾经进行过重构的情况下。但是,请小心避免使用baklava code模式(比您所能跟踪的层薄得多的可渗透层)。
  • 继续使用部分模拟。

    尽管这确实泄漏了内部调用的实现细节,并且确实违反了Mockito警告,但总的来说,部分模拟可以提供最佳的实用选择。如果您有一个方法多次调用其同级对象,那么内部方法是助手还是对等对象可能无法很好地定义,并且部分模拟可以使您不必制作该标签。

    如果您的代码有有效期限,或者具有足够的实验性,不能保证完整的设计,那么这可能是一个有吸引力的选择。

  • (旁注:的确,除非MyClass是最终版本,否则它可以进行子类化,这是部分模拟成为可能的一部分。如果要记录deleteOrganization的总契约(Contract)涉及对deleteUser的多次调用,那么这将是完全公平的创建子类或部分模拟的游戏。如果未记录,则为实现细节,应将其视为此类。)

    关于java - 如何避免上课自用,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/31456610/

    有关java - 如何避免上课自用的更多相关文章

    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 - 如何从 ruby​​ 中的字符串运行任意对象方法? - 2

      总的来说,我对ruby​​还比较陌生,我正在为我正在创建的对象编写一些rspec测试用例。许多测试用例都非常基础,我只是想确保正确填充和返回值。我想知道是否有办法使用循环结构来执行此操作。不必为我要测试的每个方法都设置一个assertEquals。例如:describeitem,"TestingtheItem"doit"willhaveanullvaluetostart"doitem=Item.new#HereIcoulddotheitem.name.shouldbe_nil#thenIcoulddoitem.category.shouldbe_nilendend但我想要一些方法来使用

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

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

    4. ruby-on-rails - 如何验证 update_all 是否实际在 Rails 中更新 - 2

      给定这段代码defcreate@upgrades=User.update_all(["role=?","upgraded"],:id=>params[:upgrade])redirect_toadmin_upgrades_path,:notice=>"Successfullyupgradeduser."end我如何在该操作中实际验证它们是否已保存或未重定向到适当的页面和消息? 最佳答案 在Rails3中,update_all不返回任何有意义的信息,除了已更新的记录数(这可能取决于您的DBMS是否返回该信息)。http://ar.ru

    5. 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

    6. ruby - 如何将脚本文件的末尾读取为数据文件(Perl 或任何其他语言) - 2

      我正在寻找执行以下操作的正确语法(在Perl、Shell或Ruby中):#variabletoaccessthedatalinesappendedasafileEND_OF_SCRIPT_MARKERrawdatastartshereanditcontinues. 最佳答案 Perl用__DATA__做这个:#!/usr/bin/perlusestrict;usewarnings;while(){print;}__DATA__Texttoprintgoeshere 关于ruby-如何将脚

    7. ruby - 如何指定 Rack 处理程序 - 2

      Rackup通过Rack的默认处理程序成功运行任何Rack应用程序。例如:classRackAppdefcall(environment)['200',{'Content-Type'=>'text/html'},["Helloworld"]]endendrunRackApp.new但是当最后一行更改为使用Rack的内置CGI处理程序时,rackup给出“NoMethodErrorat/undefinedmethod`call'fornil:NilClass”:Rack::Handler::CGI.runRackApp.newRack的其他内置处理程序也提出了同样的反对意见。例如Rack

    8. ruby - 如何每月在 Heroku 运行一次 Scheduler 插件? - 2

      在选择我想要运行操作的频率时,唯一的选项是“每天”、“每小时”和“每10分钟”。谢谢!我想为我的Rails3.1应用程序运行调度程序。 最佳答案 这不是一个优雅的解决方案,但您可以安排它每天运行,并在实际开始工作之前检查日期是否为当月的第一天。 关于ruby-如何每月在Heroku运行一次Scheduler插件?,我们在StackOverflow上找到一个类似的问题: https://stackoverflow.com/questions/8692687/

    9. ruby-on-rails - 如何从 format.xml 中删除 <hash></hash> - 2

      我有一个对象has_many应呈现为xml的子对象。这不是问题。我的问题是我创建了一个Hash包含此数据,就像解析器需要它一样。但是rails自动将整个文件包含在.........我需要摆脱type="array"和我该如何处理?我没有在文档中找到任何内容。 最佳答案 我遇到了同样的问题;这是我的XML:我在用这个:entries.to_xml将散列数据转换为XML,但这会将条目的数据包装到中所以我修改了:entries.to_xml(root:"Contacts")但这仍然将转换后的XML包装在“联系人”中,将我的XML代码修改为

    10. ruby - 如何使用文字标量样式在 YAML 中转储字符串? - 2

      我有一大串格式化数据(例如JSON),我想使用Psychinruby​​同时保留格式转储到YAML。基本上,我希望JSON使用literalstyle出现在YAML中:---json:|{"page":1,"results":["item","another"],"total_pages":0}但是,当我使用YAML.dump时,它不使用文字样式。我得到这样的东西:---json:!"{\n\"page\":1,\n\"results\":[\n\"item\",\"another\"\n],\n\"total_pages\":0\n}\n"我如何告诉Psych以想要的样式转储标量?解

    随机推荐