ramovsky Sep 2 2009 at 16:53

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

3 min

2.2K

Comments 25

ramovsky Sep 2 2009 at 17:07

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

kurokikaze Sep 2 2009 at 17:07

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

ramovsky Sep 2 2009 at 17:15

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

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

kurokikaze Sep 2 2009 at 17:16

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

ramovsky Sep 2 2009 at 17:23

начало:)

YasonBy Sep 2 2009 at 17:27

public Bot(int energy, String name, World world) {

 //...  

 if (this.energy < 1) {

  this.live = false;

 } else {

  live = true;

 }

 //...

}

У Вас боты создаются мёртвыми независимо от переданной в конструктор энергии.
Ну и World.Turn() их обнуляет.

YasonBy Sep 2 2009 at 17:30

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

this.live = (energy>=1)

ramovsky Sep 3 2009 at 12:50

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

ramovsky Sep 2 2009 at 17:34

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

YasonBy Sep 2 2009 at 17:43

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

GubkaBob Sep 2 2009 at 17:36

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

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

ramovsky Sep 2 2009 at 17:39

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

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

GubkaBob Sep 2 2009 at 17:51

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

ramovsky Sep 2 2009 at 17:54

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

GubkaBob Sep 2 2009 at 18:46

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

ramovsky Sep 2 2009 at 19:38

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

GubkaBob Sep 2 2009 at 19:40

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

ramovsky Sep 2 2009 at 19:43

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

GubkaBob Sep 2 2009 at 19:47

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

ramovsky Sep 2 2009 at 20:17

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

ramovsky Sep 3 2009 at 12:12

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

ramovsky Sep 2 2009 at 19:42

Нужно вместо
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 Sep 2 2009 at 17:45

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

voicer Sep 2 2009 at 17:38

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

ramovsky Sep 2 2009 at 17:44

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

Show the best of all time