0

このようなエラーが表示されます

$ gcc -o app llistdriver.c llistlib.c -Wall
 llistdriver.c:7: warning: return type defaults to `int'
 llistdriver.c: In function `printinsts':
 llistdriver.c:15: warning: control reaches end of non-void function
 llistlib.c: In function `insert':
 llistlib.c:25: warning: statement with no effect
 llistlib.c:41: warning: control reaches end of non-void function
 /tmp/ccqYmK3M.o:llistdriver.c:(.text+0x157): undefined reference to `_search'
 collect2: ld returned 1 exit status

このエラー メッセージが表示される理由がわかりません。リンク リストのメソッドを含むライブラリ ファイルを作成しました。

  int search( struct node *front, int val)// 
 {
     while(front != NULL)
    {
       if( front->modmark == val ) //this is to find the value 
       return 1;  // found it

    front=front->next;
    }
    return 0;  // Not found
  }

メインファイルの唯一の参照について不平を言っていると思います

        case 5: 
                printf("please enter a search value?\n");
                scanf("%i",&mark);
                choice = search(first,mark);//here
                if (choice == -1)
                {

                    printf("the modmark was not found\n");
                }
                else 
                {   
                    printf("the first occurence of the mark is at the %i th index of the list",choice); 
                };
                    break;

first と mark は、このように定義された変数です

                 struct node *first;
              int mark = 0;

このエラーのデバッグに何時間も費やしました。C コードを書くことは、現時点では私の人生の悩みの種です。

Ps llistlib.c ファイルと llistdriver.c ファイルに含まれているすべての #include "llistlib.h" を確認しました。

わかりました多分これはそれをより明確にするでしょう

// llistlib.c - listlib.h ファイルで定義された関数を実装するライブラリ #include #include #include "llistlib.h"

void initialise(struct node **ps)
{

printf("initialise function\n");

*ps = NULL;   //initialise list to empty

}


int insert(struct node **fn)
{
printf("insert function\n");
struct node *newnode;
newnode =  (struct node*) malloc(sizeof (struct node)); //make new node and give it memory
if (newnode == NULL) //if fsiled return to the program saying it failed
    return -1;
if (*fn == NULL) // for the first of the list
{
    newnode -> next == NULL; // sets the next value to nothing
    printf("what mark do you want to enter\n");
    scanf("%i",&newnode -> modmark); // this gets the modmark
    *fn = newnode; //the head of the list is set to the new node
}
else
{
    newnode -> next = *fn; // sets the newnodes next value to the head node
    printf("what mark do you want to enter\n");
    scanf("%i",&newnode -> modmark); // gets the modmark
    *fn = newnode; // the head becomes the new node


    printf("%p %i",newnode,newnode->modmark); // just for debugging

}
 }



void traverse(struct node *ps)
{   

if (ps != NULL) // if the current node is null 
{
    printf("%p - %i\n",ps,ps -> modmark);
    traverse(ps -> next); 

}
else
{
    printf("your list is empty\n");
}
}


void delete(struct node **dn)
{
printf("delete function\n");
struct node *delnode;
delnode = *dn; //sets the head node so that it can be deleted later
if (delnode != NULL)
{
    *dn = delnode -> next; // sets the head to the next node
    free(delnode); //deletes the previous head node
}

}


void finish(struct node **ps) //this function goes through the list and removes all items
{
printf("finish function\n");
struct node *finishnode;
finishnode = *ps;
while (finishnode != NULL)
{
    *ps = finishnode -> next;
    free(finishnode);
    finishnode = *ps;
}

int search(struct node *front,int val)
{
 while(front != NULL)
 {
    if( front->modmark == val ) // Assuming struct node has val member which you are trying to compare to
       return 1;  // found it

    front=front->next;
 }
  return 0;  // Not found
}

}

そしてllistdriver.cファイルは

#include <stdio.h>
#include <stdlib.h>
#include "llistlib.h"


printinsts()
{
printf("Enter:- \n");
printf("\t0 to exit\n");
printf("\t1 to initialise list\n");
printf("\t2 to insert item at front of list\n");
printf("\t3 to delete item from front of list\n");
printf("\t4 to traverse list\n");
printf("\t5 to search through the list\n");
}




int main()
{
int choice;
struct node *first;
int mark = 0;

printinsts();
scanf("%d",&choice);

while (1)
{
    switch (choice)
    {
        case 0 : finish(&first);
                    exit(0);
                    break;

        case 1 : initialise(&first);
                    break;

        case 2 : if (insert(&first)== -1)
                    {
                        printf("insert not successful");
                        choice = 0;
                     };
                    break;

        case 3 : delete(&first);
                    break;

        case 4 : traverse(first);
                     break;
        case 5: 
                printf("please enter a search value?\n");
                scanf("%i",&mark);
                choice = search(first,mark);
                if (choice == -1)
                {

                    printf("the modmark was not found\n");
                }
                else 
                {   
                    printf("the first occurence of the mark is at the %i th index of the list",choice); 
                };
                    break;

        default : printf("Invalid choice of %d \n", choice);

    };  // end of switch

    printinsts();
    scanf("%d", &choice);

}  // end of while

}

ヘッダーファイル llistlib.h は次のように読み取ります

struct node {
int     modmark;
struct node *next;
 };


void initialise(struct node **ps);


int insert(struct node **fn);


void traverse(struct node *ps);


void delete(struct node **dn);


void finish(struct node **ps);


int search(struct node *front,int val);

どのコードにもまだ明らかな問題を見つけることができませんでした

4

4 に答える 4

1

最初のエラー:

 llistdriver.c:7: warning: return type defaults to `int'
 llistdriver.c: In function `printinsts':
 llistdriver.c:15: warning: control reaches end of non-void function

関数は次のようになります。

printinsts() {}

この関数に戻り型を指定してください。無効でない場合は、関数を返します。

第二に:

llistlib.c: In function `insert':
llistlib.c:25: warning: statement with no effect

その行は次のとおりです。

newnode -> next == NULL; // sets the next value to nothing

コメントは正しくありません。この行は、代入演算子ではなく等式演算子を使用しているため、何も設定されていません。

(一部は好みの問題ですが、コード自体と同じことを単に述べようとするこのようなコメントは好きではありません。このような場合、コメントを読んで信じるように、誤解を招くようになります。 、バグを探すときにコードを解析するのではなく。)

それで:

llistlib.c:41: warning: control reaches end of non-void function

これもまた、次のことを指します。

int insert(struct node **fn)
{
printf("insert function\n");
struct node *newnode;
newnode =  (struct node*) malloc(sizeof (struct node)); //make new node and give it memory
if (newnode == NULL) //if fsiled return to the program saying it failed
    return -1;
...
}

-1以外のこの関数からの戻りはありません。何が起こっても何かを返すはずです。

最後に:

/tmp/ccqYmK3M.o:llistdriver.c:(.text+0x157): undefined reference to `_search'

finish()の終わりから閉じ中括弧を逃したため。

コードがコンパイルされるという理由だけで警告を無視しないでください。エラーだけを修正するだけでは不十分です。

于 2012-12-21T09:16:08.410 に答える
1

次のようllistlib.hにファイルに含めるだけです。llistlib.c

#include "llistlib.h"

コンパイルには次のコマンドを使用します。

gcc -o app llistdriver.c llistlib.c

お役に立てれば...

于 2012-12-21T06:43:07.907 に答える
0

デフォルトの GCC 設定でコンパイルしています。これは、「C99 標準にほぼ準拠し、非標準の GNU goo を追加する」です。C99 標準は、現在の C 標準と同様に、関数がデフォルトの型を持つことを許可していません。そのため、-Wall で警告が表示されます。戻り値の型は明示的でなければなりません。

printinsts() {}は有効な C ではvoid printinsts (void)ありません。これを as として宣言/定義する必要があります。

将来的には、常に -std=c99 または -std=c11 を指定してコンパイルし、コードを標準の C 言語としてコンパイルしていることを確認してください。

于 2012-12-21T07:42:50.353 に答える
0

finish関数に右中括弧 { がありません。またsearch、追加の閉じブレースがあります。

于 2012-12-21T07:46:11.283 に答える