0

そのため、ビットマップ情報を扱う別の構造体を含む、いくつかの変数を持つ構造体を渡そうとしています。ただし、構造内に含まれる情報へのポインターの逆参照に関して、「不完全な型へのポインターの逆参照」というエラーを吐き出すため、私のコードはどこかで失敗します。ここでこれに対処する多くの質問があることは知っていますが、そこに記載されていることを実装しようとしましたが失敗しました.

初期化に関する編集を含む main() からの関連コードは次のとおりです。

pthread_t threads[thread_num];
pthread_attr_t attr;
int rc;
void *status;
pthread_attr_init(&attr);
pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);

// Create a bitmap of the appropriate size.
struct bitmap *bitm = bitmap_create(image_width,image_height);

struct thread_args *arguments = (struct thread_args*) malloc(sizeof(struct thread_args));
arguments->bm = bitm;
arguments->xmin = xcenter-scale;
arguments->xmax = xcenter+scale;
arguments->ymin = ycenter-scale;
arguments->ymax = ycenter+scale;
arguments->max = max;
// Compute the Mandelbrot image
for(int i=0;i<thread_num;i++){
    arguments->thread_id = i;
if(pthread_create(&threads[i], NULL, compute_image, (void *)arguments)<0){
   printf("ERROR; return code from pthread_create() is %d\n", rc);
}
 }

 pthread_attr_destroy(&attr);
 for(int t=0; t<thread_num; t++) {
    rc = pthread_join(threads[t], &status);
    if (rc) {
        printf("ERROR; return code from pthread_join() is %d\n", rc);
     exit(-1);
     }
 }

pthread_create の引数として渡される関数の関連コードは次のとおりです。

void* compute_image(void *threadargs ){
   int i,j;
   struct thread_data *my_data = (struct thread_args*) malloc(sizeof(struct thread_args));

   my_data = (struct thread_data *) threadargs;
   int width = bitmap_width(my_data->bm);
   int height = bitmap_height(my_data->bm);
   int threads = my_data->threads;
int thread_id = my_data->thread_id;
double xmin = my_data->xmin;
double xmax = my_data->xmax;
double ymin = my_data->ymin;
double ymax = my_data->ymax;
int max = my_data->max;


// For every pixel in the image...

for(j=height/threads*thread_id;j<height/threads*(thread_id+1);j++) {

    for(i=0;i<width;i++) {

        // Determine the point in x,y space for that pixel.
        double x = xmin + i*(xmax-xmin)/width;
        double y = ymin + j*(ymax-ymin)/height;

        // Compute the iterations at that point.
        int iters = iterations_at_point(x,y,max);

        // Set the pixel in the bitmap.
        bitmap_set(my_data->bm,i,j,iters);
    }
}

}

そして、ここに構造があります：

struct thread_args{
    int thread_id;
    int threads;
    struct bitmap *bm;
    double xmin;
    double xmax;
    double ymin;
    double ymax;
    int max;
};
4

2 に答える 2

2

あなたのコードには 2 つの問題があります。

  1. を定義しますstruct thread_argsが、compute_image()では を使用しますstruct thread_data。これらは同じではなく、(私は推測していますが) あなたが意味したのはthread_argsではなくthread_dataです。これは、おそらくあなたが得たコンパイルエラーを説明しています。struct image_data代わりにこれに名前を付けることを検討してください。それはおそらく混乱を引き起こす可能性が低いです。
  2. d メモリで初期化my_dataしmalloc()、すぐに のキャストから再割り当てしthreadargsます。これはメモリ リークです。正当な理由はありません。malloc初期化を削除するだけです。
于 2011-03-27T05:04:20.707 に答える
0

これは、 の定義をどこに置くかの問題ですstruct thread_args。コンパイラが利用可能なプロトタイプしか持っていない場合、コンパイラは構造体メンバーについて何も知りません。インクルードされたヘッダー ファイルに構造体定義を保持するか、定義を含む src ファイルにアクセサー関数を作成する必要があります。

于 2011-03-23T15:02:36.470 に答える