Javaは既に追加されたオブジェクトに変更を加えます

Question

私は現在、学校向けの小さなプロジェクトに取り組んでいます。私はまだ始まっています。私はちょうどJFrameとそのすべてのものに読んで始めたので、私があなたが私に示すすべてのものにそれほど精通していないかもしれないのだろうかと思ってはいけません。

ここでの目標は、画像を出力し、その画像のすべての単一ピクセルを手動で変更できるプログラムを作成することです。まあ、コードはこれまでのところ（xDさんをスポッティング厄介なエラーの多くの時間後に）動作しますが、私は何をしたい

public class Pixel extends Component { 
    private BufferedImage img; 
    private int width; 
    private int height; 
    private Graphics graphics; 

    public Pixel(int w, int h) { 
     width = w; 
     height = h; 
    } 

    public void create() { 
     img = new BufferedImage(width, height, BufferedImage.TYPE_INT_RGB); 

     //Set any color for now 
     for(int wc = 0; wc < width; wc++) { 
      for(int hc = 0; hc < height; hc++) { 
       img.setRGB(wc, hc, new Color(0xAAFFBB).getRGB()); 
      } 
     } 
    }  

    @Override 
    public void paint(Graphics g) { 
     super.paint(g); 
     graphics = g; 
     create(); 
     update(); 
    } 

    public void update() { 
     graphics.drawImage(img, 0, 0, null); 
    } 

    public void testChange() { 
     for(int i = 50; i < 80; i++) { 
      for(int j = 80; j < 120; j++) { 
       img.setRGB(i, j, new Color(0xFF8876).getRGB()); 
      } 
      for(int j = 460; j < 493; j++) { 
       img.setRGB(i, j, new Color(0xFF8876).getRGB()); 
      } 
     } 
    } 
} 

：

public class JavaGraphicsTest { 
    private static Pixel pixel; 

    public static void main(String[] args) { 
     JFrame frame = new JFrame(); 
     frame.setExtendedState(JFrame.MAXIMIZED_BOTH); 
     frame.setVisible(true); 
     pixel = new Pixel(1600, 900); 
     frame.getContentPane().add(pixel); 
     //pixel.testChange(); 
    } 
} 

と：したがって、私は、次のコードを書いています今は動作していないようです：mainメソッド（現在コメントされています）でメソッド "pixel.testChange（）"を呼び出したいと思います。しかし、私がJFrameの仕組みを理解している限り、JFrameに追加したオブジェクトは何もできません。しかし、それは誰のために働くべきですか？アクティブなオブジェクトを削除したり再追加したりすることなく、どのように変更できますか？

PS：testChangeメソッドが想定していることを理解できない場合は、画像の2ブロックを別の色に変更する必要があります。基本的に画像を正常に変更したかどうかを確認するテストです。

プロジェクトに関する詳細な情報が必要な場合は、私に聞いてください:)事前に

おかげで、 ジュリアン

Answer 1

をごtestChange方法（または、それを呼び出した後）の終わりには、あなたが上invalidateを呼び出す必要がありますPixelオブジェクトです。これは、コンポーネントが変更されたときに再描画されることをコンポーネントに通知するためのシグナルです。

また、this SO questionも確認してください。

Answer 2

Graphicsオブジェクトを保管しないでください。これまでグラフィックスオブジェクト

public class Pixel extends Component { 
    private Graphics graphics; // <<-- DO NOT DO THIS 

    @Override 
    public void paint(Graphics g) { 
     // ... 
     graphics = g; // <<-- DO NOT DO THIS 
     // ... 
    } 
} 

は#paint(Graphics g)が呼び出されるたびに再作成することができ、場合#paint(Graphics g)終了し、無効に破壊または破損することができます。

の呼び出し中に画像を作成しないでください。これは、Pixelが作成されたときに1回だけ実行する必要があります。

public class Pixel extends Component { 

    @Override 
    public void paint(Graphics g) { 
     // ... 
     create(); // <<-- DO NOT DO THIS, EITHER. 
     // ... 
    } 
} 

しかし、あなたが安全にpaint(Graphics g)から呼び出された他の方法にGraphicsオブジェクトを渡すことができます。

public class Pixel extends Component { 
    // ... 

    public Pixel(int w, int h) { 
     width = w; 
     height = h; 
     create(); 
    } 

    @Override 
    public void paint(Graphics g) { 
     super.paint(g); 
     update(g); 
    } 

    private void update(Graphics g) { 
     graphics.drawImage(img, 0, 0, null); 
    } 
} 

です。手元のお仕事に - #testChange()メソッド。画像を変更した後、#repaint()呼び出しを使用してSwingにコンポーネントを再度描画するように要求するだけです。

public class Pixel extends Component { 

    public void testChange() { 
     int rgb = new Color(0xFF8876).getRGB()); // Cached, for efficiency. 

     for(int i = 50; i < 80; i++) { 
      for(int j = 80; j < 120; j++) { 
       img.setRGB(i, j, rgb); 
      } 
      for(int j = 460; j < 493; j++) { 
       img.setRGB(i, j, rgb); 
      } 
     } 

     repaint(); // <<-- Ask Swing to repaint the component. 
    } 
} 

---

最後の注意：あなたが本当にユーザーインターフェイススレッド（EDT）以外でのSwingオブジェクトを変更しないでください。例外は通常、最初のウィンドウを作成するときに表示可能になる前に許可されます。

public static void main(String[] args) { 
    JFrame frame = new JFrame(); 
    frame.setExtendedState(JFrame.MAXIMIZED_BOTH); 
    frame.setVisible(true);    // <<-- Window becomes visible here 
    pixel = new Pixel(1600, 900); 
    frame.getContentPane().add(pixel); // <<-- DANGEROUS!! 
} 

代わりに、あなたはこのようなコード再構築することができます

public static void main(String[] args) { 
    JFrame frame = new JFrame(); 
    frame.setExtendedState(JFrame.MAXIMIZED_BOTH); 
    pixel = new Pixel(1600, 900); 
    frame.getContentPane().add(pixel); // <<-- Safe - window not visible yet. 
    frame.setVisible(true);    // <<-- Window becomes visible here 
} 

ベター、SwingUtilities.invokeLater(...)を使用して、実際にEDTに切り替えることである。

public static void main(String[] args) { 
    SwingUtilities.invokeLater(new Runnable() { 
     @Override 
     public void run() { 
      JFrame frame = new JFrame(); 
      frame.setExtendedState(JFrame.MAXIMIZED_BOTH); 
      frame.setVisible(true);    // <<-- Window becomes visible here 
      pixel = new Pixel(1600, 900); 
      frame.getContentPane().add(pixel); // <<-- Safe - running on EDT. 
      pixel.testChange(); // <<-- Also safe - running on EDT 
     } 
    }); 
} 

ああ、Java8と、あなたは多くのRunnableボイラープレートコードを取り除くことができます：

public static void main(String[] args) { 
    SwingUtilities.invokeLater(() -> { 
     JFrame frame = new JFrame(); 
     frame.setExtendedState(JFrame.MAXIMIZED_BOTH); 
     frame.setVisible(true);    // <<-- Window becomes visible here 
     pixel = new Pixel(1600, 900); 
     frame.getContentPane().add(pixel); // <<-- Safe - running on EDT. 
     pixel.testChange(); // <<-- Also safe - running on EDT 
    }); 
}