Conversation
|
|
||
| case class Frame( | ||
| number: Int, | ||
| throws: List[Int] = List(), |
There was a problem hiding this comment.
better use List.empty - it doesn't allocate object
|
|
||
| case class Frame( | ||
| number: Int, | ||
| throws: List[Int] = List(), |
There was a problem hiding this comment.
throws - bad name, it can be an issue if used in JAVA code
| ) { | ||
| def score = throws.sum; | ||
|
|
||
| def isStrike = if (throws.isEmpty) false else throws(0) == 10; |
There was a problem hiding this comment.
better use def isStrike = throws.nonEmpty && throws.head == 10 (without unneccesary if\else)
| def score = throws.sum; | ||
|
|
||
| def isStrike = if (throws.isEmpty) false else throws(0) == 10; | ||
| def isSpare = if (throws.length < 2) false else throws(0) + throws(1) == 10; |
There was a problem hiding this comment.
Same as previous comment.
Also, the best approach is prevent creating empty lists while parsing(probably create some domain object or add validation while parsing - both cases are ok)
| class Game(input: String) { | ||
| def score(): Int = { | ||
| @tailrec | ||
| def calculateScore( |
There was a problem hiding this comment.
Алгоритм рабочий, но выглядит ОЧЕНЬ императивно.
В целом задача может быть решена двумя способами - 1) с разбиением на доменные объекты и классы. Тогда первым действием программы должны идти парсинг и валидация и создание непротиворечивого дерева объектов, которое уже потом в цикле обрабатывается на основе правил. В этом случае множество проверок типа isEmpty и т.д отпадают. Например у нас есть объекты Frame, которые содержат список Ball(енум Strike, Spare, Regular) и т.д.
2) Функциональный стиль - когда ты рекурсивно идешь по строке, откусывая по символу за раз. Тут конечно надо подумать как учитывать бонусные фреймы и баллы за страйки, но в этом и суть:)
Чтобы немного навести на мысль - посмотри реализация на хаскелле, там не должно быть сложно для понимания - https://github.com/cskeppstedt/haskell-bowling-kata/blob/master/src/Bowling.hs
There was a problem hiding this comment.
Я конечно подумаю ещё, но мне пока не понятна разница между императивным и декларативным. Если можешь статью какую посоветовать, то было бы хорошо.
Кажется, что можно сделать как то иначе, сохраняя рекурсию, но получится ли хоть немного легче в понимании пока не знаю, может и можно, но сложнова-то пока :)
2 participants
No description provided.