看了一集小甲鱼的数据结构,讲的双向循环链表,总感觉代码不对劲,后来去论坛看到有人也提出质疑。百度了一下,网上有源码,copy完一运行,果断不对啊,不管输入正负,输出结果都是正数的预期输出结果,也就是输入3和-3是一样的。错误的源码如下
#include <iostream>
#define OK 1
#define ERROR 0
typedef char ElemType;
typedef int Status;
typedef struct DualNode
{
ElemType data;
DualNode *prior;
DualNode *next;
} DualNode, *DuLinkList;
Status InitList(DuLinkList *L)
{
DualNode *p, *q;
*L = new DualNode;
p = *L;
if(*L == NULL)
return ERROR;
(*L)->next = (*L)->prior = NULL;
for(int i = 'A'; i <= 'Z'; i++)
{
q = new DualNode;
if(q == NULL)
return ERROR;
q->data = i;
p->next = q;
q->prior = q;
p = q;
}
p->next = (*L)->next;
(*L)->next->prior = p;
return OK;
}
void Caesar(DuLinkList *L, int i)
{
if( i > 0)
{
do
{
*L = (*L)->next;
}while( --i );
}
else if( i < 0 )
{
do
{
(*L) = (*L)->next;
}while( ++i );
}
}
int main ()
{
DuLinkList L;
int i, n;
InitList(&L);
std::cout<< "请输入一个整数 :\n";
std::cin>> n;
Caesar(&L, n);
for(i = 0; i < 26; i++)
{
L = L->next;
std::cout<< L->data<< " ";
}
return 0;
}
what the F!怎么只能在代码里写,跳不出去了,自己太菜了,还没学会怎么写博客,就这样写吧。
然后我发现是在初始化时候出现的错误,没有进行每个节点的prior的设置。而且Caesar()函数中也有错。改正后如下:
#include <iostream> #define OK 1 #define ERROR 0 typedef char ElemType; typedef int Status; typedef struct DualNode { ElemType data; DualNode *prior; DualNode *next; } DualNode, *DuLinkList; Status InitList(DuLinkList *L) { DualNode *p, *q; *L = new DualNode; p = *L; if (*L == NULL) return ERROR; (*L)->next = (*L)->prior = NULL; for (int i = 'A'; i <= 'Z'; i++) { q = new DualNode; if (q == NULL) return ERROR; q->data = i; p->next = q; q->prior = p; p = q; } p->next = (*L)->next; (*L)->next->prior = p; return OK; } void Caesar(DuLinkList *L, int i) { if (i > 0) { do { *L = (*L)->next; } while (--i); } else if (i < 0) { i -= 1; (*L)->prior = (*L)->next->prior; do { (*L) = (*L)->prior; } while (++i); } } int main() { DuLinkList L; int i, n; InitList(&L); std::cout << "请输入一个整数 :\n"; std::cin >> n; Caesar(&L, n); for (i = 0; i < 26; i++) { L = L->next; std::cout << L->data << " "; } return 0; }
这样就好了,虽然改的不太雅观。