这个嘛，好多地方都不对阿，
比如说变量名字用大写，少头文件，变量的定义和调用不一样。。。。。

个人看法，可以用枚举来改进。

再说几句

但是我很佩服写程序的这个人。他的注释，空格，缩进之类的作得都很好。这种水平的人应该不会犯代码里面的错误。 估计是别人拿过来给改成错的的。

另外，这个程序应该是Uniex/Linux下面编写的，我看他的变量的命名方式不是Windows下的命名方式。

完全不明白...
主函数都没有...

没有主函数不算什么啊。

我觉得设计错误在结构体

typedef struct _RAINBOW
{
int num_cols;
int cols[7];
char *cols_str[7];
} RAINBOW;


int cols[7];char *cols_str[7];都给定了大小是7了
设置num_cols有什么意义
init()中cols_str[i]前面应该加rbow.吧

char *get_rainbow_colour_str(int colour)中char col_str[5];5个肯定不够
strcpy(col_str, rbow.cols_str[colour]);应该为
strcpy(col_str, rbow.cols_str[colour-1]);

int get_rainbow_colour(char *colour_str)中也有同样错误
for (col_idx = RBOW_RED; col_idx <= RBOW_VIOLET; col_idx++)应为
for (col_idx = RBOW_RED-1; col_idx <= RBOW_VIOLET-1; col_idx++)
而且没有异常处理
比如输入颜色为black就没有返回值

int test_rainbow_colour()中测试的每个语句都有返回值，我觉得应该把他们赋给程序中的变量，不然有什么意义

static char rev[] = "$Revision: $";
static char hdr[] = "$Header: $";
这是什么意思，不太明白