0

何らかの理由で、問題に対して劇的に間違った答えが返ってきたので、このprintfを入れてデバッグを試みました。

for (s = 0; s <= 100; s++) {
    for (t = 0; t <= 100; t++) {
        printf("At (%f,%f), spl = %f\n", s, t, spl(loc_data, s, t)); */
            if (spl(loc_data, s, t) > 80) { 
                     p++;
            }
        }  
}

printf行を省略すると、間違った答えが返されます。メモリ割り当てや初期化されていない変数と関係があると思いますが、これは私の能力を超えています。どんな助けでもいただければ幸いです。

コード全体：

#include <stdio.h>
#include <math.h>

typedef struct {
double x;
double y;
double W;
} data_t;

double spl(data_t *, double, double);

int main(int argc, char **argv) {

data_t loc_data[1000];

double spl0, p = 0, pp;

int i = 0, j = 0;

double s, t;

while (scanf("%lf %lf %lf", &loc_data[i].x, &loc_data[i].y, &loc_data[i].W) == 3) {
    i++;
}

printf("\nStage 1\n=======\n");
printf("Number of sound sources: %d\n", i);
for (j = 0; j < i; j++) {
    printf("%.1f meters east, %.1f meters north, power %1.5f Watts\n", 
    loc_data[j].x, loc_data[j].y, loc_data[j].W);
}






printf("\nStage 2\n=======\n");
spl0 = spl(loc_data, 0, 0);
printf("SPL at (0.0,0.0): %.1f dB\n", spl0);



printf("\nStage 3\n=======\n");


for (s = 0; s <= 100; s++) {
    for (t = 0; t <= 100; t++) {
        printf("At (%f,%f), spl = %f\n", s, t, spl(loc_data, s, t)); 
        if (spl(loc_data, s, t) > 80) { 
            p++;
        }
    }
}
pp = p / 102.01;

printf("Points sampled: 10201\nAbove 80.0 dB: %.1f%%\n", pp);

return 0;
}

double spl(data_t *loc_data, double pointx, double pointy) {
int i = 0;
double r_sq, powi, spli, spl;
while (loc_data[i].W != 0) {
    r_sq = pow(loc_data[i].x - pointx,2) + pow(loc_data[i].y - pointy,2);
    powi = 10*log10(loc_data[i].W / pow(10,-12));
    spli = powi + 10*log10((2 / (4 * M_PI * r_sq)) + (4 / (2.5 * M_PI * r_sq)));
    spl = 10*log10(pow(10, spl/10) + pow(10, spli/10));
    i++;
}
return spl;
}

フォーマットが悪いことをお詫びします。

4

2 に答える 2

3

spl関数で初期化される前に変数（）を使用していますspl()：

double spl(data_t *loc_data, double pointx, double pointy) {
int i = 0;
double r_sq, powi, spli, spl;
while (loc_data[i].W != 0) {
    r_sq = pow(loc_data[i].x - pointx,2) + pow(loc_data[i].y - pointy,2);
    powi = 10*log10(loc_data[i].W / pow(10,-12));
    spli = powi + 10*log10((2 / (4 * M_PI * r_sq)) + (4 / (2.5 * M_PI * r_sq)));
    spl = 10*log10(pow(10, spl/10) + pow(10, spli/10));
    //                     ^^^
    i++;
}
return spl;
}

呼び出しprintf()は、おそらく変数が持つ値に影響を与えています。

また、配列にいくつかのエントリを読み込みますが、その情報を関数loc_dataに渡さないでください。spl()ここではspl()、フィールドを含む配列エントリをW「センチネル」（配列の終わり）として扱います。入力データの最後のエントリの値がゼロになるのは当然ですか？もしそうなら、あなたはおそらく私たちに知らせて、おそらく入力を読み終えたらそれをチェックするべきです。

于 2012-05-20T06:41:01.077 に答える
2

エントリを初期化していないためloc_data、ガベージが含まれています。エントリを作成したらscanf（1000行を読み取った後、必ず読み取りを停止してください）、配列の残りの要素をゼロで埋めます。すでにこれを試したとおっしゃっていましたが、投稿したコードには表示されません。

さらに、関数iに（より適切な名前を付けた後）渡す必要splがあります。これにより、ターミネーターとして0を探すのではなく、適切なソースがいくつあるかがわかります。これは、1000の適切なソースがある場合は存在しないか、存在する可能性があります。一部のソースの累乗が0の場合は、早すぎます。

于 2012-05-20T06:40:30.247 に答える