My Output, cant get it right using sscanf in parsing, whats my problem??

I read a couple of threads lying around the net on parsing strings and such. I came about sscanf as a best method, im gettin close to what i want but my output is not completely correct. My input text is as such:

citystats.txt

San Ignacio;
Cayo;
4000;
5;

Thats my text file im reading infrom from, and on my output i get this with some other zeros which i cant seem to take off, whats my problem. Im kinda new to structures. Between if u guys have a better of suggestions on doing this please feel free to advice. Thanks in advance loves! I know it has something to do with my looping, i believe my i's not sure though

Sorry forgot to put the output, this is it:

San Ignacio
00Cayo
004000
00500000000000000

Thats what i get!

Here is my code:

Code:

---


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

int main (void)
{

  struct stats {
    char town [32];
  char district[32];
    int population;
    int quality;
  };

  struct stats stat [100];
  char line [50];
  int i=0;

 char filename[]="citystats.txt";
 FILE *file=fopen(filename,"r");
  if (file)
  {
    while (fgets(line,sizeof line,file))
   
    sscanf(line,"&#37;[^]%[^;];%[^;];%[^;]",stat[i].town,stat[i].district,&stat[i].population,&stat[i].quality);
 
    i++;
}

  fclose (file);

 for (i=0;i<10;i++){

 printf("%s%s%d%d",stat[i].town,stat[i].district,stat[i].population,stat[i].quality);
}

return 0;

}

---

Im a bit frustrated, i dnt have my orignal codes with me, i had copied it on paper to post, but i thnk i might have copied parts wrong!

You would help us by also posting your output so we know what to look for without having to parse your whole program ourselves.

Dino - official representative of the lazy masses.

Is that your code?
Because it doesn't even compile.

Oh, and add some more braces, so your i++ is part of the while loop.

Quote:

---

Originally Posted by Dino

You would help us by also posting your output so we know what to look for without having to parse your whole program ourselves.

Dino - official representative of the lazy masses.

---

Sorry i forgot bout the output! The output is this:

San Ignacio
00Cayo
004000
00500000000000000

Quote:

---

Originally Posted by Salem

Is that your code?
Because it doesn't even compile.

Oh, and add some more braces, so your i++ is part of the while loop.

---

What do u mean it doesnt compile, i get output and all. i just posted the output i get!

Quote:

---

Originally Posted by Salem

Is that your code?
Because it doesn't even compile.

Oh, and add some more braces, so your i++ is part of the while loop.

---

Sorry matey, i had done a mistype on the codes i posted i already changed it. U were right it wudnt compile!

Quote:

---

Originally Posted by Dino

You would help us by also posting your output so we know what to look for without having to parse your whole program ourselves.

Dino - official representative of the lazy masses.

---

Hey i had mistyped parts of my code, i re did it, its there as i have it now

This won't compile...

Code:

---

printf("%s%s%d%d,stat[i].town,stat[i].district,stat[i].population,stat[i].quality);

---

Try again.

Quote:

---

Originally Posted by Dino

This won't compile...

Code:

---

printf("%s%s%d%d,stat[i].town,stat[i].district,stat[i].population,stat[i].quality);

---

Try again.

---

Sorry forgot the double quotes after the %d, i remodified it AGain! Sorry, its cuz i dnt have my orignal codes here, could u have another look pls love, Hey again matey, really sorry i had an o instead of a zero, i mistyped, i just typed my codes like i said, i have no access to a compiler right now!

You shouldn't be doing any retyping, it should be a cut-paste job.
If you continue to post approximate questions, expect approximate answers.

Either wait until you have the original code to hand, or find a way to get the code.

> sscanf(line,"%[^]%[^;];%[^;];%[^;]",stat[i].town,stat[i].district,&stat[i].population,&stat[i].quality);
If your input file is across 4 lines, as it seems to be, then you need to read 4 lines, and extract each line into each field.

The last two conversions in this sscanf should have been %d anyway.

Like I said, we're just pulling random faults, because it's impossible to tell which part is the real problem

Quote:

---

Originally Posted by Salem

You shouldn't be doing any retyping, it should be a cut-paste job.
If you continue to post approximate questions, expect approximate answers.

Either wait until you have the original code to hand, or find a way to get the code.

> sscanf(line,"%[^]%[^;];%[^;];%[^;]",stat[i].town,stat[i].district,&stat[i].population,&stat[i].quality);
If your input file is across 4 lines, as it seems to be, then you need to read 4 lines, and extract each line into each field.

The last two conversions in this sscanf should have been %d anyway.

Like I said, we're just pulling random faults, because it's impossible to tell which part is the real problem

---

Hey Salem i understand u, i thnk ill get to my pc better in a while and start a new thread, and get rid of this confusion, just my code and paste it like u said. One short question though, on the sscanf. do u mean the last %[^;] should actually be something like %d[^;]

just asking cuz im not really sure how to use sscanf, i came about on it on a recent thread! Could u just pls let me know!

Quote:

---

Originally Posted by AvaGodess

Hey Salem i understand u, i thnk ill get to my pc better in a while and start a new thread, and get rid of this confusion, just my code and paste it like u said. One short question though, on the sscanf. do u mean the last %[^;] should actually be something like %d[^;]

just asking cuz im not really sure how to use sscanf, i came about on it on a recent thread! Could u just pls let me know!

---

If you know how to use scanf, you know how to use sscanf.

Quote:

---

Originally Posted by AvaGodess

Thats what i get!

---

What do you want? If it's taking out the zero's, your output will be almost exactly the same as your input (minus some semi-colons).