简单的子弹发射队列链表实现问题

jimmy77

Directx2d下的程序最近做个发射子弹的小实验。。。运行的时候最后一颗子弹出屏时发生错误啊～～result CXX0030Error: expression cannot be evaluated 懂的请帮我看下程序：

  子弹发射和出屏用的是队列链表来实现
  //move all the bullets
  for(head = bullet_list.front;head != NULL;head = head->next)
  {
  head->ypos -= dy;

  if(head->ypos < 0)
  {
  head = DeleteBullet(bullet_list,head);
  }
  
  }
我用的是vc6.0。。。
源文件都在附件里。谢谢指导啊!!! [em6]

HuangYZ

错误在于 当双向链表只有一个的时候，DeleteBullet 会返回 NULL 所以删除
执行后，for循环需要执行 head=head->next 而 head 为 NULL 所以发生访问空指针的错误。
这是对 for 循环了解不深入所造成的初级错误。

建议：
1 将遍历表和删除表的操作放在2个循环之内。
2 一般射击游戏，子弹列表可以预先分配一个固定的长度，并且运行时通过状态来判断子弹是否无效。这样减少运行时内存的NEW和DELETE操作。具体可以参考 FREELIST 的设计模式
3 建议使用STL。

jimmy77

恩～～多谢指导～～～会继续改进自己的程序的

jimmy77

修改成功了～～～代码如下：
head = bullet_list.front;
while(head != NULL)
{
   head->ypos = dy;
   if(head->ypos < 0)
   {
      head = DeleteBullet(bullet_list,head);
   }
   if(head != NULL)
      head = head->next;
}
这样子就可以无措运行了～～～帮忙看看还有什么需要改进的没？再次谢谢 HuangYZ

ps:发现是自己对空指针的访问不了解造成的～～呵呵