Conversation
SerVB
left a comment
SerVB
left a comment
There was a problem hiding this comment.
Прошу поправить моменты. Скорее всего, будет не одна итерация проверки. На всякий случай упомяну, что стоит помечать сделанные комменты как resolved.
SerVB
left a comment
SerVB
left a comment
There was a problem hiding this comment.
Мне нравится! Есть еще несколько моментов.
SerVB
left a comment
SerVB
left a comment
There was a problem hiding this comment.
Хорошая работа. Засчитываем. Если хочешь, можешь посмотреть еще пару комментов. Ну и можешь продолжить прогать этот проект, если хочешь) Если что, спрашивай.
| fun printField() { | ||
| println(" |abcdefghi|\n-|${"-".repeat(FIELD_SIZE - 2)}|") | ||
| fun getField(): String { | ||
| var fieldString = "" |
There was a problem hiding this comment.
Глянь на StringBuilder. В отличие от строки, он мутабелен, поэтому более эффективен (ну этот SB вообще прикол Java и ,NET мира). При конкатенации строки создается новый объект и это не очень.
| CellState.MARKED -> visibleField[rowId][colId] = CellState.EMPTY | ||
| CellState.EMPTY -> visibleField[rowId][colId] = CellState.MARKED | ||
| CellState.OPENED -> return "Cell is opened" | ||
| CellState.NUMBERED -> return "Cell is opened" |
There was a problem hiding this comment.
Одинаковые ветки можно не дублировать, написав через запятую условия.
2 participants
No description provided.