-2

我用C 语言完成了一个电话簿程序(带有动态结构等)。

它在 linux 下完美运行,但是当我尝试在 Windows 中运行它(在devc++中编译之后)它启动正常,但是在我将一些数据添加到列表后它只是停止并退出。只有在电话簿中添加新数据才会发生这种情况。当我使用其他功能时,没有中断等等。

在模块的开头,我附加了windows.hand conio.hfor getch()and systen("cls")。也许我使用了一些 Windows 中不允许的功能或类似的东西。

那是我的第一个c-project,通常我在linux下工作,那么我应该注意什么?

/*SCREEN*/
#ifdef __WIN32__                                           /*for Windows*/
#include<windows.h>
#define GETCH printf("press ENTER...\n");getch() 
#define CLEAR system("cls")
#endif
#ifdef __unix__                                             /*for unix*/
#include<unistd.h> 
#define GETCH printf("\n\npress ENTER...");getchar()
#define CLEAR system("clear")
#endif

我也以同样的方式添加 name 和 pnonenumber

printf("++ ADD DATA: ++\n");
         /*LASTNAME*/
                 printf(">> add lastname :  ");
                 fgets(buf,128,stdin);
                 lastname = malloc(strlen(buf));     /*allocate memory for lastdane*/ 
                 if( lastname != NULL)               /*successful allocation*/
                 {
                   strcpy( lastname, buf );          /*copy data from buf to lastname variable*/
                   lastname[strlen(lastname)-1]='\0';/*get the number of elements and add '\0' at the end*/
                 }
                 else                                /*allocation failed*/
                 {
                   printf("Memory Allocation Error!");
                 }
                 memset(buf,0,128);    

void AddDATAsorted( char* lastname, char* firstname, char* telnumber )
{
  tDATA* pDATA;

  if( pTELBOOK )
  {
      pDATA = malloc( sizeof( tDATA ));         /*allocate memory for new item*/

      pDATA->lastname   = lastname;    
      pDATA->firstname  = firstname;
      pDATA->telnumber  = telnumber;

      addItemToList( pTELBOOK, pDATA, fcmp );   /*add a new item sorted - see list.c*/
  }
} /*END 6*/

它在另一个模块中。


int fcmp( void* pItemList, void* pItemNew )
{
  tDATA* pDATA_list;
  tDATA* pDATA_new;
  int diff;

  if( pItemList != NULL ) 
  {
      pDATA_list = ( tDATA* ) pItemList;          /*because void-pointer  */
      pDATA_new  = ( tDATA* ) pItemNew;

      diff = strcmp( pDATA_list->lastname, pDATA_new->lastname );
      if( diff != 0 ) return diff;
      else                                        /*if items have the same lastname*/                                    
      {
          diff = strcmp( pDATA_list->firstname, pDATA_new->firstname );
          if( diff != 0 ) return diff;
          else                                    /*if items have also the same firstname*/
          {
              diff = strcmp( pDATA_list->telnumber, pDATA_new->telnumber );
              if( diff != 0 ) return diff;
              else  return FAIL;                  /*if items - equal*/
          }
      }
  }
  else return OK;

调试器总是在这里显示错误...

/* 2 * 功能:删除数据*/

int RemoveDATA( void )
{
  tDATA* pDATA = NULL;
  if( pTELBOOK == NULL ) return FAIL;       
  else                                      /*if pTELBOOK's current element - not empty - free allocated memory*/
  {
     if( pTELBOOK->pCurr != NULL )
     { 
           pDATA = pTELBOOK->pCurr->pItem;
           free( pDATA->lastname );
           free( pDATA->firstname );
           free( pDATA->telnumber );
           free( pTELBOOK->pCurr->pItem );
           RemoveItem( pTELBOOK );            /*and remove element from the list*/
           return OK;                        
     }
  }
} /*END 2*/
4

1 回答 1

5
             lastname = malloc(strlen(buf)); /* problem */ 
             if( lastname != NULL)
             {
               strcpy( lastname, buf);
               lastname[strlen(lastname)-1]='\0';
             }

使用strdup。lastname 的长度比需要的短一。这可能是一个问题:

       lastname = strdup(buf);

此外,您不应该使用 128 或其他任何内容。使用sizeof buforsizeof (buf)代替,或者也许是一个定义:

             #define BUF_SIZE   128

其余的我没有检查。

于 2013-04-23T13:55:51.813 回答