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

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

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

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

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

メモリリーク

メモリリークは、プログラムファイルがメモリの解放に失敗した時に起こります。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Q&A

解決済

3回答

33383閲覧

【C言語/初心者】スタック破壊エラーの解決法について

ak_

総合スコア12

C

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

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

メモリリーク

メモリリークは、プログラムファイルがメモリの解放に失敗した時に起こります。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

0グッド

0クリップ

投稿2020/02/22 03:57

前提・実現したいこと

C言語初心者で配列の勉強をしています。
Visual Studio 2019にて”デバッグの開始”を行ったところ
下記エラーが発生します。
エラーを解決する方法と、このようなエラーを発生させないためには
なにを勉強すればいいか教えて頂けないでしょうか。

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

Run-Time Check Failure #2 - Stack around the variable 'a' was corrupted.

該当のソースコード

#include<stdio.h> #define N 20 /*要素の数*/ int main(void) { int a[N] = {1,5,4,9,23,16,2,13}; int i, temp, flag; do { flag = 0; for (i = 0; i < N + 1; i++) { if (a[i] > a[i + 1]) { temp = a[i]; a[i] = a[i + 1]; a[i + 1] = temp; flag = 1; } } } while (flag == 1); for (i = 0; i < N; i++) { printf("%d ", a[i]); printf("\n"); } return 0; }

試したこと

webで調べたところ、スタック破壊が起きているような記載を見かけました。
そこで、defineのNを大きくしてみましたが(10とか)、解決しませんでした。
ちなみにpaizaのwebエディタで同様の記述を行ったところ、エラーが
出なかったのも腑に落ちず、VS2019の操作方法に誤りがあるのかも疑っています。

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

Visual Studio 2019

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

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

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

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

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

guest

回答3

0

int a[N]; として宣言した配列の大きさは N 個です。 つまり 0 から N-1 のインデクスしか使えません。

ループの条件が for (i = 0; i < N + 1; i++) であれば i は最大で N になるのでこのとき a[i] a[i + 1] とすると確保している範囲外にアクセスしていることになります。

C はその設計理念として「プログラマを信じる」と「高速に動くべき」というということを挙げています。 なので、実行時に間違ったアクセスがあるかどうかをチェックすることを言語仕様は要求していません。 プログラマは間違ったアクセスはしないことを信じているので。 そしてチェックをしない方が高速に動作するので。

デタラメなアクセスをした結果は (言語仕様上は) 未定義です。 たまたまそこにあった値が読みだされるかもしれませんしクラッシュするかもしれません。

とはいえ、やっぱり人間は間違うものなのである程度はチェックしてくれるような機能を持つ処理系もあり、 VS2019 はそういった親切な機能をもっているという話なのです。 間違っていることをしても必ずしもエラーとして教えてくれないのは C の面倒なところで、実際に動かしながら学ぶというだけではそのような間違いに気づけない場合があります。 そこはきちんと仕様 (またはそれをわかりやすく解説したもの) を読まないと正しく使えません。

投稿2020/02/22 04:18

SaitoAtsushi

総合スコア5437

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

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

0

添え字として正常なのは、0からN-1までです。
for (i = 0; i < N + 1; i++)
で、i=Nがあり得るので、その時点で駄目です。

とりあえずのエラーの回避は、int a[N+2] = ~~~;にすればいいですが、それはあくまで「回避」なので、本来どうすればいいかはよく考えましょう。
int a[N+2] = ~~~;に書き直すことを勧めているわけじゃなくて、よく考えることを勧めています)

投稿2020/02/22 04:21

otn

総合スコア84423

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

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

0

ベストアンサー

Nを幾つにしようが"N + 1"は領域外に成ります。
領域の大きさは、0からN-1です。

投稿2020/02/22 04:05

編集2020/02/22 04:12
cateye

総合スコア6851

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

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

ak_

2020/02/22 04:18

うっかりしていました。 指摘いただいて気付くことができました。 ありがとうございました。
cateye

2020/02/22 04:31 編集

int a[N] = {1,5,4,9,23,16,2,13}のあと(初期化子13の後ろ?)は、0で初期化されますがいいのでしょうか? 参考→https://www.javadrive.jp/cstart/array/index4.html
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問