C++ メモリ共有 について

Question

C++ で「Windows のアプリケーション間の連携を」学習しているのですがいまいちわかりません。

C++ のメモリ共有というものでできるらしいのですが、参考にしているサンプルコードがコンパイルできず先に進めません。。

なかなかこのあたりの情報がなく学習が進みません。。こちらでは以前先達の方々にやさしく教えていただけてるので、また質問させていただきました。。

下記コードを動くことを確認した上で、サンプルコードをいじったりして学習したいと思っています。

当方プログラミング初心者（非プログラマー＆趣味レベル）ですので、わかりやすく教えていただけると幸いです。よろしくお願いいたします。

（環境）
Visual Studio Community 2015 / C++

参考サイトのコードはこれ

/* filemapping01.c */

#include <stdio.h>
#include <windows.h>
#include <conio.h>


int FRead(LPSTR);
int FWrite(LPSTR);

HANDLE hFMWrite;


int main()
{

	static LPSTR lpAddress1, lpAddress2;

	char szNo[8];
	BOOL bEnd = FALSE;


	while (1) {
		printf("***************\n");
		printf("1. Writre\n");
		printf("2. Read\n");
		printf("0. End\n");
		printf("***************\n");
		printf("No. = ");
		gets(szNo);

		switch (szNo[0]) {
		case '1':
			FWrite(lpAddress1);
			break;
		case '2':
			FRead(lpAddress2);
			break;
		case '0':
			bEnd = TRUE;
			break;
		default:
			printf("正しくない番号です\n\n");
			break;
		}
		if (bEnd)
			break;
	}


	if (hFMWrite) {
		if (CloseHandle(hFMWrite) == 0)
			printf("書き込みハンドルクローズ失敗\n");
		else
			printf("書き込みハンドルクローズ成功\n");
	}

	return 0;
}

int FWrite(LPSTR lpStr)
{
	char szStr[1024];

	if (!hFMWrite)
		CloseHandle(hFMWrite);


	hFMWrite = CreateFileMapping(
		(HANDLE)-1,
		NULL,
		PAGE_READWRITE,
		0,
		1024,
		"NEKODEMOWAKARU");
	if (hFMWrite == NULL)
		return -1;
	if (GetLastError() == ERROR_ALREADY_EXISTS)
		printf("すでにマッピングオブジェクトが存在しました\n");

	lpStr = (LPSTR)MapViewOfFile(hFMWrite,
		FILE_MAP_ALL_ACCESS, 0, 0, 0);
	if (lpStr == NULL)
		return -2;

	printf("書き込み==");
    gets(szStr);
	strcpy(lpStr, szStr);
	strcat(lpStr, "\n");

	UnmapViewOfFile(lpStr);

	printf("書き込みました\n");
	return 0;
}

int FRead(LPSTR lpStr)
{
	HANDLE hFM;

	hFM = OpenFileMapping(FILE_MAP_ALL_ACCESS, FALSE, "NEKODEMOWAKARU");

	lpStr = (LPSTR)MapViewOfFile(hFM,
		FILE_MAP_ALL_ACCESS, 0, 0, 0);

	if (lpStr == NULL) {
		printf("受信失敗\n");
		CloseHandle(hFM);
		return -1;
	}
	printf(lpStr);
	if (UnmapViewOfFile(lpStr) == 0) {
		printf("読み込み用アドレスアンマップ失敗\n");
	}
	else {
		printf("読み込み用アドレスアンマップ成功\n");
		lpStr = NULL;
	}

	CloseHandle(hFM);

	return 0;
}

引用：第１１４章 メモリマップトファイル その１

（エラー内容）
● 'gets': 識別子が見つかりませんでした

●'strcat': This function or variable may be unsafe. Consider using strcat_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.

Answer 1

'gets': 識別子が見つかりませんでした

gets()はセキュリティの観点から危険な関数であり、Visual Studio 2015以降で廃止されました。代わりに下記のように書き換える必要があります。

fgets(szNo, sizeof(szNo), stdin);

'strcat': This function or variable may be unsafe. Consider using strcat_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.

メッセージどおり、プログラム先頭に #define _CRT_SECURE_NO_WARNINGS とすれば警告が表示されなくなります。

Comments

[manbo]

yohhoyさん

早速コメントいただき誠にありがとうございます。
そういうことでしたか、gets()はそもそも使えなくなっていたんですね。
解決法もご記入いただきありがとうございます。

おかげさまで、エラーは消えました。

ただ、strcat()については、警告ではなくエラーでして、#defineでは消えませんでした。strcat_s（）によって自己解決できましたので、後程書かせていただきたいと思います。

とても参考になりました。ありがとうございました。

Answer 2

エラーはメモリ共有などには特に関係のない内容です。
getsはバッファオーバーランを簡単に起こす危険な関数のため最新のC言語の仕様から削除されました。
そのためVS2015でもgetsは削除されているので関数がないとなります。
fgetsで同じことをできるのでgetsをそちらに置き換えてください。

gets(szNo)
↓
fgets(szNo, sizeof(szNo), stdin);

2つ目のワーニングですが、strcatも気を付けないとバッファオーバーランを起こすのであまり推奨されておらず代わりに安全な strcat_s を使ってね、というものです。学習中というだけならいったん無視しておいても大丈夫です。
このワーニングがうっとうしかったらプロジェクトのプロパティから C/C++ -> プリプロセッサにあるプリプロセッサの定義に_CRT_SECURE_NO_WARNINGSを追加することで出なくなりますよ。

Comments

[lib]

あ、目から鱗が落ちました。　なるほど　2013で実行できたのはその関係でしたか。ためになります。
https://msdn.microsoft.com/ja-jp/library/bb531344.aspx

[manbo]

tokiさん

さっそくコメントいただき誠にありがとうございます。

そういうことだったんですね。そもそもgets()がなくなっていたんですね。しかもVS2015からとは初心者にはなんとも不運でした。

strcat_sも教えていただきありがとうございます。プリプロセッサがどうやるのかわからなかったので、strcat_sを使って自己解決いたしました。のちほど自己解決を書きたいと思いますので、まずいところがあればぜひご指摘をお願いいたします。

Answer 3

ご回答をくださった方々、迅速なご回答誠にありがとうございました。

以下に解決を書かせていただきます。じつはgetsをfgetsで回避することができましたが、strcatは、警告ではなくエラーになっており #define では回避できませんでした。ですので、ご教示いただいたとおり、strcatはstrcat_sによって回避することができました。

以下がやったことです。getsはfgetsにstrcatとstrcpyに_sを付けて、引数にキャラ型のbufを追加しました。エラーは回避し「'buf': ローカル変数は 1 度も使われていません」という警告はでたもののプログラムは正常に動作しました。もしまずいところがあればぜひご指摘お願いいたします。

●gets(szNo); → fgets(szNo, sizeof(szNo), stdin);

●strcat(lpStr, "\n"); → strcat_s(lpStr, sizeof(buf), "\n");

●strcpy(lpStr, szStr); → strcpy_s(lpStr, sizeof(buf), szStr);

（以下が修正版です）

/* Visual Studio Community 2015*/
/* Source.cpp */

//#include <stdio.h>
#include <iostream>
#include <windows.h>
#include <conio.h>


int FRead(LPSTR);
int FWrite(LPSTR);

HANDLE hFMWrite;


int main()
{

	static LPSTR lpAddress1, lpAddress2;

	char szNo[8];
	BOOL bEnd = FALSE;


	while (1) {
		printf("***************\n");
		printf("1. Writre\n");
		printf("2. Read\n");
		printf("0. End\n");
		printf("***************\n");
		printf("No. = ");
		//gets(szNo);
		fgets(szNo, sizeof(szNo), stdin);


		switch (szNo[0]) {
		case '1':
			FWrite(lpAddress1);
			break;
		case '2':
			FRead(lpAddress2);
			break;
		case '0':
			bEnd = TRUE;
			break;
		default:
			printf("正しくない番号です\n\n");
			break;
		}
		if (bEnd)
			break;
	}


	if (hFMWrite) {
		if (CloseHandle(hFMWrite) == 0)
			printf("書き込みハンドルクローズ失敗\n");
		else
			printf("書き込みハンドルクローズ成功\n");
	}

	return 0;
}

int FWrite(LPSTR lpStr)
{
	char szStr[1024];

	if (!hFMWrite)
		CloseHandle(hFMWrite);


	hFMWrite = CreateFileMapping(
		(HANDLE)-1,
		NULL,
		PAGE_READWRITE,
		0,
		1024,
		"NEKODEMOWAKARU");
	if (hFMWrite == NULL)
		return -1;
	if (GetLastError() == ERROR_ALREADY_EXISTS)
		printf("すでにマッピングオブジェクトが存在しました\n");

	lpStr = (LPSTR)MapViewOfFile(hFMWrite,
		FILE_MAP_ALL_ACCESS, 0, 0, 0);
	if (lpStr == NULL)
		return -2;

	char buf[30];
	printf("書き込み==");
	//gets(szStr);
	//strcpy(lpStr, szStr);
	//strcat(lpStr, "\n");
	fgets(szStr, sizeof(szStr), stdin);
	strcpy_s(lpStr, sizeof(buf), szStr);
	strcat_s(lpStr, sizeof(buf), "\n");

	UnmapViewOfFile(lpStr);

	printf("書き込みました\n");
	return 0;
}

int FRead(LPSTR lpStr)
{
	HANDLE hFM;

	hFM = OpenFileMapping(FILE_MAP_ALL_ACCESS, FALSE, "NEKODEMOWAKARU");

	lpStr = (LPSTR)MapViewOfFile(hFM,
		FILE_MAP_ALL_ACCESS, 0, 0, 0);

	if (lpStr == NULL) {
		printf("受信失敗\n");
		CloseHandle(hFM);
		return -1;
	}
	printf(lpStr);
	if (UnmapViewOfFile(lpStr) == 0) {
		printf("読み込み用アドレスアンマップ失敗\n");
	}
	else {
		printf("読み込み用アドレスアンマップ成功\n");
		lpStr = NULL;
	}

	CloseHandle(hFM);

	return 0;
}

Comments

[yohhoy]

strcpy_s/strcat_sの第二引数が不適切です。正しくは第一引数に渡したバッファ(lpStr)のサイズを指定すべきです。この例ではCreateFileMappingで作成した1024バイトのファイルマッピングオブジェクトサイズになるかと思います。

[退会済みユーザー]

xx

[manbo]

yohhoyさん

改訂ポイントをお教えいただき誠にありがとうございます。
大変参考になりました。

Answer 4

すでに皆さん回答されている内容でエラーは回避できると思いますので、別の観点でひとこと。

参考にされているサイトは私もかつてはお世話になったものですが、さすがにすべての記事を更新し続けることはできないでしょうから情報が古いものも多いです。キャプチャー画像はWindows 2000でしょうか。ものによってはWindows 95時代のものもあるようです。
最初のうちはできるだけ新しい入門書・解説書で勉強するのがお勧めなのですが、ネットの情報だけで勉強されているのだとしたら同じような問題に遭遇する可能性もあるので、いつ書かれた記事なのかということも気にして、情報が古い場合には注意が必要です。

Comments

[manbo]

catsforepawさん

さっそくコメントいただき誠にありがとうございます。

なるほど、このサイトはそんなに前からあるんですね。。。95ということは２０年も経つんですか。。そんなに前とは知りませんでした。それは動かないわけですね。その時代からHP上がってるというのもサーバーがずっと動いてるのもなんだかすごいです。

たしかに、おっしゃるように、バージョンやいつの記述かは非常に重要なんだなと最近わかってきました。pythonをいじっていて痛感させられました。ネットの情報はほとんど古くほとんど動きませんでした。

Answer 5

こんばんは

実はgets()は今の環境では使えなくなりました（仕様から削除されました）．
バッファオーバーフローという言葉を聞いたことありませんか？
gets()はプログラマが想像している以上の入力文字数をもらうと具合が悪く，その対応が出来ないためにこれを使えなくしました（ま，事故防止ですな）．

ここだけ修正してもコンパイルは通りませんが，まずは一歩ずついきましょうか

	while (1) {
		printf("***************\n");
		printf("1. Writre\n");
		printf("2. Read\n");
		printf("0. End\n");
		printf("***************\n");
		printf("No. = ");
		// gets(szNo);  // これはやめる
		fgets(szNo, sizeof(szNo), stdin);  // こんな感じで修正する

		switch (szNo[0]) {
		case '1':
	

Comments

[manbo]

Oh-ZAKさん

さっそくコメントいただき誠にありがとうございます。

そういうことだったんですね、事故防止で廃止されたんですね。。そういう情報はやはりベテランの方でないとわからないですよね。学習中の私はこれひとつで何週間も悩んでいて笑、はやくこちらで質問すればよかったとおもいました。。

ありがとうございます。

Answer 6

こんにちは。

やってみましたが、コンパイルもできますし、動作もしました。(msvc 2015 update1)
拡張子をcppにしていませんか？ C++モードでコンパイルするとエラーになるのだと思います。
拡張子をcにしてC言語としてコンパイルしてみてください。

---

【余談ですが】
もし、C++を使うのであれば、Cの標準ライブラリ(stdio.h等)を使うのはやめて、C++の標準ライブラリ(iostream等)を使うことをお勧めします。それなりに勉強は必要ですが、安全性が飛躍的に高まります。
Windows APIはCの時とほぼ同じように呼べます。

Comments

[manbo]

chironianさん

さっそくコメントいただき誠にありがとうございます。

なるほど、このコードはC言語のコードだったんですね。。たしかにファイル名がCとなっていますね。あのサイトのコードはすべてC++だと思っていました。。

確かに、拡張子をCにするとエラーが消えるものがありましたが、違うエラーが続出しました。。いろいろ試してみましたが、CPPに戻して、iostream を使うことにしました。
ありがとうございます。

[Chironian]

？　不思議ですね。
私は、Visual Studio 2015 update1を使い、cl filemapping01.cでコンパイルしました。
しかし、Cに拘っても得るものはありませんので、その方針がお薦めです。

Answer 7

どのプロジェクトで作成したのかによって変わってきますが・・・。
・Win32プロジェクト
・コンソール アプリケーション
追加のオプションは、
空のプロジェクト
Security Development Lifecycle (SDL)チェックは外す
共通ヘッダーファイルを追加は、すべてチェック外す
で作成したプロジェクトで書いてますか？

Comments

[manbo]

libさん

さっそくコメントいただき誠にありがとうございます。
プロジェクトの作成は、C++のコンソールアプリケーションで空の状態からつくっていました。

ちなみに、SDLチェックは外しても問題ないのですか？？