Интересная работа сборщика мусора

[ramovsky]

АААА!!! Помогите.
Что с хабром??? Я хотел сделать правку в статье, а он еще одну создал. Немогу скрыть. Немогу в черновики публикануть. Что с ним????

[kurokikaze]

Я конечно полный лузер в Java, но разве в последнем листинге не должно быть b.Turn();?

[ramovsky]

Я хочу что бы все турны выполнялись через Ворлд.

ПС. И нахрена мне такое счастье… через эти глюки карма в говне… спасибо дорогое НЛО. два раза за день…

[kurokikaze]

Ох, туплю, действительно :) Это конец хода видимо.

[ramovsky]

начало:)

[YasonBy]

public Bot(int energy, String name, World world) {
 //...  
 if (this.energy < 1) {
  this.live = false;
 } else {
  live = true;
 }
 //...
}
У Вас боты создаются мёртвыми независимо от переданной в конструктор энергии.
Ну и World.Turn() их обнуляет.

[YasonBy]

Ну и сама эта конструкция являет собой отличный образчик индийского кода :)
Может лучше всё же

this.live = (energy>=1)

[ramovsky]

Забавно. У меня стоит IntelliJ IDEA. Инспектор кода предложил заменить мой if на this.live = (energy>=1). Забавно…

[ramovsky]

Bot b = new Bot(20, «B», w)
Создавался с енергией 20, тоесть живой.

[YasonBy]

Сорри, не заметил «this.energy = energy» в первой строке конструктора.

[GubkaBob]

ссылка не теряется — у вас же она хранится в residents[1][2]. поэтому никто её собрать не должен.

код не падает, и не должен. хотя баги есть (например размерность массива [height][width], а используется как [width][height])

[ramovsky]

Но все же он падает… Причем как я говорил не на каждом компе.

Милости просим. botworld.googlecode.com/svn/trunk/BotWorld/

[GubkaBob]

я бы предположил, что падает из-за world.ResidentAction(this, (ACTION) moves.lastElement()) в Bot.
при этом обнуляется ссылка в массиве — что приведет к NRE при вызове isLive()

[ramovsky]

Да падает на isLive().
А что плохого в world.ResidentAction(this, (ACTION) moves.lastElement())?

[GubkaBob]

там обнуляется элемент массива в некоторых кейсах (разве нет?), а в турн вы пытаетесь его использовать

[ramovsky]

ААА!!! Точно. residents[res.X()][res.Y()] = null;
Большое спасибо!
А как же оно работало на других компах????

[GubkaBob]

также, просто этот кейс не выпадал на исполнение скорее всего

[ramovsky]

выпадал 100%. код не менялся. ВОт там сборщик работал действительно странно.:)

[GubkaBob]

забудьте про сборщик. в 99.99% случаях код фреймворка работает верно

[ramovsky]

Завтра протестирую на том компе, на котором не вылитало исключение и отпишусь.
Еще раз спасибо за полезный опыт.

[ramovsky]

Все оказалось просто, ошибка в логике. Мой напарник, который тестил этот код и у которого небыло ошибки, просто удалил кусок кода в котором вносились хода в лист. Ходов небыло, перемещений небыло, исключений небыло… вот и все.

[ramovsky]

Нужно вместо
residents[i][j]).Turn();
if(!residents[i][j].isLive()) residents[i][j] = null;

написать
if(!residents[i][j].isLive()) residents[i][j] = null;
else{
residents[i][j]).Turn();
}

[ramovsky]

Спасибо. Сам бы не нашел.

[voicer]

Название топика звучит как отличная вакансия =)

[ramovsky]

ага. забавно:)