数组指针方面逻辑方面的问题...

[cdkrrr]

题目是这样的：
从myfile.txt文件中读取数据，并将数据存在构造数组中。
myfile.txt中的文件内容如下;

Tom@M@91
Lily@F@93

构造类型如下：
struct Family_People
{
char name[20];/*存放姓名*/
char sex[4];/*存放性别*/
char english[20];/*存英语成绩*/
};

我写了一段代码，但是并不能把数据读入到构造类型的数组中：可能有逻辑方面的错误，请各位大侠指点一二：

#include <stdio.h>
#include <string.h>
#include <stddef.h>
#include <stdlib.h>

struct Family_People
 {  
  char name[20];/*存放姓名*/
  char sex[4];/*存放性别*/
  char english[20];/*存英语成绩*/ 
  };


void Fun_Get(struct Family_People *Stu);

int main(void)
{
 struct Family_People Tom_Cat[2];
  
 Fun_Get(Tom_Cat);
 printf("%s,%s,%s,%s,%s,%s",Tom_Cat[0].name,Tom_Cat[0].sex,Tom_Cat[0].english,Tom_Cat[1].name,Tom_Cat[1].sex,Tom_Cat[1].english);
 
 return 0;
}



void Fun_Get(struct Family_People *Stu)
{    
 char temp[20];
 char more[10000];
 int x=0;/*用x控制Stu数组下标*/    

 FILE *pfile2=NULL;
 if(!(pfile2=fopen("myfile.txt","r")))/*声明读取文件指针*/
 {printf("Can not read myfile.");
 exit (1);
 }

 while(fgets(more,10000,pfile2));/*读取一个字符串到more,直到读到了'\n'字符或者10000-1个字符为止*/
 { 
 size_t i;
 int on=0;
 int p=0;
 
 for(i=0;i<strlen(more);i++)
 {
     if(on==0)
     {
         if (more[i]!='@')
         {     
             temp[p++]=more[i];
         }
         else if(more[i]=='@')
         {
          temp[p]='\0';
          strcpy(Stu[x].name,temp);
           on=1;
           p=0;
        }
     }
     
     if(on==1)
     {  
          if (more[i]!='@')
         {     
             temp[p++]=more[i];
         }
         else if(more[i]=='@')
         {
          temp[p]='\0';
          strcpy(Stu[x].sex,temp);
          on=2;
          p=0;
          }     
     }


     if(on==2)
     {  
          if (more[i]!='@')
         {     
             temp[p++]=more[i];
         }
         else if(more[i]=='@')
         {
          temp[p]='\0';
          strcpy(Stu[x].english,temp);
          on=3;
          p=0;
          }     
     }
    
 }
    x++;
 }

 fclose(pfile2);
     
}

[cdkrrr]

感谢 brookmill！！！！

一语惊醒梦中人！！！！！

我原以为是逻辑方面的问题，没想到还是自己对语法掌握的不行啊....

谢谢！！

[brookmill]

首先，while后面多了个分号，造成的后果是只能读取最后一行
while(fgets(more,10000,pfile2)) ; /*读取一个字符串到more,直到读到了'\n'字符或者10000-1个字符为止*/

后面还有其他错误，得慢慢查了

[brookmill]

加两个else
else if(on==1)
else if(on==2)

[brookmill]

91和93后面没有@，所以就不能和前面的一样了
else if(on==2)
{
if (more[i]!='\n' && more[i] != '\0')
{
temp[p++]=more[i];
}
else
{
temp[p]='\0';
strcpy(Stu[x].english,temp);
on=3;
p=0;
}
}