最近,我对这个问题进行了热烈的讨论。
假设我在 Java 中创建了这个方法:
public Set<String> getRich() {
return ImmutableSet<String> ....;
}
每当我在拉取请求中看到这一点时,我都会大喊并试图解释为什么它是错误的。通过这样做,我通过 promise 他们将获得 Set 误导了我的方法的消费者。这意味着他们可以删除或添加元素。 javac 会愉快地编译它,但会抛出 RuntimeException。此外,它违反了“里氏替换原则”。
就个人而言,我总是这样做:
public ImmutableSet<String> getRich() {
return ImmutableSet<String> ....;
}
这样一来,没人会搬起石头砸自己的脚。
一种建议的方法是返回 Iterable。我认为这很糟糕,因为该方法的使用者将失去 HashSet、HashMap 或其他任何东西的潜在力量(因为无法调用 set.get(hash) )。
另一种方法是 - 作为消费者 - 创建 getRich() 方法输出的副本。但是你如何确定消费者会这样做呢?
当然,有些人会坚持 OOP 设计原则并说:“始终针对接口(interface)编程”。对我来说 - 在这种特殊情况下 - 这是有史以来最糟糕的选择。
你会如何处理这个案子?
(题外话:这个案例是静态类型可以确保正确性但永远无法确保正当性的完美示例)。
最佳答案
技术上,将返回类型声明为 Set<T>不违反 LSP。
Set<T>.Add 的文档明确指出实现可能会抛出 UnsupportedOperationException .
这意味着检查集合是否可变是客户的责任。
现在,是否制作ImmutableSet<T>扩展 Set<T>是个不错的选择,值得商榷。
我个人认为这是一个非常糟糕的选择——客户端代码不应该乱七八糟地使用 try/catches 来确定一个集合是否可变。此外,Set<T>甚至不提供 isMutable()让客户轻松完成检查的方法!为不可变集合提供一个单独的接口(interface)会是一个更简洁的设计。
我个人会解决这个设计缺陷,并且按照您的建议,将返回类型声明为 ImmutableSet<T> .
你问题的第二部分是你是否应该返回 ImmutableSet<T>或 Iterable<T> :这实际上取决于您认为客户对您的 API 的期望。需要考虑的事项:
Iterable<T> ;ImmutableSet<T> ?如果没有,请使用 Iterable<T> ;关于java - 一种将可变数据结构声明为输出并实际返回不可变数据结构的方法,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/25423039/
我正在学习如何使用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
总的来说,我对ruby还比较陌生,我正在为我正在创建的对象编写一些rspec测试用例。许多测试用例都非常基础,我只是想确保正确填充和返回值。我想知道是否有办法使用循环结构来执行此操作。不必为我要测试的每个方法都设置一个assertEquals。例如:describeitem,"TestingtheItem"doit"willhaveanullvaluetostart"doitem=Item.new#HereIcoulddotheitem.name.shouldbe_nil#thenIcoulddoitem.category.shouldbe_nilendend但我想要一些方法来使用
类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
我正在尝试设置一个puppet节点,但rubygems似乎不正常。如果我通过它自己的二进制文件(/usr/lib/ruby/gems/1.8/gems/facter-1.5.8/bin/facter)在cli上运行facter,它工作正常,但如果我通过由rubygems(/usr/bin/facter)安装的二进制文件,它抛出:/usr/lib/ruby/1.8/facter/uptime.rb:11:undefinedmethod`get_uptime'forFacter::Util::Uptime:Module(NoMethodError)from/usr/lib/ruby
给定这段代码defcreate@upgrades=User.update_all(["role=?","upgraded"],:id=>params[:upgrade])redirect_toadmin_upgrades_path,:notice=>"Successfullyupgradeduser."end我如何在该操作中实际验证它们是否已保存或未重定向到适当的页面和消息? 最佳答案 在Rails3中,update_all不返回任何有意义的信息,除了已更新的记录数(这可能取决于您的DBMS是否返回该信息)。http://ar.ru
我想将html转换为纯文本。不过,我不想只删除标签,我想智能地保留尽可能多的格式。为插入换行符标签,检测段落并格式化它们等。输入非常简单,通常是格式良好的html(不是整个文档,只是一堆内容,通常没有anchor或图像)。我可以将几个正则表达式放在一起,让我达到80%,但我认为可能有一些现有的解决方案更智能。 最佳答案 首先,不要尝试为此使用正则表达式。很有可能你会想出一个脆弱/脆弱的解决方案,它会随着HTML的变化而崩溃,或者很难管理和维护。您可以使用Nokogiri快速解析HTML并提取文本:require'nokogiri'h
我想了解Ruby方法methods()是如何工作的。我尝试使用“ruby方法”在Google上搜索,但这不是我需要的。我也看过ruby-doc.org,但我没有找到这种方法。你能详细解释一下它是如何工作的或者给我一个链接吗?更新我用methods()方法做了实验,得到了这样的结果:'labrat'代码classFirstdeffirst_instance_mymethodenddefself.first_class_mymethodendendclassSecond使用类#returnsavailablemethodslistforclassandancestorsputsSeco
我主要使用Ruby来执行此操作,但到目前为止我的攻击计划如下:使用gemsrdf、rdf-rdfa和rdf-microdata或mida来解析给定任何URI的数据。我认为最好映射到像schema.org这样的统一模式,例如使用这个yaml文件,它试图描述数据词汇表和opengraph到schema.org之间的转换:#SchemaXtoschema.orgconversion#data-vocabularyDV:name:namestreet-address:streetAddressregion:addressRegionlocality:addressLocalityphoto:i
为了将Cucumber用于命令行脚本,我按照提供的说明安装了arubagem。它在我的Gemfile中,我可以验证是否安装了正确的版本并且我已经包含了require'aruba/cucumber'在'features/env.rb'中为了确保它能正常工作,我写了以下场景:@announceScenario:Testingcucumber/arubaGivenablankslateThentheoutputfrom"ls-la"shouldcontain"drw"假设事情应该失败。它确实失败了,但失败的原因是错误的:@announceScenario:Testingcucumber/ar
我在我的项目中添加了一个系统来重置用户密码并通过电子邮件将密码发送给他,以防他忘记密码。昨天它运行良好(当我实现它时)。当我今天尝试启动服务器时,出现以下错误。=>BootingWEBrick=>Rails3.2.1applicationstartingindevelopmentonhttp://0.0.0.0:3000=>Callwith-dtodetach=>Ctrl-CtoshutdownserverExiting/Users/vinayshenoy/.rvm/gems/ruby-1.9.3-p0/gems/actionmailer-3.2.1/lib/action_mailer