JFrame расширяется при экспорте в Jar - PullRequest
Новая
       11

JFrame расширяется при экспорте в Jar

0 голосов
/ 09 ноября 2019

Добрый день, я создаю какую-то простую змеиную игру, чтобы узнать больше о Threading и Graphics в Java. И в моем проекте Eclipse все, что касается кода, работает не идеально, но достаточно хорошо для моих предпочтений. Теперь я попытался экспортировать этот проект из Eclipse в исполняемый файл .jar, и почему-то внезапно кадр стал другого размера, чем я его назначил, и все неожиданно оказалось больше, чем должно быть, что также смешалось с прямоугольником, который я установил как «рамку». «для игрового поля и пр. Я экспортирую как Runnable JAR File с опцией «Упаковать необходимые библиотеки в сгенерированный jar». У кого-нибудь есть идея относительно того, почему это так, и что я могу сделать, чтобы это исправить или оптимизировать мою программу для учета этих вещей? Вот мой код для всего, что используется в фрейме: 

import java.awt.BasicStroke;
import java.awt.Canvas;
import java.awt.Color;
import java.awt.Font;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.image.BufferStrategy;

import javax.swing.JFrame;
import java.awt.Rectangle;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class Grid extends Canvas{

    private JFrame _container;
    private int _size;

    private int[] food;

    private boolean running;
    private boolean alive;

    private Snake s;
    private Rectangle _border;

    private int score;
    private int highscore;

    private BufferStrategy bs;

    public Grid() {
        super();

        _container = new JFrame("Snake Final");
        _container.setSize(622,656);
        _container.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        _container.setVisible(true);

        setSize(600, 600);
        setVisible(true);

        _container.add(this);

        createBufferStrategy(2);
        bs = getBufferStrategy();

        food = new int[2];
        highscore = 0;

        init();
    }

    private void init() {
        _size = 30;
        _border = new Rectangle(0,0,getWidth(),getHeight());
        s = new Snake(_size, _border, this);

        setKeyListener();
        running = true;

        generateFood();

        run();

    }

    public void run() {
        running = true;
        while(running) {
            s.update();
            if(s.checkFood(food)) {
                generateFood();
            }
            draw();

            try {
                Thread.sleep(100);
            } catch (InterruptedException e) {}
        }
    }

    public void gameOver() {
        alive = false;
        if(score > highscore) {
            highscore = score;
        }
        Graphics g = this.getGraphics();
        g.setColor(Color.black);
        g.fillRect(0, 0, getWidth(), getHeight());
        g.setColor(Color.white);

//      g.drawLine(getWidth() / 2, 0, getWidth() / 2, getHeight());
//      g.drawLine(0, getHeight() / 2, getWidth(), getHeight() / 2);


    }

    private void setKeyListener() {
        this.addKeyListener(new KeyListener() {

            @Override
            public void keyPressed(KeyEvent arg0) {

                if(arg0.getKeyCode() == KeyEvent.VK_W) {
                    if(running) {
                        if(s.getYDir() == 0) {
                            s.changeDirection(0, -1);
                        }
                    }
                }
                else if(arg0.getKeyCode() == KeyEvent.VK_S) {
                    if(running) {
                        if(s.getYDir() == 0) {
                            s.changeDirection(0, 1);
                        }
                    }
                }
                else if(arg0.getKeyCode() == KeyEvent.VK_A) {
                    if(running) {
                        if(s.getXDir() == 0) {
                            s.changeDirection(-1, 0);
                        }
                    }
                }
                else if(arg0.getKeyCode() == KeyEvent.VK_D) {
                    if(running) {
                        if(s.getXDir() == 0) {
                            s.changeDirection(1, 0);
                        }
                    }
                }
                else if(arg0.getKeyCode() == KeyEvent.VK_ENTER) {
                    alive = true;
                    s.setPosition(4 * _size, 4 * _size);
                    s.changeDirection(1, 0);
                }
            }

            @Override
            public void keyReleased(KeyEvent arg0) {}

            @Override
            public void keyTyped(KeyEvent arg0) {}
        });
    }

    private void generateFood() {       
        int gridParts = (getWidth()/_size) - 4;
        for(int i = 0; i < food.length; i++) {
            food[i] = ((int) (Math.random() * (gridParts)) * _size) + 2 * _size;
        }
    }

    private void draw() {
        Graphics bg = bs.getDrawGraphics();
        bg.clearRect(0, 0, getWidth(), getHeight());
        if(alive) {

            Graphics2D bg2d = (Graphics2D) bg;

            bg2d.setStroke(new BasicStroke(_size*4));
            bg2d.drawRect(0,0,getWidth(),getHeight());

            s.show(bg2d);
//          bg2d.fillRect(food[0] * _size, food[1] * _size, _size, _size);

            score = s.getSize();

            bg.setColor(Color.white);
            bg.setFont(new Font("SansSerif", Font.PLAIN, 20));
            bg.drawString(String.valueOf(score), 56, 35);

            bg2d.setColor(Color.RED);
            bg2d.fillRect((food[0] + 5), (food[1] + 5), _size - 10, _size - 10);

            bg2d.dispose();
        }
        else {
            bg.setColor(Color.black);
            bg.fillRect(0, 0, getWidth(), getHeight());

            bg.setColor(Color.white);
            bg.setFont(new Font("SansSerif", Font.PLAIN, 20));
            bg.drawString("Game Over", getWidth() / 2 - 55, getHeight() / 2 - 50);
            bg.drawString("Score: " + String.valueOf(score), getWidth() / 2 - 55, getHeight() / 2 - 20);
            bg.drawString("Highscore: " + String.valueOf(highscore), getWidth() / 2 - 55, getHeight() / 2 + 10);
            bg.drawString("Press Enter to restart", getWidth() / 2 - 55, getHeight() / 2 +40);
        }
        bs.show();
        bg.dispose();
    }

}

И извините, это действительно грязный код, но, как гласит старая поговорка, зачем реорганизовывать его, если он на самом деле работает вроде как хорошо

1 Ответ

0 голосов
/ 09 ноября 2019 
    _container = new JFrame("Snake Final");
    _container.setSize(622,656);
    _container.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    _container.setVisible(true);

    setSize(600, 600);
    setVisible(true);

    _container.add(this);

Вы не должны жестко задавать размер компонента. Размер вашей панели игнорируется, потому что Swing был разработан для использования с менеджерами компоновки. BorderLayout по умолчанию установит размер вашего холста на основе пространства, доступного в кадре.

Вместо этого вы даете предложение менеджеру по расположению. Код должен выглядеть примерно так: 

    _container = new JFrame("Snake Final");
    _container.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    //setSize(600, 600);
    //setVisible(true); // not needed components are visible by default
    setPreferredSize( new Dimension(600, 600) );
    _container.add(this);

    //_container.setSize(622,656);
    _container.pack();
    _container.setVisible(true);

Таким образом, компонент добавляется в фрейм перед упаковкой () и фрейм становится видимым. Тогда размер фрейма будет правильно определен методом pack (). Он будет включать в себя предпочтительный размер холста и позволит декорировать рамку (границу и строку заголовка).

Не знаю, решит ли это вашу проблему с созданием файла Jar (возможно,есть какое-то свойство, которое переопределяет упакованный размер?), но это лучший подход к проектированию структуры Swing.

...