0

環境

Windows10 64bit
VisualStudio2019


現象

MFCにて、ダイアログベースでプログラムを作成し、
ダイアログを1つ追加し、ダイアログ間で値を受け渡そうとしています。

 ※2つのダイアログを下記で表現します。
   ベースのダイアログ:ダイアログA
   追加したダイアログ:ダイアログB

ダイアログAから、ダイアログBにint型の値を渡し
ダイアログBの関数内ではポインタを受け取るようにしました。

すると、ダイアログBのOnInitDialogのときにはポインタの値が
変化してしまい値を渡すことに失敗しました。

なぜかわからないので教えていただけないでしょうか。


プログラム概要

【ダイアログA】
 エディットコントロールがあり、そこに数値を入力できる。
 DialogBを開くボタンがあり、そのボタンが押下されるとエディットの値をDialogBに渡したあとDoModalする。

【ダイアログB】
 ディットコントロールがあり、DialogBから受け取った値をそこに表示する。


ソースコード

/*DialogA.h*/

public:
    afx_msg void OnBnClickedButton1();

private:
    int m_no;
/*DialogA.cpp*/

// DialogBを開くためのボタンを押下したときの処理
void CDialogADlg::OnBnClickedButton1()
{
    DialogB dlg;
    CString strNo;
    CWnd* Edit = GetDlgItem(IDC_EDITA);

    //エディットに表示されている文字列を取得
    Edit->GetWindowText(strNo);

    //文字列を数値型に変換
    m_no = _ttoi(strNo);

    //DialogBに数値を渡す
    dlg.setData(m_no);

    dlg.DoModal();
}
/*DialogB.h*/

public:
    void setData(int no);

private:
    int* m_pNo;
/*DialogB.cpp*/

void DialogB::setData(int no)
{
    //受け取った数値型のポインタをメンバポインタ変数に格納
    m_pNo = &no;
}

BOOL DialogB::OnInitDialog()
{
    CDialogEx::OnInitDialog();

    /* この時点で、DialogAから受け取ったポインタ変数の値(*m_pNo)が変わっている・・・なぜ?? */

    CString strNo;

    CWnd* Edit = GetDlgItem(IDC_EDITB);

    strNo.Format(_T("%d"), *m_pNo);

    Edit->SetWindowText(strNo);

    return TRUE;
}


やりたかったこと

ダイアログ間でデータをやりとりしたかった。
ポインタでやりとりすることにより、DialogBで入力した値がDialogAの値に反映されるようにしたかった。

2 件の回答 2

1
void DialogB::setData(int no)// noはスタック上の整数

がやりたいことと一致していないようです。
この場合、変数「no」の中身は「DLG-Aのm_no」の「値のコピー」ですが、
「no」のアドレスはスタック上に取られてしまい「DLG-Aのm_no」のアドレスとは異なります。以下の様に変更してみてはどうでしょう。

// DLG-A
CDialogADlg::OnBnClickedButton1()
{
    :
  dlg.setData( &m_no); // アドレスを渡す
  dlg.DoModal();
}

// DLG-B
void DialogB::setData(
   int *     ot_pNo) // アドレスを受け取る
{
   m_pNo =  ot_pNo; // アドレスのコピーを取る
}

ただし、個人的には
「DLGの寿命は一時的で短命なので、そのメンバのアドレス=ポインタを参照すべきではありません」
と、思ってます。
この場合、

// DLG-A
CDialogADlg::OnBnClickedButton1()
{
    :
  dlg.setData( m_no);   // 初期値を渡す
  dlg.DoModal();        // 編集する
  m_no = dlg.getData(); // 編集した値を取得する
}
// DLG-B
void DialogB::setData(
   int      ot_No) // 初期値値を受け取る
{
   m_Value =  ot_No;
}
int  getData() const 
{
   return m_Value; // 編集結果値を戻す
}

の様な仕組みの方が簡単かつ安全です。

1
  • ありがとうございました。C、C++でしかスタックって考え方使ったことはありませんがおもしろいですね。また、寿命の観点からgetで返すほうがいいという意見もありがとうございます。今後の参考にさせていただきます。
    – moriken
    Commented 2020年5月14日 6:01
1

ダイアログAから、ダイアログBにint型の値を渡し
ダイアログBの関数内ではポインタを受け取るようにしました。

C++言語では値がコピーされます。つまり1行目の時点で、ダイアログB側は一時的な領域(スタック)にコピーしています。一時的な領域なためDialogB::setData関数を抜け出た段階で領域が開放され再利用されます。ですので2行目の段階でアドレスを取得しポインタとしても既に遅いです。

ダイアログAから、ダイアログBにポインタを渡す必要があります。


一般的なC++言語ではポインタではなく参照を使います。ただし、MFCは参照への対応が不十分なため参照を使用するとかえって混乱するため、MFCと併用される際には無理に参照を使用せず引き続きポインタを使用されることをお勧めします。

1
  • ありがとうございます。渡した値のアドレスがスタック領域にできてしまうんですね・・・ 解決しました。
    – moriken
    Commented 2020年5月14日 5:59

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。