0 
int get_name()
{
    char cName[] = "hello";
    int iCode, i = 0;
    struct sign_in items[6];//array of six structure variables

    Fpointin =fopen("namepass.txt","r");

    if (Fpointin == NULL)
    {
        printf ("File does not exist.\n");
    }
    else
    {
        for (i=0;i<6;i++)
        {
            fscanf(Fpointin,"%s %d",items[i].name,items[i].password);//read all values from the file into th structure
        }
        printf("Here is the sign_in structure\n");//print the entirety of the sign_in structure
        for (i=0;i<6;i++)
        {
            printf("name: %s\ncode: %d\n\n", items[i].name, items[i].password);
        }
    }
    fclose(Fpointin);
}

大家好。所以我从一个项目中得到了这部分代码，每当我尝试运行它时它就会崩溃。我正在尝试将名称及其各自的密码从文件读取到结构中，但它不起作用。在这一fscanf行中，我%s %d交换了标识符并运行了，但它打印了随机的东西，这些东西甚至没有接近文件中的内容。有任何想法吗？

[评论更新：]

struct sign_in
{ 
  int password; //The password for each player 
  char name[]; //Name of the people who can sign in
}
4

4 回答 4

2

看看结构 sign_in 的样子会很有帮助。但是，快速浏览一下代码，一个明显的错误是密码被扫描的方式。

fscanf(Fpointin,"%s %d",items[i].name,items[i].password);

这一行应该是：

fscanf(Fpointin,"%s %d",items[i].name, &items[i].password);

您需要传入变量 items[i].password 的地址，以便 fscanf 可以在该地址指向的内存位置存储一个值。

希望有帮助。

于 2016-04-21T05:30:03.920 回答
1

char name[];在_

struct sign_in
{
  ...
  char name[];

是不完整的类型。它不分配内存。

利用

#define NAME_LEN_MAX 42

...

struct sign_in
{
  ...
  char name[NAME_LEN_MAX + 1];

例如，像这样调整扫描：

fscanf(Fpointin, "%42s %d", items[i].name, &items[i].password);
于 2016-04-21T06:16:50.710 回答
1

首先，你不应该char name[]在你的结构中使用。因为数组应该在使用它之前清除它的内存大小。

因此，您可以更改char name[]为：

  1. char *name: 你必须为它分配一个内存大小。在下面检查我的代码。
  2. char name[NAME_SIZE]

其次，如果你想改变另一个函数中的值，你应该将它的内存地址传递给它。例如：

示例 1： 如果您只传递值而不传递地址

void foo(int in)
{
    in = 5;
}

int main(int argc, char const *argv[])
{
    /* code */
    int a = 10;
    foo(a);
    printf("after foo, a: %d\n", a);
    return 0;
}

输出： after foo, a: 10

例2： 如果你传递内存地址。

void foo(int *in)
{
    *in = 5;
}

int main(int argc, char const *argv[])
{
    /* code */
    int a = 10;
    foo(&a);
    printf("after foo, a: %d\n", a);
    return 0;
}

输出： after foo, a: 5

因此，您必须将items[i].password的内存地址传递给 fscanf ，例如：
fscanf(Fpointin,"%s %d",items[i].name, &items[i].password);

因此，您的代码应如下所示： 

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

struct sign_in {
    int password;//The password for each player
    char *name;//Name of the people who can sign in
};

void init_myStruct(struct sign_in *s_in) {
    s_in->name = calloc(1, sizeof(char)*1024);
    return;
}

void destroy_myStruct (struct sign_in *s_in) {
    free(s_in->name);
    s_in->name = NULL;
    return;
}

int get_name()
{
    int i = 0;
    FILE *Fpointin = NULL;
    struct sign_in items[6];//array of six structure variables

    for ( i = 0; i < 6; i++)
        init_myStruct(&items[i]);

    Fpointin =fopen("namepass.txt","r");
    if (Fpointin == NULL) {
        printf ("File does not exist.\n");
        goto end_of_use;
    }
    else
    {
        printf("Here is the sign_in structure\n");//print the entirety of the sign_in structure
        for ( i = 0; i < 6; i++) {
            fscanf(Fpointin,"%s %d",items[i].name, &items[i].password);//read all values from the file into th structure
            printf("name: %s\ncode: %d\n\n", items[i].name, items[i].password);
        }
    }
    fclose(Fpointin);

end_of_use:
    for ( i = 0; i < 6; i++)
        destroy_myStruct(&items[i]);
    return;
}

int main(int argc, char const *argv[])
{
    get_name();
    return 0;
}
于 2016-04-21T06:42:25.500 回答
0

它可以是 char * 类型，您可能没有为它分配空间。然后它肯定会崩溃

于 2016-04-21T05:36:55.773 回答