Skip to main content
removeu 1 caractere do conteúdo
Fonte Link
Victor Stafusa
  • 64,9mil
  • 9
  • 120
  • 189
  • Só criar, interagir, usar e acessar componentesscomponentes Swing dentro da EDT. Para isso, uso o EventQueue.invokeLater(Runnable). Veja mais sobre isso nessa pergunta e nessa resposta.

  • Preferir usar composição do que herança, e portanto não herdar de JFrame. Falo sobre isso nessa resposta. Também abordo essa questão (e muitas outras coisas) nessa outra resposta.

  • Usar números mágicos não é uma boa prática de programação. Fiz o seu código sem usar números mágicos e também de uma forma que se o tamanho da tela ou o tamanho do label mudarem, ele não vai ficar bagunçado e vai se ajustar à mudança sozinho.

  • Pintei o JLabel de amarelo para que você possa ver exatamente onde ele está e qual é o seu tamanho.

  • Alterei o nome de algumas variáveis para ficarem mais representativos.

  • Use KeyAdapter e MouseAdapter ao invés de implementar diretamente KeyListener e MouseListener para não pecisar ter aquele monte de métodos vazios.

  • Só criar, interagir, usar e acessar componentess Swing dentro da EDT. Para isso, uso o EventQueue.invokeLater(Runnable). Veja mais sobre isso nessa pergunta e nessa resposta.

  • Preferir usar composição do que herança, e portanto não herdar de JFrame. Falo sobre isso nessa resposta. Também abordo essa questão (e muitas outras coisas) nessa outra resposta.

  • Usar números mágicos não é uma boa prática de programação. Fiz o seu código sem usar números mágicos e também de uma forma que se o tamanho da tela ou o tamanho do label mudarem, ele não vai ficar bagunçado e vai se ajustar à mudança sozinho.

  • Pintei o JLabel de amarelo para que você possa ver exatamente onde ele está e qual é o seu tamanho.

  • Alterei o nome de algumas variáveis para ficarem mais representativos.

  • Use KeyAdapter e MouseAdapter ao invés de implementar diretamente KeyListener e MouseListener para não pecisar ter aquele monte de métodos vazios.

  • Só criar, interagir, usar e acessar componentes Swing dentro da EDT. Para isso, uso o EventQueue.invokeLater(Runnable). Veja mais sobre isso nessa pergunta e nessa resposta.

  • Preferir usar composição do que herança, e portanto não herdar de JFrame. Falo sobre isso nessa resposta. Também abordo essa questão (e muitas outras coisas) nessa outra resposta.

  • Usar números mágicos não é uma boa prática de programação. Fiz o seu código sem usar números mágicos e também de uma forma que se o tamanho da tela ou o tamanho do label mudarem, ele não vai ficar bagunçado e vai se ajustar à mudança sozinho.

  • Pintei o JLabel de amarelo para que você possa ver exatamente onde ele está e qual é o seu tamanho.

  • Alterei o nome de algumas variáveis para ficarem mais representativos.

  • Use KeyAdapter e MouseAdapter ao invés de implementar diretamente KeyListener e MouseListener para não pecisar ter aquele monte de métodos vazios.

Fonte Link
Victor Stafusa
  • 64,9mil
  • 9
  • 120
  • 189

Consegui fazer assim:

import java.awt.Color;
import java.awt.EventQueue;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import javax.swing.ImageIcon;
import javax.swing.JFrame;
import javax.swing.JLabel;

public class Jogo1 {

    private static final int KEY_CODE_DOWN = 38;
    private static final int KEY_CODE_UP = 40;
    private static final int KEY_CODE_LEFT = 37;
    private static final int KEY_CODE_RIGHT = 39;

    private static final int DISTANCIA_PASSO = 20;
    private static final int LARGURA_INICIAL_TELA = 750;
    private static final int ALTURA_INICIAL_TELA = 700;
    private static final int POSICAO_X_INICIAL = 300;
    private static final int POSICAO_Y_INICIAL = 300;
    private static final int LARGURA_FLAPPY_INICIAL = 180;
    private static final int ALTURA_FLAPPY_INICIAL = 90;

    private int posicaoX;
    private int posicaoY;

    public Jogo1() {
        JFrame tela = new JFrame("Magisterix");
        tela.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); // Encerra ao fechar.
        tela.setSize(LARGURA_INICIAL_TELA, ALTURA_INICIAL_TELA); // Ajusta o tamanho.
        tela.setLocationRelativeTo(null); // Centraliza.
        tela.setVisible(true); // Torna visível.
        tela.setResizable(false); // Impossibilita o redimensionamento pelo usuário.
        tela.setLayout(null); // Permite redimensionamento de cada componente.

        JLabel labelFlappy = new JLabel("0");
        labelFlappy.setOpaque(true);
        labelFlappy.setBackground(Color.YELLOW);
        tela.add(labelFlappy);
        posicaoX = POSICAO_X_INICIAL;
        posicaoY = POSICAO_Y_INICIAL;
        labelFlappy.setBounds(posicaoX, posicaoY, LARGURA_FLAPPY_INICIAL, ALTURA_FLAPPY_INICIAL);

        tela.addKeyListener(new KeyAdapter() {
            @Override
            public void keyPressed(KeyEvent e) {
                System.out.println(e.getKeyCode());
                if (e.getKeyCode() == KEY_CODE_DOWN) {
                    posicaoY -= DISTANCIA_PASSO;
                } else if (e.getKeyCode() == KEY_CODE_UP) {
                    posicaoY += DISTANCIA_PASSO;
                } else if (e.getKeyCode() == KEY_CODE_LEFT) {
                    posicaoX -= DISTANCIA_PASSO;
                } else if (e.getKeyCode() == KEY_CODE_RIGHT) {
                    posicaoX += DISTANCIA_PASSO;
                }
                posicaoX = limitar(0, posicaoX, tela.getContentPane().getWidth() - labelFlappy.getWidth());
                posicaoY = limitar(0, posicaoY, tela.getContentPane().getHeight() - labelFlappy.getHeight());
                labelFlappy.setBounds(posicaoX, posicaoY, labelFlappy.getWidth(), labelFlappy.getHeight());
            }
        });

        tela.addMouseListener(new MouseAdapter() {
            @Override
            public void mouseClicked(MouseEvent e) {
                posicaoX = (int) (Math.random() * (tela.getContentPane().getWidth() - labelFlappy.getWidth()));
                posicaoY = (int) (Math.random() * (tela.getContentPane().getHeight() - labelFlappy.getHeight()));
                labelFlappy.setBounds(posicaoX, posicaoY, labelFlappy.getWidth(), labelFlappy.getHeight());
            }
        });
    }

    private static int limitar(int min, int meio, int max) {
        return meio < min ? min : meio > max ? max : meio;
    }

    public static void main(String[] args) {
        EventQueue.invokeLater(Jogo1::new);
    }
}

O truque está em limitar posPrincX e posPrincY ao tamanho da janela (ou posicaoX e posicaoY, vez que renomeei as variáveis). Para isso, uso o método limitar(int, int, int).

Fiz mais algumas outras mudanças no seu código:

  • Só criar, interagir, usar e acessar componentess Swing dentro da EDT. Para isso, uso o EventQueue.invokeLater(Runnable). Veja mais sobre isso nessa pergunta e nessa resposta.

  • Preferir usar composição do que herança, e portanto não herdar de JFrame. Falo sobre isso nessa resposta. Também abordo essa questão (e muitas outras coisas) nessa outra resposta.

  • Usar números mágicos não é uma boa prática de programação. Fiz o seu código sem usar números mágicos e também de uma forma que se o tamanho da tela ou o tamanho do label mudarem, ele não vai ficar bagunçado e vai se ajustar à mudança sozinho.

  • Pintei o JLabel de amarelo para que você possa ver exatamente onde ele está e qual é o seu tamanho.

  • Alterei o nome de algumas variáveis para ficarem mais representativos.

  • Use KeyAdapter e MouseAdapter ao invés de implementar diretamente KeyListener e MouseListener para não pecisar ter aquele monte de métodos vazios.