複数インスタンスを通して一意なIDを生成する方法

Question

一意なIDをCustomerCardインスタンスに割り当てたいと思っています。
以下のコードだと一つのCustomerManagerインスタンスの中では一意なIDを生成できるのですが、
CustomerManagerインスタンスを複数生成すると一意になりません。
複数のCustomerManagerを通して一意なIDを生成する方法がわかりません。

public class CustomerManager {

	private int id = 1;
	private List<CustomerCard> list = new ArrayList<>();

	CustomerManager(){}

	public void addCustomerCard (String name){
			CustomerCard customerCard = new CustomerCard(id, name);
			list.add(customerCard);
			id++;
	}

	public void printAll() {
		for(CustomerCard customerCard : list) {
		customerCard.print();
		}
	}

	public static void main(String[] args) {
		CustomerManager customerManager = new CustomerManager();

		customerManager.addCustomerCard("山田一郎");
		customerManager.addCustomerCard("鈴木太郎");
		customerManager.addCustomerCard("田中次郎");

		CustomerManager customerManager2 = new CustomerManager();
		customerManager2.addCustomerCard("yamada");
		customerManager2.addCustomerCard("suzuki");
		customerManager2.addCustomerCard("tanaka");

		customerManager.printAll();
		customerManager2.printAll();

	}
}

Answer 1

こんにちは

現状では、CustomerCard　のインスタンスを作るときに、コンストラクタの引数として id を与えていますが、これを止めて、CustomerCard のインスタンスのidが一意になるようにCustomerCard を修正すればよいかと思います。

これを実現する、一例としては、CustomerCard の staticフィールドとして、次のインスタンスに与えるid値を保持しておいて、インスタンスが作られるたびにこれを更新することです。

以下、サンプルです。（Main.java として作成しています）

import java.util.List;
import java.util.ArrayList;

class CustomerCard {
    private static int _id = 100;

    private int id;
    private String name;

    CustomerCard(String name) {
        this.name = name;   
        this.id = _id;
        _id += 10;             
    }

    void print() {
        System.out.println("CustomerCard: id:" + this.id + ", name:" + this.name);
    }
}

class CustomerManager {

    private List<CustomerCard> list = new ArrayList<>();

    public void addCustomerCard (String name){
        CustomerCard customerCard = new CustomerCard(name);
        list.add(customerCard);
    }

    public void printAll() {
        for(CustomerCard customerCard : list) {
            customerCard.print();
        }
    }
}


public class Main {

    public static void main(String[] args) {
        CustomerManager customerManager = new CustomerManager();

        customerManager.addCustomerCard("山田一郎");
        customerManager.addCustomerCard("鈴木太郎");
        customerManager.addCustomerCard("田中次郎");

        CustomerManager customerManager2 = new CustomerManager();
        customerManager2.addCustomerCard("yamada");
        customerManager2.addCustomerCard("suzuki");
        customerManager2.addCustomerCard("tanaka");

        customerManager.printAll();
        customerManager2.printAll();
    }
}

• 動作確認用Repl.it: https://repl.it/@jun68ykt/Q253068

上記では、idの初期値を100にして、以後 10ずつ増えるようにしています。

Comments

[nino8752]

解決しました、ありがとうございます！

[jun68ykt]

どういたしまして
> 解決しました
とのことでよかったです????

Answer 2

static変数を使えばいいって回答が複数ありますが、

static変数を使うこと自体は異論ないですが、
static変数を単純に使っただけでは
マルチスレッドだと、値がユニークにならない可能性あるからね。

counterのインクリメント「counter++;」の行はJavaのソースコードでは1行ですが、実際には 読み出し・変更・書き込み（RMW：Read-Modify-Write操作）という複数の処理で構成されていて 複数スレッドの実行が入り乱れる可能性があります。

まぁ。困ってないのだろうけど

Answer 3

private int id = 1;はインスタンスフィールドなのでprivate static int id = 1;と、staticなフィールドにすればいいかと。
インスタンスフィールドはインスタンスごとにフィールドが生成されますが、staticなフィールドはインスタンスの数に関係なく1つだけ生成されます。

Answer 4

staticな変数を別途用意しておけばいいのでは？

    private static int serialId = 1;
    private int id;
    private List<CustomerCard> list = new ArrayList<>();

    CustomerManager(){}

    public void addCustomerCard (String name){
            CustomerCard customerCard = new CustomerCard(id, name);
            list.add(customerCard);
            id = serialId++;
    }