プロパティ(フィールド)の値を変更されたくない場合について

初歩的な質問かもしれませんが、以下のような状況に困っています。
現在kotlinにて実装しているのでサンプルはkotlinになりますが、回答はJavaでも問題ありません。
オブジェクト指向として？正しい実装はどれか教えてください。

現在の実装

Userクラスがあります。

kotlin
1class User {
2    val condition: Condition = Condition()
3
4    fun grow() {
5        condition.age++
6        condition.height++
7        condition.weight++
8    }
9}

UserのプロパティであるConditionクラスは以下です。

kotlin
1class Condition {
2    var age: Int = 0
3    var height: Int = 0
4    var weight: Int = 0
5}

Conditionクラスでは年齢、身長、体重をプロパティとしていて、Userクラスのgrow()メソッドにて年齢、身長、体重が1ずつ上がっていきます。(普通に考えると変ですが。。。)

困っていること

このときにMainクラスでUserクラスをインスタンス化して運用したいのですが、MainクラスからConditionの値が直接変更されてしまいます。

kotlin
1class Main {
2    val user: User = User()
3
4    fun main() {
5        // getはされてもよいと考えている
6        val age = user.condition.age
7
8        // setはさせたくない
9        user.condition.age = 100
10    }
11}

user.conditionを変更することはできません。(valとしているので)
しかし、user.condition.ageを変更することができてしまいますので、これを制止したいです。

いわゆるConditionクラスの値を、Userクラス内では自由に変更できるが、その外からは読み取り専用というようにしたいと考えています。

考えたこと

Conditionクラスをやめる

Userのプロパティに年齢などを持たせて、val とすることで、当たり前ですが上記の実現したいことができると考えています。そもそもクラス設計がおかしい、というのであればこの案に最終的にはたどり着くかと考えています。

Conditionクラスをコピーして渡す

UserクラスのConditionのgetterメソッドを加工して、別の実態を渡す。そうすればそのConditionクラスの値を書き換えられない気がする。(なんとなくです)

ほかにもval をvarに変えるなど考えましたがなかなか納得のいく実装になりませんでした。

考え方、正しい実装を教えていただきたく思います。
よろしくお願いいたします。

Zuishin

2020/08/31 00:07

Condition を変更し、メンバ変数を非公開にしてゲッターのみ公開する(つまり Condition をイミュータブルにする)方法を使うことは可能ですか？

miyabi_pudding

2020/08/31 00:35

> 回答はどの言語でも大丈夫です。言語によって、仕様も実装方法も考え方も変わってしまうので、今回はKotlinのようですし、Kotlin限定での話にした方がいいかと。（ただし、他の言語でもなんらかの方法がある、という考え方はある意味では間違っていない）

退会済みユーザー

2020/08/31 09:16

Zuishinさん返信が遅くなってしまい申し訳ございません。既に回答されているozwkさんの実装であればどうか。ということでしょうか。

退会済みユーザー

2020/08/31 09:17

miyabi_takatsukさん確かにおっしゃる通りでした。実際にはタグ付けしたJava, kotlinでの希望でした。すみません。

Zuishin

2020/08/31 09:54

そうです。Condition がイミュータブルで問題ないのならそれがスマートかつ手軽だと思います。そうできない制限があるのなら他の方法をとらなくてはいけませんが。

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

回答2件

ベストアンサー

Conditionをイミュータブルにする方法

kotlin
1class User {
2    var condition: Condition = Condition()
3        get
4        private set
5
6    fun grow() {
7        condition = Condition(condition.age+1,condition.height+1,condition.weight+1)
8    }
9}
10
11
12class Condition(val age: Int = 0, val height: Int = 0, val weight: Int = 0)
13
14
15val user: User = User()
16
17fun main() {
18    // getはされてもよいと考えている
19    val age = user.condition.age
20
21    // setはさせたくない
22    //user.condition.age = 100
23    
24    println(age) // 0
25    user.grow()
26    println(user.condition.age) // 1
27}
28

投稿2020/08/31 02:08

ozwk

総合スコア13532

退会済みユーザー

2020/09/01 00:49

回答ありがとうございます。確かにこの実装であればやりたいことが実現できていると考えています。しかし、今回のようにgrow()メソッドでは、年齢・身長・体重を全て変更していますが、別メソッドにて体重のみ変更があった場合には、体重のみ値を変更してConditionインスタンスを生成することになります。私自身、知識がないため質問しているので確実に違っているとは言えないのですが、なにか違和感があるような気がします。

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