无法使用 strcpy 对命令行参数进行排序

Question

我是 C/C++ 的新手，我正在学习命令行参数。我正在尝试使用 strcpy 对我的命令行参数进行排序，但是它给了我不好的输出。 例如

---

我/p：我是

---

o/p : 亲 我

谁能帮我解决我在这里做错了什么？请注意：我只为 argc=3 运行此程序，并且我只为上面示例中列出的输入（将被排序）运行此代码。 我刚刚删除了用于调试的循环。

#include "iostream"
#include "cstdlib"
#include "cstring"
using namespace std;

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

    char temp[100];

    //sorting my command line arguments
    if(strcmp(argv[1],argv[2])>0)
    {
        strcpy(temp,argv[1]);
        strcpy(argv[1],argv[2]);
        strcpy(argv[2],temp);
    }

    cout<<argv[1]<<endl;
    cout<<argv[2]<<endl;

    return 0;
}

Answer 1

考虑您的内存布局。当你运行$ ./a.out i am时，程序启动时会是这个样子：

a   .   o  u  t  \0 i  \0 a  m  \0
^                   ^     ^
argv[0]          argv[1]  argv[2]

在您的交换过程中写入argv[1] 会将其更改为：

a   .   o  u  t  \0 a  m  \0 m  \0
^                   ^     ^
argv[0]          argv[1]  argv[2]

然后写入argv[2] 会再次变成这样：

a   .   o  u  t  \0 a  m  i  \0 \0
^                   ^     ^
argv[0]          argv[1]  argv[2]

所以当你打印出argv[1] 时，它会一直读取到空字节，给你ami。 argv[2] 将从不同的起点读取，为您提供i。

正如 Galik 指出的那样，这是因为 argv[1] 和 argv[2] 不是某种自动调整大小的缓冲区。它们只是指向内存的指针。在这一点上我应该注意，确切的布局不是由语言正式定义的。根据您使用的平台，您可能会遇到各种不同的不可预知的行为。

要解决此问题，您应该创建一个指向要排序的字符串的指针数组，并交换指针而不是字符串的值。这将更快（需要复制的字节更少）和更安全（意外溢出缓冲区的方式更少，如果您的输入之一长于 100 个字符，则在您当前的代码中会发生这种情况）。