複数のクラスでのペイント

Question

私はいつもより簡単に私のコードをすべて1クラスで行うことができたので、Javaで複数のクラスを使うのはあまり良くありません。最近、私が作っているゲームに2番目のクラスを使う必要があることがわかりました。私はエラーに遭遇しています。

今は、ユーザーがクリックする場所と時間に敵をスポーンしようとしています。

メインクラス -

package joey.rts; 
import java.awt.Graphics; 
import java.awt.Graphics2D; 
import java.awt.event.MouseEvent; 
import java.awt.event.MouseListener; 
import java.awt.image.BufferedImage; 
import java.io.File; 

import javax.imageio.ImageIO; 
import javax.swing.JFrame; 
public class RTSMain extends JFrame implements MouseListener{ 
    /** 
    * 
    */ 
    private static final long serialVersionUID = -7122370886923000314L; 
    public static BufferedImage menu,enemy; 
    public static boolean onmenu,oneenemy; 
    public static void main(String[] args){ 
     new RTSMain(); 
    } 
    public RTSMain(){ 
     init(); 
    } 
    public void init(){ 
     setSize(1700,1100); 
     setVisible(true); 
     setLocationRelativeTo(null); 
     setResizable(false); 
     setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     setTitle("RTS"); 
     addMouseListener(this); 
    } 
    public void paint(Graphics g){ 
     Graphics2D g2 = (Graphics2D) g; 
     if(onmenu == true){ 
      g2.drawImage(menu,0,0,this); 
     } 
    } 
    @Override 
    public void mouseClicked(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 
    @Override 
    public void mouseEntered(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 
    @Override 
    public void mouseExited(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 
    @Override 
    public void mousePressed(MouseEvent e) { 
     Enemy enemy = new Enemy(); 
     int x = e.getX(); 
     int y = e.getY(); 
     enemy.spawnEnemy(x, y); 

    } 
    @Override 
    public void mouseReleased(MouseEvent arg0) { 
     // TODO Auto-generated method stub 

    } 
} 

敵クラス - 自分のメインクラスで

package joey.rts; 

import java.awt.Graphics; 
import java.awt.Graphics2D; 
import java.awt.image.BufferedImage; 
import java.io.File; 

import javax.imageio.ImageIO; 

public class Enemy{ 
    /** 
    * 
    */ 
    public static BufferedImage enemy; 
    private static final long serialVersionUID = 7898827977636314494L; 
    public static RTSMain rts; 
    public static void main(String[] args){ 
     try{ 
      enemy = ImageIO.read(new File(javax.swing.filechooser.FileSystemView.getFileSystemView().getHomeDirectory() + "\\enemy.png")); 
     } catch (Exception e){ 
      e.printStackTrace(); 
     } 

    } 
    public static void spawnEnemy(int x, int y){ 
     Graphics g = rts.getGraphics(); 
     Graphics2D g2 = (Graphics2D) g; 
     g2.drawImage(enemy,x,y,null); 
    } 
} 

Answer 1

であるためにあなたのmouseClicked機能を更新する：あなたが必要な場合は

@Override 
    public void mouseClicked(MouseEvent e) { 
     Enemy enemy = new Enemy(); 
     int x=e.getX(); // get mouse positionX 
     int y=e.getY();//get mouse positionY 
     enemy.spawnEnemy(x,y);//spawn Enemy 
    } 

は、敵オブジェクトの保存を検討後で再利用する。 また、敵クラスで何かを拡張する必要はありません。

私が更新しましたあなたのメインとあなたの敵のクラス：

import java.awt.Graphics; 
import java.awt.Graphics2D; 
import java.awt.event.MouseEvent; 
import java.awt.event.MouseListener; 
import java.awt.image.BufferedImage; 
import javax.swing.JFrame; 
public class Main extends JFrame implements MouseListener{ 
    /** 
    * 
    */ 
    private static final long serialVersionUID = -7122370886923000314L; 
    public static BufferedImage menu,enemy; 
    public static boolean onmenu,oneenemy; 
    public static void main(String[] args){ 
     new Main().setVisible(true); 
    } 
    public Main(){ 
     init(); 
    } 
    public void init(){ 
     setSize(1700,1100); 
     setVisible(true); 
     setLocationRelativeTo(null); 
     setResizable(false); 
     setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     setTitle("RTS"); 
     addMouseListener(this); 
    } 

    @Override 
    public void paint(Graphics g){ 
     Graphics2D g2 = (Graphics2D) g; 
     if(onmenu == true){ 
      g2.drawImage(menu,0,0,this); 
     } 
    } 
    @Override 
    public void mouseClicked(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 
    @Override 
    public void mouseEntered(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 
    @Override 
    public void mouseExited(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 
    @Override 
    public void mousePressed(MouseEvent e) { 
     Enemy enemy = new Enemy(); 
     int x = e.getX(); 
     int y = e.getY(); 
     enemy.spawnEnemy(x, y,this.getGraphics()); 

    } 
    @Override 
    public void mouseReleased(MouseEvent arg0) { 
     // TODO Auto-generated method stub 
    } 
} 

そして、これは敵のクラスである：私は敵のクラスにあったメインのインスタンス変数を削除しました

import java.awt.Graphics; 
import java.awt.Graphics2D; 
import java.awt.image.BufferedImage; 
import java.io.File; 
import java.io.IOException; 
import java.util.logging.Level; 
import java.util.logging.Logger; 
import javax.imageio.ImageIO; 

public class Enemy{ 

    public static BufferedImage enemy; 
    private static final long serialVersionUID = 7898827977636314494L; 
    public Enemy(){ 
     try { 
      //MAKE SURE THAT THIS IS THE CORRECT IMAGE PATH 
     enemy = ImageIO.read(new File(javax.swing.filechooser.FileSystemView.getFileSystemView().getHomeDirectory() + "\\enemy.png")); 
     } catch (IOException ex) { 
      Logger.getLogger(Enemy.class.getName()).log(Level.SEVERE, null, ex); 
     } 
    } 
    public void spawnEnemy(int x, int y,Graphics g){ 
     Graphics2D g2 = (Graphics2D) g; 
     g2.drawImage(enemy,x,y,null); 
    } 
} 

。

私はspawnEnemy関数の静的修飾子を削除しました。

私はグラフィックオブジェクトをspawnEnemy関数の属性として送信しました。

私はEnemyクラスのmainメソッドにあったコードをEnemy Constructorに移動しました。

希望すると助かります！

Answer 2

答えを貼り付けるのは、コメントで説明するよりも簡単だったようです。あなたのオブジェクト指向は少しばかりです。あなたのRTSMainクラスにこれを追加します。

protected ArrayList<Enemy> enemies = new ArrayList<Enemy>(); 
protected BufferedImage enemyImage = null; 
... 
public void init() { 
    ... 
    // everything you already have... 
    enemyImage = //read in your enemy image here 
    ... 
} 
... 

@Override 
public void mouseClicked(MouseEvent e) { 
    // this takes place of Enemy.spawn(), get rid of it 
    int x = e.getX(); // get mouse positionX 
    int y = e.getY(); //get mouse positionY 
    Enemy enemy = new Enemy(enemyImage, x, y); 
    Enemies.add(enemy); 
    // Iterate over list above to draw each enemy in your paintComponent method 
} 

また、JFrameのはあなたが（paint()が推奨されません）あなたの絵を行うためにオーバーライドする必要がpaintComponent()方法を、持っていません。 JPanelを追加し、paintComponent()などを上書きします。ゲームの場合、非常にターンベースでない限り、あなたのペインティングを行うためのゲームループを作成することをお勧めします。

ハッピーコーディング。