常連

### 状況
課題としてJavaEEを用いてWebアプリを制作しているのですが、
複数のテーブルからデータを取得し、一覧として表示する機能を実装するにあたり、
どのようにBO、DAO、DTOを組むのが最適か悩んでおります。

また、ファクトリの理解ができておらず、
DAOをハードコーディングしてしまっている状態です。

### 試したこと
1. DAOをテーブル単位、DTOはテーブル単位のものに加えてサービス単位のものを作り、BOでテーブル単位のDTOからサービス単位のDTOにデータをコピーする。
0. サービス単位のDAO、DTOを作り、SQLで複数テーブルからデータを取得する。

### 他不明な点
- リスト試したこと①において、サービス単位のDTOを作成する際にどのようにして作ればいいのか？（継承やファクトリ）

### 補足情報（FW/ツールのバージョンなど）
Eclipse (4.7.3a)
Java (9.0.4)
Tomcat (9.0)
MySQL (8.0.28)

### 最後に
お世話になっております。

独学での学習のため、オブジェクト指向やDAOパターンに対する知見が浅く、
問題が多々あるとは思いますが、ご指摘頂けますと幸いです。

初歩的な質問とは思いますが、ご教示頂けますよう、よろしくお願いいたします。


複数テーブルを扱う際のDAO、DTO設計に悩んでいます

Javaを使って簡素なCUIのDBアプリを作成しています。
現在、ユーザ入力値をDBに入れるInsertの部分に取り組んでいるのですが、
下記エラーが出てしまいます。

エラー内容
```
Exception in thread "main" java.lang.NullPointerException: Cannot invoke "com.jdbc.budgettracker.core.BudgetTrackerDto.setId(int)" because "budgetTrackerDto" is null
	at BudgetTrackerCui/com.jdbc.budgettracker.main.BudgetTrackerMain.main(BudgetTrackerMain.java:100)
```
BudgetTrackerMain.javaの100行目「budgetTrackerDto.setId(insertScannerInt);」の部分で、int型の変数の値をdtoにセットする形にしたいのですが、上記エラーが出てしまい、原因が掴みきれておりません。
97行目の「System.out.println(insertScannerInt);」でユーザ入力値の数値が渡せているかと推測しているのですが。。。
お手数ですが、解決の為、ヒントを頂けると幸いです。
よろしくお願いいたします。

BudgetTrackerMain.java
```ここに言語を入力
package com.jdbc.budgettracker.main;

import java.io.FileNotFoundException;
import java.io.IOException;
import java.sql.Date;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Scanner;

import com.jdbc.budgettracker.core.BudgetTrackerDto;
import com.jdbc.budgettracker.dao.BudgetTrackerDao;

public class BudgetTrackerMain {

	@SuppressWarnings("null")
	public static void main(String[] args) throws FileNotFoundException, IOException, ParseException {
		// TODO Auto-generated method stub
		BudgetTrackerDao budgetTrackerDao = null;
		BudgetTrackerDto budgetTrackerDto = null;

		Map<Integer, String> initialSwitchMap = new HashMap<>();
		initialSwitchMap.put(1, "Select");
		initialSwitchMap.put(2, "Insert");
		initialSwitchMap.put(3, "Update");
		initialSwitchMap.put(4, "Delete");
		initialSwitchMap.put(5, "Test");
		Scanner initialSwitchScanner = new Scanner(System.in);
		int initialNumInt = 0;
		do {
			// String[] initialList = new String[] {initialSwitchMap.get(1),
			// initialSwitchMap.get(2), initialSwitchMap.get(3), initialSwitchMap.get(4)};

			for (Map.Entry<Integer, String> list : initialSwitchMap.entrySet()) {
				System.out.println(list.getKey() + ":" + list.getValue());
			}

			System.out.print("Select a number: ");
			String initialNumStr = initialSwitchScanner.next();
			initialNumInt = Integer.parseInt(initialNumStr);

		} while (initialNumInt >= 5 || initialNumInt <= 0);

		switch (initialNumInt) {
		case 1:
			// Select
			System.out.println("You chose " + initialSwitchMap.get(1));
			budgetTrackerDao = new BudgetTrackerDao();
			Map<Integer, String> selectSwitchMap = new HashMap<>();
			selectSwitchMap.put(1, "Select All");
			selectSwitchMap.put(2, "Select by Date");
			selectSwitchMap.put(3, "Select by Store");
			selectSwitchMap.put(4, "Select by Product");
			selectSwitchMap.put(5, "Select by Type");
			selectSwitchMap.put(6, "Select by Price");

			Scanner selectScanner = new Scanner(System.in);
			int selectScannerNumInt = 0;
			do {
				// String[] selectScanList = new String[] {selectSwitchMap.get(1),
				// selectSwitchMap.get(2), selectSwitchMap.get(3), selectSwitchMap.get(4),
				// selectSwitchMap.get(5)};

				for (Map.Entry<Integer, String> selectScanlist : selectSwitchMap.entrySet()) {
					System.out.println(selectScanlist.getKey() + ":" + selectScanlist.getValue());
				}

				System.out.print("Select a number: ");
				String selectScannerNumStr = selectScanner.next();
				selectScannerNumInt = Integer.parseInt(selectScannerNumStr);

			} while (selectScannerNumInt > 7 || selectScannerNumInt <= 0);

			do {
				if (selectScannerNumInt == 1) {
					System.out.println("Select All----------");
					List<BudgetTrackerDto> btd = budgetTrackerDao.selectAll();
					for (BudgetTrackerDto b : btd) {
						System.out.println(b.getId() + ", " + b.getDate() + ", " + b.getStoreName() + ", "
								+ b.getProductName() + ", " + b.getProductType() + ", " + b.getPrice());
					}

				}

			} while (selectScannerNumInt > 7 || selectScannerNumInt <= 0);
			break;
		case 2:
			// Insert
			int insertcannerInt = 0;
			System.out.println("You chose " + initialSwitchMap.get(2));

			Scanner insertScanner = new Scanner(System.in);
			System.out.print("Input an ID: ");
			int insertScannerInt = insertScanner.nextInt();
			System.out.println(insertScannerInt);
//			insertcannerInt = Integer.parseInt(insertScannerStr);
//			System.out.println(insertcannerInt);
			budgetTrackerDto.setId(insertScannerInt);
			
			System.out.print("Input Date (yyyy-MM-dd): ");
			String insertScannerStr = insertScanner.next();
			insertScannerStr = insertScanner.next();
			Date insertDate=(Date) new SimpleDateFormat("yyyy-MM-dd").parse(insertScannerStr);  
			budgetTrackerDto.setDate(insertDate);
			
			System.out.print("Input a store name: ");
			insertScannerStr = insertScanner.next();
			budgetTrackerDto.setStoreName(insertScannerStr);
			
			System.out.print("Input a product name: ");
			insertScannerStr = insertScanner.next();
			budgetTrackerDto.setProductName(insertScannerStr);
			
			System.out.print("Input a product type: ");
			insertScannerStr = insertScanner.next();
			budgetTrackerDto.setProductType(insertScannerStr);
			
			System.out.print("Input price: ");
			insertScannerStr = insertScanner.next();
			insertcannerInt = Integer.parseInt(insertScannerStr);
//			System.out.println(insertcannerInt);
			budgetTrackerDto.setPrice(insertcannerInt);
			
			budgetTrackerDao.insertIntoTable(budgetTrackerDto);

		}

	}
}
```

BudgetTrackerDto.java
```ここに言語を入力
package com.jdbc.budgettracker.core;

import java.sql.Date;

public class BudgetTrackerDto {
    private int id;
    private Date date;
    private String storeName;
    private String productName;
    private String productType;
    private int price;

    public BudgetTrackerDto() {
    }

    public BudgetTrackerDto(int id, Date date, String storeName, String productName, String productType, int price) {
        super();
        this.id = id;
        this.date = date;
        this.storeName = storeName;
        this.productName = productName;
        this.productType = productType;
        this.price = price;
    }

    public int getId() {
        return id;
    }

    public void setId(int id) {
        this.id = id;
    }

    public Date getDate() {
        return date;
    }

    public void setDate(Date date) {
        this.date = date;
    }

    public String getStoreName() {
        return storeName;
    }

    public void setStoreName(String storeName) {
        this.storeName = storeName;
    }

    public String getProductName() {
        return productName;
    }

    public void setProductName(String productName) {
        this.productName = productName;
    }

    public String getProductType() {
        return productType;
    }

    public void setProductType(String productType) {
        this.productType = productType;
    }

    public int getPrice() {
        return price;
    }

    public void setPrice(int price) {
        this.price = price;
    }

}
```

JavaのDTOで発生したNullPointerExceptionを解決したいです。

バージョン VB.net 4.5.2

閲覧ありがとうございます。
単票形式の帳票で12件ずつのデータを出力します。
Daoで取得したDataTableをFoeRoopでDtoに詰め替えるのですが
どうしてもコードが冗長になるので、もっとスッキリ書ける方法はないかアドバイスを頂きたいです。

**DTOクラス**
```VB

Class RPT_0060SettingDto

    Public Property biz As String
    Public Property rack_id As String

    Public Property disp_color_1 As String
    Public Property parts_no_1 As String
    Public Property name_1 As String
    Public Property parts_name_1 As String
    Public Property location_no_1 As String
    Public Property order_point_1 As String

    Public Property disp_color_2 As String
    Public Property parts_no_2 As String
    Public Property name_2 As String
    Public Property parts_name_2 As String
    Public Property location_no_2 As String
    Public Property order_point_2 As String

  'プロパティが連番で12まで続く・・・
   
    Public Property disp_color_12 As String
    Public Property parts_no_12 As String
    Public Property name_12 As String
    Public Property parts_name_12 As String
    Public Property location_no_12 As String
    Public Property order_point_12 As String
   
End Class


```

**問題のコード**
```VB
Class RPT_0060Util

    Shared Function GetSettingDto(ByVal table As DataTable) As List(Of RPT_0060SettingDto)

        Dim dtoList As New List(Of RPT_0060SettingDto)
        Dim settingDto As RPT_0060SettingDto

        Dim i As Integer = 0

        For Each row As DataRow In table.Rows
            '改ページしたら初期化
            If i > 0 And i Mod 12 = 0 Then
                dtoList.Add(settingDto)
                settingDto = New RPT_0060SettingDto
            End If


            If i Mod 12 = 0 Then
                settingDto.disp_color_1 = row("disp_color").ToString
                settingDto.parts_no_1 = row("parts_no").ToString
                settingDto.parts_name_1 = row("parts_name").ToString
                settingDto.location_no_1 = row("location_no").ToString
                settingDto.order_point_1 = row("order_point").ToString
            ElseIf i Mod 12 = 1 Then
                settingDto.disp_color_2 = row("disp_color").ToString
                settingDto.parts_no_2 = row("parts_no").ToString
                settingDto.parts_name_2 = row("parts_name").ToString
                settingDto.location_no_2 = row("location_no").ToString
                settingDto.order_point_2 = row("order_point").ToString
            End If

            i = i + 1
        Next
        dtoList.Add(settingDto)

        Return settingDto

    End Function

End Class
```

DTOのプロパティが1から12までの連番になっているので
settingDto.disp_color_ + 変数・・・で取得できそうなのですがうまくいきませんでした。
何か良い方法をご教授いただけると助かります。

VB.net Dtoの詰め替えで効率の良い方法が分からない

以下のようにgetter / setterが同じ名前のDTOが二つあります。

```Java
public class Test1Dto {
    private String userId;
    private String userName;
    private String address;
    
    // ======getter / setter は省略 ======
}
```

```Java
public class Test2Dto {
    private String userId;
    private String userName;
    private String address;
    
    // ======getter / setter は省略 ======
}
```

現在、下記のように三項演算子を使って片方のDTOがNullであれば、もう片方から取得するようにしているのですが、**DTOから取得する値が増えた場合、冗長になるなぁ**と思っています。

```Java
Test1Dto test1Dto = null;
Test2Dto test2Dto = null;

// =======DTO値取得部分は省略========

String userId = test1Dto != null ? test1Dto .getUserId() : test2Dto .getUserId();
String userName= test1Dto != null ? test1Dto .getUserName() : test2Dto .getUserName();
String address= test1Dto != null ? test1Dto .getAddress() : test2Dto .getAddress();
```

三項演算子以外に、上記の冗長なコードを回避する方法はありますでしょうか？

異なるDTOから同じ名前のgetterを使って値を取得することはできますか？

設計の勉強をしています。
Clean Architectureでアプリを作っていて、Viewにリストの内容を表示するために、ViewDataとして、配列を持っているのですが、これは、コレクションオブジェクトにした独自の構造体にした方がいいのでしょうか？例えばgithubのリポジトリ一覧を表示したいとき
```swift
class Repository{
   let title:String
   let description:String
   let isFavorite:Bool
}
```
のような型があった場合、viewDataはコレクションオブジェクトで表現すると
```swift
struct RepositoryList{
     let elements:[Repository]
}
```
のように表現するのかそれとも単に[Repository]という型で表現するのかどちらの方がいいのでしょうか？？

また、上記のコレクションは一種のValue Objectだと思うのですが、お気に入りフラグ（isFavorite）を切り替えようとしたとき、どのような処理を書けばいいのでしょうか？？ValueObjectはイミュータブルな処理のため、一つのリポジトリ情報更新ごとに新しい配列を返すようなメソッドがいいかなと思ったのですが、このような処理でいいのでしょうか？？

```swift
struct RepositoryListViewDataModel{
    let elements:[Repository]
    
    func updateBookmark(repository:Repository)->Self{
        if let index = elements.firstIndex(where: {$0 == repository}){
            var elements = elements
            elements[index] = repository
            return RepositoryListViewDataModel(elements: elements)
        }else{
            throw Error
        }
    }
}
```

また、DTOについて書籍で下記のように記載がありました。
``これはDTO(Data Transfer Object)と言ってデータ転送用途に使われる設計パターンです。値変更の必要がないのでインスタンス変数はfinalで宣言し、コンストラクタで値が確定するようにします。参照系のみの用途であるため、更新系で使ってはいけません。更新系で用いると低凝集に陥ります。``

これとValue Objectの違いがよくわからないのですが、どのような違いがあるのでしょうか？
また、上記のような ViewdataはDTOかVOのどちらなのでしょうか？

CleanArchitectureでの扱いについて

### 前提

laravelを始めて1年くらい経ちます。
ドメイン駆動設計というのを聞いて、
EloquentモデルをviewやControllerで使い回さない設計方法をしたいです。

laravelで通常開発を行う場合、/student/{student}/addなどと記載して、
コントローラーのメソッドに以下のように記載すれば良いかと思います。
public function show(Student $student)
{
     ///////
    return view('student.add', compact('student'));
}

### 実現したいこと

これをモデル以外のデータをルートパラメータに追加するにはどのようにすれば良いでしょうか？
また、例えば、Entitiy/Studentをパラメータに取り扱う場合はそもそもEloquentモデルではないので、パラメータに追加はできないのでしょうか？（試して見たところ使えませんでした。）

もしそうでない場合以下のことで困っております。
--------------------------

/student/{id}/addなどとして、コントローラーは以下のようにする

public function show($id)
{
     //idからStudentEntitiyを引っ張るuseCaseを呼び出す。
}

ただ、その場合はですが、わざわざこのために
useCase、それに紐づくRepository、Interfaceを作成するのはコストがかかるかなと思います。

ですが、このようにルートパラメータを使用するケースは多くあると思いますが、
このようなケースの場合どのようにすれば良いのでしょうか？

ドメイン駆動設計と書いてありますが、EloquentモデルをDTOの形にして、
ルートパラメータで取り扱う方法を知りたいです。

稚拙な内容ですがご回答よろしくお願いいたします。



laravel ドメイン駆動設計の際のルートパラメータについて

Servlet/JSPでWebアプリケーションを作っています。
顧客が使用する画面と、経営者が使用する画面があり、現在経営者用の画面で顧客が入力した情報を更新する機能を作っています。
顧客の入力した情報を保持するDTOに更新後の値を保持するフィールドを追加して使いまわそうかと思っているのですが、そうすると顧客が情報を入力する際に更新用に追加したgetter、setter(getUpdatedName()など)を使わないことになります。
DTOの一部使わずに使いまわしていいものなのか、更新用の値を追加したDTOを新たに作成するべきなのか迷っています。
現在更新用のフィールドを追加したDTOで問題なく動いておりますが、実装的にどうなのか気になったので質問させていただきました。
ご回答お待ちしております。


```Java
public class TestDTO implements java.io.Serializable {

	private String name;
	private int age;
	private String tel;
	
	public String getName() {
		return name;
	}
	public int getAge() {
		return age;
	}
	public String getTel() {
		return tel;
	}
	public void setName(String name) {
		this.name = name;
	}
	public void setAge(int age) {
		this.age = age;
	}
	public void setTel(String tel) {
		this.tel = tel;
	}
	
}

```

↓　更新用のフィールドを追加する

```Java
public class TestDTO implements java.io.Serializable {

	private String name;
	private int age;
	private String tel;
	private String updatedName;
	private int updatedAge;
	private String updatedTel;

	public String getName() {
		return name;
	}
	public int getAge() {
		return age;
	}
	public String getTel() {
		return tel;
	}
	public String getUpdatedName() {
		return updatedName;
	}
	public int getUpdatedAge() {
		return updatedAge;
	}
	public String getUpdatedTel() {
		return updatedTel;
	}
	public void setName(String name) {
		this.name = name;
	}
	public void setAge(int age) {
		this.age = age;
	}
	public void setTel(String tel) {
		this.tel = tel;
	}
	public void setUpdatedName(String updatedName) {
		this.updatedName = updatedName;
	}
	public void setUpdatedAge(int updatedAge) {
		this.updatedAge = updatedAge;
	}
	public void setUpdatedTel(String updatedTel) {
		this.updatedTel = updatedTel;
	}

}
```

※質問用に変数名や変数の数を変えています

DTOのgetter、setteの一部を使わずに使いまわしてもいいのか

Javaで家計簿アプリを作っています。
DBはMySQLで、DTO,DAOパターンでデータの受け渡しを行う使用にしています。

今回、問題があったのはDate型の取扱についてです。

下記MainClassの「budgetTrackerDto.setDate(date);」でエラーが起きており、
The local variable date may not have been initializedと書かれており, 
data変数が宣言されていない的なメッセージが警告文として出ているのですが、
私としましては、Data変数を宣言している認識なのですが、
何故、当該エラーが起きているか、ヒントがあればご教示頂けますと幸いです。

ご回答のほど、よろしくお願いいたします。

MainClass.java
```ここに言語を入力
package DatabaseProgram;

import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Scanner;

public class MainClass {

	public static void main(String[] args) {
		// TODO Auto-generated method stub
		BudgetTrackerSelectDao budgetTrackerSelectDao;
		BudgetTrackerDto budgetTrackerDto;
		
		Map<Integer, String> initialSwitchMap = new HashMap<>();
        initialSwitchMap.put(1, "Select");
        initialSwitchMap.put(2, "Insert");
        initialSwitchMap.put(3, "Update");
        initialSwitchMap.put(4, "Delete");
        
        Scanner initialSwitchScanner = new Scanner(System.in);
        int initialNumInt = 0;
        do {
            String[] initialList = new String[] {initialSwitchMap.get(1), initialSwitchMap.get(2), initialSwitchMap.get(3), initialSwitchMap.get(4)};

            for(Map.Entry<Integer, String> list : initialSwitchMap.entrySet()) {
                System.out.println(list.getKey() + ":" + list.getValue());
            }

            System.out.print("Select a number: ");
            String initialNumStr = initialSwitchScanner.next();
            initialNumInt = Integer.parseInt(initialNumStr);

        } while (initialNumInt >= 5 || initialNumInt <= 0);
        
        
        
        switch (initialNumInt) {
        
        	case 1:
        		...
   　//長いのでここは省略
                        
        	case 2:
        		System.out.println("You chose " + initialSwitchMap.get(2));
        		BudgetTrackerInsertDao budgetTrackerInsertDao = new BudgetTrackerInsertDao();
        		
        		//Type ID
        		System.out.print("Type id: ");
                Scanner idScan = new Scanner(System.in);
                int id = idScan.nextInt();
                
               //setting date format
               SimpleDateFormat sdf = new SimpleDateFormat("yyyy-mm-dd");
               Scanner dateScan = new Scanner(System.in);
               
               System.out.print("Type Date(yyyy-mm-dd): ");
               String str = dateScan.nextLine();
               
               Date date;
				try {
					date = sdf.parse(str);
					sdf = new SimpleDateFormat("yyyy-mm-dd");
		               //System.out.println("Date: " + sdf.format(date));
		               String dateFormat = sdf.format(date);
				} catch (ParseException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
				
				//type Store name
                System.out.print("Type Store name: ");
                Scanner storeScan = new Scanner(System.in);
                String storeName = storeScan.nextLine();

                //type Product name
                System.out.print("Type Product name: ");
                Scanner productScan = new Scanner(System.in);
                String productName = productScan.nextLine();

                //type Type name
                System.out.print("Type Type name: ");
                Scanner typeScan = new Scanner(System.in);
                String typeName = typeScan.nextLine();

                //type Price name
                System.out.print("Type Price: ");
                Scanner priceScan = new Scanner(System.in);
                int price = priceScan.nextInt();
               
                budgetTrackerDto = new BudgetTrackerDto();
                budgetTrackerDto.setId(id);
                budgetTrackerDto.setDate(date);
                budgetTrackerDto.setStoreName(storeName);
                budgetTrackerDto.setProductName(productName);
                budgetTrackerDto.setType(typeName);
                budgetTrackerDto.setPrice(price);
                
        }
	}

}

```