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.