2

我正在自学 C，我遇到了一个问题，即需要根据行的长度对行进行排序。这部分将输入流中的字符存储到一个数组中，并将每一行放入一个指针数组中。然后尝试打印第一行。

#include <stdio.h>
#include <string.h>
#define LINES 5
void main()
{
   char c;
   char* str1 = (char*)malloc(30);
   char* str[LINES];
   int i = 0,temp;
   while ((c = getchar()) != EOF) //storing all characters from input stream into an array
      *(str1 + i++) = c;
   *(str1 + i) = '\0';
   temp = i;//total number of characters
   i = 0;
   int j = 0, k = 0;
   str[j] = (char*)malloc(30);
   while (i < temp)//storing each line in separate pointers of the array of pointers
   {
      if (j + 1 == LINES)
         break;
      if (*(str1 + i) == '\n')
      {
         *(*(str + k) + j++) = '\0';
         str[j] = (char*) malloc(30);
         k = 0;
      }
      else
         *(*(str + k++) + j) = *(str1 + i);
      i++;
   }
   printf("%s\n", str[0]);//printing the first line

}

这是我的输出屏幕的样子：

iiii
iii
ii
i
i
^Z
Press any key to continue . . .

在给出输入并输入 EOF 后的输入屏幕中，程序崩溃。为什么它不起作用？

顺便说一句，它在 EOF 之后崩溃。

4

1 回答 1

1

问题是您已经交换了j和k索引。

*(*(str + k++) + j) = *(str1 + i);
*(*(str + k) + j++) = '\0';

相当于：

str[k++][j] = str1[i];
str[k][j++] = '\0';

这不是你想做的。你想要的是：

str[j][k++] = str1[i]; /* *(*(str + j) + k++) = *(str1 + i); */
str[j++][k] = '\0';    /* *(*(str + j++) + k) = '\0';        */

其他评论（其中一些来自对您问题的评论）：

  1. void main()应该int main(void)
  2. 编译使用-Wall -Wextra，特别是如果你正在学习

  3. 你的第一个while是for：

    for (i = 0; (c = getchar()) != EOF) && i < temp-1; i++)
    *(str + i) = c;

    或者

    for (i = 0; (str[i] = getchar()) != EOF) && i < temp-1; i++)
    ;

  4. 你的第二个while也是for：for (i = 0, j = 0, k = 0; i < temp && j < LINES; i++)

  5. 因此，您会错过最后一行j + 1 == LINES。
  6. 除非您正在练习指针，否则不要这样做*(*(str + k++) + j)；即使您正在练习指针，也不要这样做，因为您可以练习使用指针访问，而不是使用指针表示法的索引。
  7. 阅读string.h文档，您会发现很多功能可以让您的生活更轻松：fgets、strchr、strdup等。
于 2018-06-25T17:02:55.263 回答