Морской бой. Пишу метод для поворота корабля.
tt
- это текущее положение корабля:
0 - вертикально (по умолчанию)
1 - горизонтально (0 + 90°)
2 - вертикально (0 вверх ногами)
3 - горизонтально (0 - 90°)
ckw
- это флаг поворота:
true
- по часовой стрелке
false
- против часовой стрелки
Код метода поворота:
public void turn (boolean ckw) {
if ((tt == 0 && !ckw) || (tt == 1 && ckw)) {
//b -> c
}
else if ((tt == 1 && !ckw) || (tt == 2 && ckw)) {
//c -> d
}
else if ((tt == 2 && !ckw) || (tt == 3 && ckw)) {
//d -> a
}
else if ((tt == 3 && !ckw) || (tt == 0 && ckw)) {
//a -> b
}
if (ckw)
tt++;
else
tt--;
}
Комментариями обозначены 4 цикла for-each
с логикой разворота. Мне бы вот эту лесенку if
-ов сократить. Это возможно?
Логика поворота
Корабль представляет собой ArrayList
позиций в сетке. Например, 4-палубный, находящийся в середине поля: 35, 45, 55, 65
. Чтобы его повернуть по часовой стрелке, надо прибавить к каждой позиции её индекс, умноженный на 11
.
b -> c (90° -> 180°) или (0° -> 270°)
for (int i = 0; i < col.size(); i++) {
int num = col.get(i);
int new_num = num + 11 * i;
col.set(i, new_num);
}
И т.д.
b -> c
,c -> d
и прочие звери? и что завертикально (0 вверх ногами)
? Какие еще ноги у кораблей? – Алексей Шиманский 18 фев '17 в 17:49a -> b
и другие) можно объединить в единый код и необходимость вif-else
(илиswitch-case
) вообще отпадёт. – Regent 18 фев '17 в 18:37tt == 0 && !ckw
иtt == 1 && ckw
одинаковый код поворота, если в первом случае это поворот от 0 до 270 градусов, а во втором случае - от 90 до 180. – Regent 18 фев '17 в 18:44Вполне возможно, что эти четыре поворота (a -> b и другие) можно объединить в единый код
- да, спасибо, что за меня это высказали)) А я хотел по кусочкам в коменте хотел довести до конца подсказки...) – Алексей Шиманский 18 фев '17 в 18:54