ポインタに関するトラブル

質問

pointという座標を表す変数x,y,zつ保持するクラス型があったとして

point *a = point型のポインタを返す関数();

という式を作ったんですが、なぜか、
右辺と左辺はどちらのポインタも指すアドレスは確実に同じなのに。
値を参照しようとすると違う結果が出ます。具体的には

右辺から参照した結果が 50(正しい結果)　(*a).x を表示した結果
左辺から参照した結果が -9.25596e+061(？) (*point型のポインタを返す関数()).x を表示した結果

という結果で、完全に同じアドレスを保持するポインタなのに
実行した結果が違って混乱しています。まったく原因がわかりません

説明不足で申し訳ありませんが、なにが問題か分かる方がいらっしゃったらお願いしますm(_ _)m

回答No.4

kmee · ベストアンサー

> 自動変数のアドレスをもらって、直ぐにコピーしない

違います。
(*a).c[0] も「たまたま」値が上書きされなかっただけです。
アドレスが自動変数が使用する領域を指していること、そのアドレスにあったクラスは既に無効になっていること、には違いはありません。

現行のプロトタイプを使うなら、関数を抜けても消えないもの(グローバル変数や、newで確保した領域など)を返す必要があります。
どちらも、別の点で扱いに注意が必要です。

あるいは、呼び出し側をポインタではなく
point a;
と実体を用意して

・アドレス渡しや参照渡しを使う
void 関数(point * ret,point 原点,double w,double d);
void 関数(point & ret, point 原点,double w,double d);
→ point a ;
　関数(&a, 原点,100.,100.); // point * ret,～
　関数(a, 原点,100.,100.); // point &ret,～

・実体を返す関数にする
point 関数(point 原点,double w,double d);
→ point a = 関数(原点,100.,100.);
※ pointクラスが実際にはもっと複雑なら、コピーコンストラクタや =のオーバーライドが必要かも。

・pointクラスのメッソドとして定義する
class point {
point & method(point 原点,double w,double d){
～
return *this; //a.method(原点,100,100).c[0] みたいな使い方をしたいなら、自分自身の参照を返すようにする)
}
→ point a ;
　a.method(原点,100,100) ;

回答No.6

wormhole

＞自動変数のアドレスをもらって、直ぐにコピーしないと、

自動変数のアドレスを戻り値とすること自体が間違いです。

回答No.5

noname#164673

point型のポインタを返す関数()が返した結果を参照する時点で、実体が無くなっているとその様になる事があります。

point型のポインタを返す関数(){
point ans(0,0,0);
return &ans;
}

↑この様に関数を記述すると、関数の実行されている時点では、ansの値は正常ですが、リターンした後、スタックが上書きされるとansの値も上書きされます。

point型のポインタを返す関数(){
static point ans(0,0,0);
return &ans;
}

とすれば、値は静的領域に残るので、いつまでもansは有効です。

とりあえず、思いついたので、参考にして下さい。

回答No.3

επιστημη（@episteme）

point* 関数(point 原点,double w,double d){
point p = {0,0,0}; // 自動変数の
point* a = &p; // アドレスを
return a; // 返してますよ!
}

補足

なるほど・・・自動変数ってそういう意味なんですか・・・
いろいろ考えて分かりました。自動変数のアドレスをもらって、直ぐにコピーしないと、もともとそこにあったアドレスの指す変数は、スコープをでてしまっているから、上書きされちゃってたんですね・・・

回答No.2

Tacosan

「自動変数のアドレスを返す」というボケをかましているとか?

補足

それはどういう意味なのでしょうか・・・

説明不足だったので、補足にソースコードを付け足させて頂いたので、それを見てなにか分かることがあればよろしくお願いします。

回答No.1

wormhole

具体的にどういうコードを書かれているのか書いた方がよいかと思います。
とりあえず現状いえることは書かれているコードのどこかがおかしい(あなたが思った動きをするコードにはなってない)　くらいです。

補足

説明不足ですみません。 問題が再現されるギリギリまでソースコードを削ったものがこれです

//////////////////////////////////////////////////////////
#include <iostream>
#include <string>
#include <stdio.h>
using namespace std;

class point{
public:
double c[3];
};

point* 関数(point 原点,double w,double d){
point p = {0,0,0};
point* a = &p;
return a;
}
int main(void){
point 原点;

原点.c[0] = 0;
原点.c[1] = 0;
原点.c[2] = 0;

////////////////////////////////////////////////////////////
//左辺と右辺は同じはずなのに、表示すると違う結果に
point *a = 関数(原点,100,100);
////////////////////////////////////////////////////////////

printf("■アドレスの比較 \n");
printf("%p \n",a);
printf("%p \n",関数(原点,100,100));
printf("■値の比較 \n");
printf("%f \n",(*a).c[0]);
printf("%f \n",(*関数(原点,100,100)).c[0]);
getchar();
return 0;
}
/////////////////////////////////////////////////////////////

これを実行すると、アドレスは同じなのに、値が違うという結果になります。