Java 対象の一部を引用したい

Question

前提・実現したいこと

Twitterの簡易的な疑似アプリを作成する途中で、リプライやリツイートをするときに元ツイートのユーザーIDを表示させたい

該当のソースコード

import java.text.SimpleDateFormat;
import java.util.Calendar;

public abstract class Base {
	public int tweetId = 0;
	public long datetime = 0;
	public String userId = "";

	public Base(int tweetId,String userId) {
		this.tweetId = tweetId;
		this.userId = userId;
		this.datetime = getNow();
	}
	public static long getNow(){
	Calendar cl = Calendar.getInstance();
	SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMddhhmmss");
	return Long.parseLong(sdf.format(cl.getTime()));
	}
	public int getTweetId() {
		return tweetId;
	}
	public abstract String getText();

public class Tweet extends Base{
	public static String text;
	public Tweet(int tweetId,String userId,String text) {
		super(tweetId, userId);
		this.text = text;
	}
	@Override
	public String getText(){
		String tweet = text + " from  @" + userId + "[" + datetime + "]";
		return tweet;
	}
}

public class Reply extends Tweet{
	private String target = super.userId;
	public Reply(int tweetId,String userId,String text, Tweet target) {
		super(tweetId,userId,text);
	}
	@Override
	public String getText() {
		String reply = "@" + target + " " + super.getText();
		return reply;
	}
}

補足情報（FW/ツールのバージョンなど）

例えばMainで
Tweet t1 = new Tweet(0, "gaju" , "わからない");
を実行して
わからない from @gaju [20210113175300]
とツイートした時に、このツイートに対するリプライを
@gaju わかる from @someone [20210113175400]
とさせるようにしたいのですが、
Replyクラスのtargetがどうしてもリプライの送り主と同じになってしまい、
@someone わかる from @someone [20210112175400]
のようになってしまいます。
eclipseを使って開発をしています。
プログラミング自体が学びたてなので色々と拙くわかりづらい部分も多々あると思いますが、回答いただけると幸いです。

Comments

[hoshi-takanori]

Reply クラスの target フィールドの型をTweet にすべきでは？

[gaju]

返信していただきありがとうございます。
早速hoshi-takanori様の仰る通りtargetフィールドの型をString からTweetに変えてみましたが、=の後に型の不一致:StringからTweetに変換出来ませんというエラーが出てしまいます。
targetの対象がおかしいのでしょうか。

[hoshi-takanori]

ごめんなさい、ソースよく見てませんでした。問題は static ですね。
私のコメントは忘れて momon-ga さんの回答をよくお読みください。

[gaju]

分かりました。返信とアドバイスありがとうございました。
よろしければまたアドバイスを頂けると幸いです。

Answer 1

現状、userIdがstaticのため、userIdは１つしか存在させることができません。

public static String userId = "";

例えばMainで

Tweet t1 = new Tweet(0, "gaju" , "わからない");
Tweet t2 = new Tweet(1, "hoge" , "ほげ");

System.out.println(t1.getText()); 

とした場合t1のユーザIDで表示されるのは、gajuでなくhogeです。
※あとから作成したTweetのuserIdで上書きされる。Tweet.textも同様

上記でt1の内容を表示しようとしていますが、期待と異なる結果が表示されるはずです。

まずはインスタンス変数を利用するようにして
コンストラクタの処理も

this.userId = userId;

のように修正する必要があります。

あとは、

String reply = "@" + target + " " + super.getText();

も、たぶん勘違いされていると思います。が、まずはstaticを修正しないことには・・・

Comments

[gaju]

回答して頂きありがとうございます。
最初はthis.で書いていたのですが、エラー修正の際に修正したままにしていたようです。ご指摘頂きありがとうございます。
仰っていただいた通りコンストラクタの処理を修正致しました。
Tweet自体は別userIdでも更新されないようにはなったのですが、Replyはまた同じuserIdが表示されています。
最後のString reply =　...　の勘違いというのはどういったところでしょうか。

[hoshi-takanori]

横から失礼します。
Base クラスのフィールドは static じゃなくなりましたが、Tweet クラスの text フィールドは static のままですね。これも修正しましょう。
また、Reply クラスの target フィールドは宣言と同時に target = super.userId; と初期化されてますが、この super.userId というのは Reply 自身の userId になってしまいます。Reply 先の Tweet の userId を取得する必要があるのでは？

[gaju]

回答ありがとうございます。
Tweetクラスのstaticを修正し忘れておりました。ご指摘ありがとうございます。
Replyクラスのtargetフィールドを対象のTweetのuserIdとするためにいろいろと試してみたのですが、うまくいきません。
この場合Mainクラスで作ったt1のuserIdを呼び出せられればいいというのは分かったのですが、どうやって参照すればよいのでしょう…
質問ばかりで申し訳ないです。

[hoshi-takanori]

たとえば t1 に対する Reply を生成する場合、
Tweet t1 = new Tweet(0, "gaju" , "わからない");
Reply t2 = new Reply(1, "hoge" , "ほげ", t1);
という感じになると思いますが、Reply のコンストラクタには t1 が渡ってきてるので、その情報を利用すればいいと思います。

[gaju]

返信ありがとうございます。
フィールドを
private String target;
Replyコンストラクタを
public Reply(int tweetId,String userId,String text, Tweet target) {
super(tweetId,userId,text);
this.target = target.userId;
}
とすることで解決いたしました。
稚拙な質問ばかりで煩わしかったかもしれませんが解決までアドバイス頂き本当にありがとうございます！