引数の中身（値が変わっている）

メソッドの引数にaList, bListというものを追加して
returnで返ってきたあとに
再度、情報を構築しようとすると、引数の値が変わる理由がわからないです。
ConstrucTionListの引数では判定しているだけ、上書きしたりしていないので
何故このようになるのか教えていただきたいです。。


List<Model> aList = this.Service~

List<Model> bList = this.Service~

List<Bean> makeList = this.make(aList , bList);

 private List<Bean> make(
            List<Model> aList,
            List<Model> bList ) {

hogeList = this.ConstrucTionList(aList, bList);

if ( CollectionUtils.isEmpty(bList)) {
   ~aListとbListを使って情報を構築する～
　　　※ここで情報構築するとaListの情報が変更されている

}

■ConstrucTionList
public List<Bean> ConstrucTionList(
            List<Model> aList,
            List<Model> bList) {
        
     List<Model> tmpList = new ArrayList<>();
        ~aListとbListを使って情報を構築する～
　　　for ( Model dt : aList) {
              
                dt.setA
                dt.setB
                dt.setC
                dt.setD
                dt.setE
       
                tmpList.add(dt);
            }
    for ( Model dt: bList) {

           dt.setA
                dt.setB
                dt.setC
                dt.setD
                dt.setE
    

    return  tmpList ;

azuapricot

2019/02/15 08:56

圧倒的情報量の少なさなような

dice142

2019/02/15 09:02

「～…情報を構築する～」の中でメソッドの中身を書き換えるメソッドを使っているということはないですか？

riomakopa

2019/02/15 09:07

お伺いしたいことは、メソッドの中に入ってかえってきたら　引数の値が変わることってあるのでしょうか？

riomakopa

2019/02/15 09:12

すみません。もう少し具体的に記入します

riomakopa

2019/02/15 09:20

更新いたしました。

行動規範の内容に同意します

回答3件

お伺いしたいことは、メソッドの中に入って

かえってきたら　引数の値が変わることってあるのでしょうか？

あります。
オブジェクトを渡した場合、呼び出したメソッドでその値が書き換えられれば、
呼び出し元でも書き換わってしまいます。

書き換えを防ぎたいのであれば、オブジェクトをコピーして渡すことになりますが、
基本的には表面だけのコピーになるので、注意が必要です。
（オブジェクトの中に更にオブジェクトがある場合は中のオブジェクトは参照のまま）

投稿2019/02/15 09:17

dice142

総合スコア5158

riomakopa

2019/02/15 09:39

ご回答ありがとうございます。ただ、呼び出したメソッドではaListは判定で使っておりfor文で回して返却用の箱につめているので書き換えをしているような所は見当たらないのですよね。少し、調べましたがcloneメソッドを使って実装すれば書き換えを防げるということでしょうか？

dice142

2019/02/15 09:48

> 書き換えをしているような所は見当たらないのですよね。 dt.setAなどは書き換えているわけではないのでしょうか？ > cloneメソッドを使って実装すれば > 書き換えを防げるということでしょうか？回答にも書いていますが、完全に防げるわけではないです。 cloneメソッドはシャローコピーなので、オブジェクト内部全部をコピーしてくれません。

riomakopa

2019/02/15 09:57

色々、教えていただきありがとうございます。確かにsetしているところで情報は変わっているのです（ただaList自体は値は変わるはずないと思い込んでいました）少しずつ理解できてきました。情報を再構築する場合 makeメソッド内でもう一度 List<Model> aList = this.Service~で情報を取ってきた方がベターな解決策でしょうか？

dice142

2019/02/15 10:04

> aList自体は値は変わるはずないと思い込んでいました aList自体が変わらなくてもaListが保持しているオブジェクトが変われば、結果は変わることに違いないです。（ただしプリミティブ型は除く） > 情報を再構築する場合 > makeメソッド内でもう一度 > List<Model> aList = this.Service~で > 情報を取ってきた方がベターな解決策でしょうか？ this.Service～がどういう取得しているのかわからないのでベターかどうかは不明です。コードの規模や設計によって何がいいかは変わりますが、とりあえず「ディープコピー」というキーワードで調べてみてください。

行動規範の内容に同意します