Question

求大神帮忙看一下这段C语言代码哪有问题，自己做的一道编程题

题目是这样的：
古罗马帝国开创了辉煌的人类文明，但他们的数字表示法的确有些繁琐，尤其在表示大数的时候，现在看起来简直不能忍受，所以在现代很少使用了。之所以这样，不是因为发明表示法的人的智力的问题，而是因为一个宗教的原因，当时的宗教禁止在数字中出现0的概念！

罗马数字的表示主要依赖以下几个基本符号：

I 1
V 5
X 10
L 50
C 100
D 500
M 1000

这里，我们只介绍一下1000以内的数字的表示法。

单个符号重复多少次，就表示多少倍。最多重复3次。比如：CCC表示300 XX表示20，但150并不用LLL表示，这个规则仅适用于I X C M。
本题目的要求是：请编写程序，由用户输入若干个罗马数字串，程序输出对应的十进制表示。

输入格式是：第一行是整数n,表示接下来有n个罗马数字(n<100)。以后每行一个罗马数字。罗马数字大小不超过999。

要求程序输出n行，就是罗马数字对应的十进制数据。

例如，用户输入：
3
LXXX
XCIII
DCCII

则程序应该输出：
80
93
702
我自己先按照实现一个转换编的，算法没有任何问题，但是改成：第一行是整数n,表示接下来有n个罗马数字，要求程序输出n行这种形式就出问题了，我用VS2010编译调试后输完第一个数回车就这样了

以下是我的代码，求大神帮忙看看这段代码哪里有问题，应该是指针有问题，但我看不出来啊。应该怎么改，大体不用动，函数里的算法不用看懂也不影响，可能有点长，在此先谢谢了！
#include<stdio.h>
#include<stdlib.h>
void turn(char *a);
main()
{
int i,n;
char *d[100];
scanf("%d",&n);
for(i=0;i<n;i++)
scanf("%s",d[i]);
for(i=0;i<n;i++)
turn(d[i]);
return 0;
}

void turn(char *a)
{
int b[20]={0},i,j,*p,sum;
for(i=0;*(a+i)!='\0';i++)
{
switch(a[i]){
case 'I':b[i]=1;break;
case 'V':b[i]=5;break;
case 'X':b[i]=10;break;
case 'L':b[i]=50;break;
case 'C':b[i]=100;break;
case 'D':b[i]=500;break;
case 'M':b[i]=1000;break;
default:{
printf("Error\n");
exit(1);
}
}
}
sum=0;
p=b;
for(j=0;j<20;j++)
{
if(*p>=*(p+1))
sum+=*p;
else
sum-=*p;
p++;
}
printf("%d\n",sum);
}

Answer 1

原因：

 char *d[100];

 scanf("%d",&n);

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

 scanf("%s",d[i]);

没有给d这个数组中的各元素分配存储空间(也即d[0]到d[99]这100个指针变量指向不明确)，又强行给这个元素指向的地址写入值

所以，程序崩溃

 char *d[100];

改成

 char d[100][100];即可

运行结果：

追问

指针数组到底怎么用啊，为什么还要申请空间啊，我觉得定义的时候系统不就自动分配了吗？
还有就是这个只能用二维字符数组用双循环引用吗？指针数组不行吗？
谢谢了！

追答

你想一下
scanf("%s",d[i]);这句是从键盘读取字符串，存到d[i]地址开始的内存中
由于char *d[100];你的元素没有指向任何空间的话(也就是没有分配存储空间的话)，输入的字符串应该存向哪里呢？所以肯定会崩溃
其实你最主要的问题就是没弄明白，指针和指针指向的空间，char *d[100]只是定义了一个100长度的指针数组，每个元素都是一个指针，但由于你没初始化，这些指针不能明确知道，自己到底指向哪个内存空间。
他只是自动分配了存a[0]~a[99]这些指针变量的空间，而指针变量a[0]~a[99]的指向不确定。

追问

哦，我明白了，谢谢你的热心帮助，你能帮我看看
for(i=0;i<n;i++) { d[i]=new char[100];//加上这一句 scanf("%s",d[i]); }
中的d[i]=new char[100]是什么意思吗？可以吗？最后问一下，分可以追加，谢！

追答

可以，这时就有空间来存你输入的数据了
d[i]=new char[100];用new操作符，动态开一个100个char那么大的空间，然后让d[i]指向这个空间的首地址，这样也行，只是注意，用完记得释放你手动申请的内存，不然会内存溢出。不过你目前不用考虑，windows会自动帮你释放。

Answer 2

定义了100的指针数组，但是没有申请内存，scanf的时候会报错，
请改成如下方式试试：
char d[100];
scanf("%d",&n);
for(i=0;i<n;i++)
scanf("%s",&d[i]);

Answer 3

把主程序中的char * d[100] 这一行改一下试试，改成 char d[100][100];

Answer 4

char *d[100];你申明个指针数组干嘛？有且当做一般的数组来用。。不出问题才怪。。
char d[100][100];才对

Answer 5

代码太多，没看，不过看你的错误提示。意思是，你访问了不存在或没有指向的内存。在看看自己的指针用法是否正确。最好用F10和F11进行调试，出错的地方，就应该能查的到。

Answer 6

for(i=0;i<n;i++)
{
d[i]=new char[100];//j加上这一句
scanf("%s",d[i]);
}

char *d[100]; 只是100个空指针而已 ，即d[i]空指针，要去new一个空间 d[i]=new char[100]