没有参数无法执行，输入未正确解析

Question

在下面的函数中，我试图从标准输入读取几个句子，并在用户按两次输入后处理它们。如图所示，我将内存动态分配给整个缓冲区（行集）。我遇到了：

zsh: illegal hardware instruction  ./myfunc

最小编译sn-p：

void myfunc(void) {

    int textsize = BUF_SIZE;
    char **lines = (char **) malloc(sizeof(char *) * textsize);
    int linecount = 0;
    char text[BUF_SIZE];

    while ((strcpy(text, fgets(text, BUF_SIZE, stdin))) != NULL) {
        if (text[0] == '\n') {
            break;
        }
        lines[linecount] = (char *) malloc(sizeof(char *) * strlen(text));
        strcpy(lines[linecount], text);
        linecount++;
    }
    for (int index = linecount - 1; index >= 0; --index) {
        fprintf(stdout, "%s", lines[index]);
        /* if line has newline stripped, else printf("%s", lines[index]);*/
    }

    free(*lines);
    exit(0);
}

Answer 1

在本次内存分配中使用 textsize 的值

char **lines = (char **) malloc(sizeof(char *) * textsize);

没有意义。

在while循环的条件下使用strcpy

while ((strcpy(text, fgets(text, BUF_SIZE, stdin))) != NULL) {

也没有意义，因为 1) fgets 可以返回 NULL 2) 如果 fgets 没有返回 NULL 那么它已经用字符串填充了数组文本。

所以循环至少应该写成这样

while ( fgets(text, BUF_SIZE, stdin) != NULL) {

在此声明中

lines[linecount] = (char *) malloc(sizeof(char *) * strlen(text));

您需要分配char 类型的对象数组，而不是char * 类型，并且分配的元素数量应等于tp strlen( text ) + 1。

这个电话

free(*lines);

仅释放指针lines 指向的第一个分配的元素，但您需要释放所有已分配的内存。

而这个调用在函数的最后

exit(0);

再次没有任何意义。至少你可以写

return;

或者只是删除exit的调用。

该函数可以如下面的演示程序所示。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define BUF_SIZE    100

void myfunc( void ) 
{
    char **lines = NULL;
    size_t linecount = 0;
    
    char text[BUF_SIZE];
    int success = 1;

    while ( success && fgets( text, BUF_SIZE, stdin ) != NULL && text[0] != '\n' ) 
    {
        char **tmp = realloc( lines, ( linecount + 1 ) * sizeof( char * ) );
        success = tmp != NULL;
        
        if ( success )
        {
            lines = tmp;
            lines[linecount] = malloc( ( strlen( text ) + 1 ) * sizeof( char ) );
            
            success = lines[linecount] != NULL;
            
            if ( success )
            {
                strcpy( lines[linecount], text );
                ++linecount;
            }
        }
    }
    
    if ( !success ) fputs( "Nor all input records were successfully stored\n", stdout );
    
    for ( size_t i = linecount; i != 0; ) 
    {
        fprintf( stdout, "%s", lines[--i]);
    }

    for ( size_t i = 0; i < linecount; ++i )
    {
        free( lines[i] );
    }
    free( lines );
}

int main(void) 
{
    myfunc();
    
    return 0;
}

如果要输入字符串

Hello
World 

那么输出将是

World 
Hello