0

問題は、STRUCTURE1 と STRUCTURE2 の 2 つの構造が必要であると述べていました (元は私が知っています)。STRUCTURE1 は、5 つの float 値 (配列ではない) と 1 つのポインターを保持します。STRUCTURE2 は、1 つの float 値と、別の STRUCTURE2 型への 1 つのポインターを保持します。また、各要素が STRUCTURE1 を指すポインターの配列も必要でした。次に、最初のSTRUCTURE1はそれぞれ別のSTRUCTURE1を指しますが、そのSTRUCTURE1はSTRUCTURE2を指す必要があります（これが型キャストの出番です）。

以下は私のコードの重要な部分です。問題 (少なくともエラーが発生する場所) はtrav2 = (STRUCT2*)trav1->next;、セグメンテーション違反が発生する部分の printToFile() にあります。

   void main(){
      STRUCT1 *ptr[20] = {NULL};  //Array of pointers that point to type STRUCT1
      float min, max;
      int i, j, answer;
      for(i = 0; i < 19; i++){   //19 to leave the 20th array spot for NULL
        printf("\nWould you like to add 2 more sets of numbers?\n");
        printf("1: Yes\n");
        printf("2: No\n");
        scanf("%d", &answer);
        if(answer == 1){
          printf("\nFirst set of numbers:\n");
          ptr[i] = getData();
          printf("\nSecond set of numbers:\n");
          ptr[i]->next = getData();
          min = getMin(ptr[i]);
          max = getMin(ptr[i]->next);
          STRUCT2 *trav;
          trav = (STRUCT2*)malloc(sizeof(STRUCT1));
          trav = (STRUCT2*)ptr[i]->next;
          trav->next = assignMin(min);
          trav = trav->next;
          trav->next = assignMax(max);
        }else{
          break;
        }
      }
      printToFile(ptr);
      printf("\nProgram Done\n");
    }

STRUCT1* getData(){
  STRUCT1 *newStruct;
  newStruct = (STRUCT1*)malloc(sizeof(STRUCT1));
  float num;
      printf("Enter number 1: ");
      scanf("%f", &num);
      newStruct->f1 = num;
      printf("Enter number 2: ");
      scanf("%f", &num);
      newStruct->f2 = num;
      printf("Enter number 3: ");
      scanf("%f", &num);
      newStruct->f3 = num;
      printf("Enter number 4: ");
      scanf("%f", &num);
      newStruct->f4 = num;
      printf("Enter number 5: ");
      scanf("%f", &num);
      newStruct->f5 = num;

      return newStruct;
}

void printToFile(STRUCT1 *ptr[]){
  FILE *f;
  f = fopen("structure.txt", "w");
  int i, j;
  STRUCT1 *trav1;
  STRUCT2 *trav2;
  trav1 = (STRUCT1*)malloc(sizeof(STRUCT1));
  trav2 = (STRUCT2*)malloc(sizeof(STRUCT2));

  for(i = 0; i < 20; i++){
    fprintf(f, "%.3f,%.3f,%.3f,%.3f,%.3f\n", ptr[i]->f1, ptr[i]->f2, ptr[i]->f3, ptr[i]->f4, ptr[i]->f5);
    trav1 = ptr[i]->next;
    fprintf(f, "%.3f,%.3f,%.3f,%.3f,%.3f\n", trav1->f1, trav1->f2, trav1->f3, trav1->f4, trav1->f5);
    trav2 = (STRUCT2*)trav1->next;
    fprintf(f, "Minimum of first number set: %.3f\n", trav2->f1);
    trav2 = trav2->next;
    fprintf(f, "Maximum of second number set: %.3f\n", trav2->f1);
    fprintf(f, "\n");    
  }
}

これはすべて非常に間違っている可能性がありますが、私の教授はさまざまな型を指すポインターについて話したことはなく、構造に関するセクションも短かったです。うまくいけば、完全に失敗するわけではなく、あちこちでいくつかのことだけが必要です.

どこが間違っていたのか分かると思う人はいますか？

4

2 に答える 2

0

あなたの問題はこの行にあると思います：

trav = (STRUCT2*)malloc(sizeof(STRUCT1));

ここでtrayは、STRUCT2ポインタとして にメモリを割り当てますSTRUCT1。より良い使用sizeof(STRUCT2)。

于 2012-12-10T06:46:00.550 に答える
0

質問には十分な情報がありません。構造定義とこれらの構造を設定するコードが欠落しています。キャストは必ずしも間違っているわけではありません。ただし、異なるポインターを使用して、どちらのタイプの構造体にもアクセスできます。

   if (ptr->next_st1 != NULL) {
      trav1 = ptr->next_st1;
   } else if (ptr->next_st2 != NULL) {
      trav2 = ptr->next_st2;
   }

補足として、ここで行う割り当ては次のとおりです。

  trav1 = (STRUCT1*)malloc(sizeof(STRUCT1));
  trav2 = (STRUCT2*)malloc(sizeof(STRUCT2));

  for(i = 0; i < 20; i++){
    /* ... */
    trav1 = ptr[i]->next;
    /* ... */

すでに割り当てられている領域 ( ) にポインターを割り当てているため、これらは必要ありませんtrav1 = ptr[i]->next。この割り当てにより、malloc されたチャンクへのポインターが失われるため、メモリ リークが発生します。この場合mallocは必要ありませんが、必要な場合は対応するfree()呼び出しが必要です。

于 2012-12-10T06:05:19.940 に答える