2011-12-20 16 views
0

次の問題があります。次の関数は、競合をNULLに初期化して呼び出されます。ポインタの配列へのポインタをパラメータとして渡します。

fooの最後には、競合が正しい値をとります。この例の場合、* conflictはfprintfsに従って4,8、および2の値を含むべきです。

しかし、fooを呼び出す関数(2番目のコードの抜粋を参照)で再度テストすると、競合配列は変更されません。私は配列へのポインタを渡しているので、なぜこのテクニックがmultiDBと再帰的なポインタでうまく動作するのかは分かりません。どんな助けもありがとう。 (これは途中で完全なコードではなく、関連する部分のみを示しています)。ありがとう!

int foo(
    /*==================*/ 
    uchar* buf, 
    uint* multiDB, 
    uint* recursive, 
    uint** conflict) { 

select_consistent= conflict; 
bool finished; 

if (((start_of_scan == 1) && (*multiDB != 1)) || ((start_of_scan== 0) && (select_consistent == NULL))) { 
    fprintf(stderr, "Not doing select consistent \n "); 

    finished = TRUE; 
} 
else { 
    *multiDB=0; 
    fprintf(stderr, "Doing select consistent \n "); 
    finished = FALSE; 
    uint n; 
    int i = 0 ; 
    if (select_consistent == NULL) { /*This is the first round */ 
      next_round = FALSE; 
     fp = popen("./algorithm '[t(1,[t(2,||),t(3,[t(8,||),t(10,||)])]).t(1,[t(4,||),t(6,||)]).t(1,[t(2,||),t(7,||)])]'", "r"); /* Issue the command.  */ 
     finished = FALSE; 
    } 
    if (next_round == TRUE) { 
     goto parse_records; 
    } 

     fscanf(fp, "%lu", &n); 
     uint* conflict_ ; 
     if (n!=0) conflict_ = (uint*) malloc(sizeof (uint) * n); 
     conflict = &conflict_; 
     next_round = TRUE; 
     int j= 0 ; 
     while (fscanf(fp, "%lu", &n) != EOF) { 
      if (n == 0) { 
       select_consistent=conflict; 
       goto parse_records; 
      } 
      else { 
       (*conflict)[j] = n; 
      } 
      i++; 
      j++; 

     } 
     finished = TRUE;  
} 
parse_records:; 
int error; 

.... [other code] foo2(multiDB, recursive); 

fprintf(stderr, "Array states %lu %lu \n ", (*conflict)[0], (*conflict)[1]); 
fprintf(stderr, "Array states %lu %lu \n ", (*select_consistent)[0], (*select_consistent)[1]); 



return error 
} 

機能を呼び出すfooがこのようにそれをしない:

uint** conflict = NULL ; 
    error = foo(buf, multiDB, recursive, conflict); 
    fprintf(stderr, "value is %lu \n", (*conflict)[0]); //This is still unitialised. 

答えて

0

コードはいくつかのポインタの間接問題があることが表示されます。 は、uintの一次元配列を意味すると思われる。もしそうなら、最初の宣言は次のようになります。

uint* conflict = NULL; 

次に、このようにそれを渡す次のようにそれを割り当てるfooに続いて

foo(... &conflict); 

*conflict = (uint*)malloc(...); 

それとも、まだしたい場合にconflict_を使用するには、次のように割り当てます。

*conflict = conflict_; 
関連する問題