セッターゲッターメソッドを使って値を変更するにはどうしたらいいのでしょうか？

Question

###実現したいこと
SampleModelの値をセッターゲッターメソッドを使って変更するにはどうしたらいいのでしょうか？
値の変更は元の値から―1したいです。
今の書き方ではうまくいきません。なぜでしょうか？

package sample;

public class SampleModel {

	private int battery;

	public SampleModel(int battery) {
		this.battery = battery;
	}

	public int getBattery() {
		return battery;
	}

	public void setBattery(int battery) {
		this.battery = battery;
	}
}

package sample;

public class SampleUse {
		SampleModel m = new SampleModel(5);
		int battery = m.getBattery();

	public void useBattery(){
		int useBattery = battery -1;
		System.out.println(useBattery);
		m.setBattery(useBattery);
	}

	public void nowBattery() {
		System.out.println(battery);
	}

}

package sample;

public class SampleController {

	public static void main(String[] args) {

		SampleUse u = new SampleUse();
		u.nowBattery();

		u.useBattery();
		u.nowBattery();

	}

}

###そのほか
Eclipseの2020年版を使っています

元の値というのは
SampleModel m = new SampleModel(5);
で入れたSampleModelクラス内の変数「battery」の「5」です

Comments

[m.ts10806]

「元の値」ってどこのことでしょう。

Answer 1

おそらく、期待通りに -1 できています。
ただ、SampleUse クラスの nowBattery() メソッドが返しているのが、SampleModel クラスの battery プロパティではなく、 SampleUse クラスの battery プロパティであるため、変化していないように見えるのだと思います。

public class SampleUse {
    // ...省略
    public void nowBattery() {
        System.out.println(m.getBattery());
    }
}

としたら期待通りの動作になりませんか？

補足：SampleUse クラスで battery プロパティを定義するのはやめたほうがいいと思います。

Answer 2

get-1をsetすれば良いのでは。

setHoge(getHoge()-1);

今の書き方ではうまくいきません。なぜでしょうか？

「うまくいかない」が何が起きているかによるのですけど、
現状のコードはSampleModel の値ではなくSampleUse のプロパティbattery を更新してるだけです。
どこのどの情報を更新したいのでしょうか。

整理

この作りだとSampleUse に必要なのはprivateにしたSampleModelだけだと思います。
SampleUseにデータそのものを持ってしまってはModelに持っているものとは別物になります。

SampleModel

package sample.model;

import lombok.Data;

@Data
public class SampleModel {
	private int battery;
}

SampleUse

package sample.use;

import sample.model.SampleModel;

public class SampleUse {
	private SampleModel model;
	public SampleUse(int defaultBattery){
		this.model = new SampleModel();
		this.model.setBattery(defaultBattery);
	}
	public void useBattery() {
		this.model.setBattery(this.model.getBattery()-1);
	}
	public void outputBattery() {
		System.out.println(this.model.getBattery());
	}

}

SampleController

package sample.controller;

import sample.use.SampleUse;

public class SampleController {

	public static void main(String[] args) {
		SampleUse use = new SampleUse(5);
		use.outputBattery();
		use.useBattery();
		use.outputBattery();
	}

}

※Getter,Setterを自身の書くコードに置きたくないのでlombokを利用しました。
※sampleプロジェクトとはいえ、役割できちんとパッケージをわけるべきと考え、勝手にパッケージ追加しました
※個人的にプロパティでnewしたりデータ代入したりロジックっぽいの書くの嫌いなのでコンストラクタで行っています
※nowBattery()では名前から「今の値を返す」ように見えるので、メソッド名変えてます。

Comments

[yyy000]

いつもありがとうございます。
教えていただいたように

```java
public class SampleUse {
SampleModel m = new SampleModel(5);

public void useBattery(){
m.setBattery(m.getBattery() -1);
}

public void nowBattery() {
System.out.println(m.getBattery());
}

}
```

にしたところ値を変更することができました！ありがとうございます。
しかし、なぜ私の最初のやつではうまくいかなかったのでしょうか？
変数useBatteryには同じようにgetBattery-1の値である4が入っていると思ったのですが、ここのところがよくわかりません。
今後は教えていただいた書き方をしますが、うまくいかなかった理由が知りたいので、もしよろしければ教えていただけないでしょうか

[yyy000]

すいません、うまく表示させることができませんでした。

[m.ts10806]

「うまく」「うまくいかない」では何も伝えない表現ですので、具体的に現象を記載してください。プログラムは思った通りに動くのではなく書いた通りに動きます。
「元の値」が何なのか分かりません。
ModelなのかUseなのか。
今だとUseの値しか対象になっていないように見えますが。

[m.ts10806]

「Modelの値」としてざっと書きなおしてみます。
少々お時間ください。

[m.ts10806]

「SampleUse 」は何の役割ですか？直接Modelを呼び出したくないだけ？

[yyy000]

「すいません、うまく表示させることができませんでした」というのは、
↓のMarkdownと呼ばれる記法での表示がうまくできませんということです。
```html
コードの表示(ブロック)
```
具体的に書くことを今後は意識したいと思います

[m.ts10806]

コメント欄では使えません。

[yyy000]

SampleUseを使っているのは、SampleControllerでできるだけ減らしたいと考えたからです。
できる限り、SampleControllerではmainメソッド以外は定義しないようにしたいです。

[m.ts10806]

現在の内容くらいでしたら、SampleModelを直接newしたほうがシンプルで分かりやすくなるし、短く済みますよ。今のところSampleUserの役割が冗長です。

[m.ts10806]

書き直しました。
「なぜ想定の結果を得られなかったか」は
nowBatteryで出力している値はあくまでSampleUseのプロパティだから。
そしてuseBatteryでは
int useBattery = battery -1;
としていてローカル変数に格納しているのでbatteryが更新されないから。
○○Useをビジネスロジックのための役割で扱いたいのでしたら、データそのものを持たないのが吉です。
あくまでデータを呼び出したり受け取ったり渡したり計算するためだけに使い、自身ではデータを持たないことが役割分担の考え方です。
設計をしてみると分かると思います。
どこでデータを管理し、ロジックを持ち、それらをどこで扱うのか。
実際のアプリケーションはもっと複雑で入り組んで、クラスも沢山作りますから、現在の小さい構成からでも考えるようにしておくと、規模が大きくなっても迷うことなく作れます。