Question

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

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 );

Answer 1

如果达到限制中的字符数，则

strncpy()不会终止。你需要null自己终止。这使得语句strncpy( writeline, "", 0 );什么也不做（甚至没有终止字符串）。然后你添加到那个未初始化的字符串，它会很糟糕。

Answer 2

难道你告诉它确切要复制多少个字符？

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

我个人从不使用它;实际上始终生成字符串的替代方案包括strcpy，sprintf，snprintf或memcpy以及手动附加空终止符。

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

strncpy（）分段错误（再次）

解决

2 个答案: