Java, メソッドを呼び出しオブジェクトを生成する

Question

前提・実現したいこと

Javaで店クラスと商品クラスを定義し，店に商品を配置した後，店情報に加えて，商品情報を表示するシステムを作っています．
表示する情報として，店の名前と電話番号，商品の名前と値段です．

店の名前: Hogehoge-ya
店の電話番号: XX-XXXX-XXXX
商品の名前: hoge
商品の値段: 1000

発生している問題・エラーメッセージ

画面表示に合わせると設計シーケンス図の順に実行しておらず，
設計シーケンス図通りに実行したい.

例えば,StartUpクラスにおいて,createProducts()メソッド呼び出しをStoreオブジェクト生成の次に実行され,実行結果画面が同じになるようにしたい.

該当のソースコード

public class StartUp {

	public static void main(String[] args) {

		Store store = new Store("Hogehoge-ya","XX-XXXX-XXXX");
		Store.printStore();
		Store.createProducts("hoge",1000);


	}

}

public  class Store {

	//メンバ変数を宣言
	private String storeName;
	private String tel;
	//メンバ変数を宣言(集約)
	private Products products;

	//引数がないコンストラクタを生成
	public Store() {}

	//引数がStoreName,telとするコンストラクタを生成
	public Store(String storeName,String tel) {
		this.storeName = storeName;
		this.tel = tel;
	}

	//Productsクラスを生成するcreateProducts()メソッドを定義
	public void createProducts(String productsName, int price) {
		products = new Products(productsName, price);
		products.printProducts();
	}

	//店名や電話番号を表示するメソッドを定義
	public void printStore() {
		System.out.println("(Store)"+storeName+", TEL:"+tel);
	}

	//StoreName:Setter&Getter
	public void setStoreName(String storeName) {
		this.storeName = storeName;
	}

	public String getstoreName() {
		return storeName;
	}

	//tel:Setter&Getter
	public void setTel(String tel) {
		this.tel = tel;
	}

	public String getTel() {
		return tel;
	}

	//products:Setter&Getter
	public void setProducts(Products products) {
		this.products = products;
	}

	public Products getProducts() {
		return products;
	}

}

public class Products{


	//メンバ変数を宣言
	private String productsName;
	private int price;

	//引数がないコンストラクタを生成
	public Products() {}

	//引数がStoreName,telとするコンストラクタを生成
	public Products(String productsName,int price) {
		this.productsName = productsName;
		this.price = price;
	};

	//商品名や値段を表示するメソッドを生成
	public void printProducts() {
		System.out.println("(products)"+productsName+" "+price+"円");
	};

	//productsName:Setter&Getter
	public void setproductsName(String productsName) {
		this.productsName = productsName;
	};

	public String getproductsName() {
		return productsName;
	};

	//price:Setter&Getter
	public void setPrice(int price) {
		this.price = price;
	};

	public int getPrice() {
		return price;
	};


}

画面表示

(Store)Hogehoge-ya, TEL:XX-XXXX-XXXX
(products)hope 1000円

###設計クラス図

###設計クラス図

###補足
SetterとGetterはテスト段階で使うため記述しています.
まだまだ初心者でわからないことが多いのですが,ご教授よろしくお願いいたします．

Comments

[dodox86]

StartUpクラスのmainメソッドで書いているままに動いているだけだと思うのですが、
> Store Store = new Store("Hogehoge-ya","XX-XXXX-XXXX");
> Store.printStore();
> Store.createProducts("hoge",1000);

この処理で何を行っているか分からないということでしょうか。

尚、直接関係ないのですが、変数名をクラス名と同じにして宣言するのは悪い習慣です。
> Store Store = new Store("Hogehoge-ya","XX-XXXX-XXXX");
だと、Storeと言う変数がStoreクラスなのか変数名なのか強力に分かりづらいです。
一般的には
Store store = new Store("Hogehoge-ya","XX-XXXX-XXXX");
とします。

[alastashim]

dodox86様
ご指摘ありがとうございます．
変数名とクラス名については直していきたいと思います．
mainメソッドで動いていることはわかっているのですが，
設計シーケンス図において,Storeクラスを生成したのちcreateProducts()メソッドを呼び出すと書かれているので，
> Store Store = new Store("Hogehoge-ya","XX-XXXX-XXXX");
> Store.printStore();
> Store.createProducts("hoge",1000);
この処理だと先にprintStore()メソッドを呼び出してしまっているのではないかと考えています．
そこで，先にcreateProducts()メソッドを呼び出したのちにprintStore()メソッドを呼び出すソースコードはどのようにして書けばいいのかというのが今回の投稿した理由となります．
拙い文章ではありますが，どうぞよろしくお願いいたします．

Answer 1

createproductの前にprintStoreしてるからとしか言いようがないのですが？
コードは目で追えてますか？追えないのならeclipseなどのデバッガーで一行ずつ実行しては？

あと、一般的にproductsとされると、複数あるように思われるので、
クラス名ならProduct,複数ありうるなら配列かListを使ってください。
Product配列のあるProductsクラスは何か特別な操作がしたい場合に作るのがベターです。（多分）

Comments

[alastashim]

snowshink様
ご指摘ありがとうございます．複数形の使い方など次回から生かしていきたいと思います．
createproductしたのちprintStoreを行うと,実行結果が

>(products)hope 1000円
>(Store)Hogehoge-ya, TEL:XX-XXXX-XXXX

となってしまうので望んでいる実行結果にはならないのです．
そこでどこを書き直したらシーケンス図通りかつ理想の実行結果が得られるかというのをよろしければお聞きしたいです．

[snowshink]

シーケンス通りのコードにそもそもなってませんよ。
コードからシーケンス図を作るとこうではないですか？http://www.plantuml.com/plantuml/uml/TP112iCW44NtEOLh8HV8egJG0mGymJ0PYj0QcNvx_eg9fKgpKNpp_aUTNo15UZw69U-UVQG0umGfjWIRIaCMtTR46AuC_oR84Igh6SVB7h5sCdS9eWJvglkeLoV3AWIfnGMMyK6iDV_AJlriH_K1LRW9TPd-2ZNXE1z-VOS6rCVE4jRyTny0

[alastashim]

ご返信ありがとうございます．
snowshinkさんはどの部分を直せばシーケンス図通りになると思いますか.
お手数ですが,具体的によろしくお願いいたします．

[snowshink]

え。そこを考えてください。
何をしたくて、今どうなってるのか整理すれば自然とわかるはずです。
ざっくり言えば、printするタイミングが違うんですよね？ならお望みどおりに変えては？
正常に動くようにしてから、リファクタリングするのをよいのですが、
シーケンス図があっていて、これに合わせるのが前提なら、自分のと見比べて正しく修正してください。