. #75
. #75
Conversation
lib/tree.rb
Outdated
| end | ||
|
|
||
| def add_apples | ||
| rand(5..15).times do @apples.push(Apple.new()) end |
There was a problem hiding this comment.
A couple ruby style points in here:
- For blocks that fit on a single line, the preferred syntax is
{ @apples.push(Apple.new()) }.do-endis preferred for multi-line blocks. - It's standard to omit the option
()for method calls with no arguments.
| end | ||
|
|
||
| avg_diameter = # It's up to you to calculate the average diameter for this harvest. | ||
| avg_diameter = diameter_sum/basket.size |
There was a problem hiding this comment.
Looks like this is doing integer division. Is that what is intended?
There was a problem hiding this comment.
Yes, that's what I meant to do. Am I doing it incorrectly?
There was a problem hiding this comment.
Suppose you a had basket with two apples in it, with diameters of 7 and 8, respectively. What would you expect the average diameter to be? What does this code say the average diameter is?
| expect(tree.age).to eq(1); | ||
| expect(tree.apples).to eq(0); | ||
| expect(tree.alive).to eq(true); | ||
| end |
| describe Fruit do | ||
| it 'should be a Class' do | ||
| expect(described_class.is_a? Class).to eq true | ||
| end |
There was a problem hiding this comment.
This kind of test is quite possibly not worth having in a project. If Fruit isn't a class, then all kinds of stuff should be blowing up in your tests.
| it 'apple should be a an instance of Apple and subclass of Fruit' do | ||
| apple = Apple.new() | ||
| expect(apple).to be_a_kind_of(Fruit) | ||
| end |
There was a problem hiding this comment.
If you find yourself putting 'and' in the description of your test, then you should consider writing two tests instead.
There was a problem hiding this comment.
I've changed it to be two tests, one for Apple < Fruit, and one for apple to be_an_instance_of(Apple)
|
:-) |
2 participants
No description provided.