代码是以螺旋方式打印给定的数字，如
1 2 3
8 9 4
7 6 5
在调用函数godown()时出现segfault。
#include<stdio.h>
int o[100][100],c[100][100],p=0,q=0,m=0,n=0,a,i,j,r,d,l,u;
void goright(int r);
void goleft(int l);
void goup(int u);
void godown(int d);
int main()
{
int r;
printf("nenter the no. of rowsn");
scanf("%d",&a);
printf("nenter the elements u want to print in a spiral fashionn");
for(i=0;i<a;i++)
{
for(j=0;j<a;j++)
{
scanf("%d",&o[i][j]);
getchar();
}
}
for(i=0;i<a;i++)
{
for(j=0;j<a;j++)
{
printf("%d ",o[i][j]);
}
printf("n");
}
printf("n");
r=a;
goright(r);
for(i=0;i<a;i++)
{
for(j=0;j<a;j++)
{
printf("%d ",c[i][j]);
}
printf("n");
}
return 0;
}
void goright(int r)
{
for(i=0;i<r;i++)
{
c[m][n]=o[p][q];
q++;
n++;
}
if(q>a)
{
q=0;
p++;
}
else
{
q--;
p++;
}
n--;
m++;
d=r-1;
godown(d);
}
void godown(int d)
{
for(i=0;i<d;i++)
{
c[m][n]=o[p][q];
q++;
m++;
}
if(q>a)
{
q=0;
p++;
}
else
{
q--;
p++;
}
m--;
n--;
l=d;
goleft(l);
}
void goleft(int l)
{
for(i=0;i<l;i++)
{
c[m][n]=o[p][q];
q++;
n--;
}
if(q>a)
{
q=0;
p++;
}
else
{
q--;
p++;
}
n++;
m--;
u=l-1;
goup(u);
}
void goup(int u)
{
for(i=0;i<u;i++)
{
c[m][n]=o[p][q];
q++;
m--;
}
if(q>a)
{
q=0;
p++;
}
else
{
q--;
p++;
}
m++;
n++;
r=u;
goright(r);
}
我不明白我为什么会犯这个错误。调试时，goright()函数似乎工作正常，因为我尝试打印螺旋形数组"c"的元素。我尝试过更改变量的名称，但似乎没有任何帮助。
您的递归代码或多或少是这样的：
void goright(int r)
{
...
godown(d);
}
void godown(int d)
{
...
goleft(l);
}
void goleft(int l)
{
...
goup(u);
}
void goup(int u)
{
...
goright(r);
}
我省略的内容(...)不包含任何return语句，因此一旦进行，这种设置将永远重复出现。您可以通过在第一行中打印goxxx函数的自变量来验证这一点：它们将随着幅度的增长而变为负值。
所以你需要一个终止你的rcrsion的标准。您的go函数可能如下所示：
void goright(int r)
{
if (r > 0) {
...
godown(d);
}
}
或者像这样：
void goright(int r)
{
if (r == 0) return;
...
godown(d);
}
您的代码仍然存在问题，主要是因为您将要插入的值存储在二维数组中。将这些存储在a*a大小的线性数组中会容易得多——您只需要跟踪一个以可预测的方式移动的索引。
请不要将所有变量定义为全局变量。它们中的许多都不是必需的，特别是goxxx函数参数的同名者。像i和j这样的迭代器应该是本地的。
您仍然可以使用递归方法。下面是一个例子。它只是用从1开始向上计数的数字填充螺旋。请注意，te打印循环不会完全进行，它们会短于一个，这样正方形就会打印成这样：
r r r d
u d
u d
l l l d
这意味着您不必在做得太过分之后调整变量m和n。只有当你绕了一整圈，也就是说，在你再次开始向右走之后，才需要进行这种调整。这种情况下还有一个额外的检查，用于打印奇数边缘长度的最后一个中心值。
#include<stdio.h>
int c[100][100], p=0, m=0, n=0;
void goright(int r);
void goleft(int l);
void goup(int u);
void godown(int d);
int main()
{
int a;
printf("nenter the no. of rowsn");
scanf("%d",&a);
goright(a);
for(int i = 0; i < a; i++) {
for(int j = 0; j < a; j++) {
printf("%3d",c[i][j]);
}
printf("n");
}
return 0;
}
void goright(int x)
{
if (x == 1) c[m][n] = ++p;
if (x <= 1) return;
for(int i = 1; i < x; i++) {
c[m][n++] = ++p;
}
godown(x);
}
void godown(int x)
{
for(int i = 1; i < x; i++) {
c[m++][n] = ++p;
}
goleft(x);
}
void goleft(int x)
{
for(int i = 1; i < x; i++) {
c[m][n--] = ++p;
}
goup(x);
}
void goup(int x)
{
for(int i = 1; i < x; i++) {
c[m--][n] = ++p;
}
m++;
n++;
goright(x - 2);
}