編集履歴

回答編集履歴

friend についての説明を追加

2020/03/09 15:21

投稿

スコア8224

test CHANGED Viewed

@@ -331,3 +331,35 @@
 ```
 main は省略しました。
+**追記4**
+friend の意味を誤解していませんか？
+質問のコードでは、class Iterator の中に frined class Vector; がありますが、
+これは、Iterator の中のメンバ mCurrentIndex や mMyVector が private であっても
+Vector の中のメンバ関数から参照できるようにするものです。
+そんなものが必要でしょうか？
+逆に class Vector の中に、frined class Iterator; と書けば、
+Iterator の中のメンバ関数が Vector の中のメンバ mData などを参照できます。
+例えば、**追記3** のコードでそう書けば、Vectorのメンバ関数 At がなくても、
+Iterator のメンバ関数 GetData が次のように書けます。
+```C++
+        T& GetData() { return mMyVector->mData[mCurrentIndex]; }
+```

Vector<T> へのポインタをもつコードを追加

2020/03/09 15:21

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -237,3 +237,97 @@
 IsEqual は Vector のすべての要素が等しいかどうかではなく、
 Iterator が同じ要素を指しているかどうかを調べるものです。
+**追記3**
+Iterator が Vector<T> へのポインタを持つように書くこともできます。
+```C++
+#include <iostream>
+using namespace std;
+template<typename T>
+class Vector {
+    T* mData;
+    int mSize;
+    int mCapacity;
+public:
+    Vector() {
+        mSize = 0; mCapacity = 15; mData = new T[mCapacity];
+    }
+    ~Vector() { delete[] mData; }
+    void PushBack(const T& tItem) {
+        if (mSize < mCapacity)
+            mData[mSize++] = tItem;
+        else
+            cout << "to be implemented later\n";
+    }
+	T& At(int i) { return mData[i]; }
+    class Iterator {
+        int mCurrentIndex = 0;
+        Vector<T> *mMyVector = nullptr;
+    public:  // コンストラクタは public に
+        Iterator(Vector<T> *tWho, int tLocation) {
+            mMyVector = tWho;
+            mCurrentIndex = tLocation;
+        }
+        T& GetData() { return mMyVector->At(mCurrentIndex); }
+        void Next() { mCurrentIndex += 1; }
+        bool IsEqual(const Iterator& rhs) const {
+            return mMyVector == rhs.mMyVector && mCurrentIndex == rhs.mCurrentIndex;
+        }
+    };  // end of class Iterator definition
+    Iterator Begin() { return Iterator(this, 0); }
+    Iterator End() { return Iterator(this, mSize); }
+};
+```
+main は省略しました。

質問者のコードの問題点を指摘

2020/03/09 06:45

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -115,3 +115,125 @@
 ```
 さらに、main の a は、Vector<int> a(20); にします。
+**追記2**
+main が少しずつ実行できるように作っていきましょう。
+```C++
+#include <iostream>
+using namespace std;
+template<typename T>
+class Vector {
+    T* mData;
+    int mSize;
+    int mCapacity;
+public:
+    Vector() {
+        mSize = 0; mCapacity = 15; mData = new T[mCapacity];
+    }
+    ~Vector() { delete[] mData; }
+    void PushBack(const T& tItem) {
+		if (mSize < mCapacity)
+			mData[mSize++] = tItem;
+		else
+			cout << "to be implemented later\n";
+	}
+    class Iterator {
+        int mCurrentIndex = 0;
+        T *mMyVector = nullptr;  // not a pointer to Vector<T>
+    public:  // コンストラクタは public に
+        Iterator(T *tWho, int tLocation) {  // not a pointer to Vector<T>
+            mMyVector = tWho;
+            mCurrentIndex = tLocation;
+        }
+        T& GetData() { return mMyVector[mCurrentIndex]; }
+        void Next() { mCurrentIndex += 1; }
+        bool IsEqual(const Iterator& rhs) const {
+			return mMyVector == rhs.mMyVector && mCurrentIndex == rhs.mCurrentIndex;
+        }
+    };  // end of class Iterator definition
+    Iterator Begin() { return Iterator(mData, 0); }
+    Iterator End() { return Iterator(mData, mSize); }
+};
+int main()
+{
+    Vector<int> tTester;
+    tTester.PushBack(0);
+    tTester.PushBack(1);
+    tTester.PushBack(2);
+    for (Vector<int>::Iterator iter = tTester.Begin();
+			!iter.IsEqual(tTester.End()); iter.Next()) {
+        cout << iter.GetData() << " ";
+    }
+    cout << endl;
+}
+```
+IsEqual は Vector のすべての要素が等しいかどうかではなく、
+Iterator が同じ要素を指しているかどうかを調べるものです。

Vectorのコンストラクタの修正

2020/03/09 05:28

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -95,3 +95,23 @@
 ```
 Vector のコピーコンストラクタや代入演算子の定義は省略しました。
+**追記**
+質問のコードの Vector のコンストラクタの 15 は mSize ではなく mCapacity のデフォルト値だったんですね。
+コンストラクタを次のように修正します。
+```C++
+    Vector(int n = 0) : mSize(n), mCapacity(n < 15 ? 15 : n) {
+		mData = new T[mCapacity];
+	}
+```
+さらに、main の a は、Vector<int> a(20); にします。

無意味な改行の削除

2020/03/08 16:09

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -70,8 +70,6 @@
 		a[i] = i + 1;
-	cout << endl;
 	for (Vector<int>::Iterator it = a.Begin(); it != a.End(); ++it)
 		cout << " " << *it;