C++(MFC)で自作クラスをクラス定義(.h)と関数実装(.cpp)に分割したい

Question

前提・実現したいこと

C++(MFC)でsqliteをDLLとして読み込んで使いたいです。
DLLの読み込み処理をクラスにまとめてヘッダファイル(.h)に書いた際は動作しましたが、
そのクラスのコンストラクタの実装を.cppに分割したところコンパイルが通らなくなりました。
開発環境は次の通りです

• Windows8.1(64bit)
• VisualStudioCommunity2017
• MFCアプリケーション(ダイアログベース)
• SQLite version 3.27.1 こちらを元にDLLを読み込む処理を書いた

該当のソースコード

ダイアログにはボタンが一つあり、それが押されたときにsqliteのDLL読み込んで.dbファイルを作ります
sqliteを扱うクラスはSaverという名前でsaver.hにクラス定義、saver.cppにコンストラクタの実装、と分割したいと考えています。

saver.h

#pragma once

#include "sqlite3.h"
#include "stdafx.h"


// sqlite3.dll内関数ポインタ型定義
typedef int(__cdecl * pOpen)(char *, sqlite3 **);
typedef int(__cdecl * pExec)(sqlite3 *, const char *, sqlite3_callback, void *, char **);
typedef int(__cdecl * pGetTable)(sqlite3 *, const char *, char ***, int *, int *, char **);
typedef int(__cdecl * pFreeTable)(char **);
typedef int(__cdecl * pClose)(sqlite3 *);

class Saver {



public:
	Saver();


public:

	// 関数ポインタ変数
	pOpen sqlite3_open;
	pExec sqlite3_exec;
	pGetTable sqlite3_get_table;
	pFreeTable sqlite3_free_table;
	pClose sqlite3_close;


	HINSTANCE hDll;
	sqlite3 *db;
	char *err;
	char **result;
	int row, col;
	CString str;

};

saver.cpp

#include "saver.h"
#include "stdafx.h"
#include <windows.h>
Saver::Saver(){

	AfxMessageBox(_T("saver constructed"));
	// sqlite3.dllをロード
	hDll = LoadLibrary("sqlite3");
	if (hDll == NULL) {
		FreeLibrary(hDll);
		AfxMessageBox(_T("dll cant be loaded."));
		return;
	}


	// sqlite3関数のアドレスを取得
	sqlite3_open = (pOpen)GetProcAddress(hDll, "sqlite3_open");
	sqlite3_exec = (pExec)GetProcAddress(hDll, "sqlite3_exec");
	sqlite3_get_table = (pGetTable)GetProcAddress(hDll, "sqlite3_get_table");
	sqlite3_free_table = (pFreeTable)GetProcAddress(hDll, "sqlite3_free_table");
	sqlite3_close = (pClose)GetProcAddress(hDll, "sqlite3_close");



	// データベースファイルを作成
	sqlite3_open("test.db", &db);

	// テーブル作成
	sqlite3_exec(db, "create table test(num integer,name string);", NULL, NULL, &err);
	// データを格納
	sqlite3_exec(db, "insert into test values(1,'test1');", NULL, NULL, &err);


	// データベースファイルを閉じる
	sqlite3_close(db);


}

memoDlg.cpp

文字数が足りないので抜粋です。
デフォルトの状態からボタンを追加しており、ヘッダのインクルードとボタン押下時のクラスを実体化を追加しています。

// memoDlg.cpp : 実装ファイル
//

#include "stdafx.h"
#include "memo.h"
#include "memoDlg.h"
#include "afxdialogex.h"

#include "saver.h"

void CmemoDlg::OnBnClickedadd()
{
	// TODO: ここにコントロール通知ハンドラー コードを追加します。
	Saver sv;
}

エラー

1>------ ビルド開始: プロジェクト: memo, 構成: Debug x64 ------
1>saver.cpp
1>c:\users\master\desktop\memo\memo\saver.cpp(6): error C2653: 'Saver': 識別子がクラス名でも名前空間名でもありません。
1>c:\users\master\desktop\memo\memo\saver.cpp(6): error C4430: 型指定子がありません - int と仮定しました。メモ: C++ は int を既定値としてサポートしていません
1>c:\users\master\desktop\memo\memo\saver.cpp(10): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(11): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(12): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(14): warning C4508: 'Saver': 関数に戻り値の型が指定されていません。戻り値を void 型と見なします。
1>c:\users\master\desktop\memo\memo\saver.cpp(19): error C2065: 'sqlite3_open': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(19): error C2065: 'pOpen': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(19): error C2146: 構文エラー: ';' が、識別子 'GetProcAddress' の前に必要です。
1>c:\users\master\desktop\memo\memo\saver.cpp(19): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(20): error C2065: 'sqlite3_exec': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(20): error C2065: 'pExec': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(20): error C2146: 構文エラー: ';' が、識別子 'GetProcAddress' の前に必要です。
1>c:\users\master\desktop\memo\memo\saver.cpp(20): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(21): error C2065: 'sqlite3_get_table': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(21): error C2065: 'pGetTable': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(21): error C2146: 構文エラー: ';' が、識別子 'GetProcAddress' の前に必要です。
1>c:\users\master\desktop\memo\memo\saver.cpp(21): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(22): error C2065: 'sqlite3_free_table': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(22): error C2065: 'pFreeTable': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(22): error C2146: 構文エラー: ';' が、識別子 'GetProcAddress' の前に必要です。
1>c:\users\master\desktop\memo\memo\saver.cpp(22): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(23): error C2065: 'sqlite3_close': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(23): error C2065: 'pClose': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(23): error C2146: 構文エラー: ';' が、識別子 'GetProcAddress' の前に必要です。
1>c:\users\master\desktop\memo\memo\saver.cpp(23): error C2065: 'hDll': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(28): error C2065: 'db': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(28): error C3861: 'sqlite3_open': 識別子が見つかりませんでした
1>c:\users\master\desktop\memo\memo\saver.cpp(31): error C2065: 'db': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(31): error C2065: 'err': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(31): error C3861: 'sqlite3_exec': 識別子が見つかりませんでした
1>c:\users\master\desktop\memo\memo\saver.cpp(33): error C2065: 'db': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(33): error C2065: 'err': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(33): error C3861: 'sqlite3_exec': 識別子が見つかりませんでした
1>c:\users\master\desktop\memo\memo\saver.cpp(37): error C2065: 'db': 定義されていない識別子です。
1>c:\users\master\desktop\memo\memo\saver.cpp(37): error C3861: 'sqlite3_close': 識別子が見つかりませんでした
1>c:\users\master\desktop\memo\memo\saver.cpp(40): error C2617: 'Saver': 'return' ステートメントが正しくありません。
1>c:\users\master\desktop\memo\memo\saver.cpp(6): note: 'Saver' の宣言を確認してください
1>memoDlg.cpp
1>コードを生成中...
1>スキップ中... (関連する変更は検出されませんでした)
1>memo.cpp
1>プロジェクト "memo.vcxproj" のビルドが終了しました -- 失敗。
========== ビルド: 0 正常終了、1 失敗、0 更新不要、0 スキップ ==========

試したこと

saver.hにsaver.cppの内容を書き、saver.cpp内はコメントアウトした場合は想定通りに動きます。

saver.h

#pragma once

#include "sqlite3.h"
#include "stdafx.h"


// sqlite3.dll内関数ポインタ型定義
typedef int(__cdecl * pOpen)(char *, sqlite3 **);
typedef int(__cdecl * pExec)(sqlite3 *, const char *, sqlite3_callback, void *, char **);
typedef int(__cdecl * pGetTable)(sqlite3 *, const char *, char ***, int *, int *, char **);
typedef int(__cdecl * pFreeTable)(char **);
typedef int(__cdecl * pClose)(sqlite3 *);

class Saver {



public:
	Saver();


public:

	// 関数ポインタ変数
	pOpen sqlite3_open;
	pExec sqlite3_exec;
	pGetTable sqlite3_get_table;
	pFreeTable sqlite3_free_table;
	pClose sqlite3_close;


	HINSTANCE hDll;
	sqlite3 *db;
	char *err;
	char **result;
	int row, col;
	CString str;
};

Saver::Saver() {

	AfxMessageBox(_T("saver constructed"));
	// sqlite3.dllをロード
	hDll = LoadLibrary("sqlite3");
	if (hDll == NULL) {
		FreeLibrary(hDll);
		AfxMessageBox(_T("dll cant be loaded."));
		return;
	}


	// sqlite3関数のアドレスを取得
	sqlite3_open = (pOpen)GetProcAddress(hDll, "sqlite3_open");
	sqlite3_exec = (pExec)GetProcAddress(hDll, "sqlite3_exec");
	sqlite3_get_table = (pGetTable)GetProcAddress(hDll, "sqlite3_get_table");
	sqlite3_free_table = (pFreeTable)GetProcAddress(hDll, "sqlite3_free_table");
	sqlite3_close = (pClose)GetProcAddress(hDll, "sqlite3_close");



	// データベースファイルを作成
	sqlite3_open("test.db", &db);

	// テーブル作成
	sqlite3_exec(db, "create table test(num integer,name string);", NULL, NULL, &err);
	// データを格納
	sqlite3_exec(db, "insert into test values(1,'test1');", NULL, NULL, &err);


	// データベースファイルを閉じる
	sqlite3_close(db);


}

質問のMFCのコードではなぜ、クラスやそのメンバ変数を見つけられないのでしょうか？
よろしくお願い致します。

Comments

[KoichiSugiyama]

プロジェクトをビルドするのではなく、単純にsaver.cppをコンパイルした場合どうなりますか？
もし同じエラーが出るとしたら、saver.hとsaver.cppのどちらかに問題があると絞り込めると思います。
ヘッダにコピーしてビルドが通った状態のものから、再度*.hと*.cppに分けても同じエラーになりますか

[exp]

情報ありがとうございます！
.cppのみをコンパイルしてみると、エラーはでませんでした。

Answer 1

こんにちは。

外しているかも知れませんが、プリコンパイル済みヘッダで問題が起きている可能性があります。

#include "stdafx.h"の前にはコメント以外書いて大丈夫でしたっけ？　stdafx.hがプリコンパイルされた後で、その前にある定義が変更されると問題が発生しそうな気がします。それが原因であればリビルドすれば良いはずです。

ただ、過去リビルドでも改善しなかったことがあります。その時は、プリコンパイル済みヘッダを「使用しない」と設定して逃げました。設定方法は「プリコンパイル済みヘッダ」で検索すると多数でてきます。

Comments

[exp]

情報ありがとうございます！
コンストラクタの前方に`class Saver;`を追加して解決できました。
プリコンパイル済みヘッダについては知らなかったので、今後の参考にさせて頂きます。

[Chironian]

う～ん、少なくともそれは正しい対策ではないですよ。
本質的に解決出来るはずがない対処なので、後日、謎のトラブルに巻き込まれるリスクが結構あると思います。

[exp]

おっしゃる通り、リビルドしてみたところ、`class Saver;`がなくても動作しました。
先ほど解決したと思ったのは`class Saver;`は関係なく、
気づかないうちにリビルドされていたということでしょうか？

以後`#include "stdafx.h"`を最初に書くようにします。
ご指摘ありがとうございます！

Answer 2

この方法では解決していませんでした。
~~saver.cppでコンストラクタの実装より前に次を追加したところ解決しました。
前方宣言というのですね。
~~

class Saver;

ご協力頂いたy_waiwaiさん、KoichiSugiyamaさん、Chironianさん、ありがとうございました。

Answer 3

Saver.hはどこに入れてますか？
Saver.cppは、そのSaver.h ではないファイルを読んでいるんじゃないでしょうか

C/C++では、すでに定義されたものしか見つけることができません。
そのキーワードを使用するときには、それより以前（それより手前の行）で定義されていなければなりません。

Comments

[exp]

ご回答ありがとうございます！
saver.hは.cpp等と同じ階層にありました。
もしやと思い、#include "./saver.h"のようにしてみましたが同様の結果でした。