質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
MFC

MFC (Microsoft Fouondation Class)とは、MicrosoftがVC++用に開発したWindows用アプリケーションのフレームワークです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

2回答

883閲覧

構造体にデータを格納することが出来ない

Grayi

総合スコア26

MFC

MFC (Microsoft Fouondation Class)とは、MicrosoftがVC++用に開発したWindows用アプリケーションのフレームワークです。

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

0クリップ

投稿2019/06/15 06:31

編集2019/06/15 10:07

iniファイルでページ数を指定し、コンボボックスに「1、2、3・・・、(最大ページ数)」と表示させ、最終的にコンボボックスで選択したページに画面が切り替わるようなツールを作ろうとしています。
以下のようなコードを書き、デバックして確認したところ、
構造体(m_ComboPageNum)にページ数が入っていないようだということが分かりました。
領域の確保の仕方が間違っているのが、データの格納の仕方が間違っているのか分かりません。
どなたか教えてくださると幸いです。よろしくお願いいたします。

ini

1//Setting.ini 2//ページ数を設定する 3 4[PAGE] 5PageNum=7

C++

1//test.h 2 3//コンボボックスに項目名と値を設定する構造体 4typedef struct{ 5 LPCSTR strName//項目名。コンボボックスに表示される 6 int iData//項目名ごとの値 7} COMBO_PAGE 8 9 10//test.cpp 11BOOL CTest::OnInitDialog() 12{ 13 //iniファイルから設定されたページ数を取得する 14 int iPage; 15 iPage = GetPrivateProfileInt(_T("PAGE"), T("PageNum"), 4, _T("Setting.ini")); 16 17 //ページ数+NULL分の領域を確保する 18 COMBO_PAGE *m_ComboPageNum; 19 m_ComboPageNum = new COMBO_PAGE[iPage + 1]; 20 21 int iPageNum = 0; 22 CString strPage; 23 24 for(int iCount = 0; iCount<iPage; iCount++) 25 { 26 iPageNum = iCount + 1; 27 strPage.Format(_T("%d"), iPageNum); 28 29 m_ComboPageNum[iCount] = {(LPCSTR)strPage, iPageNum}; 30 } 31 32333435

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

m_ComboPageNum[iCount] = {(LPCSTR)strPageNum, iPageNum};

strPageNumは何でしょうか?

それと

typedef struct{

LPCSTR strName//項目名。コンボボックスに表示される int iData//項目名ごとの値

} COMBO_PAGE

strName はポインターですがメモリの管理がいろいろ面倒なので、CStringとかにした方がいいんじゃないでしょうか?

投稿2019/06/15 10:02

Bull

総合スコア986

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Grayi

2019/06/15 10:12

申し訳ありません、strPageNumはstrPageと書くべきところでした。 コードを書くのが久しぶりなためか、手落ちが多くてすみません。質問内容を修正します。ご指摘ありがとうございます。 COMBO_PAGEは他のソースでも使用している構造体であるため、できればLPCSTRのままで使いたいと考えています。LPCSTRだから、COMBO_PAGEにデータが格納できないのでしょうか?
Bull

2019/06/15 10:17

できなくはないですが単純に代入するというわけにはいかないです。LPCSTRのまま文字列を格納するには、new 等でメモリーを確保してから、strcpy などで文字列をコピーする必要があります。当然ですが、必要なくなった段階で、delete しなければなりません。
Grayi

2019/06/15 10:35

m_ComboPageNum = new COMBO_PAGE[iPage + 1]; ↑ここでメモリーを確保しているので、あとはstrcpyなどで文字列をコピーすれば良い、というこでしょうか? 今すぐソースを変更して試すことが出来ないので、確認できたらコメントします。
episteme

2019/06/15 10:44

ちがいます。strNameもnew/deleteすべし。
Bull

2019/06/15 10:52

いえ違います。ここでメモリーを確保しているのは COMBO_PAGE 型オブジェクトを確保しているだけです。内部の strName はポインターなので、(LPCSTR)strPage を代入しても、strPage は使い回されているので、正しい文字列は保存されないのではないかと。
Grayi

2019/06/15 10:53

COMBO_PAGEをnewするだけではだめだったのですね。 strName, iData両方new/deleteしてみます。Bullさん、epistemeさん、アドバイスありがとうございます。
Bull

2019/06/15 10:56

iData は new/delete する必要はないというか、new/delete してはダメです。
Grayi

2019/06/15 12:00

そうだったのですね。教えてくださってありがとうございます。 strNameだけnew/deleteしてみます。
guest

0

*m_ComboPageNum = new COMBO_PAGE[iPage + 1];

m_ComboPageNum = new COMBO_PAGE[iPage + 1];

*が余計です。

投稿2019/06/15 06:44

asm

総合スコア15147

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Grayi

2019/06/15 06:47

すみません、今確認したらソースには*を付けていませんでした。 書き間違えてしまいました。質問内容を修正します。 ご指摘ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問