Using fscanf with a structure

I don't really use fscanf, so I need some help here.

I need to read in data from file like this:
A 0 7 B 24 67 C 67 6 D 82 9 E ...

letters should be char & #s be int, and I'm using this structure:

Code:

---

typedef struct pcb{
        char pid;
        int arrivetime;
        int jobtime;
        int timeleft;
        int cputime;
        int waitime;
        struct pcb *next;
}PCB;

PCB *plist = NULL; //later points to structs in heap,etc.

---

so this is what I'd like to do:

Code:

---

while(!feof(dfile)){
        fscanf(dfile,"%c",plist->pid);
        fscanf(dfile,"%d",plist->arrivetime);
        fscanf(dfile,"%d",*(plist->jobtime));
        plist->next = new PCB; //make next PCB & update list
}

---

But that doesn't seem to work. I get runtime error with 1st fscanf line. the others don't seem to read the value in at all.

I've tried writing it this way but no good:

Code:

---

fscanf(dfile,"%c",((PCB *)plist)->pid);
fscanf(dfile,"%c",*(PCB *).(char *)plist.pid);

---

What's the right way to use fscanf in this scenario? Or should I use some other method? I definitely don't want to manually parse the data.

Beware of naming instances of structs that similarly to the type name. Especially when you end up declaring pointers of type PCB, when PCB is an instance.

??? so you mean I should use "pcb" for typecasting or rename PCB to remove possible confusions?

Both

edit: I just saw the "typedef" before your structure definition - and it's been too long since I used typedef for me to know what I'm talking about here, so just disregard what I just said, unless someone who uses C more confirms it.

Quote:

---

Originally Posted by daluu

Code:

---

typedef struct pcb{
        char pid;
        int arrivetime;
        int jobtime;
        int timeleft;
        int cputime;
        int waitime;
        struct pcb *next;
}PCB;

PCB *plist = NULL; //later points to structs in heap,etc.

---

so this is what I'd like to do:

Code:

---

while(!feof(dfile)){
        fscanf(dfile,"%c",plist->pid);
        fscanf(dfile,"%d",plist->arrivetime);
        fscanf(dfile,"%d",*(plist->jobtime));
        plist->next = new PCB; //make next PCB & update list
}

---

But that doesn't seem to work. I get runtime error with 1st fscanf line. the others don't seem to read the value in at all.

---

No. You don't dereference pointers to scan into them for fscanf. You give the address-of the variable you want to scan into. You do know how to scan into an int, right?

Code:

---

fscanf( foo, "%d", &myint );

---

Quzah.

> while(!feof(dfile)){
FAQ

quzah, I know that but I'm not good at passing addresses of members within a structure. If pid were by itself I would use &pid. But how do I do it within the context of plist->pid?

I guess I can resort to using a temp variable for fscanf and then pass that to the structure.

salem, I'm not sure what your comment means.

consult the faq as to why using feof () for loop control is bad.

You can lead a horse to water, but you can't make him think
http://faq.cprogramming.com/cgi-bin/...&id=1043284351

Quote:

---

Originally Posted by daluu

If pid were by itself I would use &pid. But how do I do it within the context of plist->pid?

---

&plist->pid

I had thought with the illustration of using scanf, that you'd be able to put two and two together. Apparently not.

Quzah.