testdrivendevelopment at Yahoo! Groups

Re: I already did that!

Sean Corfield — Sat, 07 Nov 2009 20:18:46 GMT

... If I'm understanding you correctly, no, you don't really have two places. You have a method that filters on whether something is paused or not - but it

Re: CRAP for C++ ?

Sean Corfield — Sat, 07 Nov 2009 00:17:24 GMT

On Mon, Nov 2, 2009 at 7:35 AM, Srdjan Todorovic ... You might want to look at QA C++ from Programming Research as a good source code analysis tool for

Re: CRAP for C++ ?

James Martin — Fri, 06 Nov 2009 22:25:06 GMT

Take a look at http://cppdepend.com. They have a command line version if you're not using VS. It includes most of the same metrics as crap4j and is probably

Re: I already did that!

Erlis Vidal — Thu, 05 Nov 2009 15:53:57 GMT

Hi guys, Thanks for all your responses. Putting together John, gutzofter and Franz ideas I could do a better solution. When I was reading all your responses,

Re: I already did that!

gutzofter — Thu, 05 Nov 2009 08:30:38 GMT

... It's really simple state is just data. When I say 'behavior', I mean the algorithm for transforming data. ... FindRunningRealeases is now just an iterator.

Re: I already did that!

Franz Allan Valencia See — Thu, 05 Nov 2009 08:03:34 GMT

If I understood your problem correctly, initially, you have tests for FindRunningReleases that are something like these

Re: I already did that!

Donaldson, John (GEO) — Wed, 04 Nov 2009 22:01:02 GMT

Erlis, You have separated the InPause logic, and you can test that separately. This seems to be the right direction. I don't see that you need to test again

Re: I already did that!

Erlis Vidal — Wed, 04 Nov 2009 21:35:56 GMT

Hi gutzofter, Thanks for your answer. I have to admit that I didn't understand well what is the difference between behaviour vs state. Can you send me some

Re: I already did that!

Erlis Vidal — Wed, 04 Nov 2009 21:35:25 GMT

Hi John, You are right when referred to remove the old X tests, that is what I usually do. But in this case what seems to happen is that all X appear to be

Re: Too many injections

Nat Pryce — Wed, 04 Nov 2009 21:33:24 GMT

2009/11/4 Donaldson, John (GEO) ... There could be several solutions to your problem. One way to get a different perspective is to

Re: Too many injections

Donaldson, John (GEO) — Wed, 04 Nov 2009 21:07:51 GMT

Alan, I don't think this addresses the difficulty. Your suggestion is a good example of how to construct a complex object using fakes. But it doesn't treat the

Re: I already did that!

Donaldson, John (GEO) — Wed, 04 Nov 2009 18:40:57 GMT

Erlis, I'm a bit confused. Here's the basic position: duplicated code is bad - remove it. It seems as though you have something duplicated, so I'm wondering

Re: I already did that!

gutzofter — Wed, 04 Nov 2009 17:02:36 GMT

@Erlis, I use a rule of thumb when using extract class: Extract behavior not state. What it looks like is you are extracted state to a new class. So now, not

Re: I already did that!

Erlis Vidal — Wed, 04 Nov 2009 14:44:58 GMT

Hi Olof, Thanks for your response. I like your point of view, and I'm really trying to follow that idea, that's why I'm confused. I created a method, with the

Re: I already did that!

Olof Bjarnason — Wed, 04 Nov 2009 07:12:21 GMT

2009/11/3 Erlis Vidal ... The single responsibility principle states that each unit should have a single purpose/responsibility.