来自 Kent Beck 的 TDD 的货币示例

发布于 2024-12-09 05:32:34 字数 453 浏览 0 评论 0原文

因此，我已经完成了 Kent Beck 的《测试驱动开发示例》一书中的 Money 示例，并且能够使代码正常工作，直到他编写的最后一个测试：

@Test
public void testPlusSameCurrencyReturnsMoney(){
    Expression sum = Money.dollar(1).plus(Money.dollar(1));
    assertTrue(sum instanceof Money);
}

这是这个函数当我运行这个时，

public Expression plus(Expression addend) {
    return new Sum(this, addend);
}

它给出了 java.lang.AssertionError ，所以我的问题是为什么它会给出这个错误以及如何修复它？

原文

So I have worked through the Money example in Kent Beck's book Test Driven Development by Example and have been able to get the code to work up until the last test that he writes:

@Test
public void testPlusSameCurrencyReturnsMoney(){
    Expression sum = Money.dollar(1).plus(Money.dollar(1));
    assertTrue(sum instanceof Money);
}

and here is the function that this calls

public Expression plus(Expression addend) {
    return new Sum(this, addend);
}

When I run this, it gives java.lang.AssertionError, so my question is why is it giving this error and how do I fix it?

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

情绪少女 2024-12-16 05:32:34

Lunivore 已经回答了如何解决问题的问题，但我认为如果您想更多地了解 Beck 试图传达的内容，您应该重新阅读代码块（和测试）之前和之后的段落。

最后一句是“这是我们必须修改才能使其工作的代码：”。该代码块首先在第 75 页上输入（带有测试用例）。第 79 页的最终效果没有任何更改。这只是表明如果我们想保留此测试，我们可以更改哪些内容。

“当且仅当它是金钱时，没有明显、干净的方法来检查参数的货币。实验失败了，我们删除了测试，然后就走了”。

他还表示这个测试很丑陋，并在下一页总结道“尝试了一个简短的实验，但没有成功就放弃了”。

我写这篇文章是为了防止您认为所有示例都有效并且应该保留。

回复收藏 0 原文

梦回梦里 2024-12-16 05:32:34

您正在检查 sum 变量是否为 Money，但在 plus 方法中返回 Sum。

因此，除非 Sum 是 Money 的子类，否则该断言将始终失败。

为了使其通过，您可能需要执行以下操作：

public Expression plus(Expression addend) {
    return new Money(...<whatever>...);
}

当然，Money 也必须是一个Expression。

或者您可能想评估总和以从中获利。或者甚至可以使用 sum instanceof Sum 来代替。这取决于您实际想要实现的行为。

顺便说一下，要小心 instanceof 运算符。

You're checking that the sum variable is a Money, but returning a Sum in the plus method.

So, unless Sum is a subclass of Money, that assertion will always fail.

To make it pass, you might want to do something like:

public Expression plus(Expression addend) {
    return new Money(...<whatever>...);
}

Of course, Money would then have to be an Expression too.

Or you might want to evaluate the sum to get the money out of it. Or maybe even do sum instanceof Sum instead. It depends on what behavior you're actually trying to achieve.

By the way, beware the instanceof operator.

回复收藏 0 原文

~没有更多了~