他クラスで作成したMapを使用したい

Question

簡易的なチャットシステムを作成したいです。
ログインしたらその人が作成したチャンネルとチャットが表示される、discordに近いものにしたいです。

ユーザー→名前(name)、パス(pass)、チャンネルリスト(Map<String,List>data)
チャンネル→チャンネルの名前、チャット
を持たせればよいと考えたのですが、Channelクラスで作ったリストをUserクラスにどのように渡したらいいかわかりません。また、順序関係的にまずユーザーが作成されてその後チャンネルを追加していく形になるのですが、上記のような構造でも問題はないでしょうか？

つたない文章で申し訳ないです。java初心者なのですが現状できる最大限のことをしたつもりです。迷走してしまって拙い箇所多々あると思いますが、ご教授いただけますと幸いです。
よろしくお願いいたします。

該当のソースコード

//User.java
@Controller
public class User {//ユーザを登録する
    String name;//登録する名前
    String pass;//登録するパス
    String loginName;//ログインの時に入力した名前
    String loginPass;//ログインの時に入力したパス
    String returnPass;//HashMapから返されたパスを格納するところ
    Map<String,String> users = new HashMap<>();
    Map<String,Map>data = new HashMap<>();//ユーザーとチャンネルマップを結びつけるところ

    User(String name,String pass,Map data){
        this.name = name;
        this.pass = pass;
        this.data = data;
    }

    @RequestMapping("/addUser")
    public String addUser(@RequestParam("name")String name, @RequestParam("pass")String pass){
        users.put(name,pass);
        return "login.html";
    }


    public String getName(){
        return name;
    }

    public String getPass(String a){//名前を入力してマップからパスを返してもらう
        return users.get(a);
    }
 

    public void loginUser(String loginName,String loginPass){//ログイン情報が入る
        this.loginName = loginName;
        this.loginPass = loginPass;
    }

    //ログイン処理

    }

//Channel.java
public class Channel{
String channelName;//チャンネルの名前
    String checkName;//チャンネルの名前が存在するかMapを調査するため
    Boolean check;//あるかないかの判定


    List<String>chats = new ArrayList();//チャットのリスト
    Map<String,List>channel = new HashMap<>();//チャンネルのマップ Stringはチャンネル名、Listはチャット　→こことUserを関連させたい



    public Channel(String channelName,List<String>chats){
        this.channelName = channelName;
        this.chats = chats;
        channel.put(channelName,chats);//チャンネルに名前とチャットを入れる
    }
    @RequestMapping(value = "/create")
    public String addChannel(@RequestParam("channelName") String channelName){//チャンネルを新しく追加する
        channel.put(channelName,chats);
        //data.put(User.getName(), (Channel) channel);
        return "Chat.html";
    }


@RequestMapping(value = "/serect")
    public void showing(@RequestParam("checkName") String checkName){//チャンネルが既にあるかどうかを確かめるところ
       check = channel.containsKey(checkName);
       if(check = true){
           System.out.println("チャンネルを選択");
           channel.get(checkName);//チャンネルにあるリストを返す
       }else if(check = false){
           System.out.println("チャンネルを作成");
           channel.put(channelName,chats);//チャンネル一覧に入れる

       }else{
           System.out.println("エラー");
       }


    }


    public Map getMap(){
    return channel;
    }
    public String getChannelName(){
        return channelName;
    }
}

//chat
@Controller
public class Chat {
    private List<String> chats = new ArrayList();

    String logName;


    String chatlog(Model model) {
        //追加処理
    }
        @RequestMapping(value="/addchat")
        String addChat(@RequestParam("name") String name,
                @RequestParam("chat") String chat) {
            
          //追加処理
        }

    void addChat(String name,String chat, Channel channel) {
        var nowChat = channel.chats;
        if (!chat.isBlank()) {
            nowChat.add(name + ":" + chat);
        }
    }

public class ChatApplication {

	public static void main(String[] args) {
		SpringApplication.run(ChatApplication.class, args);
	}

}

Comments

[jimbe]

なぜ channnel.java や main メソッドのコードがクラスのテイを成していないのでしょうか。
また、 アノテーションが付いていますが、何でしょうか。

[hiyori_]

channel,mainは両方ともこちらに記載する際に落ちてしまったようです。ご指摘ありがとうございます、修正します。
アノテーションはhtmlと結合させるためにつけています。

[jimbe]

修正ありがとうございます。

> アノテーションはhtmlと結合させるため
つまり、これらのコードはこれらダケでは動作しないものですね。
フレームワークをご利用であればそれらについても言及して頂かなければ、こちらには何も分かりません。
最後の main の中身からは Spring をご利用のようです。であれば、ご質問のタグに Spring を付けたほうが良いように思います。

[hiyori_]

記載不足、申し訳ないです┏●
ご助言大変ありがとうございます！
そのように致します。

Answer 1

Spring Boot にさほど詳しくないため、Spring Boot 向きではない話に仕上がっているのかもしれませんが、他に回答が付いていないようだったので答えてみます。

上記のような構造でも問題はないでしょうか？

ベストな方法かどうかはさておいて、とにかく必要なデータが最低限どこかにあればいいわけですから、そういう意味では問題ないと思います。

Channelクラスで作ったリストをUserクラスにどのように渡したらいいか

そもそものところ、データ的な情報を Controller に持たせるのは、設計上好ましくないんじゃないかと思います。

今回は、データベース等は使わず、すべてインメモリで済ませて良いものとお見受けしますので、データを持たせるクラスを１つ作り、その static フィールドにデータを集約してしまうのが、１つの方策になると思います。データの登録・取得メソッドも、ここに集めます。

こんな感じで良いと思います。（イメージです。完成品ではありません）

public class ChatSystem {


	/** ユーザ情報： Map<ユーザ名, パスワード> */
	private static final Map<String, String> USERS = new HashMap<>();

	/** ログイン済ユーザ情報： Set<ユーザ名> */
	private static final Set<String> USERS_LOGGED_IN = new HashSet<>();

	/** チャンネル情報： Map<チャンネル名, Set<ユーザ名>> */
	private static final Map<String, Set<String>> CHANNELS = new HashMap<>();

	/** チャット内容： Map<チャンネル名, List<チャットメッセージ>> */
	private static final Map<String, List<String>> MESSAGES = new HashMap<>();

	/* 初期設定 */
	static {

		// 初期ユーザー
		String user = "管理者";
		createUser(user, "password");

		// 初期チャンネル
		String channel = "テスト用チャンネル";
		createChannel(user, channel);

		// 初期投稿
		String message = "これはテスト用の投稿です";
		post(user, channel, message);
	}


	/**
	 * @return true: 作成成功、false: 作成失敗
	 */
	public static boolean createUser(String user, String pass) {

		USERS.put(user, pass);

		return true;
	}

	/**
	 * @return true: ログイン成功、false: ログイン失敗
	 */
	public boolean login(String user, String pass) {

		USERS_LOGGED_IN.add(user);

		return true;
	}

	/**
	 * @return true: 作成成功、false: 作成失敗
	 */
	public static boolean createChannel(String user, String channel) {

		CHANNELS.put(channel, new HashSet<>());

		MESSAGES.put(channel, new ArrayList<>());

		return true;
	}

	/**
	 * チャンネルに参加する。
	 *
	 * @return true: 参加成功、false: 参加失敗
	 */
	public static boolean join(String user, String channel) {

		CHANNELS.get(channel).add(user);

		return true;
	}

	/**
	 * チャットを投稿する。
	 *
	 * @return true: 投稿成功、false: 投稿失敗
	 */
	public static boolean post(String user, String channel, String message) {

		MESSAGES.get(channel).add(user + " さん： " + message);

		return true;
	}

	public static Set<String> getChannels() {

		return CHANNELS.keySet();
	}

	public static Set<String> getMembers(String channelName) {

		return CHANNELS.get(channelName);
	}

	public static Set<String> getUsersLoggedIn() {

		return USERS_LOGGED_IN;
	}

	public static List<String> getMessages(String channelName) {

		return MESSAGES.get(channelName);
	}
}

Controller の方では、基本的には、その static メソッドを呼ぶだけにします。

@Controller
public class ChatController {


	@RequestMapping("/createUser")
	public String createUser(@RequestParam("user") String user, @RequestParam("pass") String pass, Model model) {

		if (ChatSystem.createUser(user, pass)) {

			model.addAttribute("result", user + " を登録しました。");
		}

		model.addAttribute("channels", ChatSystem.getChannels());

		return "index.html";
	}
}

Comments

[momodx]

ところで、画面表示の方はうまくいっていますか？

老婆心ながら、

① ユーザ作成機能を（画面周りまで）動くようにする
② ログイン機能を（同じく）動くようにする
③ チャンネル表示機能
④ チャンネル作成機能

の様にひとつずつ仕上げていかないと、だんだんと何をやって良いか分からない状態になっていってしまうと思いますよ。

[hiyori_]

とても分かりやすいご説明ありがとうございました。
参考にさせていただきます┏●