Question

我有一个包含3行的文件，我正在尝试读取此文件并将每行另存为单独的字符串。这是我尝试执行的操作，它确实保存了第一行，但通过保存第一行和第二行覆盖了它，并且我无法理解如何将每一行保存为单独的字符串，并且我也遇到错误->

*检测到堆栈粉碎* ：/home/ubuntu/workspace/ex12.c.o已终止中止

 #include <stdio.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include<stdio.h> 
 #include<fcntl.h> 
 #include<errno.h> 
 #include <unistd.h>
 extern int errno;                               
int main( int argc, char *argv[] )  {
    char *path1;   
    char  firstline[80];
    char  secondline[80];
    char  thirdline[80];  

    printf("Program name %s\n", argv[0]);

    if( argc == 2 ) {
         printf("The path of the config file that you supplied is %s\n", argv[1]);
    }
    else if( argc > 2 ) {
        printf("Too many arguments supplied.\n");
    }
    else {
        printf("One argument expected.\n");
    }

    int fd1 = open(argv[1], O_RDONLY | O_CREAT);  

    if (fd1 ==-1) 
    { 
        // print which type of error have in a code 
        printf("Error Number % d\n", errno);  

        // print program detail "Success or failure" 
        perror("Program"); 
        exit(EXIT_FAILURE);
    }       
    else {            
        char c;
        int i=0;            

        while ((read(fd1, &c, 1) == 1) )
        {                
            firstline[i++]=c;                
            if(c=='\n')
            {
                //printf("end of line"); 
                printf("%s",firstline);
            }

        }
    }
    int close(int fd1);         
    return 0;        
}

注意：我不想使用fopen，fgets，sscanf或getline。任何帮助将不胜感激

Answer 1

printf打印，直到看到NULL终止字符。您的printf遍历了未分配的内存区域，因为您没有在字符串末尾插入NULL（值0）。另外，您忘记了将i重新初始化为0。

while ((read(fd1, &c, 1) == 1) )
{                
    firstline[i++]=c;                
    if(c=='\n')
    {
        firstline[i] = 0;
        i = 0;
        //printf("end of line"); 
        printf("%s",firstline);
    }
}

Answer 2

出现错误的原因很可能是因为您从未将i重置为0，所以您继续在firstline中读取80个以上的字符

关于将每一行保存到自己的字符串中，您只需要使用其他变量，而不是一直使用firstline。

有两种方法可以做到：

当您检测到行尾时，退出循环（使用break），并开始另一个循环，将字符放入secondline中。对第三者做同样的事情。
如果您已经了解了一些有关指针的知识，则可以保留一个循环，但可以使用一个额外的变量，例如char *currentLine，该变量将保存您要读取的数组的地址。每次检测到行尾时都要更改地址，例如：currentLine = secondline。

还请记住在读取的每一行末尾都添加'\ 0'，否则当您尝试在屏幕上打印读取的内容时，程序可能会打印垃圾。

Answer 3

以下建议的代码：

干净地编译
执行所需的功能
正确检查并处理错误
赋予“魔术”数字（3、80）有意义的名字

现在，建议的代码：

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <errno.h>
#include <unistd.h>

#define MAX_LINES 3
#define MAX_LINE_LEN 80


int main( int argc, char *argv[] )  
{
    char Lines[ MAX_LINES ][ MAX_LINE_LEN ] = { '\0' };


    if( argc != 2 )
    {
        fprintf( stderr, "USAGE: %s <configFileName>\n", argv[0] );
        exit( EXIT_FAILURE );
    }

    // implied else, correct number of command line arguments


    int fd1 = open(argv[1], O_RDONLY );  

    if (fd1 ==-1) 
    { 
        perror( "open failed" ); 
        exit(EXIT_FAILURE);
    }  

    // implied else, open successful        

    char c;

    for( int i = 0; i < MAX_LINES; i++ )
    {
        for( int j=0; j< MAX_LINE_LEN; j++ )
        {
            ssize_t bytecount = read(fd1, &c, 1 );
            if( bytecount < 0 )
            {
                perror( "read failed" );
                close( fd1 );
                exit( EXIT_FAILURE );
            }

            // implied else, read successful 

            Lines[ i ][ j ] = c;

            if( c == '\n' )
            {
                break;
            }   
        } 
    }

    for( int i = 0; i< MAX_LINES; i++ )
    {
        printf( "%s\n", Lines[i] );
    }

    close( fd1 );         
    return 0;        
}

注意：此代码假定每行少于80个字符

针对其自身的源文件运行建议的代码将导致：

#include <stdio.h>

#include <stdlib.h>

#include <sys/types.h>

Answer 4

下面是一个示例，演示您的示例从第一个循环开始的制动过程：

#define MAXLENGTH 80    
...

char  firstline[MAXLENGTH + 1] = {0};
char  secondline[MAXLENGTH + 1] = {0};
char  thirdline[MAXLENGTH + 1] = {0};  
....
else {            
    char c;
    int i=0;            
    while ((read(fd1, &c, 1) == 1 && i < MAXLENGTH)
    {                
        firstline[i++]=c;                
        if(c=='\n')
        {
            break; /* break from first loop */
        }
    }
    /* add a '\0' to the end of the string! */
    firstline[i] = '\0';
    //printf("end of line"); 
    printf("%s",firstline);
    i=0;            
    while ((read(fd1, &c, 1) == 1 && i < MAXLENGTH)
    {                
        secondline[i++]=c;                
        if(c=='\n')
        {
            break; /* break from second loop */
        }
    }
    /* add a '\0' to the end of the string! */
    secondline[i] = '\0';
    printf("%s",secondline);
    i = 0;
    int i=0;            
    while ((read(fd1, &c, 1) == 1 && i < MAXLENGTH)
    {                
        thirdline[i++]=c;                
        if(c=='\n')
        {
            break; /* break from third loop */
        }
    }
    /* add a '\0' to the end of the string! */
    thirdline[i] = '\0';
    //printf("end of line"); 
    printf("%s",thirdline);
}
...

在C中逐行读取文件（不使用fgets）

4 个答案: