如何在另一個函數中更改字符串指針數組？

Question

我一直在試圖找出如何修改char指針數組的方法，但是無論我做什么，下面似乎都沒有改變，我正在嘗試更改的三個數組包括對正在使用的函數的調用。

char*cm1[5];
char*cm2[5];
char*cm3[5];
setupCommands(&cm1,commands,file,0);
setupCommands(&cm2,commands,file,1);
setupCommands(&cm3,commands,file,2);

下面的代碼是函數本身，我當時在想它可能包含一個雙指針，但是如果我嘗試使用* cmd更改數組，則會遇到分段錯誤。

void setupCommands(char **cmd[], char* commands[],char file[],int index){



    char str1[255];
    strcpy(str1,commands[index]);
    char newString [5][255];
    int j = 0;
    int ctr = 0;
    int i;
    //printf("str1 %s\n" ,str1);

    for(i = 0; i <= strlen(str1); i++){

        if(str1[i] == ' '|| str1[i] =='\0'){
            newString[ctr][j] = '\0';
            ctr++;//next row
            j=0;// for next word, init index to 0
        }else{
            newString[ctr][j]=str1[i];
            j++;
        }
    }


    for(i = 0; i < ctr; i++){
            //printf(" test2 %s \n", newString[i]);
        cmd[i] = newString[i];
            //printf(" test2 %d %s \n", i,cmd[i]);
    }
 //printf("index %d", i);
  cmd[i]= file;
  cmd[i + 1] = NULL;

  //execvp(cmd[0],cmd);
   //cmd
}

Answer 1

首先-成為三星級指針程序員並不好:)

您將其分配給指向局部變量的指針，該局部變量在函數返回后不再可用

但是，如果您仍然想要三顆星的指針：

char **cm1;
char **cm2;
char **cm3;
setupCommands(&cm1,commands,file,0);
setupCommands(&cm2,commands,file,1);
setupCommands(&cm3,commands,file,2);

---

#define MAXWORD 256

int setupCommands(char ***cmd, const char *commands,const char *file,int index){

    char str1[255];
    strcpy(str1,commands[index]);

    int j = 0;
    int ctr = 0;
    int i;
    //printf("str1 %s\n" ,str1);

    *cmd = malloc(sizeof(char *));
    **cmd = malloc(MAXWORD);
    if(!*cmd || !**cmd) 
    {
        /* do spmething if mallocs failed*/
        return -1;
    }
    for(i = 0; i <= strlen(str1); i++){

        if(str1[i] == ' '|| str1[i] =='\0'){
            (*cmd)[ctr][j] = '\0';
            ctr++;//next row
            *cmd = realloc((ctr + 1) * sizeof(int));
            (*cmd)[ctr] = malloc(MAXWORD);
            if(!*cmd || !*cmd[ctr]) 
            {
            /* do spmething if mallocs failed*/
                return -1;
            }

            j=0;// for next word, init index to 0
        }else{
            (*cmd)[ctr][j]=str1[i];
            j++;
        }
    }


    *cmd = realloc(sizeof(char *)  * ctr + 2)  
    (*cmd)[ctr - 2] = malloc(MAX);
    if(!*cmd || !*cmd[ctr - 2]) 
    {
        /* do spmething if mallocs failed*/
        return -1;
    }
    strcpy((*cmd)[ctr - 2], file);
    (*cmd)[ctr - 1] = NULL;

    return 0;

  //execvp(cmd[0],cmd);
   //cmd
}

您可以改善很多事情（例如，不是每次都重新分配，而是在較大的塊中重新分配），並且我沒有更改您的代碼邏輯中的任何內容。

Answer 2

1. 您的代碼存在一些問題：
   • 您正在嘗試在函數退出時返回對本地'char newString [5] [255]'的引用。 在簡單的世界中-永遠不要返回堆棧上本地分配的任何內容。 這就是您遇到分段錯誤的原因。
   • 必須將char **cmd[]聲明為char *cmd[] -即使您assignment from incompatible pointer type收到編譯器assignment from incompatible pointer type的警告，代碼仍將正確運行和執行（基本上** cmd []會執行相同的工作如果您未返回對本地對象的引用，則為* cmd []，即使它的類型不正確）；

簡單而簡單的優化只是刪除數組str1並直接對數組commands 。

除了這種簡單的優化之外，我還更改了代碼以克服分段錯誤，方法是在堆上分配，而不是在堆棧上分配（直到程序終止時才存在）多維數組，並且我還計算了它的大小，因此我將知道有多少內存分配。 現在可以安全地返回對其的引用。

請注意，可以進行更多優化，但是為了簡單起見，這是此代碼正常工作的最低要求。

int setupCommands(char *cmd[], char *commands[], char file[], int index)
{
int j = 0;
int ctr = 0;
int i = 0;

int rows = 0;
int cols = 0;

char **newString = NULL;

while(commands[index][i])
{
    if (commands[index][i] == ' ')
    {
        ++rows;
    }

    ++i;
}
++rows;

cols = strlen(commands[index]) + 1;

newString = malloc(rows * sizeof(*newString));
if (newString == NULL)
{
    return -1;
}

for (i = 0; i < rows; ++i)
{
    newString[i] = malloc(cols * sizeof(*newString));
    if (newString[i] == NULL)
    {
        return -1;
    }
}

for(i = 0; i <= strlen(commands[index]); i++){

    if(commands[index][i] == ' '|| commands[index][i] =='\0'){
        newString[ctr][j] = '\0';
        ctr++;//next row
        j=0;// for next word, init index to 0
    }else{
        newString[ctr][j]=commands[index][i];
        j++;
    }
}

for(i = 0; i < ctr; i++){

    cmd[i] = newString[i];

}

cmd[i]= file;
cmd[i + 1] = NULL;

return 0;
}