0

我正在尝试编写一个文字格式化程序，该程序需要一个数据文件，并对其进行格式化，以便每行最多有一定数量的字符。当我尝试遍历文件每一行中的单词时遇到分段错误错误。

int readFile(  )
{
    int rc = 1;
    char readBuffer[ 256 ];
    char writeLine[ 256 ];
    char *word = malloc( numChars * sizeof( char ) );
    
    writeFile = fopen( writeName, "a" );
    
    if ( ( openFile = fopen( readName, "r" ) ) == NULL ) 
    {
        printf( "Unable to open %s file for read\n", readName );
        rc = 0;
    } 
    else 
    {       
        while( fgets( readBuffer, sizeof( readBuffer ), openFile) != NULL ) 
        {
            strncpy( writeLine, "", 0 );
            if ( readBuffer[0] != EOF ) 
            {
                strncpy( word, strtok( readBuffer, " " ), numChars -1 );
                while( word != NULL )
                {
                    if( strlen( word ) + strlen( writeLine ) < numChars )
                    {   
                        strcat( writeLine, word );

我在这里得到错误：

                        strncpy( word, strtok( NULL, " " ), numChars -1 );
                    }
                    else
                    {
                        fprintf( writeFile, "%s", writeLine );
                        strcpy( writeLine, word );
                        strncpy( word, strtok( NULL, " " ), numChars -1 );
                    }
                }
            }
        }
    }
    
    return rc;
}

我不明白为什么会这样。strncpy 的重点不是告诉它要复制多少个字符吗？如果我明确告诉它只将字符复制到字符串长度以下一个，为什么会出现分段错误？编辑：我已将代码修改为以下内容：

            strcpy( writeLine, strtok( readBuffer, " " ) );
            
            while( strcpy( word, strtok( NULL, " " ) ) != NULL )
            {
                if( strlen( word ) + strlen( writeLine ) < numChars )
                {   
                    strcat( writeLine, word );
                }
                else
                {
                    fprintf( writeFile, "%s", writeLine );
                    strcpy( writeLine, word );
                }
            }

这不会改变操作，它仍然是分段错误。如何检查令牌是否有效并且如果有效仍访问相同的令牌？

解决了

我开发了一个可行的解决方案：

    writeLine[0] = '\0';
    
    while( fgets( readBuffer, sizeof( readBuffer ), openFile) != NULL ) 
    {
        if ( readBuffer[0] != EOF ) 
        {               
            for( word = readBuffer; NULL != ( word = strtok( word, " \t\n" ) ); word=NULL )
            {
                printf("%s\n", writeLine );
                if( strlen( word ) + strlen( writeLine ) < numChars )
                {   
                    strcat( writeLine, word );
                    strcat( writeLine, " " );
                }
                else
                {
                    printf("%s", "print\n");
                    fprintf( writeFile, "%s\n", writeLine );
                    strcpy( writeLine, word );
                    strcat( writeLine, " " );
                }
            }
        }
    }
    fprintf( writeFile, "%s\n", writeLine );
4

2 回答 2

1

strncpy 的重点不是告诉它要复制多少个字符吗？

不。关键是你告诉它你有多大的缓冲区可用。然后它将尽可能多的字符复制到该缓冲区中（将其余字符归零），并且如果至少有与缓冲区大小一样多的可用字符，它不会以空值终止。

就我个人而言，我从不使用它；实际上总是生成字符串的替代方法包括strcpy, sprintf, snprintf, 或memcpy手动附加空终止符。

始终建议阅读标准功能的文档。strncpy( writeline, "", 0 );什么也没做。

于 2014-04-25T04:29:20.603 回答
1

strncpy()如果达到限制中的字符数，则不为空终止。您需要自己终止。这使得语句strncpy( writeline, "", 0 );什么都不做（甚至不终止字符串）。然后你添加到那个未初始化的字符串上，它会很糟糕。

于 2014-04-25T04:24:31.300 回答