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

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

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

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

C++

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

Q&A

解決済

1回答

2668閲覧

動的多次元配列におけるstack overflow

Weapon

総合スコア106

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

C++

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

0グッド

0クリップ

投稿2020/09/26 11:51

編集2020/09/26 13:01

前提・実現したいこと

C言語で3次元配列を確保するにあたりstack使いすぎとの警告が出ます.
流石にhoge[1000000]などということはやっていないのですが
コードの書き方に問題があるのでしょうか?

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

C6262
関数はスタックの '40644' バイトを使用します: /analyze:stacksize '16384' を超えています。
データの一部をヒープに移動することを考慮してください。

簡単なソースコード

C

1#include <stdlib.h> 2#include <random> 3 4int hogefunc(void* lp, int x, int y, int z) 5{ 6 int*** lptmp = (int***)malloc(z * sizeof(int**)); 7 8 if(lptmp==nullptr)return -1; 9 10 for(int i = 0; i < z; i++) 11 { 12 lptmp[i]=(int**)malloc(y * sizeof(int*)); 13 if(lptmp[i] == nullptr) 14 { 15 for(int j=0;j < i; j++)free(lptmp[i]); 16 free(lptmp); 17 return -2; 18 } 19 } 20 21 for(int i = 0; i < z; i++)for(int j = 0; j < y; j++) 22 { 23 lptmp[i][j] = (int*)calloc(x, sizeof(int)); 24 if(lptmp[i][j] == nullptr) 25 { 26 for(int k = 0; k < j; k++)free(lptmp[i][k]); 27 for(int k = 0; k < i - 1; k++)for(int l = 0; l < y; l++)free(lptmp[k][l]); 28 for(int k = 0; k < z; k++)free(lptmp[k]); 29 free(lptmp); 30 return -3; 31 } 32 } 33 34 lp = (void*)lptmp; 35 36 { 37 /* Mersenne Twister */ 38 std::random_device seed_gen; 39 std::mt19937_64 engine(seed_gen()); 40 std::uniform_int_distribution<> dist(0, 1); 41 for (int i = 0; i < z; i++) 42 { 43 for (int j = 0; j < y; j++) 44 { 45 for (int k = 0; k < x; k++) 46 { 47 int tmpi = dist(engine); 48 if (tmpi == 0)tmpi = -1; 49 lptmp[i][j][k] = tmpi; 50 } 51 } 52 } 53 } 54 55 return 0; 56}

補足情報

Windows10 Pro
VisualStudio2019 Community

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

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

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

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

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

SHOMI

2020/09/26 12:10 編集

for(lptmp); コンパイルが通りません。free(lptmp);でしょうか? スタックを消費するようなコードも見当たりません。 質問の際に省略したコード部分に問題があるのでは?
y_waiwai

2020/09/26 12:17

全体のコードを提示しましょう。 一部分だけ出されてもどうしようもありません
Weapon

2020/09/26 12:18

ご指摘ありがとうございます. 他の部分を少し確認してみます.
SHOMI

2020/09/26 12:28 編集

C6262が検出されている行番号を確認して下さい。問題のある関数の先頭行になっていると思います。間違いなくhogefunc()なのでしょうか。
Weapon

2020/09/26 12:40 編集

本来のコードは引数をもとにswitchであらゆる型に切り替えるコードとなっており 似たようなコードなので全コードはお許しください.(どのみちC++混合のコードの為templateを使ったほうがいいのでしょうが) エラーは間違いなく動的確保をしているこの関数です. 追記しましたMersenneTwisterのコードをコメントアウトしたところエラーが消えましたがここなのでしょうか
y_waiwai

2020/09/26 12:39

タグはCとなってますが、C++なんでしょうか? あまりにも意味不明です
Weapon

2020/09/26 12:45

追記したMTはともかく 初期に掲示したコードでは書き方がC likeすぎてC++に分類するべきか自信がありませんでした.
SHOMI

2020/09/26 13:12 編集

si[i][j][k]は何者でしょうか。これを除いてコード分析をかけてもC6262は検出されませんでした。 他の省略箇所に問題が有るのでしょう。
guest

回答1

0

自己解決

Mersenne Twisterの複数の定義が原因で配列の動的確保自体には問題がないようなので一応質問は閉じさせていただきます.
ありがとうございました.

投稿2020/09/26 13:50

Weapon

総合スコア106

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

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

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

質問する

関連した質問