Question

我正在为下面的代码获取分段错误。为了编译，我键入了gcc -std=c99 -g alphacode.c。这是我从here解决的问题，我不确定问题是什么。

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

int catchar(char a, char b) {

  char a1[2];
  a1[1] = a;
  a1[0] = b;

  return atoi(a1);
}

int processNumber(char *num) {
 int size = strlen(num);
 int p[size];

 if (num[size-1] != 0)
   p[size-1] = 1;
 else
   p[size-1] = 0;

 int i;
 for (i = size-2; i>=0; i--)
    {

      if (catchar(num[i], num[i-1]) > 26 ||
      catchar(num[i] , num[i-1]) <1 || num[i] == 0) 
    p[i] = p[i-1];    
      else
    p[i] = p[i-1] + p[i-2];

    }

 return p[0];

}


int main() {

    int bytes_read;
    int nbytes = 5000;
    char *number;
    bytes_read = getline (&number, &nbytes, stdin);
    while (bytes_read != -1) {

      int out = processNumber(number);
      printf("%d\n", out);
      bytes_read = getline (&number, &nbytes, stdin);

    }
    return 0;
}

Answer 1

int catchar(char a, char b) {

  char a1[2];
  a1[1] = a;
  a1[0] = b;

  return atoi(a1);
}

atoi()需要一个字符串，一个字符串必须有一个'\0'终结符，如果没有它，atoi()只会一直查找'\0'，你可能会得到垃圾或段错误 - 如果你没有分配内存。

您应该声明大小为3的数组，并将'\0'放在索引2处。

Answer 2

String必须为null（0）终止，因此你需要3的数组（[a] [b] [\ 0]）。

int catchar(char a, char b) 
{
  char concat[3] = { a,b,NULL};

  return atoi(concat);
}

我的c代码有什么问题？

2 个答案: