Char数组指针(通过'reference'传递)不分配(在sscanf变量之后)

时间:2014-06-23 10:12:36

标签: c string pointers scanf c89

在通过sscanf更改其中一个字符串之前,我已经将这些功能正常工作了。

即如果我有一个char [] NAME,我在(被调用)函数中创建,并将其初始化为一个字符串(例如" fred&#34 ;;),然后将其分配给传递的参数 - >参数在其父函数中发生变化。

但是,如果我尝试使用来自fgets的输入更改字符串(NAME) - > sscanf,然后将其分配给传递的数组,父函数不打印任何内容('空白')。

我试图弄清楚char数组与指针的情况。我以为我拥有它;我可以使用本地(初始化)字符串更改传递的String。但是,一旦我分配了一些用户输入'对于本地字符串,参数字符串未被正确更改。

以下是我的职能......

  int enter_record(FILE *fp, const char *prompt[]) {
     char *name[BUFFER] = {"wram"};
     int score = 12; /* REPLACE : Put 0 or something ! */

     fprintf(fp, "Enter_record()\n%s\n", *prompt); /* This is only here to use variables; so wil compile */

     printf("Score: %d\n", score);
     printf("Name:%s\n", *name);

     get_name(prompt, name);

     printf("Post-Change Name:%s\n", *name);

     return 1;
  }

  int get_name(const char *prompt[], char *fullName[]) {
     int n;   /* The number of strings scanned */

     char line[BUFFER];   /* The line that is scanned */
     char firstString[BUFFER];  /* First scanned string */
     char midString[BUFFER];    /* Second scanned string (',' ?) */
     char lastString[BUFFER];   /* Last string scanned */

     /* Function call validation */
     printf("get_name()\n%s\n", *prompt); 

     if( !fgets(line, BUFFER, stdin) ) {
        clearerr(stdin);
        return 0;   /* If *EOF* return FALSE */
     }

     if( !(n = sscanf(line, " %[a-zA-Z-] %[a-zA-Z-,] %[a-zA-Z-] %*s", firstString, midString, lastString)) ) {
        printf("Wrong format!!!!\n");
     }

     printf("n:%d\n", n);
     printf("firstString:%s\n", firstString);


     *fullName= firstString;

     return 1;
  }

下面我将包含我的其余代码(主要,菜单等)

  #include <stdio.h>

  #define BUFFER 512
  #define NAMELENGTH 14

  int menu(const char *choices[], const char *prompt[]);
  int enter_record(FILE *fp, const char *prompt[]);
  int get_name(const char *prompt[], char *fullName[]);

  int main(int argc, char *argv[]) {
     FILE *ofp;

     const char *menuOptions[] = {"Enter record", "Display records", "Quit", '\0'};   
     const char *prompt[2] = {"> "}; /* Prompt for user to enter input */
     int menuResult;

   /* Command line ARGS validation */
     /* 
        VALIDATES main() is called correctly from console.
     */
           /* Ensure program envoked correctly */
           if(argc != 2) {
              fprintf(stderr, "usage: %s [destination file]\n", argv[0]);
              return 1; /* Incorrect-envocation error */
           }

           /* Open file for student records */
           if( (ofp = fopen(argv[1], "wb+")) == 0) { /*If there's an error */
              perror("fopen");
              return 2;   /* File-open error */
           }   
     /*
        ///// END VALIDATION for main() call.
     */



     printf("main()\n");

     while(1) {
        menuResult = menu(menuOptions, prompt);

        switch(menuResult) {
           case 0: /* Menu returned FALSE;  */
              return 0;
           case 1: /* Enter Record; choice */ 
              enter_record(ofp, prompt);  
              break;
           default:
              break;
        }
     }

     return 0;
  }

  int menu(const char *choices[], const char *prompt[]) {
     int i; /* Used to print menu "choice" "Number" */
     char input[BUFFER]; /* The line scanned by fgets */
     int choice = 0;   /* What the user chooses */
     int choiceLen = -1;  /* Used to track how many elements in "choices[]"
                             This is used because (in my code) 'QUIT' is always last option
                          */

     /* Calculates # elements in *choices[] */
        while (choices[++choiceLen] != NULL) {};  
        printf("\n");

        for (i=0; choices[i] != '\0'; i++){
           printf("%1d. %s\n", i+1, choices[i]);
        }

     while(1) {
        printf("%s", *prompt);

        if(!fgets(input, BUFFER, stdin)) {
           clearerr(stdin);
           return 0;
        }
        sscanf(input, "%d %*s", &choice);
        if(choice == choiceLen) /* QUIT is always last option (choiceLen) */
           return 0;   /* Return QUIT */
        else if ((choice > 0) && (choice < choiceLen)) /* 0 is invalid choice */
           return choice; 
     }

     return 0;
  }

如果有人能指出我哪里出错了; 为什么用sscanf更改变量会改变父变量的影响方式? 我的困惑在于它在“儿童字符串”中起作用。是预先初始化的(例如,字符名称[BUFFER] = {&#34; Bob&#34;};),但不是在我用sscanf改变之后。

如果我的问题出在其他地方,请告诉我。

任何方向的指针都会受到赞赏。

干杯。

1 个答案:

答案 0 :(得分:1)

问题在于:

char line[BUFFER];
char firstString[BUFFER];
char midString[BUFFER];
char lastString[BUFFER];

这些是局部变量,这意味着字符数组在堆栈框架中;当函数get_name完成时,将释放此内存。将指向此内存的指针暴露给外界是错误的(即函数enter_record)。

最简单的解决方案是在静态内存中分配字符串缓冲区:

static char line[BUFFER];
static char firstString[BUFFER];
static char midString[BUFFER];
static char lastString[BUFFER];

这种方法确实有一些缺点。

  1. 字符串缓冲区在程序的整个持续时间内占用内存。
  2. 缓冲区大小必须是静态的;它不能是在运行时确定的动态字符数。
  3. 替代方案是:

    1. enter_record而不是get_name中定义字符串缓冲区(作为局部变量),从而相应地延长其生命周期。让enter_record在调用get_name时将指针传递给缓冲区。
    2. 使用malloc。我必须说我不赞成get_name分配内存,然后让enter_record释放它;它有点伤害可维护性。