0

当我运行以下分段错误时...目标是将 argv 数组复制到 allowdip 数组。

char *allowdip;
int *allowdipcount;


int main(int argc, char *argv)
{
  int xer;

  allowdipcount = argc;

  for(xer=0; xer<allowdipcount; xer++) {
    allowdip[xer]=argv[xer];
  }

  for(xir=0; xir<allowdipcount -1; xir++) {
    printf("%s\n", allowdip[xir]);
  }


exit(EXIT_SUCCESS);
}

关于我做错了什么的任何想法？

更新

谢谢，现在我的代码是：

char **allowdip;
int allowdipcount;

int main(int argc, char *argv)
{
  int xer;
  int xir;

  allowdipcount = argc;
  char **allowdip = malloc(allowdipcount * sizeof(char*));
  for(xer=0; xer<argc; xer++) {
      allowdip[xer]=argv[xer];
  }

    for(xir=1; xir<allowdipcount; xir++)
    printf("%s\n", allowdip[xir]);
    exit(EXIT_SUCCESS);

}

它返回：

 testscript2.c:51: warning: assignment makes pointer from integer without a cast

第 51 行包含：

 allowdip[xer]=argv[xer];
4

2 回答 2

4

allowdip是一个未初始化的指针。你需要为它分配内存

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

int main(int argc, char *argv[])
{
  int xer;
  int allowdipcount = argc;
  char **allowdip = malloc(allowdipcount * sizeof(char*));

  for(xer=0; xer<allowdipcount; xer++) {
      allowdip[xer]=argv[xer];
  }

  for(xer=0; xer<allowdipcount; xer++) {
    printf("%s\n", allowdip[xer]);
  }
  free(allowdip);
  return EXIT_SUCCESS;
}

请注意，我在您的代码中进行了一些其他更改

  • allowdipcount应该是类型int（否则你也需要为它分配存储空间）
  • allowdip已将类型更改为char指针数组
  • 的签名main不太正确 -argv应该是一个char*数组
  • 将您的全局变量更改为本地变量，main因为没有明显需要它们是全局变量
  • 更改printf循环以遍历所有程序参数。它跳过了您问题中的最后一个参数。
  • allowdip完成后释放我们分配的内存
  • main文森特建议的简化回报
于 2013-04-23T14:52:23.330 回答
1

你没有分配内存，你的类型都是错误的。

计数器应该是整数，而不是指针：

int allowdipcount;

并且数组应该是指针数组，而不是字符数组：

char **allowdip;

然后你可以分配：

allowdip = malloc(argc * sizeof *allowdip);

并复制数组：

memcpy(allowdip, argv, argc * sizeof *allowdip);

请注意，这不会复制实际的参数字符串，只会复制指向字符串的指针数组。它也不包括终止数组的NULL指针。argv[argc]

于 2013-04-23T14:57:01.427 回答