警告 C6262 をなくしたい

Question

実現したいこと

警告 C6262 関数はスタックの '11000316' バイトを使用します。データの一部をヒープに移動することを考慮してください。
という警告をなくしたい

発生している問題・エラーメッセージ

警告	C6262	関数はスタックの '11000316' バイトを使用します。データの一部をヒープに移動することを考慮してください。

該当のソースコード

#include <iostream>
#include <windows.h>
#include <vector>
#include <string>
using namespace std;
#pragma comment( lib, "urlmon.lib" )


#define		DEF_NUM		1
#define		DEF_DIRECTORY	"画像データ"

typedef struct {
	vector<string>		gv_szURL;		//	トップページのURL
	vector<string>		gv_szTitle;		//	トップページのタイトル
} _tNode;

static	_tNode	gInfo;


void	step1(int num);			//	トップページの取得 １ ～ DEF_NUM
void	step2(int num,char *szPoint);	//	point の文字列を探す

int main()
{
	CreateDirectoryA(DEF_DIRECTORY, NULL);
	step1( DEF_NUM );
	static char szPoint[256];
	sprintf_s(szPoint, "entry - card - wrap a - wrap border - element cf");
	step2( DEF_NUM, szPoint );
}

void	step2( int num, char *szPoint)
{
	int i;
	FILE* file;
	for (i = 1; i <= num; i++)
	{
		vector<string>	vText;
		char szText[256];
		sprintf_s(szText, "%s\\hoge%03d.txt", DEF_DIRECTORY, i);
		fopen_s(&file, szText, "rt");
		if (file == NULL)
			continue;
		//	ファイルの内容を全部　vText	に読み込む
		vText.clear();
		while (!feof(file))
		{
			char szBuf[10000000];
			memset(szBuf, 0, sizeof(szBuf));
			fgets(szBuf, sizeof(szBuf), file);
			vText.push_back(szBuf);
		}
		fclose(file);

		int j;
		for (j = 0; j < vText.size(); j++)
		{
			char szTemp[1000000];
			memset(szTemp, 0, sizeof(szTemp));
			strcpy_s(szTemp, vText.at(j).c_str());
			printf("%s", szTemp);
		}
	}
}


void	step1( int num )
{
	int i;
	for (i = 0; i < num; i++)
	{
		char szURL[256];
		char szText[256];
		sprintf_s(szURL, "https://erokan.net/page/%d", i);
		sprintf_s(szText, "%s\\hoge%03d.txt", DEF_DIRECTORY, i);
		Download(szURL, szText);
	}
}

試したこと

step2 の関数を追加したことで、この警告が発生したと思われるので、
main関数に書いてみたがダメだった。

補足情報（FW/ツールのバージョンなど）

OS:Windows 10 Home
IDE:Visual Studio Commnunity 2022

メモリは４GBを想定
十分にあると想定しています。

Comments

[jbpb0]

> 警告 C6262 関数はスタックの '11000316' バイトを使用します。データの一部をヒープに移動することを考慮してください。

参考
https://learn.microsoft.com/ja-jp/cpp/code-quality/c6262?view=msvc-170#example
の「例」
https://qiita.com/lightnig-programer/items/71631883264f3e6be30a
の「C6262」

Answer 1

これでどうだ?

#include <iostream>
#include <fstream> // 追加
#include <windows.h>
#include <vector>
#include <string>
using namespace std;
...
void	step2( int num, char *szPoint) {
	for ( int i = 1; i <= num; i++) {
        vector<string>	vText;
        char szText[256];
        sprintf_s(szText, "%s\\hoge%03d.txt", DEF_DIRECTORY, i);
        ifstream file(szText);
        if ( !file.is_open() )
			continue;

        string buf;
        while ( getline(file, buf)) {
            vText.push_back(buf);
        }

        for ( const std::string& szTemp : vText ) {
            puts(szTemp.data());
        }
    }
}
...

Answer 2

警告メッセージのとおりです。

ローカル変数で巨大な配列を確保すると、スタックを消費しますが、スタックはそんなに容量がないので不足してしまいます。

データがスタックに来ないよう、std::arrayなどに切り替えてみるのも一案です。

Comments

[yukkuri_55]

今の自分では力不足でできなさそうです。

Answer 3

[香車]東上☆あらし☆海美「
＞　step2 の関数を追加したことで、この警告が発生したと思われるので、main 関数に書いてみたがダメだった。

いったい、何を main 関数 に 書いた/移した んでしょうか ?

Download(szURL, szText) 関数は、定義がないので、注釈にしてあります。

#include <iostream>
#include <windows.h>
#include <vector>
#include <string>

using namespace std;
//#pragma comment( lib, "urlmon.lib" )


#define		DEF_NUM		1
#define		DEF_DIRECTORY	"画像データ"

typedef struct {
	vector<string>		gv_szURL;		//	トップページのURL
	vector<string>		gv_szTitle;		//	トップページのタイトル
} _tNode;

static	_tNode	gInfo;


//	トップページの取得 １ ～ DEF_NUM
void	step1(int num)
{
	int i;
	for (i = 0; i < num; i++)
	{
		char szURL[256];
		char szText[256];

		sprintf_s(szURL, "https://erokan.net/page/%d", i);
		sprintf_s(szText, "%s\\hoge%03d.txt", DEF_DIRECTORY, i);

		//Download(szURL, szText);
	}
}


// 大域変数に変更。
vector<string>	vText;
char szBuf[10000000];
char szTemp[10000000];


//	point の文字列を探す
void	step2(int num, char* szPoint)
{
	int i;
	FILE* file;

	for (i = 1; i <= num; i++)
	{
		//vector<string>	vText;
		char szText[256];

		sprintf_s(szText, "%s\\hoge%03d.txt", DEF_DIRECTORY, i);
		fopen_s(&file, szText, "rt");
		if (file == NULL)
			continue;
		//	ファイルの内容を全部　vText	に読み込む
		vText.clear();
		while (!feof(file))
		{
			//char szBuf[10000000];

			memset(szBuf, 0, sizeof(szBuf));
			fgets(szBuf, sizeof(szBuf), file);
			vText.push_back(szBuf);
		}
		fclose(file);

		int j;
		for (j = 0; j < vText.size(); j++)
		{
			//char szTemp[1000000];

			memset(szTemp, 0, sizeof(szTemp));
			strcpy_s(szTemp, vText.at(j).c_str());
			printf("%s", szTemp);
		}
	}
}


int
main() try
{
	CreateDirectoryA(DEF_DIRECTORY, NULL);
	step1(DEF_NUM);

	static char szPoint[256];
	sprintf_s(szPoint, "entry - card - wrap a - wrap border - element cf");
	step2(DEF_NUM, szPoint);

	return EXIT_SUCCESS;
}
catch (...)
{
    return EXIT_FAILURE;
}

」

Comments

[yukkuri_55]

大域変数に変更したら警告がなくなりました。
ありがとうございました。