这代码的排版让人看了蛋疼呀
建议楼主规范下写代码的风格。。。。

可以去掉，不去掉也可以。一般不加。

应该是没有分配内存，你用的是指针形参，就应该给他分配内存
void scan_element(struct element_t *p)
可以试着改成这样
p=(struct element_t*)malloc(sizeof(struct element_t));
scanf("%d%s%s%f",&(*p).number,&(*p).name,&(*p).symbol,&(*p).weight);

我自己改倒改出自己一堆毛病..
  复制运行了下，没问题貌似.

蜘蛛侠是浩子吗