如何在c中填充指针数组

Question

我想按以下格式填充一个 char 指针数组：

[姓名、电子邮件地址数量、所有电子邮件地址]*人数。每个人的邮件地址数量一开始是未知的。

• 我必须使用 char *contacts[N]
• 打印数组时只打印联系人[0]，所以我猜我扫描输入的方式是错误的

这是我目前为止的主要内容：

int main()
{
    char *contacts[N];
    int nextAvailable = 0;
    int * const nextAvailableP = &nextAvailable;
    add(contacts, nextAvailableP);
    //Printing this way will only print contacts[0]
    int i;
    for (i = 0; i < nextAvailable; i++) {
        printf("%s", contacts[i]);
    }
}

这是添加函数：

int add(char *contacts[], int *nextAvailableP) {
    if (*nextAvailableP > 97) {
        printf("Not enough memory\n");
        return FALSE;
    }

    char tempName[50];
    char tempMail[50];
    int numberOfMails = 1;

    printf("Enter your name\n");
    fgets(tempName, 50, stdin);
    if ((strlen(tempName) > 0) && (tempName[strlen(tempName) - 1] == '\n'))
        tempName[strlen(tempName) - 1] = '\0';

    contacts[*nextAvailableP] = (char *)malloc((strlen(tempName) + 1));

    if (contacts[*nextAvailableP] == NULL) {
        printf("Not enough memory\n");
        return FALSE;
    } else {
        strcpy(contacts[*nextAvailableP], tempName);
    }

    (*nextAvailableP)++;
    int numberOfMailsIndex = *nextAvailableP;
    itoa(numberOfMails, contacts[numberOfMailsIndex], 10);
    (*nextAvailableP)++;
    

    printf("Enter your mail/s, use enter to enter a mail and '-1' to signal you finished\n");
    fgets(tempMail, 50, stdin);

    while (strcmp(tempMail, "-1") != 0) {
        if ((strlen(tempMail) > 0) && (tempMail[strlen(tempMail) - 1] == '\n')) {
            tempMail[strlen(tempMail) - 1] = '\0';
        }

        contacts[*nextAvailableP] = (char *)malloc((strlen(tempMail) + 1));
        
        if (contacts[*nextAvailableP] == NULL) {
            printf("Not enough memory");
            return FALSE;
        } else {
            strcpy(contacts[*nextAvailableP], tempMail);
        }
        (*nextAvailableP)++;
        numberOfMails++;
        itoa(numberOfMails, contacts[numberOfMailsIndex], 10);
        fgets(tempMail, 50, stdin);
    }
}

我以为我正在将联系人中的每个单元格初始化为请求的大小，然后将我从用户那里扫描的单词复制到其中 - 但显然我不是。我应该逐个字符地遍历我分配的每个内存吗？

顺便说一句，我知道不需要强制转换为 *char

Answer 1

所以这将是您更正的代码

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

#define MAX_ENTRIES 50

int add(char *contacts[], int *nextAvailable);

int main(void){
    int i;
    char *contacts[MAX_ENTRIES];
    int nextAvailable = 0;
    add(contacts, &nextAvailable);
    add(contacts, &nextAvailable);

    for(i=0; i < nextAvailable ; i++){
        printf("%s\n", contacts[i]);
    }
}

char* intToString(int val, int base){
    //source: www.strudel.org.uk/itoa/
    static char buf[32] = {0};

    int i = 30;

    for(; val && i ; --i, val /= base)

        buf[i] = "0123456789abcdef"[val % base];

    return &buf[i+1];

}

int add(char *contacts[], int *nextAvailableP) {
    if (*nextAvailableP > 97) {
        printf("Not enough memory\n");
        return 0;
    }

    char tempName[50];
    char tempMail[50];
    int numberOfMails = 1;

    printf("Enter your name\n");
    fgets(tempName, 50, stdin);
    if ((strlen(tempName) > 0) && (tempName[strlen(tempName) - 1] == '\n'))
        tempName[strlen(tempName) - 1] = '\0';

    contacts[*nextAvailableP] = (char *)malloc((strlen(tempName) + 1));

    if (contacts[*nextAvailableP] == NULL) {
        printf("Not enough memory\n");
        return 0;
    } else {
        strcpy(contacts[*nextAvailableP], tempName);
    }

    (*nextAvailableP)++;
    int numberOfMailsIndex = (*nextAvailableP);
    contacts[numberOfMailsIndex] = malloc(5);
    (*nextAvailableP)++;

    printf("Enter your mail/s, use enter to enter a mail and '-1' to signal you finished\n");
    fgets(tempMail, 50, stdin);

    while (strcmp(tempMail, "-1\n") != 0) {
        if ((strlen(tempMail) > 0) && (tempMail[strlen(tempMail) - 1] == '\n')) {
            tempMail[strlen(tempMail) - 1] = '\0';
        }

        contacts[*nextAvailableP] = (char *)malloc((strlen(tempMail) + 1));

        if (contacts[*nextAvailableP] == NULL) {
            printf("Not enough memory");
            return 0;
        } else {
            strcpy(contacts[*nextAvailableP], tempMail);
        }
        (*nextAvailableP)++;
        numberOfMails++;
        fgets(tempMail, 50, stdin);
    }
    strcpy(contacts[numberOfMailsIndex], intToString(numberOfMails - 1,10));

}

Answer 2

您的函数 add 可以在从用户那里获取字符串的子函数中剪切。该字符串将存储在分配的内存空间中，其大小为字符串。

请记住：在一个数组（姓名和电子邮件）中混合不同性质的数据并不是一个好习惯。使用结构会更好（正如@Barmar 所说）。

我会执行以下步骤：

• 从用户处获取名称（字符 *）
• 分配内存并将名称复制到其中。
• 将指向此分配内存的指针插入到您的数组（联系人）中
• 获取电子邮件地址并将其放入动态分配的内存空间中
• 将其指针添加到您的数组（联系人
• 增加电子邮件计数器
• 重复
• 检测到-1
• 将您的电子邮件计数器转换为字符串并将指针转换为您的数组

无论如何，这里有一个代码，你可以开始玩：

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

#define MAX_ENTRIES 50
#define BUFFERSIZE 50

int add(char *contacts[], int *nextAvailable);
char *getString(char *queryText);
char* intToString(int val, int base);

int main(void){
    int i;
    char *contacts[MAX_ENTRIES];
    int nextAvailable = 0;
    add(contacts, &nextAvailable);
    add(contacts, &nextAvailable);

    for(i=0; i < nextAvailable ; i++){
        printf("%s", contacts[i]);
    }
}

int add(char *contacts[], int *nextAvailable){
    if(*nextAvailable > MAX_ENTRIES - 1){
        printf("Not enough memory\n");
        return 1;
    }

    contacts[*nextAvailable] = getString("Enter your name\n");
    if(contacts[*nextAvailable] == NULL){
        printf("Malloc Error\n");
        return 1;
    }
    ++(*nextAvailable);
    int counterfield = *nextAvailable;
    ++(*nextAvailable);
    int emailCounter = 0;
    while(1){

        if(*nextAvailable > MAX_ENTRIES - 1){
            printf("Not enough memory\n");
            return 1;
        }
        char *email = getString("Enter email\n");
        if(strcmp(email, "-1\n") == 0){

            contacts[counterfield] = malloc(5);
            contacts[counterfield] = intToString(emailCounter,  10);

            return 0;
        }
        emailCounter++;
        contacts[*nextAvailable] = email;
        ++(*nextAvailable);

    }


}


char *getString(char *queryText){
    char buffer[BUFFERSIZE];
    char *ret;
    printf("%s", queryText);

    fgets(buffer, BUFFERSIZE, stdin);
    ret = malloc(strlen(buffer));

    strcpy(ret, buffer);
    return ret;
}

char* intToString(int val, int base){
    //source: www.strudel.org.uk/itoa/
    static char buf[32] = {0};

    int i = 30;

    for(; val && i ; --i, val /= base)

        buf[i] = "0123456789abcdef"[val % base];

    return &buf[i+1];

}