- 1
- 2
- 3
while (cell.getFlag() != null) {
this.flag(x, y);
}-
Java / Говнокод #7150
+147
из моего древнего "Сапёра".
заменяется наcell.setFlag(null);Запостил:
Lure Of Chaos,
05 Июля 2011
Follow us!
Мне кажется, или проект имеет странноватую архитектуру?)))
а чем странноватую?
И тут же идет обращение к this.flag(/*координаты*/); //причем даже не setFlag(int,int);
То есть, по логике, this - это поле с клеточками ^_^
Я ,конечно, понимаю, что когда-то метод flag(int,int) установит флаг клетки в null, это хоть и извращение, но, допустим, оно имеет место быть. Тогда под вопросом архитектура описанных классов.
Вообще мне было бы очень интересно посмотреть код всего проекта, если вы будете так любезны выложите его где-нибудь, буду очень признателен!
public void четотам(int x, int y){
Cell cell = cells[x][y];//я конечно утрирую и надеюсь, что это не так
while(cell.getFlag() != null) this.flag(x,y);
}
То здесь можно было бы обойтись и без класса Cell. Хоть у клетки и есть какое-то состояние и поведение, то ваша реализация класса поля лишь запутывает код)))
для работоспособности нужен также http://xp-dev.com/svn/gargoyle-util/