Валидатор слишком умный

Вопрос по задаче

Решен

Сделал код максимально красивый. Без повторений, а валидатор не принимает)

Пришло время реализовать методы, которые вызываются при нажатии клавиш. Это onKeyPress(Key) и onKeyReleased(Key).
При запуске игры поле состоит из 3x3 клеток.

Тебе нужно сделать так, чтобы:
- при нажатии клавиши Key.LEFT закрашивались три левые клетки;
- при нажатии Key.RIGHT закрашивались три правые клетки;
- при нажатии Key.UP — три верхние;
- при нажатии Key.DOWN — три нижние.
Клетки должны закрашиваться в зеленый цвет.
При отпускании любой из этих четырех клавиш все клетки должны закрашиваться в белый.

Требования:

В классе KeyboardGame должен быть переопределен метод onKeyPress(Key) родительского класса Game.
В классе KeyboardGame должен быть переопределен метод onKeyReleased(Key) родительского класса Game.
При нажатии клавиши Key.LEFT должны быть закрашены три левые клетки в зеленый цвет.
При нажатии клавиши Key.RIGHT должны быть закрашены три правые клетки в зеленый цвет.
При нажатии клавиши Key.UP должны быть закрашены три верхние клетки в зеленый цвет.
При нажатии клавиши Key.DOWN должны быть закрашены три нижние клетки в зеленый цвет.
При отпускании клавиш Key.LEFT, Key.RIGHT, Key.UP или Key.DOWN все клетки должны быть закрашены в белый.

package com.javarush.games.minigames.mini08; import com.javarush.engine.cell.*; /* Работа с клавиатурой */ public class KeyboardGame extends Game { @Override public void initialize() { setScreenSize(3, 3); for (int x = 0; x < 3; x++) { for (int y = 0; y < 3; y++) { setCellColor(x, y, Color.WHITE); } } } public void onKeyPress(Key key) { checkKeyAndSetColor(key, Color.GREEN); } public void onKeyReleased(Key key) { checkKeyAndSetColor(key, Color.WHITE); } public void fillTheColumn(int x, Color color) { for (int y = 0; y < 3; y++) { setCellColor(x, y, color); } } public void fillTheLine(int y, Color color) { for (int x = 0; x < 3; x++) { setCellColor(x, y, color); } } public void checkKeyAndSetColor(Key key, Color color) { switch(key) { case LEFT: fillTheColumn(0, color); break; case RIGHT: fillTheColumn(2, color); break; case UP: fillTheLine(0, color); break; case DOWN: fillTheLine(2, color); break; } } }

Комментарии (6)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

CodeMashineРаботает в Автосервис

10 декабря 2024, 16:47

я думал я один такой умный. валидатор тоже не пропускает. Пишет задача не скомпилирована на сервере!!!

Anonymous #2813974

Уровень 29

26 сентября 2022, 16:30

@Override public void onKeyPress(Key key) { for (int i = 0; i < 3; i++) { for (int j = 0; j < 3; j++) { if (key == Key.LEFT) { setCellColor(0, j, Color.GREEN); } if (key == Key.RIGHT) { setCellColor(2, j, Color.GREEN); } if (key == Key.DOWN){ setCellColor(i,2,Color.GREEN); } if (key == Key.UP){ setCellColor(i,0,Color.GREEN); } } } } @Override public void onKeyReleased(Key key) { for (int i = 0; i < 3; i++) { for (int j = 0; j < 3; j++) { if (key == Key.LEFT || key == Key.DOWN || key == Key.UP || key == Key.RIGHT) { setCellColor(i, j, Color.WHITE); } } } } }

Гофф

Уровень 3

Master

11 декабря 2024, 08:08

Это странное решение - перебирать все клетки вместо того, что-бы перебирать только те, которые нужно покрасить. Со свитчом пограмотнее будет ;-)

Никита Ильин

Уровень 28

20 июля 2022, 17:41

Валидатор скушал отличное решение. Ни капли не перебор:

public class KeyboardGame extends Game {

    @Override
    public void initialize() {
        setScreenSize(3, 3);

        for (int x = 0; x < 3; x++) {
            for (int y = 0; y < 3; y++) {
                setCellColor(x, y, Color.WHITE);
            }
        }
    }

    public void onKeyPress(Key key) {
        switch(key) {
            case LEFT: {
                for (int y = 0, x = 0; y < 3; y++) {
                    setCellColor(x, y, Color.GREEN);
                }
            }
            break;
            case RIGHT: {
                for (int y = 0, x = 2; y < 3; y++) {
                    setCellColor(x, y, Color.GREEN);
                }
            }
            break;
            case UP: {
                for (int x = 0, y = 0; x < 3; x++) {
                    setCellColor(x, y, Color.GREEN);
                }
            }
            break;
            case DOWN: {
                for (int x = 0, y = 2; x < 3; x++) {
                    setCellColor(x, y, Color.GREEN);
                }
            }
        }
    }
    public void onKeyReleased(Key key) {
        switch(key) {
            case LEFT: {
                for (int y = 0, x = 0; y < 3; y++) {
                    setCellColor(x, y, Color.WHITE);
                }
            }
                break;
            case RIGHT: {
                for (int y = 0, x = 2; y < 3; y++) {
                    setCellColor(x, y, Color.WHITE);
                }
            }
                break;
            case UP: {
                for (int x = 0, y = 0; x < 3; x++) {
                    setCellColor(x, y, Color.WHITE);
                }
            }
                break;
            case DOWN: {
                for (int x = 0, y = 2; x < 3; x++) {
                    setCellColor(x, y, Color.WHITE);
                }
            }
        }
    }
}

Anonymous #3419780

Уровень 24

14 октября 2024, 21:16

старался сделать как можно компактнее своим примитивным умом, получилось вот так public void onKeyPress(Key key) { for (int i = 0; i < 3; i++) { if (key == Key.LEFT) { setCellColor(0, i, Color.GREEN); } else if (key == Key.UP) { setCellColor(i, 0, Color.GREEN); } } for (int i = 2; i >= 0; i--) { if (key == Key.RIGHT) { setCellColor(2, i, Color.GREEN); } else if (key == Key.DOWN) { setCellColor(i, 2, Color.GREEN); } } } public void onKeyReleased(Key key) { for (int i = 0; i < 3; i++) { if (key == Key.LEFT) { setCellColor(0, i, Color.WHITE); } else if (key == Key.UP) { setCellColor(i, 0, Color.WHITE); } } for (int i = 2; i >= 0; i--) { if (key == Key.RIGHT) { setCellColor(2, i, Color.WHITE); } else if (key == Key.DOWN) { setCellColor(i, 2, Color.WHITE); } } }

Exaltyr777

Уровень 25

18 августа 2025, 17:35

мммм, вкуснятинааа)