Searcher #3
Searcher #3
Conversation
| @@ -1,9 +1,11 @@ | |||
| package ru.spbstu.terrai.core | |||
|
|
|||
| data class Condition(val items: List<Item>, val exitReached: Boolean) { | |||
| constructor(): this(false) | |||
| data class Condition(val items: List<Item>, val exitReached: Boolean, val exitFind: Boolean) { | |||
There was a problem hiding this comment.
Кроме этого, поскольку данная штука нужна конкретному игроку, корректнее запоминать её в нём, не меняя внутренние классы игры
|
|
||
| private var wormholes = 0 //считает количество уже пройденных червоточин | ||
|
|
||
| private var maxholes = 0 //хранит количество червоточин на карте |
There was a problem hiding this comment.
По Code Style -- maxHoles (и ниже такие места ещё попадаются)
|
|
||
| private var maxholes = 0 //хранит количество червоточин на карте | ||
|
|
||
| private var pathWasFinded = false //найден ли путь от вормхолла до выхода |
There was a problem hiding this comment.
В общем и целом, флагов у вас накопилось уже столько, что стоило их все убрать и завести единую переменную-перечисление "Состояние". Его и использовать
| private var pathTemp: MutableList<Direction>? = mutableListOf<Direction>() | ||
| //хранит последний результат движения | ||
| private var lastResult: MoveResult?=null //копия последней полученной информации о состоянии | ||
| private var loopwWh = false //завис ли бот в цикле |
There was a problem hiding this comment.
Это название в принципе неудачное
| //функция управления движением//работает нормально//пока | ||
| override fun getNextMove(): Move {//если есть любая клетка, в которой бот еще не был, то он идет туда | ||
|
|
||
| var rng = Random();//для выбора направления |
There was a problem hiding this comment.
Генератор случайных чисел лучше определять на верхнем уровне, не создавая его каждый раз заново
There was a problem hiding this comment.
Точки с запятой в Котлине почти никогда не нужны. Также важно употреблять val везде, где это возможно, заменяя var (IDE тут подсказывает)
| private var bindingFlag = false | ||
| private var pathTemp: MutableList<Direction>? = mutableListOf<Direction>() | ||
| //хранит последний результат движения | ||
| private var lastResult: MoveResult?=null //копия последней полученной информации о состоянии |
There was a problem hiding this comment.
Здесь в Котлине лучше так:
private lateinit var lastResult: MoveResult
Дело в том, что по факту оно всё равно используется как not-null, а null бывает только в самом начале работы
|
|
||
| if(backInTheFirstWormholeFlag){//бот нашел клад и должен вернуться ко входу | ||
| var step: Direction | ||
| if (wormholes!=1){ |
There was a problem hiding this comment.
Почему вдруг именно 1? В чём специальность этого значения?
| return lastMove | ||
| } | ||
|
|
||
| //создает копию пути от вормхолла до выхода |
There was a problem hiding this comment.
Скорее, видимо, не до выхода, а до входа в текущую комнату
| //бот следует по запомненному пути к выходу | ||
| if(lastResult?.room is Wormhole){//свалился в новый вормхолл -> скопировал маршрут заново | ||
| pathTemp = mutableListOf<Direction>() | ||
| pathTemp?.addAll(pathToExit) |
There was a problem hiding this comment.
Не очень понял смысл этой ветки. Можно обсудить при встрече
|
|
||
| private val pathToExit = mutableListOf<Direction>()//список, хранящий путь от ямы до выхода | ||
|
|
||
| private var wormholes = 0 //считает количество уже пройденных червоточин |
There was a problem hiding this comment.
Далее по коду получается, что это очень хакерское поле, и название его не совсем точно
Немного поумневший DummySearcher.
Чупин Никита, 33501/1