检查answer here并在我的环境中运行后,我发现代码仍然遇到与我的代码相同的问题。 这个问题是每当我有一个类似于此的输入文件时......
FILE A
|---------------------|
| ABCDE |
| abcde |
|---------------------|
我在目标文件中生成额外的换行空格。
FILE B
|---------------------|
| edcba |
| |
| EDCBA |
|---------------------|
在调试我的代码后,我可以看到' \ n'被复制到目标文件两次,我想了解为什么lseek正在这样做。
附件是我的代码。有问题的关键部分是在do / while循环中。我非常确定我的思维过程是合理的,因为我查找的answer代码会输出完全相同的结果。
#define CHAR_SIZE 2048
#define BUFF_SIZE 1
#define PERMS 0666
int main(int argc, char* argv[]){
if (argc < 3) {
return 1;
printf ("ERROR: not enough arguments\n");
}
int src_file = 0; int dest_file = 0;
int n = -1;
if((src_file=open(argv[1],O_RDONLY)) < -1) return 1;
if((dest_file = creat(argv[2], PERMS)) < -1) return 1;
printf ("The filesize is: %d\n", lseek(src_file, (off_t) 0, SEEK_END));
char buffer[BUFF_SIZE];
lseek (src_file,n--,SEEK_END);
do {
read(src_file,buffer,BUFF_SIZE);
printf ("%s", buffer);
write(dest_file,buffer,BUFF_SIZE);
}while (lseek (src_file,n--,SEEK_END) > -1);
printf("\n");
return 0;
}
答案 0 :(得分:1)
你有很多问题。您的额外换行符来自于POSIX将行结束定义为newline
(尽管并非所有文件都符合)。因此,要摆脱额外的换行符,您需要:
int n = -2;
而不是
int n = -1;
(显式读取POSIX行结束并将其写入输出文件)
您的其他问题是您不知道您的文件是否因为错误地检查返回而失败:
// if ((src_file = open (argv[1],O_RDONLY)) < -1) return 1;
// if ((dest_file = creat (argv[2], PERMS)) < -1) return 1;
失败时返回-1
。因此,您需要:
if ((src_file = open (argv[1],O_RDONLY)) == -1) return 1;
if ((dest_file = creat (argv[2], PERMS)) == -1) return 1;
接下来,您的缓冲区将不会包含字符串。至少您需要一个2-char
数组来保存1-char
加上nul-terminating
字符。以下内容不起作用:
#define BUFF_SIZE 1
...
char buffer[BUFF_SIZE];
...
printf ("%s", buffer);
您已向printf
发送了一个未终止的字符。如果您正在使用警告启用(例如-Wall -Wextra
)进行编译,那么您会知道,因为编译器会警告您(根据您的编译器,它甚至可能没有正确的警告标志)。至少要做上述工作,你需要:
#define BUFF_SIZE 2
并且您必须在致电buffer
之前确保 nul-terminate printf
。您可以通过以下方式实现这一目标:
char buffer[BUFF_SIZE] = "";
...
read (src_file, buffer, BUFF_SIZE - 1);
buffer[1] = 0;
printf ("%s", buffer);
write (dest_file, buffer, BUFF_SIZE - 1);
(您应该检查read
和write
的返回,以验证您正在读/写您认为自己的字节数...)
将更正放在一起(并显示您可以完全消除缓冲区并简单地使用int
作为buf char),您可以执行以下操作:
#include <stdio.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <unistd.h>
#define CHAR_SIZE 2048
#define BUFF_SIZE 2
#define PERMS 0666
int main(int argc, char **argv) {
if (argc < 3) {
return 1;
printf ("ERROR: not enough arguments\n");
}
int src_file = 0;
int dest_file = 0;
int n = -2;
int nl = '\n'; /* explicit newline when writing in reverse */
// if ((src_file = open (argv[1],O_RDONLY)) < -1) return 1;
// if ((dest_file = creat (argv[2], PERMS)) < -1) return 1;
if ((src_file = open (argv[1],O_RDONLY)) == -1) return 1;
if ((dest_file = creat (argv[2], PERMS)) == -1) return 1;
printf ("The filesize is: %ld\n", lseek(src_file, (off_t) 0, SEEK_END));
lseek (src_file, n--, SEEK_END);
#ifdef WBUFCHAR
int bufchar;
do { /* validate both read and write */
if (read (src_file, &bufchar, BUFF_SIZE - 1) == 0) {
fprintf (stderr, "error: read failure.\n");
return 1;
}
putchar (bufchar);
if (write (dest_file, &bufchar, BUFF_SIZE - 1) == 0) {
fprintf (stderr, "error: write failure.\n");
return 1;
}
} while (lseek (src_file, n--, SEEK_END) > -1);
#else
char buffer[BUFF_SIZE];
do { /* validate both read and write */
if (read (src_file, buffer, BUFF_SIZE - 1) == 0) {
fprintf (stderr, "error: read failure.\n");
return 1;
}
buffer[1] = 0; /* nul-terminate */
printf ("%s", buffer);
if (write (dest_file, buffer, BUFF_SIZE - 1) == 0) {
fprintf (stderr, "error: write failure.\n");
return 1;
}
} while (lseek (src_file, n--, SEEK_END) > -1);
#endif
/* explicity write the newline you removed earlier */
if (write (dest_file, &nl, BUFF_SIZE - 1) == 0) {
fprintf (stderr, "error: write failure.\n");
return 1;
}
putchar ('\n');
return 0;
}
注意:正常编译以使用您的buffer
。使用-DWBUFCHAR
进行编译以使用bufchar
代码。
示例编译和输出
$ cat testabcde.txt
ABCDE
abcde
$ gcc -Wall -Wextra -Ofast -o bin/extranl extranl.c
$ ./bin/extranl testabcde.txt testrev.txt
The filesize is: 12
edcba
EDCBA
$ cat testrev.txt
edcba
EDCBA
$ gcc -Wall -Wextra -Ofast -DWBUFCHAR -o bin/extranlwbc extranl.c
$ ./bin/extranlwbc testabcde.txt testrevwbc.txt
The filesize is: 12
edcba
EDCBA
$ cat testrevwbc.txt
edcba
EDCBA
尝试一下,如果您有其他问题,请告诉我。
答案 1 :(得分:0)
#include <unistd.h>
#include <fcntl.h>
#include <sys/types.h>
#define CHAR_SIZE 2048
#define BUFF_SIZE 1
#define PERMS 0666
int main(int argc, char* argv[]){
if (argc < 3) {
printf ("ERROR: not enough arguments\n");
return 1;
}
int src_file = 0; int dest_file = 0;
int n = -1;
if((src_file=open(argv[1],O_RDONLY)) == -1) return 1;
if((dest_file = creat(argv[2], PERMS)) == -1) return 1;
printf ("The filesize is: %d\n", lseek(src_file, (off_t) 0, SEEK_END));
char buffer[BUFF_SIZE];
lseek (src_file,n--,SEEK_END);
do {
read(src_file,buffer,BUFF_SIZE);
printf("%02x\n", *buffer);
if (*buffer == '\n')
n--;
write(dest_file,buffer,BUFF_SIZE);
}while (lseek (src_file,n--,SEEK_END) > -1);
printf("\n");
return 0;
}
此解决方案是在输出文件中修复额外换行空格(\ n \ n)的最佳方法。
通过将此部分添加到do / while循环,我可以补偿添加的额外\ n。
if (*buffer == '\n')
n--;