0

我正在尝试将一行 80 个字符的输入拆分为一个数组，其中每个元素都指向一个字符字符串。本质上，将 char a[80] 转为“Hello world！” 进入 char* b[64] 其中 b[0] 指向“Hello”，b[1] 指向“world！”

基本上， strsep() 将允许我使用以下代码：

while((cmd->argv[argc++] = strsep(clPtr, WHITESPACE)) != NULL);

我想知道如何修改此代码：

int parse(char* comm, char** commarray) {
  int count = 0;
  char word[80] = "";
  char ch[2] = {' ', '\0'};

  if(strlen(comm) == 0) {
    commarray[0] = "NULL";
    return 0;
  }

  for(size_t i = 0; i < strlen(comm); i++) {
    int c = int(comm[i]);
    if(!isspace(c)) {
      ch[0] = comm[i];
      strcat(word, ch);
      if(i == (strlen(comm) - 1)) {
        commarray[count] = word;
        cout << commarray[count] << endl;
        count++;
      }
    }
    else if(isspace(c) && word != "") {
      commarray[count] = word;
      cout << commarray[count] << endl;
      word[0] = '\0';
      count++;
    }
  }

 return 1;
}

//main
int main() {
  char command[80];
  char* args[64];

  while(true) {
    cout << "order>";
    cin.getline(command, 80);

    if(strcmp(command, "quit") == 0 || strcmp(command, "exit") == 0) {
      break;
    }

    parse(command, args);

    cout << args[0] << endl;

    if(strcmp(args[0], "quit") == 0 || strcmp(args[0], "exit") == 0) {
      break;
    }

    /*for(int i = 0; i < 3; i++) {
        cout << args[i] << endl;
    }*/
  }
  return 0;
}

main() 中的变量 args 不显示变量 commarray 在 parse() 中的作用。相反，我会胡言乱语。为什么是这样？我认为传递数组默认是通过引用传递？对于 commarray，我得到了适当的指向字符串的指针数组（我认为）。对于 args，我没有得到任何可用的东西。

4

1 回答 1

1

指针地狱就是你所在的地方。我可以看到代码至少有两个基本问题，但可能还有更多。

1）您将所有作业重用为 commarray。因此，您最终会得到 commarray 中的所有指针都指向同一个字数组。显然那是行不通的。

2）当您退出解析函数时，单词数组不再在范围内，因此它成为无效内存。因此，您的所有 args 数组指针都指向同一块无效（因此是垃圾）内存。

我的建议是，停止使用指针，开始使用 C++，即 std::string 类，它的行为比任何指针都更具逻辑性和直观性。

于 2013-03-28T23:59:06.363 回答