別ウインドウに画像と文字を表示しない

質問

こんにちは。
　ボタンをクリックすると、別ウインドウが開いて、画像と文字が表示される
アプレットを作りたいのですが、別ウインドウは開きますが、画像と文字は
表示されません。

　何度考えても、全く原因が分かりません。
　皆さんのご教示宜しくお願いします。

================================================================================
public class ImageTest extends Applet implements ActionListener
{
Image img1 , img2 ;
JLabel jb1 ;
int lblnum ;
MyPanel1 mp1 ;
Graphics g ;

public void init()
{
img1 = getImage( getDocumentBase() , "star1.jpg" ) ;

this.setBackground( Color.yellow ) ;
this.setLayout( null ) ;

JPanel jp1 = new JPanel() ;
jp1.setSize( 1500 , 850 ) ;
jp1.setLayout( null ) ;
jp1.setBackground( Color.yellow ) ;

JPanel jp2 = new JPanel() ;
jp2.setBackground( Color.magenta ) ;

this.add( jp1 ) ;
this.add( jp2 ) ;

JLabel jl1 = new JLabel( new ImageIcon( "star1.gif" ) ) ;
jl1.setSize( 150 , 150 ) ;
jl1.setLocation( 50 , 50 ) ;
jp1.add( jl1 ) ;

JButton jb1 = new JButton( "star" ) ;
jb1.setMargin(new Insets( 1 , 1 , 1 , 1 ) ) ;
jb1.setBorderPainted( false ) ;
jb1.setForeground( Color.red ) ;
jb1.setBackground( Color.yellow ) ;
jb1.setLocation( 75 , 210 ) ;
jb1.setSize( 100 , 50 ) ;
jb1.setFont( new Font( "SansSerif" , Font.ITALIC , 20 ) ) ;
jb1.addActionListener( this ) ;
jp1.add( jb1 ) ;
}

public void actionPerformed( ActionEvent e )
{
Object source = e.getSource() ;
if( source == jb1 ) lblnum = 1 ;

MyPanel1 mp1 = new MyPanel1( lblnum , img1 ) ;
mp1.setVisible( true );
repaint();
}

class MyPanel1 extends Frame
{
int k ;
String str ;
Image img ;

public MyPanel1( int k , Image img )
{
this.k = k ;
this.img = img ;
setSize( 500 , 500 ) ;
setBackground( Color.magenta ) ;
setLocation( 300 , 300 ) ;
}

public void init( )
{
switch( k )
{
case 1: str = "test" ; break ;
}
setVisible( false ) ;
}

public void paint( Graphics g )
{
g.drawImage( img , 0 , 0 , 30 , 30 , this ) ;
g.drawString( str , 150 , 150 ) ;
}
}
}
================================================================================

回答No.3

salsberry · ベストアンサー

> 「source == jb1」が間違いなら、どのように書くのが正解なのでしょうか。

"if( source == jb1 )"の部分自体は書き換える必要はありません。

まずは下記のことを理解してください。
・質問者さんのコードにはjb1という名前の変数が2つ存在しています。
　(A) init()のローカル変数でJButton型のもの
　(B) ImageTestのインスタンス変数でJLabel型のもの
・(A)にはinit()の中で値が代入されていますが、(B)には値が代入されていないので常にnullです。
・actionPerformed()でsourceと比較されているのは(B)であるため、条件が真になりません。
質問者さんは、sourceの値とinit()で(A)に代入した値を比較したいのですよね？

どう書けばいいかですが、(A)(B)2つのjb1を使い分ける必要はなさそうなので1つにまとめればいいだけです。init()のローカル変数をactionPerformed()から参照することは不可能なので、インスタンス変数である(B)のほうを使います。
・ImageTestのインスタンス変数のjb1の型をJLabelからJButtonに変えます。
・init()の中でローカル変数のjb1を作っているのをやめて、代わりにインスタンス変数のjb1を使うようにします。具体的には、JButton jb1 = new JButton( "star" );の行のjb1の前にあるJButtonを消します。

> しかし、imgは表示されません。

質問者さんがどのような環境でどのようにAppletを実行しているのかが分かりませんが、適切な場所(ImageTest.classと同じ場所)に画像ファイルを置けば表示されましたよ。画像ファイルを読めなくてもdrawImage()が例外を発生することはないみたいですが。

補足

salsberryさん、有難うございました。

　同じ変数を２つの型で定義していたとは全く気がつきませんでした。
　本当に有難うございました、今後とも宜しくお願いします。

回答No.2

salsberry

init()またはstringCase()が呼ばれていないことをどうやって確認しましたか？ お礼にある書き方であれば呼ばれているはずです。

もう一つの問題は、actionPerformed()の中でif( source == jb1 )の条件が成立することがなくlblnumが1にならないために、stringCase()が呼ばれてもstrへの代入が発生しないことです。stringCase()のswitchの中に
default: str="Error"; break;
という行を追加しておけば、とりあえず文字が描画されないということはなくなります。

source == jb1が真にならないのは、init()の中で使われているJButton型のjb1とactionPerformed()の中から見えているJLabel型のjb1は別物だからです。いろいろいじっている間にこうなってしまったのでしょうが、外側と内側のスコープで同名の変数を使うのは間違いの元です。

お礼

salsberryさん、回答有難う御座います。

　stringCase();の中で「default: str="Error";break;」をかいてやると、「Error」が
表示されました。

　しかし、imgは表示されません。
　同じ原因だと思いますが、imgが表示されないことについて、「NullPointException・・・」
というエラーも吐きません。

　＞source == jb1が真にならないのは、init()の中で使われているJButton型のjb1と
　＞actionPerformed()の中から見えているJLabel型のjb1は別物だからです。
　＞外側と内側のスコープで同名の変数を使うのは間違いの元です。

の意味がよく理解出来ません。

　この場合の「source」は、「 jb1」ではないということでしょうか。
　「source == jb1」が間違いなら、どのように書くのが正解なのでしょうか。
　宜しくお願いします。

回答No.1

salsberry

試してみました。
MyPanel1.paint()でg.drawImage()による画像の描画は実行されますが、その次のg.drawString()で例外が発生します。

Exception in thread "AWT-EventQueue-1" java.lang.NullPointerException: String is null
at sun.java2d.SunGraphics2D.drawString(SunGraphics2D.java:3045)
at qa7816901$MyPanel1.paint(qa7816901.java:82)

このメッセージの中に書かれているとおり、drawString()に渡しているstrがnullであることが原因です。
MyPanel1.init()で"test"という文字列を代入しているからnullではないはず、と思っているかもしれませんが、MyPanel1.init()が呼ばれることはないのでstrはnullのままです。Appletのサブクラスであればinit()が自動的に呼ばれますが、FrameのサブクラスであるMyPanel1の場合は誰かが勝手にinit()を呼んでくれるということはありません。

お礼

salsberryさん、回答有難うございます。

　・　「init();」をそのままにして、「mp1.init();」をメインクラスに書いてやりましたが、メインから
　　呼ばれません。

　・　「init();」を「stringCase( );」というメッソドに変えましたが、メインクラスからは呼ば
　　れません。

　・　「mp1.stringCase( );」をメインクラスに書いてやりましたが、やはりメインクラスから呼ばれま
　　せん。

　全く原因が分かりません、何故でしょうか。宜しくお願いします。

========================================================================

public class ImageTest extends Applet implements ActionListener
{
Image img1 , img2 ;
JLabel jb1 ;
int lblnum ;
MyPanel1 mp1 ;
Graphics g ;

public void init()
{
img1 = getImage( getDocumentBase() , "star1.jpg" ) ;

this.setBackground( Color.yellow ) ;
this.setLayout( null ) ;

JPanel jp1 = new JPanel() ;
jp1.setSize( 1500 , 850 ) ;
jp1.setLayout( null ) ;
jp1.setBackground( Color.yellow ) ;

JPanel jp2 = new JPanel() ;
jp2.setBackground( Color.magenta ) ;

this.add( jp1 ) ;
this.add( jp2 ) ;

JLabel jl1 = new JLabel( new ImageIcon( "star1.gif" ) ) ;
jl1.setSize( 150 , 150 ) ;
jl1.setLocation( 50 , 50 ) ;
jp1.add( jl1 ) ;

JButton jb1 = new JButton( "star" ) ;
jb1.setMargin(new Insets( 1 , 1 , 1 , 1 ) ) ;
jb1.setBorderPainted( false ) ;
jb1.setForeground( Color.red ) ;
jb1.setBackground( Color.yellow ) ;
jb1.setLocation( 75 , 210 ) ;
jb1.setSize( 100 , 50 ) ;
jb1.setFont( new Font( "SansSerif" , Font.ITALIC , 20 ) ) ;
jb1.addActionListener( this ) ;
jp1.add( jb1 ) ;
}

public void actionPerformed( ActionEvent e )
{
Object source = e.getSource() ;
if( source == jb1 ) lblnum = 1 ;

MyPanel1 mp1 = new MyPanel1( lblnum , img1 ) ;
// mp1.init( );
mp1.stringCase( );
mp1.setVisible( true );
repaint();
}

class MyPanel1 extends Frame
{
int k ;
String str ;
Image img ;

public MyPanel1( int k , Image img )
{
this.k = k ;
this.img = img ;
setSize( 500 , 500 ) ;
setBackground( Color.magenta ) ;
setLocation( 300 , 300 ) ;
}

// public void init( )
public void stringCase( )
{
switch( k )
{
case 1: str = "test" ; break ;
}
setVisible( false ) ;
}

public void paint( Graphics g )
{
g.drawImage( img , 0 , 0 , 30 , 30 , this ) ;
g.drawString( str , 150 , 150 ) ;
}
}

}

=============================================================================