Question

我正在编写一个程序，通过向每个字符添加10来加密文件。不知何故，一部分程序工作目录正在打印到文件中，我不明白为什么。

#include <stdio.h>
    int main(void){
        FILE *fp;
        fp=fopen("tester.csv","r+");
        Encrypt(fp);      
        fclose(fp);
    }

    int Encrypt(FILE *fp){
        int offset=10;
        Shift(fp, offset);
    }
    int Decrypt(FILE *fp){
        int offset= -10;
        Shift(fp, offset);
    }
    int Shift(FILE *fp, int offset){
        char line[50],tmp[50], character;
        long position;
        int i;
        position = ftell(fp);
        while(fgets(line,50,fp) != NULL){
            for(i=0;i<50;i++){
                character = line[i];
                character = (offset+character)%256;
                tmp[i] = character; 
                if(character=='\n' || character == 0){break;}                
            }
            fseek(fp,position,SEEK_SET);
            fputs(tmp,fp);
            position = ftell(fp);
            fseek(stdin,0,SEEK_END);
        }
      }

该文件最初读取

this, is, a, test
i, hope, it, works!

程序运行后

：

~rs}6*s}6*k6*~o}~
/alexio/D~6*y|u}+
k6*~o}~
/alexio/D

其中users / alexio / Desktop是路径的一部分。这是怎么发生的？

Answer 1

因为你“编码”了字符串，所以它不会被空终止（这是你的情况），或者它甚至在字符串结尾之前包含一个null（字符+偏移量％256 == 0）。稍后您尝试将其写为字符串，它会超出缓冲区，并输出部分程序参数。

使用fread和fwrite。

Answer 2

该行

fputs(tmp,fp);

写出一个可能非空终止的字符串。因此，它继续将内存复制到文件，直到找到空值。

如果循环在换行符上断开，则需要在'tmp'的末尾添加一个空值。

Answer 3

许多事情：

您正在对读取缓冲区中的所有50个字符进行编码，无论实际使用fgets()读取了多少个字符。回想一下fgets()读取行，而不是整个缓冲区（除非行长于缓冲区，而你的不是）。来自行文件输入的字符串长度超过堆栈垃圾。
然后你要转储所有额外的垃圾数据，而不是在用tmp[]写入之前没有终止你的fputs()字符串，你不应该使用它。还有更多堆垃圾。

解。使用fread()和fwrite()进行此编码。没有理由使用字符串函数。当您编写解码器时，您会感谢您使用fread()和fwrite()

为什么输出字符串显示我从未写过的数据？

3 个答案: