javaについて。このプログラムでラベルとボタンが表示されません。

Question

実現したいこと

・黒い背景と、画像を表示される

前提

・開発環境はVCcodeです。
・中学生で独学で学んでいます。
・webなどから参照したコードではありません。
・javaをはじめて1ヶ月ほどです。コードがきたないですが、ご了承ください

発生している問題

・フレーム以外何も表示されません

該当のソースコード

・javaです

import javax.swing.*;
import java.awt.*;

public class A {
    public static void main(String[] args) {
        B b = new B();
        b.AB();
    }
}

class B extends JFrame{
    void AB() {
                JFrame f = new JFrame("GAME");
                f.setBounds(0,0,1200,700);
                f.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
                f.setLocationRelativeTo(null);
        
                JLabel l1 = new JLabel("label1");
                JLabel l2 = new JLabel("label2");
                
                ImageIcon icon1 = new ImageIcon("./img/ShootingHome1.png"); 

                l1.setBounds(100,100,1000,300);
                l2.setBounds(0,0,1200,700);
        
                l2.setOpaque(true);
                l2.setBackground(Color.BLACK);
                       
                l1.setIcon(icon1);

                JPanel p1 = new JPanel();
                p1.setBounds(0, 0, 1200, 700);
                p1.setLayout(null);
                         
                p1.add(l2);
                p1.add(l1);

                Container contentPane = getContentPane();
                contentPane.add(p1);
                
                f.setVisible(true);
            }
}

実行結果

試したこと

・特にないです。問題点がわかりませんでした

補足情報

・VCcode使用
・jdk-21.0.6.7-hotspotです。

Comments

[jimbe]

コードのご提示にはコードのマークダウン(```で上下を囲む)をご利用ください。
通常の文として書いてしまうと、 * 等が他のマークダウンと解釈されて消えたりしてしまいます。

[momijimomikko]

申し訳ございません。初めての投稿だったので、これからは修正します。ありがとうございました！

[jimbe]

編集ありがとうございます。

[momijimomikko]

こちらこそ、教えていただきありがとうございました。
これからは気を付けるようにします。

Answer 1

クラス B は JFrame クラスを継承していますから(class B extends JFrame { ... })、新たに JFrame クラスのインスタンスを作成する必要はない様に思います。(JFrame f = new JFrame("GAME"); 部分)

以下のコードではクラス A のコンストラクタで表示処理を行う様にしています。

import javax.swing.*;
import java.awt.*;

class A extends JFrame {
  public static void main(String[] args) {
    new A();
  }

  A() {
    setTitle("Game");
    setBounds(0, 0, 1200, 700);
    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    setLocationRelativeTo(null);

    JLabel l1 = new JLabel("label1");
    JLabel l2 = new JLabel("label2");
    l1.setBounds(100, 100, 1000, 300);
    l1.setForeground(Color.WHITE);
    l2.setBounds(0, 0, 1200, 700);
    l2.setForeground(Color.WHITE);
    l2.setBackground(Color.BLACK);
    l2.setOpaque(true);

    ImageIcon icon1 = new ImageIcon("./img/ShootingHome1.png"); 
    l1.setIcon(icon1);
                   
    JPanel p1 = new JPanel();
    p1.setBounds(0, 0, 1200, 700);
    p1.setLayout(null);
    p1.add(l1);
    p1.add(l2);
    // p1.setComponentZOrder(l1, 0);
    add(p1);

    setVisible(true);
  }
}

Comments

[momijimomikko]

回答ありがとうございます。
ベストアンサーは別の方を選ばせていただきましたが、こちらの回答も非常に参考になりました。
べつのところでも生かせるようにします。
ありがとうございました！

Answer 2

どのオブジェクトを表示するのかをキチンと把握する必要があります。
B は JFrame を継承していますが実際に表示されるのは B では無く B.AB() 内で作成(JFrame f = new JFrame("GAME");)している JFrame です。(なので B の extends JFrame は消したほうが良いです。)
従って各コンポーネントはこの f に設定しなければなりませんが、 JPanel p1 を載せる際に

    Container contentPane = getContentPane();
    contentPane.add(p1);

としてしまっており、これでは contentPane は B のコンテンツペインです。
前述のように実際に表示されるのは f ですので、

    Container contentPane = f.getContentPane();
    contentPane.add(p1);

とする必要があります。

---

ついでに余計なお世話ですが、いくら初心者とはいえ公開する(ことになった)コードに A とか B とかいうクラス・メソッド名はテキトウ過ぎです。
"コードがきたない" と思われたのでしたら、整理しましょう。汚い所為でバグが見え難い場合もあります。
整理するなら、処理内容毎では無く対象オブジェクト毎にするほうがオブジェクト指向的にやり易いと思います。(個人の見解です。)

    JLabel l1 = new JLabel("label1");
    JLabel l2 = new JLabel("label2");
            
    ImageIcon icon1 = new ImageIcon("./img/ShootingHome1.png"); 

    l1.setBounds(100,100,1000,300);
    l2.setBounds(0,0,1200,700);

    l2.setOpaque(true);
    l2.setBackground(Color.BLACK);

    l1.setIcon(icon1);

↓

    JLabel l1 = new JLabel("label1");
    l1.setBounds(100, 100, 1000, 300);
    l1.setIcon(new ImageIcon("./img/ShootingHome1.png"));

    JLabel l2 = new JLabel("label2");
    l2.setBounds(0, 0, 1200, 700);
    l2.setOpaque(true);
    l2.setBackground(Color.BLACK);

Comments

[momijimomikko]

回答ありがとうございます。
無事、解決しました！
あと、これからはクラス、メソッドの名前は気を付けるようにします。
本当にありがとうございました！