配列データのsetter/getterの扱いについて

Question

JAVAのバージョン：JDK8

array1.java

package test;

import java.io.IOException;
import java.util.ArrayList;

import test.CommodityPrice;

public class array1 {
	
	public ArrayList<CommodityPrice> getList()throws IOException{
		
		String SYONAME[] = {"はまち","マグロ","マグロ","マグロ","うに","サーモン","サーモン"};
		String Pr[] = {"100円","200円","","200円","300円","250円","250円"};
		
		ArrayList<CommodityPrice> adapterList = new ArrayList<CommodityPrice>();
		for (int i = 0; i < SYONAME.length; i++) {
			CommodityPrice adapter = new CommodityPrice();
			adapter.setName(SYONAME[i]);
			adapter.setPrice(Pr[i]);
			adapterList.add(adapter);
		}
		
		return adapterList;
		
	}

}

CommodityPrice.java

package test;

public class CommodityPrice {
	
	//「商品名」を保持するフィールド
		private String name;

	//「商品価格」を保持するフィールド
		private String price;

	//以下各フィールドのgetter/setter
		public String getName() {
			return name;
		}

		public void setName(String name) {
			this.name = name;
		}

		public String getPrice() {
			return price;
		}

		public void setPrice(String price) {
			this.price = price;
		}

}

このようなプログラムにて、【SYONAME】の長さ分の値を
セットしたく考えています。

for (int i = 0; i < SYONAME.length; i++) {
CommodityPrice adapter = new CommodityPrice();
・
・
・
}

として、毎回[CommodityPrice]クラスのインスタンスを生成しています。
これ以外の方法でより良い方法がありましたら、ご教授いただければと思います。
というのも、テストデータは数件ですが、数百件のデータを扱う場合、
毎回インスタンス生成してしまっては、メモリ管理に負荷がかかってしまうか
と思い、何か良い方法がないかをお聞きしました。

どなたかご教授を宜しくお願いします。

Comments

[tetutetu]

マグロ（およびサーモン）が複数あったり、マグロ3つのうち一つだけEmptyなのが気がかりなのですが
これは何か意図があるのでしょうか。
確実に「マグロ」と「200円」を紐付けたいのであれば、nerianighthawk様の回答の通り
HashMapでも実装可能ですが、Emptyが存在する可能性があるのであれば不適切になります。
Emptyはどういった意図で入っているのでしょうか？

[kumakumatan]

ご回答ありがとうございます。
値がない場合を想定してのEmptyになります。
宜しくお願いします。

Answer 1

毎回インスタンス生成してしまっては、メモリ管理に負荷がかかってしまうか

検証結果後述します。
使いまわしができるものであれば、ループ内でインスタンスを生成しない方が良いですが
このような使い方の場合、インスタンスを生成してしまって問題ないかと。

ちょっと試してみました

（Objクラスには２つのStringのgetter、setterを定義）

    long startMs = System.currentTimeMillis();
    List<Obj> list = new ArrayList<>();

    int count = 1000000;   //100万

    for(int i=0;i<count;i++) {
      Obj o = new Obj();
      o.setA("a");
      o.setB("b");
      list.add(o);
    }

    long endMs = System.currentTimeMillis();

    System.out.println((endMs - startMs) + "ミリ秒");

100万回ループするよう設定しても、130～170ミリ秒で終了します。（一般的なノートPC利用）

Comments

[kumakumatan]

ありがとうございました。
お陰様で解決できました。

Answer 2

CommodityPrice というクラスをどうしても使いたいのであれば kumakumatan さんの書いているコードで問題ないかと思います。数百件でもそんなに遅くはないのではないでしょうか。
ただ、SYONAME と Pr の一対一対応を作りたいだけであればハッシュマップというものがあります。
以下のサイトが参考になります。
【HashMap】Javaで連想配列を扱う！サンプルつき

kumakumatan さんのコードで実装するのであれば、

package test;

import java.io.IOException;
import java.util.HashMap;

import test.CommodityPrice;

public class array1 {

    public HashMap<String, String> getList()throws IOException{

        String SYONAME[] = {"はまち","マグロ","マグロ","マグロ","うに","サーモン","サーモン"};
        String Pr[] = {"100円","200円","","200円","300円","250円","250円"};

        HashMap<String, String> hashmap = new HashMap<String, String>();
        for (int i = 0; i < SYONAME.length; i++) {
            hashmap.put(SYONAME[i], Pr[i]);
        }

        return hashmap;

    }

}

と言った形になるかと思います。
こちらであれば CommodityPrice クラスは使いません。

Comments

[kumakumatan]

ご回答ありがとうございます。
この場合、「array1」クラスを呼び出す場合、どうなりますでしょうか？
ArrayListの場合は、以下のようにしています。
```
array1 P1= new array1();
ArrayList<CommodityPrice> adapterList = P1.getList();
```
宜しくお願いします。

[tetutetu]

横から失礼します。
ArrayList<CommodityPrice> adapterList = P1.getList();
とArrayList型で宣言している部分を
Map<String,String>なりHashMap<String,String>に書き換えるだけです。

[nerianighthawk]

array1のインスタンス生成の部分には影響はありません。
getList の戻り値の型が変わっているので、その部分を変更すれば大丈夫です。
以下のようになります。
```
array1 P1= new array1();
HashMap<String, String> adapterList = P1.getList();
```
ハッシュマップになってしまったので、名前は変えた方が分かりやすいかもしれません。

[kumakumatan]

ありがとうございます。