編集履歴

回答編集履歴

コード修正

2020/11/25 08:28

投稿

スコア2663

test CHANGED Viewed

@@ -62,7 +62,7 @@
     {
-        _list = Enumerable.Range(0,size).Select(x => default(T)).ToList();
+        _list = new T[size].ToList();
         this._size = size;

コード修正

2020/11/25 08:28

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -94,7 +94,7 @@
     {
-        if (_list.Count() > _size)
+        if (_list.Count() >= _size)
         {

コード修正

2020/11/25 08:27

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -234,7 +234,7 @@
    public AddVector3(Vector3 v){
-       _vectors.add(v);
+       _vectors.Add(v);
    }
@@ -242,7 +242,7 @@
    public AddQuaternion(Quaternion q){
-       _quaternions.add(q);
+       _quaternions.Add(q);
    }

文章の修正

2020/11/25 08:25

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -199,3 +199,53 @@
 }
 ```
+> 現在同じ処理をそれぞれ型の違う(Vector3/Quaternion/float)数値へかけています。
+そこで処理を共通化したい
+```C#
+public class A
+{
+   private const int ListSize = 5;
+   private FixedLengthList<Vetctor3> _vectors = new FixedLengthList<Vetctor3>(ListSize);
+   private FixedLengthList<Quaternion> _quaternions = new FixedLengthList<Quaternion>(ListSize);
+   private FixedLengthList<float> _floatValues = new FixedLengthList<float>(ListSize);
+   public void Method(){
+       //TODO: _vectors,_quaternions,_floatValues を参照して何かを処理する;
+   }
+   public AddVector3(Vector3 v){
+       _vectors.add(v);
+   }
+   public AddQuaternion(Quaternion q){
+       _quaternions.add(q);
+   }
+}
+```

そもそもAクラスの必要性があるかもよく分からない

2020/11/25 08:24

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -62,7 +62,7 @@
     {
-        _list = new T[size].ToList();
+        _list = Enumerable.Range(0,size).Select(x => default(T)).ToList();
         this._size = size;
@@ -199,27 +199,3 @@
 }
 ```
-するとAクラスが非常にシンプルになります。
-```C#
-public class A
-{
-    private const int ListSize = 5;
-    private FixedLengthList<string> _strList = new FixedLengthList<string>(ListSize);
-    private FixedLengthList<float> _floatList = new FixedLengthList<float>(ListSize);
-}
-```
-これでこの質問で抱えていたそもそもの問題は回避できます。

文章の修正

2020/11/25 08:11

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -42,7 +42,7 @@
-記載されたコードでやりたいことは、ジェネリックで指定できる最新の要素のみを保持する固定長のコレクションを用意すれば実現できませんか？
+ジェネリックで型を指定できる最新の要素のみを保持する固定長のコレクションを用意すれば実現できませんか？

そもそもListクラスで例外が出るだろうからいらないか…　なので消しました。

2020/11/25 08:05

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -132,14 +132,6 @@
     {
-        if (arrayIndex >= _size)
-        {
-            throw new IndexOutOfRangeException();
-        }
         _list.CopyTo(array, arrayIndex);
     }
@@ -170,14 +162,6 @@
     {
-        if (index >= _size)
-        {
-            throw new IndexOutOfRangeException();
-        }
         _list.Insert(index, item);
     }
@@ -198,14 +182,6 @@
     {
-        if (index >= _size)
-        {
-            throw new IndexOutOfRangeException();
-        }
         _list.RemoveAt(index);
     }

しょぼいコーディングバグ

2020/11/25 08:04

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -132,6 +132,14 @@
     {
+        if (arrayIndex >= _size)
+        {
+            throw new IndexOutOfRangeException();
+        }
         _list.CopyTo(array, arrayIndex);
     }
@@ -152,6 +160,16 @@
     {
+        return _list.IndexOf(item);
+    }
+    public void Insert(int index, T item)
+    {
         if (index >= _size)
         {
@@ -160,16 +178,6 @@
         }
-        return _list.IndexOf(item);
-    }
-    public void Insert(int index, T item)
-    {
         _list.Insert(index, item);
     }
@@ -190,6 +198,14 @@
     {
+        if (index >= _size)
+        {
+            throw new IndexOutOfRangeException();
+        }
         _list.RemoveAt(index);
     }

追記

2020/11/25 08:03

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -35,3 +35,199 @@
 言語仕様にないので出来ません。その必要がないと判断されているからです。
+---
+記載されたコードでやりたいことは、ジェネリックで指定できる最新の要素のみを保持する固定長のコレクションを用意すれば実現できませんか？
+試しに作りました。
+```C#
+public class FixedLengthList<T> : IList<T>
+{
+    private readonly List<T> _list;
+    private readonly int _size;
+    public FixedLengthList(int size)
+    {
+        _list = new T[size].ToList();
+        this._size = size;
+    }
+    public T this[int index]
+    {
+        get => _list[index];
+        set => _list[index] = value;
+    }
+    public int Count => _list.Count;
+    public bool IsReadOnly => ((IList<T>)_list).IsReadOnly;
+    public void Add(T element)
+    {
+        if (_list.Count() > _size)
+        {
+            _list.RemoveAt(0);
+        }
+        _list.Add(element);
+    }
+    public void Clear()
+    {
+        _list.Clear();
+    }
+    public bool Contains(T item)
+    {
+        return _list.Contains(item);
+    }
+    public void CopyTo(T[] array, int arrayIndex)
+    {
+        _list.CopyTo(array, arrayIndex);
+    }
+    public IEnumerator<T> GetEnumerator()
+    {
+        return ((IList<T>)_list).GetEnumerator();
+    }
+    public int IndexOf(T item)
+    {
+        if (index >= _size)
+        {
+            throw new IndexOutOfRangeException();
+        }
+        return _list.IndexOf(item);
+    }
+    public void Insert(int index, T item)
+    {
+        _list.Insert(index, item);
+    }
+    public bool Remove(T item)
+    {
+        return _list.Remove(item);
+    }
+    public void RemoveAt(int index)
+    {
+        _list.RemoveAt(index);
+    }
+    IEnumerator IEnumerable.GetEnumerator()
+    {
+        return ((IList<T>)_list).GetEnumerator();
+    }
+}
+```
+するとAクラスが非常にシンプルになります。
+```C#
+public class A
+{
+    private const int ListSize = 5;
+    private FixedLengthList<string> _strList = new FixedLengthList<string>(ListSize);
+    private FixedLengthList<float> _floatList = new FixedLengthList<float>(ListSize);
+}
+```
+これでこの質問で抱えていたそもそもの問題は回避できます。

文章の修正

2020/11/25 08:01

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -18,7 +18,7 @@
-[Convertクラス](https://docs.microsoft.com/ja-jp/dotnet/api/system.convert?view=net-5.0#methods)を見てください。`ToByte`メソッド、`ToBoolean`メソッドなどはオーバーロードしていますが、ドキュメントを読めば間違うことはありません。
+[Convertクラス](https://docs.microsoft.com/ja-jp/dotnet/api/system.convert?view=net-5.0#methods)を見てください。`ToByte`メソッド、`ToBoolean`メソッドなどはオーバーロードを多く行っていますが、ドキュメントを読めば引数を渡し間違うことはありません。

文章の修正

2020/11/25 06:46

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -12,9 +12,9 @@
-利用者に、ドキュメントを読んで使うという文化があれば間違うことはありません。
+利用者に、ドキュメントを読んで使うという意識があれば間違うことはありません。
-その文化がなければ、そもそもプログラマとしてどうかを疑います。
+その意識がなければ、そもそもプログラマとしてどうかを疑います。