トップ C#に関する質問 [C#]文字列による処理の振り分けをswitchでなくスマートに書きたい

編集履歴

回答編集履歴

キーが合っていないので修正

2019/12/02 08:59

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -314,15 +314,15 @@
         // データベースからテーブルデータをdataUnitsへ保存（下記はデバッグ用にデータを生成）
-        dataUnits.Add(new DataUnit("WalkSpeed", 10.0f));
+        dataUnits.Add(new DataUnit("Walk", 10.0f));
-        dataUnits.Add(new DataUnit("AttackSpeed", -20.0f));
+        dataUnits.Add(new DataUnit("Attack", -20.0f));
-        dataUnits.Add(new DataUnit("IsPoison", 1f));
+        dataUnits.Add(new DataUnit("Health", 1f));
-        dataUnits.Add(new DataUnit("WalkSpeed", 12f));
+        dataUnits.Add(new DataUnit("Walk", 12f));
-        dataUnits.Add(new DataUnit("AttackSpeed", 5f));
+        dataUnits.Add(new DataUnit("Attack", 5f));

コード追加

2019/12/02 08:59

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -207,3 +207,135 @@
 これで、`hitables`を列挙したときに新しく追加したインスタンスのメソッドが呼ばれるようになります。
+----
+掲示されたコードだとこうなりそうです。
+`Update`という名前が抽象的ですが、これ以上の具体的な命名は難しそうです。
+```C#
+public interface IAbility{
+    void Update(float value);
+}
+public class Walk : IAbility{
+    private float speed = 0f;
+    public void Update(float value){
+        speed += value;
+    }
+}
+public class Attack : IAbility{
+    private float speed = 0f;
+    public void Update(float value){
+        speed += value;
+    }
+}
+public class Health : IAbility{
+    private Condition condition = Condition.None;
+    public void Update(float value){
+        condition = value > 0f ? Condition.Poison : Condition.None;
+    }
+}
+public enum Condition{
+    None,
+    Poison,
+    Burning,
+    Palsy
+}
+public class Chara {
+    List<IAbility> abilities = new List<IAbility>();
+    public Chara()
+    {
+        abilities.Add(new Walk());
+        abilities.Add(new Attack());
+        abilities.Add(new Health());
+    }
+    void TestFunc() {
+        List<DataUnit> dataUnits = new List<DataUnit>();
+        // データベースからテーブルデータをdataUnitsへ保存（下記はデバッグ用にデータを生成）
+        dataUnits.Add(new DataUnit("WalkSpeed", 10.0f));
+        dataUnits.Add(new DataUnit("AttackSpeed", -20.0f));
+        dataUnits.Add(new DataUnit("IsPoison", 1f));
+        dataUnits.Add(new DataUnit("WalkSpeed", 12f));
+        dataUnits.Add(new DataUnit("AttackSpeed", 5f));
+        foreach(var unit in dataUnits) {
+            var ability = abilities.First(x => x.GetType().Name == unit.Key);
+            ability.Update(unit.Value);
+        }
+    }
+}
+```

文章の修正

2019/12/02 08:45

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -12,7 +12,7 @@
-まずは振る舞いを定義するためにインタフェースを定義します。
+まずは同一の振る舞いを持ったオブジェクトを複数定義するためにインターフェースを定義します。
@@ -134,7 +134,7 @@
-また、掲示されたコードでは、そもそもDataUnitsを使う必要性が分かりませんでした。
+また、掲示されたコードでは、そもそも`DataUnits`を使う必要性が分かりませんでした。
 下記のような書き方でも良さそうに思います。
@@ -184,13 +184,13 @@
 `fieldA`,`fieldB`,`fieldC`の違いを明確にしてください。
-配列・コレクションで持つ方が望ましいならそうするべきでしょう。
+おそらく配列やコレクションで持つ方が望ましいかもしれません。
 また、`DataUnits`の役割を明確にしてください。
-もし必要がない、もしくは`DataUnits`を使うことで却って煩雑になるようなら別の実装方法を検討すべきです。
+もし定義する必要性がない、もしくは`DataUnits`を使うことで却って煩雑になるようなら別の実装方法を検討したほうが良いかもしれません。
@@ -206,4 +206,4 @@
-これで、`hitables`を`foreach`で列挙したときにそれぞれのオブジェクト毎に処理を行います。
+これで、`hitables`を列挙したときに新しく追加したインスタンスのメソッドが呼ばれるようになります。

文章の修正

2019/12/02 08:23

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -80,7 +80,9 @@
-最後に、その振る舞いを持つオブジェクトをリストで保持するようにし、ポリモーフィズムを利用して各要素が持つ`ReceiveDamage`メソッドを呼び出してください。これで、各オブジェクトのメンバーが保持している値を変更できるようになります。
+最後に、`IHitable`なオブジェクトをリストに保持するようにし、ポリモーフィズムを利用して各要素が持つ`ReceiveDamage`メソッドを呼び出してください。
+これで、各オブジェクトのメンバーが保持している値を変更できるようになります。

文章修正とそれに対応してコードの順番を変更

2019/12/02 08:13

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -30,9 +30,37 @@
 次に、その振る舞いを持つオブジェクトを定義します。
+イメージでは、`Bair.HitPoint`が元`fieldA`、`Dog.HitPoint`が元`fieldB`、`OldMan.IsDead`が元`fieldC`に相当します。
-```C#
+```C#
+public class Bair : IHitable{
+    private int HitPoint = 300;
+    public void ReceiveDamage(int damage){
+        HitPoint -= damage / 2;
+    }
+}
+public class Dog : IHitable{
+    private int HitPoint = 150;
+    public void ReceiveDamage(int damage){
+        HitPoint -= damage;
+    }
+}
@@ -48,39 +76,11 @@
 }
-public class Bair : IHitable{
-    private int HitPoint = 300;
-    public void ReceiveDamage(int damage){
-        HitPoint -= damage / 2;
-    }
-}
-public class Dog : IHitable{
-    private int HitPoint = 150;
-    public void ReceiveDamage(int damage){
-        HitPoint -= damage;
-    }
-}
-```
+```
-最後に、その振る舞いを持つオブジェクトをリストで保持するようにし、ポリモーフィズムを利用して共通の振る舞いを持った`ReceiveDamage`メソッドを呼び出せばそれぞれ初期化ができるようになります。
+最後に、その振る舞いを持つオブジェクトをリストで保持するようにし、ポリモーフィズムを利用して各要素が持つ`ReceiveDamage`メソッドを呼び出してください。これで、各オブジェクトのメンバーが保持している値を変更できるようになります。
@@ -98,76 +98,76 @@
     void TestFunc(){
+        hitables.Add(new Bair());
+        hitables.Add(new Dog());
         hitables.Add(new OldMan());
+        List<DataUnit> dataUnits = new List<DataUnit>();
+        dataUnits.Add(new DataUnit("Bair", 20));
+        dataUnits.Add(new DataUnit("Dog", 1));
+        dataUnits.Add(new DataUnit("Oldman", 10));
+        foreach(var unit in dataUnits){
+            var hitable = hitables.First(x => x.GetType().Name == unit.Key);
+            hitable.ReceiveDamage(unit.Value);
+        }
+    }
+}
+```
+また、掲示されたコードでは、そもそもDataUnitsを使う必要性が分かりませんでした。
+下記のような書き方でも良さそうに思います。
+```C#
+public class Test{
+    List<IHitable> hitables = new List<IHitable>();
+    void TestFunc(){
+        var bair = new Bair();
+        bair.ReceiveDamage(20);
+        var dog = new Dog();
+        dog.ReceiveDamage(1);
+        var oldMan = new OldMan();
+        oldMan.ReceiveDamage(10);
         hitables.Add(new Bair());
         hitables.Add(new Dog());
-        List<DataUnit> dataUnits = new List<DataUnit>();
-        dataUnits.Add(new DataUnit("Oldman", 10));
-        dataUnits.Add(new DataUnit("Bair", 20));
-        dataUnits.Add(new DataUnit("Dog", 1));
-        foreach(var unit in dataUnits){
-            var hitable = hitables.First(x => x.GetType().Name == unit.Key);
-            hitable.ReceiveDamage(unit.Value);
-        }
-    }
-}
-```
-また、掲示されたコードでは、そもそもDataUnitsを使う必要性が分かりませんでした。
-下記のような書き方でも良さそうに思います。
-```C#
-public class Test{
-    List<IHitable> hitables = new List<IHitable>();
-    void TestFunc(){
-        var oldMan = new OldMan();
-        oldMan.ReceiveDamage(10);
-        var bair = new Bair();
-        bair.ReceiveDamage(20);
-        var dog = new Dog();
-        dog.ReceiveDamage(1);
         hitables.Add(new OldMan());
-        hitables.Add(new Bair());
-        hitables.Add(new Dog());
     }
 }

文章の修正

2019/12/02 08:11

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -196,11 +196,11 @@
-0. `IHitable`を実装した新しいクラスを定義してください
+0. `IHitable`を実装した新しいクラスを定義
-0. そのクラス内の`ReceiveDamage`メソッドに独自の処理を追加してください
+0. そのクラス内の`ReceiveDamage`メソッドに独自の処理を追加
-0. `hitables`に新しく追加したクラスのインスタンスを追加をしてください
+0. `hitables`に新しく追加したクラスのインスタンスを`Add`する

誤字

2019/12/02 08:04

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -192,7 +192,7 @@
-# 処理パターンが増やす場合
+# パターンを増やす場合

文章の修正

2019/12/02 08:02

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -196,8 +196,12 @@
+0. `IHitable`を実装した新しいクラスを定義してください
-`IHitable`を実装した新しいクラスを定義して、そのクラスに定義する`ReceiveDamage`メソッドに独自の処理を追加してください。
+0. そのクラス内の`ReceiveDamage`メソッドに独自の処理を追加してください
-そして`hitables`に追加をすれば処理が行われます。
+0. `hitables`に新しく追加したクラスのインスタンスを追加をしてください
+これで、`hitables`を`foreach`で列挙したときにそれぞれのオブジェクト毎に処理を行います。

構文エラーとマークダウンの修正

2019/12/02 08:01

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -1,4 +1,4 @@
-とりあえず、下記ケースを想定してサンプルを作りました。
+各`field`がどういう値を持つのかを明確にするため、下記ケースを想定してサンプルを作りました。
@@ -106,8 +106,6 @@
         List<DataUnit> dataUnits = new List<DataUnit>();
         dataUnits.Add(new DataUnit("Oldman", 10));
@@ -154,21 +152,17 @@
         oldMan.ReceiveDamage(10);
         var bair = new Bair();
         bair.ReceiveDamage(20);
         var dog = new Dog();
-        dog.ReceiveDamage(1)
+        dog.ReceiveDamage(1);
-        hitables.Add();
+        hitables.Add(new OldMan());
         hitables.Add(new Bair());
@@ -182,16 +176,28 @@
-少し独自路線へ向かってしまいましたので、話を戻します。
-スマートさを出すためには、
+# スマートさを出すために検討すべきこと
-- `fieldA`,`fieldB`,`fieldC`の違いを明確にしてください(配列・コレクションで持つ方が望ましいならそうする)
+`fieldA`,`fieldB`,`fieldC`の違いを明確にしてください。
+配列・コレクションで持つ方が望ましいならそうするべきでしょう。
+また、`DataUnits`の役割を明確にしてください。
-- `DataUnits`の役割を明確にしてください(必要がない、もしくは`DataUnits`を使うことで却って煩雑になるなら使わない)
+もし必要がない、もしくは`DataUnits`を使うことで却って煩雑になるようなら別の実装方法を検討すべきです。
-を心掛ける必要があります。
+# 処理パターンが増やす場合
+`IHitable`を実装した新しいクラスを定義して、そのクラスに定義する`ReceiveDamage`メソッドに独自の処理を追加してください。
+そして`hitables`に追加をすれば処理が行われます。