Question

/*                                                                           
  Program to calculate trip and plan flights                                 
*/
#define TRIP 6
#define NAMEMAX 40
#define DEST 1

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


int main(void)
{
  int i, trip_num, row, col;
  char travel_name[TRIP][DEST],
    dest_in[1];


  printf("Please enter the number of trips:");
  scanf("%d", &trip_num);

  while (trip_num > TRIP)
    {
      printf("Invalid number of trip. (Min of 3 trips and Max 6 trips).\n");\
  /*input number of trips*/
      printf("Please enter the number of trips:");
      scanf("%d", &trip_num);
      if (trip_num < TRIP)
        printf("Valid trip number. Please proceed to enter destination code.\
\n");
    }

  for (i=0; i < trip_num ; i++)

    {
      printf("Please enter name of destination(Note: use _ to for spaces in \
name):\n");
      scanf("%s", &dest_in);
      if (strlen(dest_in) < NAMEMAX)
        strcpy(travel_name[i],dest_in);
   /*   else (strlen(dest_in) > NAMEMAX) */



  for (row = 0; row < trip_num; row++)
    {
      for (col=0; col < DEST; col++)
        printf("Trip#:%d travel_name:%s \n", row+1, travel_name[row][col]);
    }

  return 0;
}

我正在尝试让用户输入一个名称作为字符串并存储它，如果名称是40个字符或更少，但它给我一个分段错误

Answer 1

trip_num未初始化。不知道for循环执行的次数是多少次：

travel_name[i][0]=dest_in[100];

可能会在数组末尾写入。此外，该声明的目的是什么？我并没有真正遵循它想要做的事情，所以我会先想出来。

Answer 2

您的代码存在很多问题。

scanf("%s", &dest_in)会将一个字符串读入dest_in，这可能会使您的缓冲区溢出，因为没有大小说明符。考虑将其更改为scanf("%99s", dest_in)，这样您最多可读取99个字符，空终止符为1，即100（您的数组大小）。此外，此处无需使用&运算符。
travel_name[i][0]=dest_in[100];您正在访问一个超出dest_in范围的字符。您应该访问的唯一索引是0。
printf("Trip#:%d travel_name:%s \n", row+1, travel_name[row][col]);您的代码表示您要打印字符串。 printf寻找指向字符数组的指针，但travel_name[row][col]是一个字符。
while (trip_num > TRIP)。您告诉用户输入3到6之间的数字，但不检查输入是否小于3.

Answer 3

我认为这是因为 - ＆gt;什么是trip_num？

通过使用调试器可以轻松解决这些类型的问题。我建议学习gdb（或cgdb），或者使用Eclipse CDT等内置调试功能的直接IDE。

字符串/分段错误

3 个答案: