Using same array to store data repeatly

Question

I'm reading data from a file word by word and trying to store in same buffer with space between every word. But words are coming out without space. Please tell me what is wrong? output coming aniljaiswalvikaskumarnikhilkumarindia but i want space between each word.

int main()
{
    char buff[200];
    int l=0;
    FILE *fp;
    fd = NULL;``
    fp = fopen("header.c","r");
    memset(buff,0,200);
    while(1)
    {
        fscanf(fp,"%s",buff+l);
        l=strlen(buff);
        printf("Word of string = %s\n",buff);
        printf("Length of string = %d\n",l);
        if(feof(fp))
        {
            break;
        }
    }
    fclose(fp);
    return 0;
}

add space character explicitly to your buff after each scanf — Omkant
– Omkant, Commented Nov 1, 2012 at 18:13
Well, you never add any spaces. Where did you expect them to come from? — Kerrek SB
– Kerrek SB, Commented Nov 1, 2012 at 18:14
i think use fetc and put each character to buff ... there will not be problem with that — Omkant
– Omkant, Commented Nov 1, 2012 at 18:15

MByD · Accepted Answer · 2012-11-01 18:30:14Z

1

Ignoring the fact that you are exposed to buffer overflow, you can simply add the space:

fscanf(fp,"%s",buff+l);
l=strlen(buff);
buff[l] = ' ';
l++;

answered Nov 1, 2012 at 18:30

MByD

138k30 gold badges269 silver badges278 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Omkant · Accepted Answer · 2012-11-01 18:27:31Z

0

The full working code :

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

     int main()
        {
            char buff[200];
            FILE *fp;
            char c;
            int len=0;
            int i=0;
            fp = fopen("header.c","r");
            memset(buff,0,200);
            while((c=fgetc(fp))!=EOF)
            {
                len++;
                buff[i++]=c;
                if(c==' ' || c=='\n')
                {
                printf("Word of string = %s\n",buff);
                printf("Length of string = %d\n",len);
                }
            }

            fclose(fp);
            return 0;
        }

edited Nov 1, 2012 at 18:27

answered Nov 1, 2012 at 18:21

Omkant

9,24411 gold badges41 silver badges61 bronze badges

2 Comments

Omkant Over a year ago

no problem..... but sometimes scanf creates problem so it's the safest way to read a file

Omkant Over a year ago

Already ... answered.. check the 2nd answer given by Binyamin

Collectives™ on Stack Overflow

Using same array to store data repeatly

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related