C++のpublic関数を使用してクラスポインタを返します

Question

文字列をパラメータとするpublic関数を使用してクラスポインタを返そうとしています。私はそれがどのように行われたのか（正しい方法）は分かりません。これは、私が持っているものです：

Room::Room* getPointer(const string &nameOfRoom){  
    return &Room(); 
} 

私はこの構造体を使用する場合、正しいポインタまたは私の関数が作成している新しいインスタンスへのポインタを返していますか？
ありがとうございます

申し訳ありませんが私の意図については非常に明確ではないの皆さん。私は部屋の中に他の部屋へのポインタのリストを持っています。私の意見では、グラフのように見えるはずです。私は、リスト内のポインタを使ってグラフを「ナビゲート」する文字を持っています。私は文字列の形でユーザーの入力を取得しています。私のユーザが "ベッドルームに移動"と言う場合、私は何とか私のリストのベッドルームを指しているポインタを取得できるように、関数にパラメータとして "ベッドルーム"を渡したいと思います。
私はそれを正しく説明しているかどうか分からないので、「新しさ」と以前の説明の欠如を許してください。
ありがとう

Answer 1

いいえ、間違いがあります。ほとんどのコンパイラは、これについて警告する必要があります。私は意図したセマンティクスがあなたの機能にどのようなものかはっきりしていないので、あなたはそれについて非常に明確ではないが、それは非常に間違いではない。

Answer 2

ポインタをオブジェクトに戻そうとしている場合は、そのようにします。

Room* getP(const std::string& sfasd) 
{ 
    return this; 
} 

しかし、それはより良い意味的にこれを行うために：あなたはあなたができる新しい部屋を作成することでしたどのような意味ならば、最終的

Room r; 
Room *rp=&r; 

：

Room& getP(const std::string& sfasd) 
{ 
    return *this; 
} 

をまた、あなたは常にこの操作を行うことができます

Room& getP(const std::string& name) 
{ 
    return new Room(name); 
} 

::://else where 
std::unique_ptr<Room> room_pointer(getP("hi")); 

Answer 3

あなたが何をしようとしているのか明確ではありません。 まず、関数がポインタを無効にすると無効になるので、まず関数はスタックベースの変数のアドレスを返します。

そして、あなたは文字列を渡してスタックスペースを無駄にしています....それを使用しないでください。マイクロソフトコンパイラを使用している場合、レベル4の警告をオンにすることができます（デフォルトではオンにならない）それについての警告。あなたが部屋の新しいインスタンスを作成したい場合は

は、単に次の操作を行います。

Room* Room::getPointer() 
{ 
    return new Room(); 
} 

だから、再び、私はあなたが何をしたいか分からないので、私はこの点だけで推測することができます。