4

我正在尝试编写一个程序，该程序在运行时需要几个参数来将文本附加到文件中。

该程序在运行时产生分段错误。这是代码：

int main(int argc, char* argv[])
{
    //error checking
    if (argc < 1 || argc > 4) {
        cout << "Usage: -c(optional - clear file contents) <Filename>, message to write"    << endl;
        exit(EXIT_FAILURE);
    }

    char* filename[64];
    char* message[256];

    //set variables to command arguments depending if -c option is specificed
    if (argc == 4) {
        strcpy(*filename, argv[2]);
        strcpy(*message, argv[3]);
    } else {
        strcpy(*filename, argv[1]);
        strcpy(*message, argv[2]);
    }

    int fd; //file descriptor 

    fd = open(*filename, O_RDWR | O_CREAT, 00000); //open file if it doesn't exist    then create one
    fchmod(fd, 00000);

    return 0;
}

我仍然是一个初学者，我在理解 c 字符串时遇到了巨大的麻烦。char* 和 char[] 和 char* [] 有什么区别？

更新：

代码仍然会引发分段错误，这是我修改后的代码：

using std::cout;
using std::endl;

 int main(int argc, char* argv[])
 {

//error checking
if (argc < 1 || argc > 4) {
cout << "Usage: -c(optional - clear file contents) <Filename>, message to write"      << endl;
 exit(EXIT_FAILURE);
 }

char filename[64];
char message[256];

 //set variables to command arguments depending if -c option is specificed
 if (argc == 4)
{
strncpy(filename, argv[2], 64);
strncpy(message, argv[3], 256);
}
 else
 {
strncpy(filename, argv[1], 64);
strncpy(message, argv[2], 256);
 }

 int fd; //file descriptor 

 fd = open(filename, O_RDWR | O_CREAT, 00000); //open file if it doesn't exist then create one
 fchmod(fd, 00000);


 return 0;

 }
4

5 回答 5

7

char* filename[64]创建一个包含 64 个指针的数组。您打算为 64 个字符的字符串创建空间 - 这将是char filename[64]. 因为您只为指针分配空间，而从未使指针指向任何内存，所以您会遇到段错误。

解决方案：使用char filename[64];

这将为您的字符串创建一个 64 字节的块；该值filename指向此块的开头，可用于复制操作

strcpy(filename, argv[2]);

我强烈建议使用“复制不超过 n 个字符”功能 - 这可以防止非常长的参数导致缓冲区溢出。因此

strncpy(filename, argv[2], 64);

会更安全。甚至更好

strncpy(filename, argv[2], 63);
filename[63] = '\0';

这保证了复制的字符串是空终止的。

你有同样的问题message。我认为您不需要重复代码...

如果您需要更多信息，请告诉我。

更新
今天我了解到存在strlcpy- 看到这个答案。NUL即使原始字符串比分配的空间长，它也会考虑包含字符串终止符。有关更完整的讨论，请参阅此内容，包括此功能并非在所有编译器上都可用的原因（如果您尝试编写可移植代码，这当然是一个主要缺点）。

于 2013-10-05T05:16:58.440 回答
3

由于您已将其标记为 C++（还没有人提到它）：

argv已经是一个 C 风格的数组，所以没有必要将它复制到另一个（除非你只是想浪费空间）。如果您真的想将其复制到某些东西中，那么std::string对象将是更好的方法：

int main(int argc, char* argv[])
{
    // assuming your conditional checks are already done here ...
    std::string filename = argv[1];
    std::string message = argv[2];
    // do something
    return 0;
}
于 2013-10-05T05:44:44.750 回答
2

您的变量filename和message是char 指针数组，而不是 C 风格的字符串（应该是空终止char数组）。所以你需要将它们的类型声明为：

char filename[64];
char message[256];

并strcpy用作：

strcpy(filename, argv[2]);
strcpy(message, argv[3]);

调用open类似：

fd = open(filename, O_RDWR | O_CREAT, 00000);
于 2013-10-05T05:16:16.223 回答
2

>>> 我仍然是一个初学者，我在理解 c 字符串时遇到了巨大的困难。char和 char[] 和 char* []有什么区别？*

第一次遇到指针时很难理解。

  • char 是内存中的单个字节
  • char* 是指向内存的指针（可以是单个字节或字符数组）
  • char[] 是一个字符数组，可以指向 char*
  • char*[] 是指向 char 的指针数组

当你有一个变量文件名时，*filename 取消引用该变量，这意味着它不是指针，而是指向的东西。

  • *filename 是 char 类型，不是 strcpy 的有效参数，这是您的 segfault 发生的地方
  • *message 是 char 类型，不是 strcpy 的有效参数，这是您的下一个段错误发生的地方
  • open(*filename 又是一个字符，它不是 open 的有效参数

你大部分的程序都是对的。问题是您不清楚如何使用指针。这是您的代码，稍作修改即可工作。我注释掉了损坏的部分，以便您可以将损坏的部分与已修复的部分进行比较。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
int
main(int argc, char* argv[])
{
    //char* filename[64];
    //char* message[256];
    char filename[64]; //declare filename, point it at char[64]
    char message[256]; //declare message, point it at char[256]
    int fd; //file descriptor 

    printf("argc %d\n",argc);
    //error checking
    if ( (argc < 1) || (argc > 4) )
    {
        //cout << "Usage: -c(optional - clear file contents) <Filename>, message to write"    << endl;
printf("Usage: -c(optional - clear file contents) <Filename>, message to write\n");
        exit(EXIT_FAILURE);
    }

    int argi=1;
    if( !strcmp(argv[argi],"-c") ) { argi++; } //clear
    //set variables to command arguments depending if -c option is specificed
    if (argc == 4)
    {
        //strcpy(*filename, argv[argi++]);
        //strcpy(*message, argv[argi++]);
        strcpy(filename, argv[argi++]);
        strcpy(message, argv[argi++]);
    }
    else
    {
        //strcpy(*filename, argv[argi++]);
        //strcpy(*message, argv[argi++]);
        strcpy(filename, argv[argi++]);
        strcpy(message, argv[argi++]);
    }

    //fd = open(*filename, O_RDWR | O_CREAT, 00000); //open file if it doesn't exist    then create one
    if( !(fd = open(filename, O_RDWR | O_CREAT, 00000)) ) //open file if it doesn't exist    then create one
    {
        //always check for failure to open
        //and emit error if file open fails
        exit(EXIT_FAILURE);
    }
    //fchmod(fd, 00000);
    write(fd,message,strlen(message));

    return 0;
}
于 2013-10-05T05:39:29.093 回答
0

我仍然是一个初学者，我在理解 c 字符串时遇到了巨大的麻烦。char* 和 char[] 和 char* [] 有什么区别？

对您的明确问题的简短回答是，char*两者char[]都可以用作 C 字符串。char* []另一方面是一个数组C-strings。

于 2013-10-05T05:43:20.390 回答