一つのクラスに複数のクラスからアクセスする正しい方法が知りたいです。

前提・実現したいこと

下記の物はウィンドウとボタンを表示するだけのプログラムなのですが、新たにTest.javaを作りその中の情報をUIクラスに与えたいときにTest.java内で
UI ui = nwe UI();
ui.createUI(情報);
としたくなるのですが合っていますでしょうか？Gameクラス内でもインスタンス化しているので違う気がしてモヤモヤしています。
正しいやり方などがあればお聞きしたいです。よろしくお願いします。

該当のソースコード

package package01;

public class Game {
	
	UI ui = new UI();
	public static void main(String[] args) {

		new Game();

	}
	
	public Game(){
		
		ui.createUI();
		
	}

}

package package01;

import java.awt.Color;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class UI {

	JFrame window;
	JPanel startButtonPanel;
	JButton startButton;
	
	public void createUI() {
		
		window = new JFrame();
		window.setSize(800,600);
		window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		window.getContentPane().setBackground(Color.black);
		window.setLayout(null);
		
		startButtonPanel = new JPanel();
		startButtonPanel.setBounds(300, 400, 200, 100);
		startButtonPanel.setBackground(Color.black);
		startButton = new JButton("start");
		startButtonPanel.add(startButton);
		
		window.add(startButtonPanel);	
		
		window.setVisible(true);
		
	}
}

m.ts10806

2020/08/03 07:34

そのクラスの情報を共有するのか独立させるかどうかで違います。

clothblue

2020/08/03 07:35

共有する方法を知りたいです。

dodox86

2020/08/03 07:37

往々にして明らかに間違った方法は存在するけど、唯一の正しい方法は無い気が。間違った方法は大抵、見た瞬間に「コレチガウ」感がありますね。

sazi

2020/08/03 08:02 編集

>ui.createUI(情報); >としたくなるとは、質問のコードをそのように変更するのは正しいアプローチなのか？という問いですか？ getter／setterの話のような気もするし・・

行動規範の内容に同意します

回答1件

ベストアンサー

質問の意味を勘違いしてるかもしれませんが・・・
Testクラスから、GameクラスのUiにアクセスしたいってことなら。
現状は、package01の中からなら何もせずに直接アクセスできます。

java
1// 作成するTestクラスが、package01に属していることが重要(Gameクラスがpackage01なので)
2package package01; 
3
4public class Test {
5    public static void main(String[] args) {
6        Game g = new Game();
7        Ui ui = g.ui;
8        ui.createUI(情報);
9    }
10}

あるクラス内のフィールド(メンバ変数)にアクセスしたい場合、上記のように
可視性を何もつけない(同じパッケージからアクセス可能)かpublicにします。

ただ、これだと何でもできちゃうので、困るという場合は
privateにして、必要な操作だけメソッドを用意します。

java
1public class Game {
2    private UI ui = new UI(); // privateにすることで直接アクセスできないようにする
3    // 必要な操作を公開する
4    void setColor(Color color) {
5        ui.startButtonPanel.setBackground(color);
6    }
7    // UIを直接操作させたいならUIを返すメソッドを用意する
8    public UI getUi() {
9        return ui;
10    }
11}