C-指向结构的指针-分段错误

我学习C指针，因此我尝试使用指针表示法创建结构数组。我将指针数组分配给struct

User

，设置名称和标识符，然后尝试打印它，然后它崩溃

Dockerfile：

FROM    ubuntu:xenial

RUN     apt-get update \
            && apt-get install -y \
                software-properties-common \
                wget \
            && add-apt-repository -y ppa:ubuntu-toolchain-r/test \
            && apt-get update \
            && apt-get install -y \
                make \
                git \
                curl \
                vim \
                vim-gnome \
            && apt-get install -y cmake=3.5.1-1ubuntu3 \
            && apt-get install -y \
                gcc-4.9 g++-4.9 gcc-4.9-base \
                gcc-4.8 g++-4.8 gcc-4.8-base \
                gcc-4.7 g++-4.7 gcc-4.7-base \
                gcc-4.6 g++-4.6 gcc-4.6-base \
            && update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-4.9 100 \
            && update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-4.9 100


COPY ./*.c /tmp/

WORKDIR /tmp/
RUN gcc -Wall main.c -o main

ENTRYPOINT ./main

我在docker容器中运行我的程序，因为我希望有未使用的内存

我的节目：

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

#define MAX_USERS_NUMBER 10

void set_users();
void print_users();

typedef struct User
{
    int id;
    char* username;
} User;

User **users;

int main(int argc, const char* argv[])
{
    set_users();
    print_users();
    return 0;
}

void set_users()
{
    users = (User **) malloc(MAX_USERS_NUMBER * sizeof(User *));
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++)
    {
        *(users + i) = (User *) malloc(sizeof(User));
        (*users + i)->id = i;
        (*users + i)->username = (char *) malloc(strlen("username") + 1);
        strcpy((*users + i)->username, "username");
        printf("%d. id: %d, username: %s\n", i, (*users + i)->id, (*users + i)->username);
    }
}

void print_users()
{
    printf("\nUsers\n");
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++) 
    {
        printf("%d. id: %d: ,username: %s\n", i, (*users + i)->id, (*users + i)->username);
    }
}

我通过命令运行它：

docker build -t gcc-poc .

docker run -it gcc-poc /bin/bash

我总是得到这样的结果：

0. id: 0, username: username
1. id: 1, username: username
2. id: 2, username: username
3. id: 3, username: username
4. id: 4, username: username
5. id: 5, username: username
6. id: 6, username: username
7. id: 7, username: username
8. id: 8, username: username
9. id: 9, username: username

Users
0. id: 0: ,username: 
1. id: 1: ,username: username
2. id: 2: ,username: username
3. id: 3: ,username: username
Segmentation fault

我读过O'Reilly的《理解和使用C指针》一书，但我无法诊断原因。

试试这个：

#define MAX_CHARS_USERNAME 40

void set_users()
{
    static/*added*/ User* pUser;
    pUser = (User *) malloc(MAX_USERS_NUMBER * sizeof(User));
    users = &pUser;
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++)
    {
        (*users + i)->id = i;
        (*users + i)->username = (char *) malloc(MAX_CHARS_USERNAME);
        snprintf((*users + i)->username, MAX_CHARS_USERNAME, "username%d",i);
        printf("%d. id: %d, username: %s\n", i, (*users + i)->id, (*users + i)->username);
    }
}

#定义MAX_CHARS_用户名40
void set_users（）
{
静态/*添加*/User*pUser；
pUser=（用户*）malloc（最大用户数*sizeof（用户））；
用户=&pUser；
int i；
对于（i=0；iid=i；
（*users+i）->username=（char*）malloc（MAX_CHARS_username）；
snprintf（（*users+i）->用户名，MAX\u CHARS\u用户名，“用户名%d”，i）；
printf（“%d.id:%d，用户名：%s\n”，i，（*users+i）->id，（*users+i）->username）；
}
}

对于指向指针的指针（如果需要的话），您可能需要使用类似于

ppUser

的东西。

---

使用后不要忘记释放内存（按相反的顺序）

由于

用户

是指向

用户

的指针，

*（users+i）

是指向

用户

的指针，是正确的指针算法。但是，

（*users+i）

不是将产生您要查找的结果的指针算法。
当指针递增时，增量将是它所指向的类型的大小。 例如这里：

users+1

将使用

sizeof（User*）增加
users
这将是系统上指针的大小

但是
*users+1
将增加
*users
的
sizeof（User）
；这是您定义的
用户
结构的大小

因此，您必须将代码中的
（*users+i）
更改为
（*（users+i））
。需要额外的一对括号，因为
->
运算符的优先级高于
*
运算符

更改
set_users
和
print_users
这样的功能，它将按预期工作：

void set_users()
{
    users = (User **) malloc(MAX_USERS_NUMBER * sizeof(User *));
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++)
    {
        *(users + i) = (User *) malloc(sizeof(User));
        (*(users + i))->id = i;
        (*(users + i))->username = (char *) malloc(strlen("username") + 1);
        strcpy((*(users + i))->username, "username");
        printf("%d. id: %d, username: %s\n", i, (*(users + i))->id, (*(users + i))->username);
    }
}

void print_users()
{
    printf("\nUsers\n");
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++) 
    {
        printf("%d. id: %d: ,username: %s\n", i, (*(users + i))->id, (*(users + i))->username);
    }
}

void set_users（）
{
用户=（用户**）malloc（最大用户数*sizeof（用户*）；
int i；
对于（i=0；iid=i；
（*（users+i））->username=（char*）malloc（strlen（“username”）+1）；
strcpy（（*（users+i））->用户名，“username”）；
printf（“%d.id:%d，用户名：%s\n”，i，（*（用户+i））->id，（*（用户+i））->用户名）；
}
}
作废打印用户（）
{
printf（“\nUsers\n”）；
int i；
对于（i=0；iid，（*（用户+i））->用户名）；
}
}
*（users+i）
和
（*users+i）
不是一回事。为什么你突然将
*（users+i）
更改为
（*users+i）
？如果您不确定数组表示法是否更清晰，下次只需使用
users[i]
。值得注意的是，使用全局传递数据是一个极坏的习惯。更好的方法是第一个函数返回第二个函数可以使用的指针。此外，您实际上不需要
User**
。您可以让
User*
为
MAX\u USERS\u NUMBER*sizeof（User）
“
USERS=&pUser；
”分配一个全局指针一个局部变量的地址是一个坏主意，事实上它根本没有意义。这是一个坏主意。这是错误的。我想知道为什么这段代码会运行。函数print_users实际上需要在函数set_users离开后存储的pUser！我只能想象在线编译器（onlinegdb.com）优化了双指针访问。要更正代码，pUser需要是“静态”或全局的。那么代码是正确的，但不是好的样式。。。我纠正了我的例子。我仍然使用双指针，尽管实际上不需要它。我可以从set_用户那里传回一个正常的指针。为什么不按照您最后的回答添加正确的代码呢？“但是（*users+I）不是指针算术。”这不是真的。您还写道，用户是双指针，所以*users是普通指针。@Mico:添加了必要的说明。