newとdeleteな気がするエラー

Question

以下のようなプログラムを書いているのですが、delete [] ang;の行を実行すると、

Debug Assertion Failed!

Program: ...\Debug\pro.exe
File: f:\dd\vctools\crt_bld\self_x86\crt\src\dbgdel.cpp
Line: 52

Expression: _BLOCK_TYPE_IS_VALID(pHead->nBlockUse)

というエラーが発生します。
fscanfを書いてからうまくいかなくなってきたので、modtorがオーバーフローしてるのかな？とも思ったのですが、特にそれらしいものは見当たりません。

いったい何が原因なのでしょうか？
よろしくお願いします。

#include<stdio.h>
#include<string>
using namespace std;
void main(void){
　　int i;
　　int ret;

　　static const int anum=3;
　　static const int LoopMax=100;
　　double *modtor=new double[LoopMax*anum];
　　double *ang=new double[LoopMax*2*anum];
　　const string strlearned="A.csv";
　　const string strlearning="_learning.csv";
　　FILE *fpl;
　　FILE *fpvnmd;
　　errno_t err;
　　double outputv[anum]={0};
　　double tempd;

　　double *tarr=new double[LoopMax];
　　//delete [] tarr;　　　　　　//ここで消すとエラーが消える
　　fopen_s(&fpl,strlearned.c_str(),"r");
　　printf("データ読み込み\n");
　　for(i=0;i<LoopMax;i++){
　　　　if(fscanf(fpl,"%[^,],%lf,%lf,%lf,%lf",&tempd,&modtor[anum*i],&modtor[anum*i+1],&modtor[anum*i+2])==EOF)break;
　　}
　　fclose(fpl);　　
　　delete [] ang;
　　delete [] tarr;
　　delete [] modtor;
}

php504 · Accepted Answer

>この意味を余りよく知らないのです＾＾；
これが原因なので意味を理解しないまま使ってはだめでしょう。
%[～]は文字列を読み込む指定子の1つです。(普通は%sですね）
フォーマット系の可変引数の関数では引数の型チェックができません。
フォーマット指定子と実際の引数の型を合わせるのはプログラマの責任で型が合わないとこのようなバグを引き起こします。

>この「*」は何を意味しているのでしょうか？
http://www.linux.or.jp/JM/html/LDP_man-pages/man3/scanf.3.html　より引用
-------------------
代入抑制文字"*"。 scanf() は変換指定に指示された通り入力を読み込むが、その入力は捨てられる。対応する pointer 引き数は必要なく、 scanf() が返す代入が成功した数にこの指定は含まれない。 
-------------------
です。

php504 · Answer

4.　「元のままだと「５番目の引数が無いので、」とありますが、もとの場合、5番目の引数は、「&modtor[anum*i+1]」となっていると思うのですが、これはどういう意味なのでしょうか？

フォーマット指定部分"%[^,],%lf,%lf,%lf,%lf"では5個の指定がありますが読み込む変数が
&tempd
&modtor[anum*i]
&modtor[anum*i+1]
&modtor[anum*i+2]
の4個しかないということでしょう。
今回は個数の問題だけでもなく1個目の型があってないのも原因ですが。

データが
0.000000,0.000000,0.010127,0.210067
なら
fscanf(fpl,"%lf,%lf,%lf,%lf",&tempd,&modtor[anum*i],&modtor[anum*i+1],&modtor[anum*i+2])
でも問題なかったのでは。

jmh · Answer

その「http://simd.jugem.jp/?eid=49」には「%[^,] を使用することにより、カンマ以外の_文_字_列_を読み込むという_指_定_が出来るわけです。」とありますが？

jmh · Answer

%[^,] に対して double tempd でいいの？

chie65535 · Answer

if(fscanf(fpl,"%[^,],%lf,%lf,%lf,%lf",&tempd,&modtor[anum*i],&modtor[anum*i+1],&modtor[anum*i+2])==EOF)break;
を
if(fscanf(fpl,"%*[^,],%lf,%lf,%lf",&modtor[anum*i],&modtor[anum*i+1],&modtor[anum*i+2])!=anum)break;
にしてみよう。

「scanfは引数が足りなくても知ったこっちゃない」と「scanfは読み捨て可能」と、２つの事を覚えましょう。

元のままだと「５番目の引数が無いので、メモリ上にあったゴミをアドレスだと思って、そこに実数値を格納し、メモリがグチャグチャに壊れる」とか「カンマ以外を読み捨てる時、読み捨てるバイト数がdoubleオブジェクトのサイズを超えてたら、メモリがグチャグチャに壊れる」って事が起きます。

つか、
＞　　//delete [] tarr;　　　　　　//ここで消すとエラーが消える
の状態で実行時エラーが出ないで動いちゃうのが不思議。偶然「壊れても構わないメモリが壊れているだけ」なんだろうなあ。

newとdeleteな気がするエラー

>この意味を余りよく知らないのです＾＾；

4. 「元のままだと「５番目の引数が無いので、」とありますが、もとの場合、5番目の引数は、「&modtor[anum*i+1]」となっていると思うのですが、これはどういう意味なのでしょうか？

その「

%[^,] に対して double tempd でいいの？

if(fscanf(fpl,"%[^,],%lf,%lf,%lf,%lf",&tempd,&modtor[anum*i],&modtor[anum*i+1],&modtor[anum*i+2])==EOF)break;

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング

4.　「元のままだと「５番目の引数が無いので、」とありますが、もとの場合、5番目の引数は、「&modtor[anum*i+1]」となっていると思うのですが、これはどういう意味なのでしょうか？

if(fscanf(fpl,"%[^,],%lf,%lf,%lf,%lf",&tempd,&modtor[anumi],&modtor[anumi+1],&modtor[anum*i+2])==EOF)break;