カスタムクラス内にベクターを定義する

Question

私は自分のクラスの中で単にベクターを使用しようとしています。ベクトルにアクセスしようとすると、定義されていないことがわかります（ただし、ヘッダーに定義しています）。

私はPersonとDogという2つのクラスを持っています。 1つの人が1つ以上の犬を所有することができるので、自分が所有する犬をアレイに追加したい。これは本当に簡単なので、この問題は本当に私に手をつけ始めているはずです。

クラスPerson.cpp：

#include "Person.h" 
#include "stdafx.h" 
#include <iostream> 

using namespace std; 

Person::Person(string name, string address, int age) 
    :name(name), 
    address(address), 
    age(age) 
    {} 
int Person::getAge(){ 
    return age; 
} 
std::string Person::getDogInfo(int index){ 
} 
void Person::addDog(string dogName, string breed){ 
    dogCollection.push_back(Dog(dogName, breed)); 
} 
std::vector<Dog> getDogs(){ 
    return dogCollection; //dogCollection undefined error here 
} 

そして、ここではPerson.hだ：ここにいくつかのコードは

#ifndef Person_H 
#define Person_H 
#include <vector> 
#include "Dog.h" 
using namespace std; 
class Person{ 
    public: 
     Person(string name, string address, int age); 
     string getName(){return name}; 
     string getAddress(){return address}; 
     void addDog(string dogName, string breed); 
     string getDogInfo(int index); 
     std::vector<Dog> getDogs(); 
     int getAge(); 

    private: 
     string name; 
     string address; 
     int age; 
     std::vector<Dog> dogCollection; 
}; 
#endif 

あなたは私の犬の授業を見てしたい場合は、私はそれらを貼り付けます同様：

Dog.cpp：

#include "stdafx.h" 
#include <iostream> 
#include "dog.h" 

Dog::Dog(string dogName, string breed) 
    :dogName(dogName), 
     breed(breed){} 

std::string Dog::Dog.getDogName(){ 
return dogName; 
} 

std::string Dog::Dog.getBreed(){ 
return breed; 
} 

とDog.h：

#ifndef Dog_H 
#define Dog_H 
#include <iostream> 
using namespace std; 

class Dog{ 
public: 
    Dog(std::string dogName, std::string breed); 
    std::string getDogName(); 
    std::string getBreed(); 
private: 
    std::string dogName; 
    std::string breed; 
}; 
#endif 

はまた、私は、これは何の宿題ではないことを追加します。私はjavaに慣れていて、将来の仕事のために必要とするので、C++を学ぶことのみを試みています。

EDIT：更新されたコード

Answer 1

これは正しくない（および不要）である：dogCollectionとして

dogCollection = new std::vector<Dog>; // Remove this line. 

がstd::vector<Dog>*ありません。

---

また、これは誤りです。dogCollectionとして

void Person::addDog(string dogName, string breed){ 
    Dog *newDog = new Dog(dogName, breed); 
    dogCollection.push_back(newDog); 
} 

はDogインスタンス、ないDog*が含まれています。変更します

void Person::addDog(string dogName, string breed){ 
    dogCollection.push_back(Dog(dogName, breed)); 
} 

---

コンストラクタのすべてに問題があります：

Person::Person(string name, string address, int age){ 
    name=name; 
    address=address; 
    age=age; 
} 

これは、それ自体に引数nameを割り当てている：それはメンバーnameに代入されていませんが。 addressおよびageと同じで、他のクラスのコンストラクタも同様です。初期化リストを使用します。

Person::Person(string name, string address, int age) : 
    name(name), 
    address(address), 
    age(age) 
{} 

---

この方法はstd::stringを返さない：

string Person::getDogInfo(int index){ 
} 

EDIT：

欠落クラスの修飾子：

std::vector<Dog> getDogs(){ 
    return dogCollection; //dogCollection undefined error here 
} 

が、これは単にであることを意味フリーe関数であり、クラスPersonに関連付けられていないため、dogCollectionへのアクセス権はありません。

変更するには：

std::vector<Dog> Person::getDogs(){ 
    return dogCollection; 
} 

Answer 2

std::vector<Dog> dogCollection; // here im defining dogCollection, no error here! 

実際がここでの問題あり - クラスDogは、この時点でコンパイラに知られていません。あなたはPerson.cppでPerson.h前Dog.h、またはより良いを含むいずれかの方法でこの問題を解決することができます

はPerson.hの上部に#include "Dog.h"を追加します。

Answer 3

あなたのコードにはいくつかの問題があります。他の回答の多くは、主にnewを使用しないことを指摘しています。 （あなたはC＃プログラマで、C++に移行していますか？）

しかし、#includeディレクティブにも問題があります。上記の通り、PersonはDogを使用しているため、そのヘッダーはPerson.hに含める必要があります。しかしPersonもvectorを使用するので、ヘッダーもそこに含めてください。だから、Person.hはその後Person.cppにあなたがそれらのファイルを含める必要はありません

#include <vector> 
#include "Dog.h" 

...で始まる必要があります。

一般的な規則（後で「前方宣言」について学ぶことができます）では、ヘッダーで参照されるすべての型は、そのヘッダー内で#includeです。