フィールド変数を使用せずにクラス型配列の中身を表示

Question

/*

• 引数、パラメータ、戻り値を使用して
• SubTest19クラスでフィールド変数を使用しないよう、
• 修正して下さい。
• また、SubTest19クラスのput()で行っている出力はfor文を使用する事。

*/

class Test19 {

public static void main(String[] args) {

	SubTest19 st19 = new SubTest19();

	st19.set();

	st19.put();

}

}

class SubTest19 {

private Test19data t19d[] = null;

protected void set(){
	t19d = new Test19data[3];
	t19d[0] = new Test19data();
	t19d[1] = new Test19data();
	t19d[2] = new Test19data();

	t19d[0].px = "K-0963";
	t19d[0].ct = 23;

	t19d[1].px = "T-523363";
	t19d[1].ct = 517;

	t19d[2].px = "J-97553";
	t19d[2].ct = 5;

}

protected void put(){
	System.out.println(t19d[0].px);
	System.out.println(t19d[0].ct);

	System.out.println(t19d[1].px);
	System.out.println(t19d[1].ct);

	System.out.println(t19d[2].px);
	System.out.println(t19d[2].ct);

}

}

class Test19data{
public String px;
public int ct;

public Test19data(){
	px = "";
	ct = 0;
}

}

フィールド変数を使用しないというのは、いわゆるprivate Test19data t19d[] = null;や、public String px;、public int ct;というコードを使わずに、引数、パラメータ、戻り値を使用してということだと思います。

ご回答よろしくお願いします。

Answer 1

class Test19 {

public static void main(String[] args) {

	SubTest19 st19 = new SubTest19();

	Test19data[] t19d = st19.set();

	st19.put(t19d);

}

}

class SubTest19 {

protected Test19data[] set(){
	Test19data t19d[] = null;
	t19d = new Test19data[3];
	t19d[0] = new Test19data("K-0963", 23);
	t19d[1] = new Test19data("T-523363", 517);
	t19d[2] = new Test19data("J-97553", 5);

	return t19d;

}

protected void put(Test19data[] t19d){
	//
	for( int i = 0; i < t19d.length; i++ ){
		System.out.println(t19d[i].px);
		System.out.println(t19d[i].ct);
	}
}

}

class Test19data{

public String px;
public int ct;

public Test19data(String px, int ct){
	this.px = px;
	this.ct = ct;
}

}

Answer 2

「SubTest19クラスでフィールド変数を使用しない」ということなので，
SubTest19の中で使う変数が全部ローカル変数になるように改変すればいいわけです．
Test19dataにはそんな指定は無いですし，そもそもこの場合
Test19dataにはフィールド変数がなければ使い物になりません．
Test19dataのコンストラクタはargiusさんのがそのまま使えます．

学校の課題ということなので完全な答えは出さずに，ここでは手がかりを示すにとどめます．

最終的にSubTest19#putメソッドでTest19dataの中身を表示することになるので，
putメソッドの中でTest19data[]を取得する必要があるわけです．
一方で，現在setメソッドではフィールド変数を操作しています．
ここで行った結果を，ローカル変数のみを用いて他でも使える様にするためには・・・？

さあ，お考えください．

Comments

[touch]

ご配慮ありがとうございます。
ヒントを参考にもう少し自分で考えてみます。

[swordone]

ちょっと編集履歴から先ほど載せていたコードを見ていたのですが，
あれが割と課題の要求に沿っていたコードだと思ったのですが(引数，パラメータ，戻り値を利用している)
何が疑問，もしくは不満だったのでしょうか？

[touch]

課題の原型を載せないと、皆様にフィールド変数を使わないという課題の意図が伝わらないと思い、すっきりさせる意味でも消してしまいました。どちらも最初から載せておくべきでしたね。

[swordone]

そういうことじゃなくて，質問の時点で載せていたコードで
課題が要求していた仕様はほぼ満たしていたと思うんです．
私は「フィールド変数を使わない」というのがあやふやで混乱していましたが．
そういうことなら，なぜ質問したのか，何が疑問で質問したのかがわからないのです．

[touch]

あともう少しという所まで来ているのかなと思っていたのですが、あのままだとTest19dataクラスのところでどうしてもフィールド変数を使わないと回答にたどりつかなくて、何か違ったアプローチはないかと質問させていただきました。

[swordone]

なるほど．ですがあなたの出された課題が正しくここに表示されているのであれば，
「SubTest19でフィールド変数を使わない」としか書かれていません．
おそらくこれは「SubTest19クラスがフィールド変数を持たない」ということでしょう．
先生が一般的なプログラムを意識しているのであれば，課題のコードのputメソッドで登場する"オブジェクト.変数名"を使わない，ということも入っていると思われます．
他のクラスでフィールド変数を持つことに関しては制限がない，と私は読み取りますね．
完全にフィールド変数を封じられたらTest19dataは本当に使い物にならなくなりますし．

[touch]

よく問題を読めば、 Test19dataクラス内では変数に関して制限が無いですね。
なら最初の方針で合ってたということですね・・
すいません、お騒がせしました。これからは問題をよく読みます。

[swordone]

ついでに，私の最初の回答に付いているベストアンサーを外していただけるとありがたいです．
質問に沿わない回答がベストアンサーになっているのは私としても本意ではないので．

Answer 3

「フィールド変数」というのが何を指しているのかがよくわかりませんが，
確かにmainメソッドが無駄に冗長な気がします．
以下推測で回答するので間違っていたらご指摘願います．

SubTest19のメンバとしてTest19data[]を定義します．
SubTest19のコンストラクタでTest19data[]を初期化します．
そしてSubTest19でtoString()をオーバーライドします．

class SubTest19 { 

    private Test19data t19d[];

    public SubTest19(){ 

        //配列の初期化 
        t19d = new Test19data[3]; 

         //インスタンス生成および情報の受け渡し 
        t19d[0] = new Test19data("K-0963", 23); 

        t19d[1] = new Test19data("T-523363", 517); 

        t19d[2] = new Test19data("J-97553", 5);  

    } 

    public String toString(){ 

        String str = "";
        //出力 
        for( int i = 0; i < t19d.length; i++ ){ 
            str += t19d[i].toString();    //argiusさんのtoStringをそのまま使うとします．
            if(i != t19d.length - 1)
                str += \n;                //データの最後でなければ改行を入れる
        } 
        return str;
    } 

} 

こうするとmainメソッドにおいて，

class Test19 { 
    public static void main(String[] args){
        System.out.println(new SubTest19());
    }
}

このように，フィールド変数を使わずに内容を出力できます．

あるいは，Test19dataにputメソッドを次のようにして，

public void put(){
    for( int i = 0; i < t19d.length; i++ ){ 
        System.out.println(t19d[i]);
    }
}

mainメソッドの中で

public static void main(String[] args){
    new SubTest19().put();
}

としても出力できると思います．

Comments

[touch]

ご回答頂きありがとうございます。
確かにこれならフィールド変数を使わずに内容を出力できますね。
勉強になります。

[swordone]

いや，質問の意味が分からない時点で回答したのでこうしましたが，
そういう質問であればこの回答は不適ですね(SubTest19がフィールド変数を持っている)

Answer 4

引数、パラメータ、戻り値を使用して、フィールド変数を使用せずに
Test19data型配列の中身をfor文で出力したいです。

「フィールド変数を使用せずに」の部分が意味不明ですが、（仰っている内容を直接解釈すると）それは実装不可能です。

というより、フィールド変数の役割／目的と、コンストラクタとの関係を勘違いされているように見えます。コンストラクタは、フィールド変数を初期化するために存在します。質問文にあるコードは、「オブジェクトnew時に与えられた値を受け取ったパラメータ変数pxとctの中身を破棄し、""と0を再代入しているだけ（そしてコンストラクタを抜けるときに全ての値が破棄される）」という、結果的に何もしていないコードになっています。

class Test19data{
  public Test19data(String px, int ct){
    px = "";
    ct = 0;
  }
}

Comments

[touch]

ご回答頂きありがとうございます。
まだ初心者ゆえご指摘の箇所含め勉強中でした。
今後の参考にさせていただきます。

Answer 5

Javaではこのような場合はtoStringメソッドをオーバーライドします。
後はTest19dataのオブジェクトをそのままprintすればOKです。

toStringはEclipseなどのIDEを使えば生成機能で作ることができます。

class Test19data{
    private String px;
    private int ct;
    //コンストラクタ
    public Test19data(String px, int ct){
        this.px = px;
        this.ct = ct;
    }

    @Override
    public String toString() {
        return "Test19data [px=" + px + ", ct=" + ct + "]";
    }

}

// System.out.println(t19d[i]); // => Test19data [px=, ct=0]

Comments

[touch]

早速ご回答頂きありがとうございます。
toStringメソッドをオーバーライドする方法があったのですね。
ちなみにフィールド変数を使用せずにエラーなく表示させることは可能でしょうか？

[argius]

フィールド変数を使わずにというのは、フィールド変数なしで、ということですか？ だとしたら無理だと思います。
なぜフィールド変数を使わずに、なのかが良く分かりませんので、その辺をご説明いただけるともうすこし的確にお答えできるかもしれません。
もしかして、タプルの配列を実現したいということでしょうか？

[touch]

言葉足らずで申し訳ないです。
private String px;やprivate int ct;といったフィールド変数を使わずにということでした。
これは学校の課題で、そのような指示をされています。これから編集で原型を載せます。