Fishing Seaweed & Caskets - #425
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## kotlin-experiments #425 +/- ##
=====================================================
Coverage ? 23.23%
Complexity ? 811
=====================================================
Files ? 643
Lines ? 11132
Branches ? 1636
=====================================================
Hits ? 2587
Misses ? 8256
Partials ? 289 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| */ | ||
| SEAWEED(id = 401, level = 16, experience = 1.0), | ||
| CASKET(id = 405, level = 16, experience = 1.0) | ||
| ; |
There was a problem hiding this comment.
Move this semicolon back to the previous line.
| * A [FishingSpot] [Option] that can provide a two different types of fish. | ||
| */ | ||
| private data class Pair(override val tool: FishingTool, val primary: Fish, val secondary: Fish) : Option() { | ||
| private class Pair(override val tool: FishingTool, val primary: Fish, val secondary: Fish, vararg val junk: Fish) : Option() { |
There was a problem hiding this comment.
Why did you remove the data modifier?
There was a problem hiding this comment.
There's also no reason for junk to be variadic. This constructor isn't a user facing API.
There was a problem hiding this comment.
The data modifier was removed as you can't have variadic values in a data class. I'll fix this though, thanks!
| } | ||
|
|
||
| fun of(tool: FishingTool, primary: Fish, secondary: Fish, junk: Array<Fish>): Option { | ||
| return when { |
There was a problem hiding this comment.
Write this as if-else instead
| return valid[rand(valid.size)] | ||
| } | ||
| } | ||
| return when { |
There was a problem hiding this comment.
Should be if-else instead of when.
|
Some style issues, otherwise LGTM |
| override val level = Math.min(primary.level, secondary.level) | ||
|
|
||
| override fun sample(level: Int): Fish { | ||
| if (junk.isNotEmpty() && rand(100) < JUNK_WEIGHTING) { |
There was a problem hiding this comment.
Hmm, we need a unit test for this, but I'm not quite sure how we go about stubbing the randomness in this case (cc @Major-).
3 participants
Addresses Issue #364 where fishing seaweed & caskets from big nets is requested