Question

我有一个关于将某些文件从一个目录移动到另一个目录的项目。我已经完成了所有工作，只是输出有点奇怪。我需要在argsv数组中提供目标路径，但是当我尝试执行我的代码时，它可以编译并运行，但是显示的错误路径中包含许多路径！这是相关的部分，如果您需要更多代码，我会添加！预先谢谢你！

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

int size = NFILES;
int index = -1;
file * files = malloc(size * sizeof(file));

listFilesRecursively(argv[1], &files, &size, &index);

if (index != -1) {
      int N = atoi(argv[2]);

if(N==1) qsort(files, index + 1, sizeof(file), compPathname);
else if(N==2) qsort(files, index + 1, sizeof(file), compPathsize);

for (int i = 0; i <= index; ++i) {
        char *dest = argv[3];
                strcat(dest, "/");
                strcat(dest, files[i].justname);
  printf("%s : %s : %ld\n", files[i].name, dest , (long)   files[i].file_info.st_size);
//  if(rename(files[i].name, dest)==0) printf("Success!\n"); else     printf("Failed!/n");
}

所以这是主要的。所需的输出是这样的（我有很多文件）：

./copyto.c : /home/nik/copyto.c : 676
Success!
./mvfilrd.c : /home/nik/mvfilrd.c : 957
Success!
./sortall.c : /home/nik/sortall.c : 992
Success!

以此类推...但是我得到了

./newdir/newfile.txt : /home/nik/Music/newfile.txt : 0
Success!
./newdir/3.exe : /home/nik/Music/newfile.txt/3.exe : 0
Failed!/n./newdir/compil : /home/nik/Music/newfile.txt/3.exe/test :     0
Failed!/n./newdir/2.c : /home/nik/Music/newfile.txt/3.exe/test/exe :         0

然后是更多的垃圾

Failed!/n./newf.exe : /home/nik/Music/newfile.txt/3.exe/test/exe    /1//Q�/~�dZ /�l�G^ /
                                                                                    ��`(/4�a^d /a.txt/range/1.txt/1.exe/print.exe/filrd.exeC/2.exre/filrd.exe/2.exe/fi.txt/fil.txt/dest.txt/sorcopy.c/filew.exe/.filer.c.swp    /progfilrd.exe/compile/myfile/.m

第一个参数似乎也崩溃了...

Answer 1

char *dest = argv[3]; 
strcat(dest, "/"); 
strcat(dest, files[i].justname);

哎呀，您修改了一个您不拥有的字符串，不要这样做，您可能会写出该字符串，然后进行复制

替换

for (int i = 0; i <= index; ++i) {
    char *dest = argv[3];
    strcat(dest, "/");
    strcat(dest, files[i].justname);
    printf("%s : %s : %ld\n", files[i].name, dest , (long)   files[i].file_info.st_size);
    if(rename(files[i].name, dest)==0) 
       printf("Success!\n");
    else
       printf("Failed!/n");
}

作者

for (int i = 0; i <= index; ++i) {
    size_t sz = strlen(argv[3]);
    char *dest = malloc(sz + strlen(files[i].justname) + 2);

    strcpy(dest, argv[3]);
    dest[sz] = '/';
    strcpy(dest + sz + 1, files[i].justname);

    printf("%s : %s : %ld\n", files[i].name, dest , (long)   files[i].file_info.st_size);
    if(rename(files[i].name, dest)==0)
      printf("Success!\n");
    else
      printf("Failed!/n");

    free(dest);
}

Answer 2

此：

    char *dest = argv[3]

使dest指向与argv[3]相同的字符串。也就是说，您已复制了指针，而不是指针所指向的数据。当您随后通过dest修改该数据时，就是在修改参数本身。这是允许的，但格式不正确。

更大的问题是您实际使用的方法。这个：

  strcat(dest, "/"); 
  strcat(dest, files[i].justname);

试图将数据追加到参数字符串的末尾，但是假设（在实践中可能是不正确的）指向的数组中有任何可用空间来存储多余的字符是不安全的。如果确实没有足够的空间，那么您将产生未定义的行为。

根据您所显示的代码和所描述的行为，我推测您未显示的内容可能包括指向files[i].justname的另一个参数指针的类似分配。这样的分配并不是天生的错误，但是它很容易使UB的表现比以前更加令人困惑。

无论如何，如果要形成程序参数的串联，则需要为结果保留单独的空间。您可以使用足够大的自动阵列或足够大的动态分配空间来实现。后者可能看起来像这样：

    size_t path_chars = strlen(argv[3]) + strlen(files[i].justname) + 2;
    char *path = malloc(path_chars);

    if (path) {
        sprintf(path, "%s/%s", argv[3], files[i].justname);
    } // else handle memory allocation failure

如果愿意，可以改用strcpy()和strcat()，但是在这种情况下，我认为sprintf()更加清晰和干净。

如何从命令行正确传递文件路径？

2 个答案: