1

こんにちは、スタックオーバーフロー。

今週末に取り組んでいる宿題があり、少し問題があります。構造体 "Record" (ディーラーの車に関する情報を含む) があり、1) メーカーと 2) モデル年に従って、リンクされたリストの特定の場所に配置されます。

これは、メインで「int insertRecordInAscendingOrder」関数が呼び出されたときに、最初にリストを作成するときに行われます。「insertRecordInAscendingOrder」では、3 番目の関数「createRecord」が呼び出され、リンク リストが作成されます。関数は次に関数「compareCars」に進み、どの要素がどこに配置されるかを判断します。この関数によって返された値に応じて、insertRecordInAscendingOrder はレコードを所属する場所に配置します。その後、リストが印刷されます。課題はまだありますが、ここまで来たら橋を渡ります。

理想的には、割り当てが正しいと見なされるためには、リンクされたリストが次のように並べられている必要があります。

    Chevrolet 2012 25
    Chevrolet 2013 10
    Ford 2010 5
    Ford 2011 3
    Ford 2012 15
    Honda 2011 9
    Honda 2012 3
    Honda 2013 12
    Toyota 2009 2
    Toyota 2011 7
    Toyota 2013 20

データが次のように並べられたテキスト ファイルから:

    Ford 2012 15
    Ford 2011 3
    Ford 2010 5
    Toyota 2011 7
    Toyota 2012 20
    Toyota 2009 2
    Honda 2011 9
    Honda 2012 3
    Honda 2013 12
    Chevrolet 2013 10
    Chevrolet 2012 25

「make」フィールドのアルファベット順が優先されることに注意してください。次に、モデルの年が古いものから新しいものへと並べられます。

ただし、プログラムはこれを最終的なリストとして生成します。

    Chevrolet 2012 25
    Chevrolet 2013 10
    Honda 2011 9
    Honda 2012 3
    Honda 2013 12
    Toyota 2009 2
    Toyota 2011 7
    Toyota 2012 20
    Ford 2010 5
    Ford 2011 3
    Ford 2012 15

私は大学院生と一緒に座って、昨日これらすべてを解決しようとしましたが、なぜそれがフォードノードをリストの最後に追いやったのか理解できませんでした.

これがコードです。お気づきのとおり、ノードの挿入の各インスタンスに printList 呼び出しを含めました。このようにして、ノードが「順序付け」されているときに何が起こっているかを確認できます。ANSI C99 です。すべての関数呼び出しは指定どおりに行わなければならないため、残念ながら、より効率的なアルゴリズムを作成してこの問題を回避する方法はありません。

    #include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MAX_LINE 50
#define MAX_MAKE 20

typedef struct record
{
    char *make;
    int year;
    int stock;
    struct record *next;
} Record;

int compareCars(Record *car1, Record *car2);
void printList(Record *head);
Record* createRecord(char *make, int year, int stock);
int insertRecordInAscendingOrder(Record **head, char *make, int year, int stock);

int main(int argc, char **argv)
{
    FILE *inFile = NULL;
    char line[MAX_LINE + 1];
    char *make, *yearStr, *stockStr;
    int year, stock, len;
    Record* headRecord = NULL;

    /*Input and file diagnostics*/
    if (argc!=2)
    {
        printf ("Filename not provided.\n");
        return 1;
    }

    if((inFile=fopen(argv[1], "r"))==NULL)
    {
        printf("Can't open the file\n");
        return 2;
    }

    /*obtain values for linked list*/
    while (fgets(line, MAX_LINE, inFile))
    {
        make = strtok(line, " ");
        yearStr = strtok(NULL, " ");
        stockStr = strtok(NULL, " ");
        year = atoi(yearStr);
        stock = atoi(stockStr);
        insertRecordInAscendingOrder(&headRecord,make, year, stock);

    }


    printf("The original list in ascending order: \n");
    printList(headRecord);
}

/*use strcmp to compare two makes*/

int compareCars(Record *car1, Record *car2)
{
    int compStrResult;
    compStrResult = strcmp(car1->make, car2->make);
    int compYearResult = 0;

    if(car1->year > car2->year)
    {
        compYearResult = 1;
    }
    else if(car1->year == car2->year)
    {
        compYearResult = 0;
    }
    else
    {
        compYearResult = -1;
    }

    if(compStrResult == 0 )
    {
        if(compYearResult == 1)
        {
            return 1;
        }
        else if(compYearResult == -1)
        {
            return -1;
        }
        else
        {
            return compStrResult;
        }
    }
    else if(compStrResult == 1)
    {
        return 1;
    }
    else
    {
        return -1;
    }
}

int insertRecordInAscendingOrder(Record **head, char *make, int year, int stock)
{
    Record *previous = *head;
    Record *newRecord = createRecord(make, year, stock);
    Record *current = *head;
    int compResult;

    if(*head == NULL)
    {
        *head = newRecord;
        printf("Head is null, list was empty\n");
        printList(*head);
        return 1;
    }
    else if ( compareCars(newRecord, *head)==-1) 
    {
        *head = newRecord;
        (*head)->next = current;
        printf("New record was less than the head, replacing\n");
        printList(*head);
        return 1;
    }
    else
    {
        printf("standard case, searching and inserting\n");
        previous = *head;

        while ( current != NULL &&(compareCars(newRecord, current)==1)) 
        {
            printList(*head);
            previous = current; 
            current = current->next; 
        }

        printList(*head);
        previous->next = newRecord;
        previous->next->next = current;
    }
    return 1;
}

/*creates records from info passed in from main via insertRecordInAscendingOrder.*/
Record* createRecord(char *make, int year, int stock)
{
    printf("CreateRecord\n");
    Record *theRecord;
    int len;

    if(!make)
    {
        return NULL;
    }

    theRecord = malloc(sizeof(Record));

    if(!theRecord)
    {
        printf("Unable to allocate memory for the structure.\n");
        return NULL;
    }

    theRecord->year = year;
    theRecord->stock = stock;

    len = strlen(make);
    theRecord->make = malloc(len + 1);
    strncpy(theRecord->make, make, len);
    theRecord->make[len] = '\0';
    theRecord->next=NULL;

    return theRecord;
}

/*prints list. lists print.*/
void printList(Record *head)
{
    int i;
    int j = 50;
    Record *aRecord;
    aRecord = head;

    for(i = 0; i < j; i++)
    {
        printf("-");
    }
    printf("\n");
    printf("%20s%20s%10s\n", "Make", "Year", "Stock");

    for(i = 0; i < j; i++)
    {
        printf("-");
    }
    printf("\n");

    while(aRecord != NULL)
    {       
        printf("%20s%20d%10d\n", aRecord->make, aRecord->year,
        aRecord->stock);        
        aRecord = aRecord->next;
    }
    printf("\n");
}

コマンド ライン引数に必要なテキスト ファイルは、任意の名前で保存できます。必要なコンテンツは次のとおりです。

    Ford 2012 15
    Ford 2011 3
    Ford 2010 5
    Toyota 2011 7
    Toyota 2012 20
    Toyota 2009 2
    Honda 2011 9
    Honda 2012 3
    Honda 2013 12
    Chevrolet 2013 10
    Chevrolet 2012 25

よろしくお願いします。私は自分でそれを耕し続けます。

4

3 に答える 3

2

問題はそれです

else if(compStrResult == 1)

の特定の戻り値に対してチェックしますstrcmp。仕様では、それぞれのケースで結果が正、負、またはゼロであるとのみ述べられており、多くの場合、最初の異なる文字の違いです。チェックする必要があります

if (compStrResult > 0)

そこの。

于 2012-10-13T21:18:12.070 に答える
0

とにかく:修正後のコード。コーディングスタイルに取り組む

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define MAX_LINE 50
#define MAX_MAKE 20

typedef struct record
{
    char *make;
    int year;
    int stock;
    struct record *next;
} Record;

int compareCars(Record *car1, Record *car2);
void printList(Record *head);
Record* createRecord(char *make, int year, int stock);
int insertRecordInAscendingOrder(Record **head, char *make, int year, int stock);

int main(int argc, char **argv)
{
    FILE *inFile = NULL;
    char line[MAX_LINE + 1];
    char *make, *yearStr, *stockStr;
    int year, stock, len;
    Record* headRecord = NULL;

    /*Input and file diagnostics*/
    if (argc!=2)
    {
        printf ("Filename not provided.\n");
        return 1;
    }

    if((inFile=fopen(argv[1], "r"))==NULL)
    {
        printf("Can't open the file\n");
        return 2;
    }

    /*obtain values for linked list*/
    while (fgets(line, MAX_LINE, inFile))
    {
        make = strtok(line, " ");
        yearStr = strtok(NULL, " ");
        stockStr = strtok(NULL, " ");
        year = atoi(yearStr);
        stock = atoi(stockStr);

       // fprintf (stderr, "Record read Make[%s] Year[%d] Stock[%d]", make, year, stock);
        insertRecordInAscendingOrder(&headRecord,make, year, stock);

    }


    printf("The original list in ascending order: \n");
    printList(headRecord);

    scanf ("%d", &len);

    return 0;
}

/*use strcmp to compare two makes*/

int compareCars(Record *car1, Record *car2)
{
    int compStrResult;
    compStrResult = strcmp(car1->make, car2->make);
    int compYearResult = 0;

    if(car1->year > car2->year)
    {
        compYearResult = 1;
    }
    else if(car1->year == car2->year)
    {
        compYearResult = 0;
    }
    else
    {
        compYearResult = -1;
    }

    if(compStrResult == 0 )
    {
        if(compYearResult == 1)
        {
            return 1;
        }
        else if(compYearResult == -1)
        {
            return -1;
        }
        else
        {
            return compStrResult;
        }
    }
    else if(compStrResult > 0)
    {
        return 1;
    }
    else
    {
        return -1;
    }
}

int insertRecordInAscendingOrder(Record **head, char *make, int year, int stock)
{
    Record *previous = *head;
    Record *newRecord = createRecord(make, year, stock);
    Record *current = *head;
    //int compResult;

    if(*head == NULL)
    {
        *head = newRecord;
        printf("Head is null, list was empty\n");
        //printList(*head);
        return 1;
    }
    else if ( compareCars(newRecord, *head)==-1)
    {
        *head = newRecord;
        (*head)->next = current;
        printf("New record was less than the head, replacing\n");
        //printList(*head);
        return 1;
    }
    else
    {
        printf("standard case, searching and inserting\n");
        previous = *head;

        while ( current != NULL &&(compareCars(newRecord, current)==1))
        {
            //printList(*head);
            previous = current;
            current = current->next;
        }

        //printList(*head);
        previous->next = newRecord;
        previous->next->next = current;
    }
    return 1;
}

/*creates records from info passed in from main via insertRecordInAscendingOrder.*/
Record* createRecord(char *make, int year, int stock)
{
    printf("CreateRecord\n");
    Record *theRecord;
    int len;

    if(!make)
    {
        return NULL;
    }

    theRecord = malloc(sizeof(Record));

    if(!theRecord)
    {
        printf("Unable to allocate memory for the structure.\n");
        return NULL;
    }

    theRecord->year = year;
    theRecord->stock = stock;

    len = strlen(make);
    theRecord->make = malloc(len + 1);
    strncpy(theRecord->make, make, len);
    theRecord->make[len] = '\0';
    theRecord->next=NULL;

    return theRecord;
}

/*prints list. lists print.*/
void printList(Record *head)
{
    int i;
    int j = 50;
    Record *aRecord;
    aRecord = head;

    for(i = 0; i < j; i++)
    {
        printf("-");
    }
    printf("\n");
    printf("%20s%20s%10s\n", "Make", "Year", "Stock");

    for(i = 0; i < j; i++)
    {
        printf("-");
    }
    printf("\n");

    while(aRecord != NULL)
    {
        printf("%20s%20d%10d\n", aRecord->make, aRecord->year,
        aRecord->stock);
        aRecord = aRecord->next;
    }
    printf("\n");
}
于 2012-10-13T21:33:45.790 に答える
0

非常に単純なテスト スイートを作成すると、compareCars にエラーがあることがわかります。

年数の比較は単純なので、strcmp のドキュメントを参照してください。0、1、または -1 は返されませんが、0 または正または負の整数が返されます。

int compareCars(Record *car1, Record *car2)
{
    int makecmp = strcmp(car1->make, car2->make);

    if( makecmp > 0 )
      return 1;

    if( makecmp == 0 )
    {
        if (car1->year > car2->year)
          return 1;
        if( car1->year == car2->year )
          return 0;
    }

    return -1;
}

その関数のテストスイートの例:

#include <assert.h>

void main()
{
  Record c1 = {"Toyota", 1991};
  Record c2 = {"Toyota", 1992};
  Record c3 = {"Ford", 1991};
  Record c4 = {"Ford", 1992};

  assert( compareCars( &c1, &c2 ) == -1 );
  assert( compareCars( &c1, &c1 ) == 0 );
  assert( compareCars( &c2, &c1 ) == 1 );
  assert( compareCars( &c2, &c3 ) == 1 );
  assert( compareCars( &c3, &c2 ) == -1 );
  assert( compareCars( &c3, &c4 ) == -1 );
  assert( compareCars( &c4, &c3 ) == 1 );
  assert( compareCars( &c4, &c1 ) == -1 );
}

次に、メモ: createRecordstrdupでは、malloc/strncpy およびその仲間の代わりに を使用することをお勧めします。

于 2012-10-13T21:27:12.243 に答える