0

信じられないほど単純なものが欠けていますか?私のコードが文字列を動的に保存していない理由が見つからないように見えるからです。最後に印刷すると、空白行しか表示されません。任意の行数の最後の「n」行を出力することになっています。ただし、実際の行の保存に問題があるようです。誰か手を貸してくれませんか?

ちなみに、これはHWではありません。これは K&R の本の問題です (その回答はオンラインです)。私は独学でCを学ぼうとしています。

void tail5_13(int n)
{

    int maxpoint = 3;
    size_t *maxlength;
    *maxlength = sizeof(char) * 10;

    char **pointptr = malloc(sizeof(void *) * maxpoint);
    char **pointcnt = pointptr;

    char *lineptr = malloc(sizeof(char) * *maxlength);

    int numlines = 0;
    int printnum = 0;
    int c;



    while((c = getline(&lineptr, maxlength, stdin)) > 1)
    {
        if(numlines < maxpoint)
        {
            storeline5_13(pointcnt, lineptr, c);
            pointcnt++;
        }
        else
        {
            maxpoint *= 2;
            printf("Increased pointer amount to %d\n", maxpoint);
            pointptr = (char **) realloc(pointptr, sizeof(void *) * maxpoint);
            storeline5_13(pointcnt, lineptr, c);
            pointcnt++;
        }
        numlines++;
    }

    if(n <= numlines)
        printnum = n;
    else
        printnum = numlines;

    while(printnum-- > 0 )
    {
        printf("%s\n", *(pointcnt-1));
    }

}



void storeline5_13(char** pointcnt, char* line, int length)
{


    *pointcnt = (char *) malloc(sizeof(char) * length);

    while(*line != '\n')
    {
        *((*pointcnt)++) = *line++;

    }

    **pointcnt = '\0';


}
4

1 に答える 1

4
size_t *maxlength;
*maxlength = sizeof(char) * 10;

ワイルド(割り当てられていない)ポインターを逆参照しています。

代わりに、次のようにする必要があります。

size_t maxlength = sizeof(char) * 10;
// ...
char *lineptr = malloc(sizeof(char) * maxlength);
// ...
while((c = getline(&lineptr, &maxlength, stdin)) > 1)

次に、自動変数を使用しているだけです。

于 2010-10-07T05:46:05.160 に答える